Download as
Browse Code »

Commit a710bcc721baf6ed9af4ff3a2e235d3d68f179c0

Authored by Gabriela Navarro
Committed by Rodrigo Souto
1 parent 9360f04a
Exists in master and in 22 other branches angular_poc, api-article-archived, article-readonly, backup-7-jan-2016, captcha_serpro_plugin, content-manager-hostspot, export-comment-paragraph, fix_event_date_issue, login-captcha, master_rails3, new_video_plugin, pg_search_rank, production, refactor_with_role, refactor_with_role_scope, remove_profile_cat_icons, resend_confirmation_email, staging, staging_rails3, tasks_keep_filter_params, theme-brasil-digital-from-staging, travis

Change logic for showing public articles in private communities 

Signed-off-by: Alvaro Fernando <alvarofernandoms@gmail.com>
Signed-off-by: Arthur Del Esposte <arthurmde@gmail.com>
Signed-off-by: David Carlos <ddavidcarlos1392@gmail.com>
Signed-off-by: Eduardo Vital <vitaldu@gmail.com>
Signed-off-by: Fabio Teixeira <fabio1079@gmail.com>
Signed-off-by: Gabriela Navarro <navarro1703@gmail.com>
Signed-off-by: Luciano Prestes <lucianopcbr@gmail.com>
Signed-off-by: Tallys Martins <tallysmartins@gmail.com>
Showing 13 changed files with 67 additions and 53 deletions   Show diff stats
app/controllers/my_profile/cms_controller.rb
  Diff comments   View file @a710bcc
... ... @@ -143,6 +143,7 @@ class CmsController &lt; MyProfileController
143 143 klass = @type.constantize
144 144 article_data = environment.enabled?('articles_dont_accept_comments_by_default') ? { :accept_comments => false } : {}
145 145 article_data.merge!(params[:article]) if params[:article]
  146 + article_data.merge!(:profile => profile) if profile
146 147 @article = klass.new(article_data)
147 148  
148 149 parent = check_parent(params[:parent_id])
... ... @@ -220,7 +221,7 @@ class CmsController &lt; MyProfileController
220 221 if @errors.any?
221 222 render :action => 'upload_files', :parent_id => @parent_id
222 223 else
223   - session[:notice] = _('File(s) successfully uploaded')
  224 + session[:notice] = _('File(s) successfully uploaded')
224 225 if @back_to
225 226 redirect_to @back_to
226 227 elsif @parent
... ...
app/controllers/public/content_viewer_controller.rb
  Diff comments   View file @a710bcc
... ... @@ -13,7 +13,7 @@ class ContentViewerController &lt; ApplicationController
13 13 @version = params[:version].to_i
14 14  
15 15 if path.blank?
16   - @page = profile.home_page
  16 + @page = profile.home_page
17 17 return if redirected_to_profile_index
18 18 else
19 19 @page = profile.articles.find_by_path(path)
... ... @@ -121,21 +121,23 @@ class ContentViewerController &lt; ApplicationController
121 121 helper_method :pass_without_comment_captcha?
122 122  
123 123 def allow_access_to_page(path)
124   - allowed = true
125 124 if @page.nil? # page not found, give error
126 125 render_not_found(path)
127   - allowed = false
128   - elsif !@page.display_to?(user)
129   - if !profile.public?
  126 + return false
  127 + end
  128 +
  129 + unless @page.display_to?(user)
  130 + if !profile.visible? || profile.secret? || (user && user.follows?(profile))
  131 + render_access_denied
  132 + else #!profile.public?
130 133 private_profile_partial_parameters
131 134 render :template => 'profile/_private_profile', :status => 403, :formats => [:html]
132   - allowed = false
133   - else #if !profile.visible?
134   - render_access_denied
135   - allowed = false
136 135 end
  136 +
  137 + return false
137 138 end
138   - allowed
  139 +
  140 + return true
139 141 end
140 142  
141 143 def user_is_a_bot?
... ... @@ -180,7 +182,7 @@ class ContentViewerController &lt; ApplicationController
180 182 if @page.forum? && @page.has_terms_of_use && terms_accepted == "true"
181 183 @page.add_agreed_user(user)
182 184 end
183   - end
  185 + end
