Download as
Browse Code »

Commit d7b0f56123d1ece9fceda25cf60b1a3bfe14ceaf

Authored by Keilla Menezes
Committed by Daniela Feitosa
1 parent dc6be7cf
Exists in master and in 28 other branches 1.2+SPB2, 2_split_by_role, add_fields_in_api, api_fixies, api_pagination, api_private_token, api_tasks, article_hit_count, blog_page_bug, checkbox_to_user_can_edit_page, communities_ratings, communities_ratings_moderation, community_ratings_rebase, fix_cloned_article_parent, fix_gallery_image_url, fix_javascript_categories, fix_optional_fields, fix_suggest_article, fix_user_email_validation, highlight_block_fix, notification_plugin, notification_plugin_rebase, script_to_update, split_by_role, stable, temp_ratings, tests, tests_fixies

Keep the feed reader block enabled after edition 

(ActionItem1798)
Showing 2 changed files with 7 additions and 1 deletions   Show diff stats
app/models/feed_reader_block.rb
  Diff comments   View file @d7b0f56
... ... @@ -12,7 +12,7 @@ class FeedReaderBlock < Block
12 12 def address=(new_address)
13 13 old_address = address
14 14 orig_set_address(new_address)
15   - self.enabled = (old_address.blank? && !new_address.blank?) || (new_address && new_address != old_address) || false
  15 + self.enabled = (new_address && new_address != old_address) || (new_address && self.enabled) || false
16 16 end
17 17  
18 18 settings_items :limit, :type => :integer
... ...
test/unit/feed_reader_block_test.rb
  Diff comments   View file @d7b0f56
... ... @@ -173,4 +173,10 @@ class FeedReaderBlockTest < ActiveSupport::TestCase
173 173 assert_equal true, reader.enabled, 'must enable when setting to new address'
174 174 end
175 175  
  176 + should 'kepp enable when address is not changed' do
  177 + reader = build(:feed_reader_block, :address => 'http://www.example.com/feed')
  178 + reader.address = 'http://www.example.com/feed'
  179 + assert_equal true, reader.enabled
  180 + end
  181 +
176 182 end
... ...