[Git][noosfero/noosfero][master] 2 commits: mailing-list-plugin: translations adjusts and error handling

[Rodrigo Souto]

Rodrigo Souto pushed to branch master at Noosfero / noosfero


Commits:
e7fa637f by Rodrigo Souto at 2017-08-11T20:05:28Z
mailing-list-plugin: translations adjusts and error handling

- - - - -
d6fc51a1 by Rodrigo Souto at 2017-09-15T13:30:42Z
Merge branch 'mailing-list-fixes' into 'master'

mailing-list-plugin: translations adjusts and error handling

See merge request noosfero/noosfero!1298
- - - - -


3 changed files:

- plugins/mailing_list/lib/mailing_list_plugin/client.rb
- plugins/mailing_list/po/pt/mailing_list.po
- plugins/mailing_list/views/mailing_list_plugin_myprofile/_subscriptions_table.html.erb


Changes:

=====================================
plugins/mailing_list/lib/mailing_list_plugin/client.rb
=====================================
--- a/plugins/mailing_list/lib/mailing_list_plugin/client.rb
+++ b/plugins/mailing_list/lib/mailing_list_plugin/client.rb
@@ -14,7 +14,11 @@ class MailingListPlugin::Client
   end
 
   def review(group)
-    subscribers = client.review(treat_identifier(group.identifier))
+    begin
+      subscribers = client.review(treat_identifier(group.identifier))
+    rescue SOAP::FaultError
+      subscribers = []
+    end
     subscribers == ['no_subscribers'] ? [] : subscribers
   end
 
@@ -64,11 +68,11 @@ class MailingListPlugin::Client
   end
 
   def subscribe_person_on_group_list(person, group)
-    add(person.email, treat_identifier(group.identifier), person.name) unless person_subscribed_on_group_list?(person, group)
+    add(person.email, treat_identifier(group.identifier), person.name) unless !group_list_exist?(group) || person_subscribed_on_group_list?(person, group)
   end
 
   def unsubscribe_person_from_group_list(person, group)
-    del(person.email, treat_identifier(group.identifier)) if person_subscribed_on_group_list?(person, group)
+    del(person.email, treat_identifier(group.identifier)) if group_list_exist?(group) && person_subscribed_on_group_list?(person, group)
   end
 
   def deploy_list_for_group(group)


=====================================
plugins/mailing_list/po/pt/mailing_list.po
=====================================
--- a/plugins/mailing_list/po/pt/mailing_list.po
+++ b/plugins/mailing_list/po/pt/mailing_list.po
@@ -450,14 +450,14 @@ msgid ""
 "           comments."
 msgstr ""
 "Essa funcionalidade irá permitir que você integre os fóruns e blogs "
-"dessegrupo com uma lista de discussão. Depois de implantar essa "
+"desse grupo com uma lista de discussão. Depois de implantar essa "
 "funcionalidade, uma nova lista de discussão será criada automaticamente e "
-"todos os membrosdesse grupo serão inscritos nessa lista de discussão. Após "
-"de a funcionalidadeser implantada, você poderá selecionar quais fóruns ou "
-"blogs devem serintegrados com a sua lista de discussão and então todos os "
-"tópicos epostagens criados nesses fóruns e blogs selecionados serão "
-"enviadospor e-mail para a lista de discussão. O mesmo irá acontecer com "
-"comentáriosnos conteúdos e respostas aos comentários."
+"todos os membros desse grupo serão inscritos nessa lista de discussão. Após "
+"a funcionalidade ser implantada, você poderá selecionar quais fóruns ou "
+"blogs devem ser integrados com a sua lista de discussão e então todos os "
+"tópicos e postagens criados nesses fóruns e blogs selecionados serão "
+"enviados por e-mail para a lista de discussão. O mesmo irá acontecer com "
+"comentários nos conteúdos e respostas aos comentários."
 
 #: ../views/mailing_list_plugin_myprofile_organization/edit.html.erb:19
 msgid "Deploy"


=====================================
plugins/mailing_list/views/mailing_list_plugin_myprofile/_subscriptions_table.html.erb
=====================================
--- a/plugins/mailing_list/views/mailing_list_plugin_myprofile/_subscriptions_table.html.erb
+++ b/plugins/mailing_list/views/mailing_list_plugin_myprofile/_subscriptions_table.html.erb
@@ -19,12 +19,21 @@
       group = related
       ongoing_subscription = MailingListPlugin::AcceptSubscription.ongoing_subscription?(person, group)
     end %>
-    <% subscribed = @client.person_subscribed_on_group_list?(person, group) %>
+    <% deployed = @client.group_list_exist?(group) %>
+    <% subscribed = deployed ? @client.person_subscribed_on_group_list?(person, group) : false %>
+    <% subscribe_html_options = {:class => ongoing_subscription || !deployed ? 'fetching' : ''} %>
+    <% if !deployed %>
+      <% td_title = _('List is not deployed yet.') %>
+    <% elsif ongoing_subscription %>
+      <% td_title = _('Subscription waiting approval') %>
+    <% else %>
+      <% td_title = '' %>
+    <% end %>
     <tr>
       <td><%= link_to_profile related.short_name(48), related.identifier, :title => _('View profile') %> </td>
       <td style="text-align: center"><%= subscribed ? _('Yes') : _('No') %></td>
-      <%= content_tag('td', :title => ongoing_subscription ? _('Subscription waiting approval') : '') do %>
-        <%= button_without_text 'subscribe', _('Subscribe'), {:action => 'subscribe', :id => related.id}, :class => ongoing_subscription ? 'fetching' : '' unless subscribed %>
+      <%= content_tag('td', :title => td_title) do %>
+        <%= button_without_text 'subscribe', _('Subscribe'), {:action => 'subscribe', :id => related.id}, subscribe_html_options unless subscribed %>
         <%= button_without_text 'unsubscribe', _('Unsubscribe'), {:action => 'unsubscribe', :id => related.id}, data: {confirm: _("Do you really want to unsubscribe?")} if subscribed %>
       <% end %>
     </tr>



View it on GitLab: https://gitlab.com/noosfero/noosfero/compare/1c40bc115cca92d6874ed385bf5e56a14704397e...d6fc51a1bf2a2322028727b4349c88056756fe95

---
View it on GitLab: https://gitlab.com/noosfero/noosfero/compare/1c40bc115cca92d6874ed385bf5e56a14704397e...d6fc51a1bf2a2322028727b4349c88056756fe95
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listas.softwarelivre.org/pipermail/noosfero-dev/attachments/20170915/ec2c0f7a/attachment-0001.html>