184 186  
185 187 def is_a_forum_topic? (page)
186 188 return (!@page.parent.nil? && @page.parent.forum?)
... ...
app/controllers/public_controller.rb
  Diff comments   View file @a710bcc
... ... @@ -3,7 +3,7 @@ class PublicController &lt; ApplicationController
3 3  
4 4 def allow_access_to_page
5 5 unless profile.display_info_to?(user)
6   - if profile.visible?
  6 + if profile.visible? && !profile.secret
7 7 private_profile
8 8 else
9 9 invisible_profile
... ...
app/models/article.rb
  Diff comments   View file @a710bcc
... ... @@ -25,6 +25,16 @@ class Article &lt; ActiveRecord::Base
25 25 :display => %w[full]
26 26 }
27 27  
  28 + def initialize(*params)
  29 + super
  30 +
  31 + if !params.blank? && params.first.has_key?(:profile)
  32 + profile = params.first[:profile]
  33 + self.published = false unless profile.public?
  34 + end
  35 +
  36 + end
  37 +
28 38 def self.default_search_display
29 39 'full'
30 40 end
... ... @@ -488,14 +498,14 @@ class Article &lt; ActiveRecord::Base
488 498  
489 499 scope :display_filter, lambda {|user, profile|
490 500 return published if (user.nil? && profile && profile.public?)
491   - return [] if user.nil? || (profile && !profile.public? && !user.follows?(profile))
  501 + return [] if user.nil? || profile.nil? || (profile && !profile.public? && !user.follows?(profile))
492 502 where(
493 503 [
494   - "published = ? OR last_changed_by_id = ? OR profile_id = ? OR ?
495   - OR (show_to_followers = ? AND ?)", true, user.id, user.id,
  504 + "published = ? OR last_changed_by_id = ? OR profile_id = ? OR ?
  505 + OR (show_to_followers = ? AND ? AND profile_id = ?)", true, user.id, user.id,
496 506 profile.nil? ? false : user.has_permission?(:view_private_content, profile),
497   - true, user.follows?(profile)
498   - ]
  507 + true, user.follows?(profile), profile.id
  508 + ]
499 509 )
500 510 }
501 511  
... ... @@ -509,7 +519,7 @@ class Article &lt; ActiveRecord::Base
509 519  
510 520 def display_to?(user = nil)
511 521 if published?
512   - profile.display_info_to?(user)
  522 + (profile.secret? || !profile.visible?) ? profile.display_info_to?(user) : true
513 523 else
514 524 if !user
515 525 false
... ...
features/article_versioning.feature
  Diff comments   View file @a710bcc
... ... @@ -80,8 +80,8 @@ Feature: article versioning
80 80  
81 81 Scenario: deny access to specific version when disabled, private and not logged
82 82 Given the article "Edited Article" is updated with
83   - | display_versions | published |
84   - | false | false |
  83 + | display_versions | published | show_to_followers |
  84 + | false | false | false |
85 85 And I am not logged in
86 86 And I go to /joaosilva/edited-article?version=1
87 87 Then I should see "Access denied"
... ...
features/edit_article.feature
  Diff comments   View file @a710bcc
... ... @@ -41,6 +41,7 @@ Feature: edit article
41 41 When I follow "Folder"
42 42 And I fill in "Title" with "My Folder"
43 43 And I choose "article_published_false"
  44 + And I uncheck "article_show_to_followers"
44 45 And I press "Save"
45 46 And I log off
46 47 And I go to /freesoftware/my-folder
... ... @@ -87,6 +88,7 @@ Feature: edit article
87 88 When I follow "Folder"
88 89 And I fill in "Title" with "My Folder"
89 90 And I choose "article_published_false"
  91 + And I uncheck "article_show_to_followers"
90 92 Then I should see "Fill in the search field to add the exception users to see this content"
91 93  
92 94 @selenium
... ...
features/secret_community.feature
  Diff comments   View file @a710bcc
... ... @@ -33,7 +33,7 @@ Feature: Use a secret community
33 33 Scenario: Non members shouldn't see secret communit's content
34 34 Given I am logged in as "maria"
35 35 And I go to mycommunity's homepage
36   - And I should see "Access denied"
  36 + And I should see "Oops ... you cannot go ahead here"
