Download as
Browse Code »

Commit 80800e40322891929e85049de33896c832edf87b

Authored by Arthur Esposte
1 parent e3817276
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

Change feed published metho to consider it's parent privacy

Showing 2 changed files with 36 additions and 1 deletions   Show diff stats
app/models/rss_feed.rb
  Diff comments   View file @80800e4
... ... @@ -14,7 +14,7 @@ class RssFeed < Article
14 14  
15 15 # store setting in body
16 16 serialize :body, Hash
17   -
  17 +
18 18 def body
19 19 self[:body] ||= {}
20 20 end
... ... @@ -89,6 +89,14 @@ class RssFeed < Article
89 89 )
90 90 end
91 91  
  92 + def published?
  93 + if self.parent
  94 + self.parent.published?
  95 + else
  96 + self.published
  97 + end
  98 + end
  99 +
92 100 def self.short_description
93 101 _('RSS Feed')
94 102 end
... ...
test/unit/rss_feed_test.rb
  Diff comments   View file @80800e4
... ... @@ -262,4 +262,31 @@ class RssFeedTest < ActiveSupport::TestCase
262 262 assert_equal [es_post], blog.feed.fetch_articles
263 263 end
264 264  
  265 + should 'a feed have the same privacy of its parent' do
  266 + profile = create_user('testuser').person
  267 + blog = create(Blog, :name => 'blog-test', :profile => profile)
  268 + feed = blog.feed
  269 +
  270 + assert blog.published?
  271 + assert feed.published?
  272 +
  273 + feed.published = false
  274 + feed.save
  275 +
  276 + assert blog.published?
  277 + assert feed.published?
  278 +
  279 + blog.published = false
  280 + blog.save
  281 + feed.reload
  282 +
  283 + assert !blog.published?
  284 + assert !feed.published?
  285 +
  286 + feed.published = true
  287 + feed.save
  288 +
  289 + assert !blog.published?
  290 + assert !feed.published?
  291 + end
265 292 end
... ...