Merge branch 'master' into staging

Leandro Santos
2 parents 03bb9bb0 a0f2837c
Showing 2 changed files with 2 additions and 6 deletions Show diff stats
app/models/approve_article.rb
test/unit/approve_article_test.rb
@@ -9,9 +9,6 @@ class ApproveArticle &lt; Task
       if target.person? && requestor != target
         self.errors.add(:requestor, _('You can not post articles to other users.'))
       end
-      if target.organization? && !target.members.include?(requestor) && target.environment.portal_community != target
-        self.errors.add(:requestor, _('Only members can post articles on communities.'))
-      end
     end
   end
@@ -462,7 +462,7 @@ class ApproveArticleTest &lt; ActiveSupport::TestCase
     assert t2.invalid?(:requestor)
   end
-  should 'allow only members to be requestors when target is a community' do
+  should 'allow anyone to be requestors when target is a community' do
     community = fast_create(Community)
     member = fast_create(Person)
     community.add_member(member)
@@ -472,8 +472,7 @@ class ApproveArticleTest &lt; ActiveSupport::TestCase
     t2 = ApproveArticle.new(:requestor => non_member, :target => community)
     assert t1.valid?
-    assert !t2.valid?
-    assert t2.invalid?(:requestor)
+    assert t2.valid?
   end
   should 'allow any user to be requestor whe the target is the portal community' do