[Git][noosfero/noosfero][master] 2 commits: Fix saving unsubscribers list

[Victor Costa]

Victor Costa pushed to branch master at Noosfero / noosfero


Commits:
7e578a25 by Evandro Junior at 2016-12-27T08:30:34-03:00
Fix saving unsubscribers list

- - - - -
99f4cb0e by Victor Costa at 2016-12-27T12:11:02+00:00
Merge branch 'fix_unsubscribe_newsletter' into 'master'

Fix saving unsubscribers list

See merge request !1076
- - - - -


2 changed files:

- plugins/newsletter/lib/newsletter_plugin/newsletter.rb
- plugins/newsletter/test/unit/newsletter_plugin_newsletter_test.rb


Changes:

=====================================
plugins/newsletter/lib/newsletter_plugin/newsletter.rb
=====================================
--- a/plugins/newsletter/lib/newsletter_plugin/newsletter.rb
+++ b/plugins/newsletter/lib/newsletter_plugin/newsletter.rb
@@ -183,6 +183,7 @@ class NewsletterPlugin::Newsletter < ApplicationRecord
 
   def unsubscribe(email)
     unsubscribers.push(email).uniq!
+    save!
   end
 
 end


=====================================
plugins/newsletter/test/unit/newsletter_plugin_newsletter_test.rb
=====================================
--- a/plugins/newsletter/test/unit/newsletter_plugin_newsletter_test.rb
+++ b/plugins/newsletter/test/unit/newsletter_plugin_newsletter_test.rb
@@ -440,7 +440,7 @@ EOS
       :person => fast_create(Person)
     )
     newsletter.unsubscribe(p2.email)
-    assert_equivalent [p1, p3], newsletter.people
+    assert_equivalent [p1, p3], newsletter.reload.people
   end
 
   should "no filter newsletter's recipients if unsubscribers list empty" do



View it on GitLab: https://gitlab.com/noosfero/noosfero/compare/73c5c09b0ccfd663e58f163afb1e6a956afd31bf...99f4cb0ebe9cb2f2cd4030faa0f25e4bd94cd5f8
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listas.softwarelivre.org/pipermail/noosfero-dev/attachments/20161227/6c98c60c/attachment-0001.html>