37 37 And I follow "Communities"
38 38 Then I should not see "My Community"
39 39  
... ...
test/functional/contact_controller_test.rb
  Diff comments   View file @a710bcc
... ... @@ -131,7 +131,7 @@ class ContactControllerTest &lt; ActionController::TestCase
131 131 post :new, :profile => community.identifier
132 132  
133 133 assert_response :forbidden
134   - assert_template :private_profile
  134 + assert_template "profile/_private_profile"
135 135 end
136 136  
137 137 should 'not show send e-mail page to non members of invisible community' do
... ...
test/functional/content_viewer_controller_test.rb
  Diff comments   View file @a710bcc
... ... @@ -257,22 +257,22 @@ class ContentViewerControllerTest &lt; ActionController::TestCase
257 257 end
258 258  
259 259 should 'not give access to private articles if logged off' do
260   - profile = Profile.create!(:name => 'test profile', :identifier => 'test_profile')
  260 + profile = Community.create!(:name => 'test profile', :identifier => 'test_profile')
261 261 intranet = Folder.create!(:name => 'my_intranet', :profile => profile, :published => false)
262 262  
263 263 get :view_page, :profile => 'test_profile', :page => [ 'my-intranet' ]
264 264  
265   - assert_template 'access_denied'
  265 + assert_template "profile/_private_profile"
266 266 end
267 267  
268 268 should 'not give access to private articles if logged in but not member' do
269 269 login_as('testinguser')
270   - profile = Profile.create!(:name => 'test profile', :identifier => 'test_profile')
  270 + profile = Community.create!(:name => 'test profile', :identifier => 'test_profile')
271 271 intranet = Folder.create!(:name => 'my_intranet', :profile => profile, :published => false)
272 272  
273 273 get :view_page, :profile => 'test_profile', :page => [ 'my-intranet' ]
274 274  
275   - assert_template 'access_denied'
  275 + assert_template "profile/_private_profile"
276 276 end
277 277  
278 278 should 'not give access to private articles if logged in and only member' do
... ... @@ -1428,7 +1428,7 @@ class ContentViewerControllerTest &lt; ActionController::TestCase
1428 1428  
1429 1429 article = TinyMceArticle.create(:name => 'Article to be shared with images',
1430 1430 :body => 'This article should be shared with all social networks',
1431   - :profile => @profile,
  1431 + :profile => community,
1432 1432 :published => false,
1433 1433 :show_to_followers => true)
1434 1434 article.parent = blog
... ...
test/functional/events_controller_test.rb
  Diff comments   View file @a710bcc
... ... @@ -60,7 +60,7 @@ class EventsControllerTest &lt; ActionController::TestCase
60 60 post :events, :profile => community.identifier
61 61  
62 62 assert_response :forbidden
63   - assert_template :private_profile
  63 + assert_template "profile/_private_profile"
64 64 end
65 65  
66 66 should 'not show events page to non members of invisible community' do
... ...
test/integration/http_caching_test.rb
  Diff comments   View file @a710bcc
... ... @@ -85,7 +85,7 @@ class HttpCachingTest &lt; ActionController::IntegrationTest
85 85  
86 86 test 'private community content should not return cache headers' do
87 87 community = create_private_community('the-community')
88   - create(Article, profile_id: community.id, name: 'Test page')
  88 + create(Article, profile_id: community.id, name: 'Test page', published: false)
89 89  
90 90 get "/the-community/test-page"
91 91 assert_response 403
... ... @@ -139,4 +139,3 @@ class HttpCachingTest &lt; ActionController::IntegrationTest
139 139 end
140 140  
141 141 end
142   -
... ...
test/unit/article_test.rb
  Diff comments   View file @a710bcc
... ... @@ -484,7 +484,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
484 484  
485 485 should 'say that member user can not see private article' do
486 486 profile = fast_create(Profile, :name => 'test profile', :identifier => 'test_profile')
487   - article = fast_create(Article, :name => 'test article', :profile_id => profile.id, :published => false)
  487 + article = fast_create(Article, :name => 'test article', :profile_id => profile.id, :published => false, :show_to_followers => false)
