decidim-core 0.27.5 → 0.27.7

data/app/presenters/decidim/admin_log/oauth_application_resource_presenter.rb

@@ -11,7 +11,7 @@ module Decidim
       #
       # Returns an HTML-safe String.
       def resource_path
-        @resource_path ||= h.decidim_admin.oauth_application_path(resource)
+        @resource_path ||= h.decidim_system.oauth_application_path(resource)
       end
     end
   end

data/app/presenters/decidim/admin_log/organization_presenter.rb

@@ -26,7 +26,7 @@ module Decidim
 
       def settings_attributes_mapping
         {
-          name: :string,
+          name: :i18n,
           default_locale: :locale,
           reference_prefix: :string,
           twitter_handler: :string,

data/app/presenters/decidim/log/resource_presenter.rb

@@ -10,6 +10,8 @@ module Decidim
     # overwrite `BasePresenter#resource_presenter` to return your custom resource presenter.
     # The only requirement for custom renderers is that they should respond to `present`.
     class ResourcePresenter
+      include Decidim::SanitizeHelper
+
       # Public: Initializes the presenter.
       #
       # resource - An instance of a model that can be located by
@@ -65,7 +67,11 @@ module Decidim
       #
       # Returns an HTML-safe String.
       def present_resource_name
-        h.translated_attribute extra["title"]
+        if resource.present? && resource.respond_to?(:presenter) && resource.presenter.respond_to?(:title)
+          resource.presenter.title(html_escape: true)
+        else
+          decidim_escape_translated(extra["title"]).html_safe
+        end
       end
     end
   end

data/app/presenters/decidim/notification_to_mailer_presenter.rb

@@ -7,11 +7,16 @@ module Decidim
   class NotificationToMailerPresenter < SimpleDelegator
     include Decidim::TranslatableAttributes
 
+    EXTENDED_NOTIFICATIONS_CLASSES = [
+      "Decidim::Comments::CommentCreatedEvent"
+    ].freeze
+
     delegate :url_helpers, to: "Decidim::Core::Engine.routes"
     delegate :resource_title, to: :event
     delegate :resource_url, to: :event
     delegate :email_intro, to: :event
     delegate :resource_path, to: :event
+    delegate :safe_resource_text, to: :event
 
     def date_time
       if frequency == :daily
@@ -21,6 +26,10 @@ module Decidim
       end
     end
 
+    def show_extended_information?
+      EXTENDED_NOTIFICATIONS_CLASSES.include?(event_class)
+    end
+
     private
 
     def event

data/app/services/decidim/events_manager.rb

@@ -46,5 +46,11 @@ module Decidim
     def self.subscribe(event, &block)
       ActiveSupport::Notifications.subscribe(event, &block)
     end
+
+    def self.subscribe_events!
+      subscribe(/^decidim\.events\./) do |event_name, data|
+        EventPublisherJob.perform_later(event_name, data)
+      end
+    end
   end
 end

data/app/services/decidim/log/diff_changeset_calculator.rb

@@ -86,7 +86,7 @@ module Decidim
         locales.flat_map do |locale|
           previous_value = values.first.try(:[], locale)
           new_value = values.last.try(:[], locale)
-          if previous_value == new_value
+          if previous_value == new_value || (previous_value.nil? && new_value.blank?)
             nil
           else
             label = generate_label(attribute, locale)

data/app/services/decidim/push_notification_message_sender.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module Decidim
+  # A wrapper for preparing push notifications messages from conversations
+  # It respects the same contract as the Decidim::Messaging::ConversationMailer
+  class PushNotificationMessageSender
+    include HasConversations
+
+    def deliver
+      SendPushNotification.new.perform(@notification, title)
+    end
+
+    private
+
+    # rubocop:disable Metrics/ParameterLists
+    # rubocop:disable Lint/UnusedMethodArgument
+    #
+    # There are some parameters thar are not used in the method, but they are needed to
+    # keep the same contract as the Decidim::Messaging::ConversationMailer
+    def send_notification(from:, to:, conversation:, action:, message: nil, third_party: nil)
+      @action = action
+      @sender = to
+      @third_party = third_party
+
+      @notification = PushNotificationMessage.new(recipient: to, conversation: conversation, message: message)
+
+      self
+    end
+    # rubocop:enable Lint/UnusedMethodArgument
+    # rubocop:enable Metrics/ParameterLists
+
+    def title
+      get_subject(action: @action, sender: @sender, third_party: @third_party)
+    end
+  end
+end

data/app/services/decidim/send_push_notification.rb

@@ -14,14 +14,19 @@ module Decidim
     # Send the push notification. Returns `nil` if the user didn't allowed push notifications
     # or if the subscription to push notifications doesn't exist
     #
-    # Returns the result of the dispatch or nil if user or subscription are empty
-    def perform(notification)
+    # @param notification [Decidim::Notification, Decidim::PushNotificationMessage] the notification to be sent
+    # @param title [String] the title of the notification. Optional.
+    #
+    # @return [Array<Net::HTTPCreated>, nil] the result of the dispatch or nil if user or subscription are empty
+    def perform(notification, title = nil)
       return unless Rails.application.secrets.dig(:vapid, :enabled)
+      raise ArgumentError, "Need to provide a title if the notification is a PushNotificationMessage" if notification.is_a?(Decidim::PushNotificationMessage) && title.nil?
+
+      user = notification.user
 
-      I18n.with_locale(notification.user.locale || notification.user.organization.default_locale) do
-        notification.user.notifications_subscriptions.values.map do |subscription|
-          message_params = notification_params(Decidim::PushNotificationPresenter.new(notification))
-          payload = build_payload(message_params, subscription)
+      I18n.with_locale(user.locale || user.organization.default_locale) do
+        user.notifications_subscriptions.values.map do |subscription|
+          payload = build_payload(message_params(notification, title), subscription)
           # Capture webpush exceptions in order to avoid this call to be repeated by the background job runner
           # Webpush::Error class is the parent class of all defined errors
           begin
@@ -36,9 +41,18 @@ module Decidim
 
     private
 
-    def notification_params(notification)
+    def message_params(notification, title = nil)
+      case notification
+      when Decidim::PushNotificationMessage
+        notification_params(notification, title)
+      else # when Decidim::Notification
+        notification_params(Decidim::PushNotificationPresenter.new(notification))
+      end
+    end
+
+    def notification_params(notification, title = nil)
       {
-        title: notification.title,
+        title: title.presence || notification.title,
         body: notification.body,
         icon: notification.icon,
         data: { url: notification.url }

data/app/views/decidim/devise/registrations/new.html.erb

@@ -60,7 +60,7 @@
 
         <div class="card" id="card__tos">
           <div class="card__content">
-            <h3><%= t(".tos_title") %></h3>
+            <h2><%= t(".tos_title") %></h2>
 
             <p class="tos-text">
               <%= strip_tags(translated_attribute(terms_and_conditions_page.content)) %>
@@ -74,7 +74,7 @@
 
         <div class="card" id="card__newsletter">
           <div class="card__content">
-            <h3><%= t(".newsletter_title") %></h3>
+            <h2><%= t(".newsletter_title") %></h2>
               <div class="field">
                 <%= f.check_box :newsletter, label: t(".newsletter"), checked: @form.newsletter %>
               </div>

data/app/views/decidim/links/new.html.erb

@@ -1,3 +1,5 @@
+<% add_decidim_page_title t("decidim.links.warning.title") %>
+
 <div class="wrapper external-link-page-content">
   <h1 class="text-center"><%= t("decidim.links.warning.title") %></h1>
   <div class="row columns large-centered large-8">

data/app/views/decidim/notifications_digest_mailer/_email_content.html.erb

@@ -8,4 +8,11 @@
       <%= link_to decidim_sanitize(notification.resource_title, strip_tags: true), notification.resource_url %>
     </p>
   <% end %>
+  <% if notification.show_extended_information? %>
+    <blockquote>
+      <p>
+      <%= notification.safe_resource_text %>
+      </p>
+    </blockquote>
+  <% end %>
 </div>

data/app/views/decidim/shared/_address_details.html.erb

@@ -1,7 +1,7 @@
 <% if geolocalizable.respond_to? :location %>
-  <strong><%= translated_attribute geolocalizable.location %></strong><br>
+  <strong><%= decidim_sanitize(translated_attribute(geolocalizable.location)) %></strong><br>
 <% end %>
 <span><%= geolocalizable.address %></span><br>
 <% if geolocalizable.respond_to? :location_hints %>
-  <span><%= translated_attribute geolocalizable.location_hints %></span>
+  <span><%= decidim_sanitize(translated_attribute(geolocalizable.location_hints)) %></span>
 <% end %>

data/app/views/layouts/decidim/_js_configuration.html.erb

@@ -1,5 +1,6 @@
 <%
 js_configs = {
+  api_path: decidim_api.root_path(locale: nil),
   icons_path: Decidim.cors_enabled ? "" : asset_pack_path("media/images/icons.svg"),
   messages: {
     "selfxssWarning": {

data/config/locales/ar.yml

@@ -91,9 +91,7 @@ ar:
     'true': 'نعم'
   carrierwave:
     errors:
-      file_resolution_too_large: دقة الملف أكبر من المسموح به
       file_size_too_large: حجم الملف أكبر من المسموح به
-      not_inside_organization: لم يتم إرفاق الملف بأي منظمة
   date:
     formats:
       decidim_short: "%d/%m/%Y"
@@ -335,7 +333,6 @@ ar:
           '1': إنشاء التعديل الخاص بك
           '2': قارن تعديلك
           '3': اكمل تعديلك
-          '4': انشر تعديلك
     anonymous_user: مجهول
     application:
       collection:
@@ -612,6 +609,7 @@ ar:
         ready: جاهز
       show:
         download_data: قم بتنزيل البيانات
+        download_data_description: سيتم إرسال ملف يحتوي على جميع المعلومات المرتبطة بحسابك إلى <strong>%{user_email}</strong>. سوف يحتوي هذا البريد الإلكتروني على ملف .zip وكلمة مرور لفتحه.<br/><br/>لفك الملف ، ستحتاج إلى <a href="https://www.7-zip.org/">7-Zip</a> (للويندوز) أو <a href="https://www.keka.io/">Keka</a> (لماكوس). إذا كنت تستخدم لينوكس، في معظم الأحيان سيتم تثبيته بشكل افتراضي. إذا لم يكن الأمر كذلك، يمكنك استخدام <a href="https://gitlab.gnome.org/GNOME/file-roller">ملف Roller</a> أو <a href="https://peazip.github.io">PeaZip</a>).
         request_data: طلب البيانات
     editor_images:
       create:
@@ -782,6 +780,7 @@ ar:
           notification_title: تم إضفاء الطابع الرسمي على المشارك %{name} (%{nickname}).
     export_mailer:
       download_your_data_export:
+        click_button: 'انقر فوق الرابط التالي لتحميل بياناتك.<br/>الملف سيكون متاحا حتى %{date}.<br/>ستحتاج إلى <a href="https://www.7-zip.org/">7-Zip</a> (للويندوز)، <a href="https://www.keka.io/en/">Keka</a> (لماكوس) أو <a href="https://peazip.github.io">PeaZip</a> (للينوكس) لفتحه. كلمة المرور: %{password}'
         download: تحميل
       export:
         ready: يرجى الاطلاع على نسخة مضغوطة من التصدير.
@@ -834,6 +833,7 @@ ar:
       file_help:
         file:
           explanation: 'إرشادات الملف:'
+          message_1: يجب أن تكون صورة أو وثيقة.
           message_2: بالنسبة للصور، يُفضل استخدام صور بالوضع العرضي، تقوم الخدمة باقتصاص الصورة.
         icon:
           explanation: 'إرشادات للأيقونة:'
@@ -1268,6 +1268,7 @@ ar:
           error: كانت هناك مشكلة في قبول الشروط والأحكام.
           success: عظيم! لقد قبلت الشروط والأحكام.
         form:
+          agreement: أوافق على هذه الشروط
           legend: توافق على شروط وأحكام الاستخدام
         refuse:
           modal_body: إذا قمت بالرفض، لن تتمكن من استخدام المنصة، يمكنك <a href="%{download_your_data_path}">تنزيل بياناتك</a> و/أو <a href="%{delete_path}">حذف حسابك</a>.