488 488 person = create_user('test_user').person
489 489 profile.affiliate(person, Profile::Roles.member(profile.environment.id))
490 490  
... ... @@ -509,15 +509,15 @@ class ArticleTest &lt; ActiveSupport::TestCase
509 509 assert article.display_to?(person)
510 510 end
511 511  
512   - should 'not show article to non member if article public but profile private' do
  512 + should 'show article to non member if article public but profile private' do
513 513 profile = fast_create(Profile, :name => 'test profile', :identifier => 'test_profile', :public_profile => false)
514 514 article = fast_create(Article, :name => 'test article', :profile_id => profile.id, :published => true)
515 515 person1 = create_user('test_user1').person
516 516 profile.affiliate(person1, Profile::Roles.member(profile.environment.id))
517 517 person2 = create_user('test_user2').person
518 518  
519   - assert !article.display_to?(nil)
520   - assert !article.display_to?(person2)
  519 + assert article.display_to?(nil)
  520 + assert article.display_to?(person2)
521 521 assert article.display_to?(person1)
522 522 end
523 523  
... ... @@ -543,7 +543,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
543 543  
544 544 should 'not allow friends of private person see the article' do
545 545 person = create_user('test_user').person
546   - article = create(Article, :name => 'test article', :profile => person, :published => false)
  546 + article = create(Article, :name => 'test article', :profile => person, :published => false, :show_to_followers => false)
547 547 friend = create_user('test_friend').person
548 548 person.add_friend(friend)
549 549 person.save!
... ... @@ -1686,7 +1686,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
1686 1686 a.allow_members_to_edit = true
1687 1687 assert !a.allow_edit?(nil)
1688 1688 end
1689   -
  1689 +
1690 1690 should 'allow author to edit topic' do
1691 1691 community = fast_create(Community)
1692 1692 admin = fast_create(Person)
... ... @@ -1905,7 +1905,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
1905 1905 end
1906 1906  
1907 1907 should 'display_filter display only public articles if there is no user' do
1908   - p = fast_create(Person)
  1908 + p = fast_create(Person)
1909 1909 Article.delete_all
1910 1910 a = fast_create(Article, :published => true, :profile_id => p.id)
1911 1911 fast_create(Article, :published => false, :profile_id => p.id)
... ... @@ -1915,7 +1915,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
1915 1915  
1916 1916 should 'display_filter display public articles for users' do
1917 1917 user = create_user('someuser').person
1918   - p = fast_create(Person)
  1918 + p = fast_create(Person)
1919 1919 user.stubs(:has_permission?).with(:view_private_content, p).returns(false)
1920 1920 Article.delete_all
1921 1921 a = fast_create(Article, :published => true, :profile_id => p.id)
... ... @@ -1926,7 +1926,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
1926 1926  
1927 1927 should 'display_filter display private article last changed by user' do
1928 1928 user = create_user('someuser').person
1929   - p = fast_create(Person)
  1929 + p = fast_create(Person)
1930 1930 user.stubs(:has_permission?).with(:view_private_content, p).returns(false)
1931 1931 Article.delete_all
1932 1932 a = fast_create(Article, :published => false, :last_changed_by_id => user.id, :profile_id => p.id)
... ... @@ -1938,7 +1938,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
1938 1938 should 'display_filter display user private article of his own profile' do
1939 1939 user = create_user('someuser').person
1940 1940 user.stubs(:has_permission?).with(:view_private_content, user).returns(false)
1941   - p = fast_create(Person)
  1941 + p = fast_create(Person)
1942 1942 Article.delete_all
1943 1943 a = fast_create(Article, :published => false, :profile_id => user.id)
1944 1944 fast_create(Article, :published => false, :profile_id => p.id)
... ... @@ -1948,7 +1948,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
1948 1948  
1949 1949 should 'display_filter show profile private content if the user has view_private_content permission' do
1950 1950 user = create_user('someuser').person
1951   - p = fast_create(Person)
  1951 + p = fast_create(Person)
1952 1952 Article.delete_all
1953 1953 user.stubs(:has_permission?).with(:view_private_content, p).returns(false)
1954 1954 a = fast_create(Article, :published => false, :profile_id => p.id)
... ... @@ -1965,8 +1965,8 @@ class ArticleTest &lt; ActiveSupport::TestCase
1965 1965 user.stubs(:has_permission?).with(:view_private_content, p).returns(false)
1966 1966 Article.delete_all
1967 1967 a = fast_create(Article, :published => false, :show_to_followers => true, :profile_id => p.id)
1968   - fast_create(Article, :published => false, :profile_id => p.id)
1969   - fast_create(Article, :published => false, :profile_id => p.id)
  1968 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
  1969 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
1970 1970 assert_equal [a], Article.display_filter(user, p)
1971 1971 end
1972 1972  
... ... @@ -1977,8 +1977,8 @@ class ArticleTest &lt; ActiveSupport::TestCase
1977 1977 user.stubs(:has_permission?).with(:view_private_content, p).returns(false)
1978 1978 Article.delete_all
1979 1979 a = fast_create(Article, :published => false, :show_to_followers => true, :profile_id => p.id)
1980   - fast_create(Article, :published => false, :profile_id => p.id)
1981   - fast_create(Article, :published => false, :profile_id => p.id)
  1980 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
  1981 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
1982 1982 assert_equal [a], Article.display_filter(user, p)
1983 1983 end
1984 1984  
... ... @@ -2057,8 +2057,8 @@ class ArticleTest &lt; ActiveSupport::TestCase
2057 2057 user.stubs(:has_permission?).with(:view_private_content, p).returns(false)
2058 2058 Article.delete_all
2059 2059 a = fast_create(Article, :published => true, :profile_id => p.id)
2060   - fast_create(Article, :published => false, :profile_id => p.id)
2061   - fast_create(Article, :published => false, :profile_id => p.id)
  2060 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
  2061 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
2062 2062 assert_equal [a], Article.display_filter(user, p)
2063 2063 end
2064 2064  
... ... @@ -2088,7 +2088,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
2088 2088 a1 = fast_create(Article, :published => true, :profile_id => user.id)
2089 2089 a2 = fast_create(Article, :published => true, :profile_id => p.id)
2090 2090 fast_create(Article, :published => false, :profile_id => p.id)
2091   - assert_equivalent [a1,a2], Article.display_filter(user, nil)
  2091 + assert_equivalent [a1,a2], Article.display_filter(nil, user)
2092 2092 end
2093 2093  
2094 2094 should 'display_filter show person public content of private person profile for user friends' do
... ... @@ -2099,8 +2099,8 @@ class ArticleTest &lt; ActiveSupport::TestCase
2099 2099 user.stubs(:has_permission?).with(:view_private_content, p).returns(false)
2100 2100 Article.delete_all
2101 2101 a = fast_create(Article, :published => true, :profile_id => p.id)
2102   - fast_create(Article, :published => false, :profile_id => p.id)
2103   - fast_create(Article, :published => false, :profile_id => p.id)
  2102 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
  2103 + fast_create(Article, :published => false, :show_to_followers => false, :profile_id => p.id)
2104 2104 assert_equal [a], Article.display_filter(user, p)
2105 2105 end
2106 2106  
... ... @@ -2130,7 +2130,7 @@ class ArticleTest &lt; ActiveSupport::TestCase
2130 2130 a1 = fast_create(Article, :published => true, :profile_id => user.id)
2131 2131 a2 = fast_create(Article, :published => true, :profile_id => p.id)
2132 2132 fast_create(Article, :published => false, :profile_id => p.id)
2133   - assert_equivalent [a1,a2], Article.display_filter(user, nil)
  2133 + assert_equivalent [a1,a2], Article.display_filter(nil, user)
2134 2134 end
2135 2135  
2136 2136 end
... ...
test/unit/folder_helper_test.rb
  Diff comments   View file @a710bcc
... ... @@ -68,7 +68,7 @@ class FolderHelperTest &lt; ActionView::TestCase
68 68 profile.public_profile = false
69 69 profile.save!
70 70 profile2 = create_user('Folder Viwer').person
71   - folder = fast_create(Folder, :profile_id => profile.id)
  71 + folder = fast_create(Folder, :profile_id => profile.id, :published => false)
72 72 article = fast_create(Article, {:parent_id => folder.id, :profile_id => profile.id})
73 73  
74 74 result = available_articles(folder.children, profile2)
... ...