decidim-core 0.28.5 → 0.29.0.rc1

data/app/controllers/concerns/decidim/filter_resource.rb

@@ -38,7 +38,7 @@ module Decidim
       end
 
       def search_collection
-        raise NotImplementedError, "A search class is neeeded to filter resources"
+        raise NotImplementedError, "A search class is needed to filter resources"
       end
 
       def filter

data/app/controllers/concerns/decidim/needs_rtl_direction.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Decidim
+  module NeedsRtlDirection
+    extend ActiveSupport::Concern
+
+    included do
+      helper_method :rtl_direction
+
+      def rtl_direction
+        return "rtl" if rtl_languages_codes.include?(I18n.locale.to_s)
+
+        "ltr"
+      end
+
+      private
+
+      def rtl_languages_codes
+        %w(ar ar-BH ar-EG ar-SA ar-YE dv he fa ks ks-PK ug ur-IN ur-PK ydd)
+      end
+    end
+  end
+end

data/app/controllers/concerns/decidim/paginable.rb

@@ -7,7 +7,7 @@ module Decidim
   module Paginable
     extend ActiveSupport::Concern
 
-    OPTIONS = [25, 50, 100].freeze
+    OPTIONS = [10, 20, 50, 100].freeze
 
     included do
       helper_method :per_page, :page_offset

data/app/controllers/concerns/decidim/participatory_space_context.rb

@@ -7,21 +7,6 @@ module Decidim
   module ParticipatorySpaceContext
     extend ActiveSupport::Concern
 
-    class_methods do
-      # Public: Called on a controller, it sets up all the surrounding methods to render a
-      # participatory space's template. It expects the method `current_participatory_space` to be
-      # defined, from which it will extract the participatory manifest.
-      #
-      # options - A hash used to modify the behavior of the layout. :only: - An array of actions on
-      #           which the layout will be applied.
-      #
-      # Returns nothing.
-      def participatory_space_layout(options = {})
-        layout :layout, **options
-        before_action :authorize_participatory_space, **options
-      end
-    end
-
     included do
       include Decidim::NeedsOrganization
 
@@ -91,7 +76,7 @@ module Decidim
       current_participatory_space_manifest.context(current_participatory_space_context).layout
     end
 
-    # Method for current user can visit the space (assembly or proces)
+    # Method for current user can visit the space (assembly or process)
     def current_user_can_visit_space?
       return true unless current_participatory_space.try(:private_space?) &&
                          !current_participatory_space.try(:is_transparent?)
@@ -100,13 +85,6 @@ module Decidim
       current_user.admin || current_participatory_space.users.include?(current_user)
     end
 
-    def check_current_user_can_visit_space
-      return if current_user_can_visit_space?
-
-      flash[:alert] = I18n.t("participatory_space_private_users.not_allowed", scope: "decidim")
-      redirect_to action: "index"
-    end
-
     def help_section
       @help_section ||= Decidim::ContextualHelpSection.find_content(
         current_organization,

data/app/controllers/concerns/decidim/payload_info.rb

@@ -5,13 +5,14 @@ require "active_support/concern"
 module Decidim
   module PayloadInfo
     extend ActiveSupport::Concern
+    include Decidim::TranslatableAttributes
 
     included do
       def append_info_to_payload(payload)
         super
         payload[:user_id] = try(:current_user).try(:id)
         payload[:organization_id] = try(:current_organization).try(:id)
-        payload[:app] = try(:current_organization).try(:name)
+        payload[:app] = translated_attribute(try(:current_organization).try(:name))
         payload[:remote_ip] = request.remote_ip
         payload[:referer] = request.referer.to_s
         payload[:request_id] = request.uuid

data/app/controllers/concerns/decidim/registers_permissions.rb

@@ -10,8 +10,8 @@ module Decidim
   module RegistersPermissions
     extend ActiveSupport::Concern
 
-    def self.register_permissions(artifact, *permission_classes)
-      ::Decidim.permissions_registry.register_permissions(artifact, *permission_classes)
+    def self.register_permissions(artifact, *)
+      ::Decidim.permissions_registry.register_permissions(artifact, *)
     end
 
     class_methods do
@@ -19,8 +19,8 @@ module Decidim
       #
       # +artifact+ is expected to be the class or module that declares `NeedsPermission.permission_class_chain`.
       # +permission_classes+ are subclasses of `DefaultPermissions` or at least should quack as one.
-      def register_permissions(artifact, *permission_classes)
-        RegistersPermissions.register_permissions(artifact, *permission_classes)
+      def register_permissions(artifact, *)
+        RegistersPermissions.register_permissions(artifact, *)
       end
     end
   end

data/app/controllers/concerns/decidim/use_organization_time_zone.rb

@@ -12,7 +12,7 @@ module Decidim
       around_action :use_organization_time_zone
       helper_method :organization_time_zone
 
-      # Executes a block of code in the context of the the organization's time zone
+      # Executes a block of code in the context of the organization's time zone
       #
       # &action - a block of code to be wrapped around the time zone
       #

data/app/controllers/concerns/decidim/withdrawable.rb

@@ -3,7 +3,7 @@
 require "active_support/concern"
 
 module Decidim
-  # A controller concern to enable withawing the controller resources. Only
+  # A controller concern to enable withdrawing the controller resources. Only
   # affects the UI, so the actual logic to withdraw the resource will still need
   # to be implemented.
   module Withdrawable

data/app/controllers/decidim/account_controller.rb

@@ -16,7 +16,7 @@ module Decidim
     def update
       enforce_permission_to(:update, :user, current_user:)
       @account = form(AccountForm).from_params(account_params)
-      UpdateAccount.call(current_user, @account) do
+      UpdateAccount.call(@account) do
         on(:ok) do |email_is_unconfirmed|
           flash[:notice] = if email_is_unconfirmed
                              t("account.update.success_with_email_confirmation", scope: "decidim")
@@ -45,7 +45,7 @@ module Decidim
       enforce_permission_to(:delete, :user, current_user:)
       @form = form(DeleteAccountForm).from_params(params)
 
-      DestroyAccount.call(current_user, @form) do
+      DestroyAccount.call(@form) do
         on(:ok) do
           sign_out(current_user)
           flash[:notice] = t("account.destroy.success", scope: "decidim")

data/app/controllers/{concerns/decidim → decidim}/amendments_controller.rb

@@ -9,8 +9,8 @@ module Decidim
     helper UserGroupHelper
 
     before_action :authenticate_user!
-    helper_method :amendment, :amendable, :emendation, :similar_emendations
-    before_action :ensure_is_draft_from_user, only: [:compare_draft, :edit_draft, :update_draft, :destroy_draft, :preview_draft, :publish_draft]
+    helper_method :amendment, :amendable, :emendation
+    before_action :ensure_is_draft_from_user, only: [:edit_draft, :update_draft, :destroy_draft, :preview_draft, :publish_draft]
 
     def new
       raise ActionController::RoutingError, "Not Found" unless amendable
@@ -34,7 +34,7 @@ module Decidim
       Decidim::Amendable::CreateDraft.call(@form) do
         on(:ok) do |amendment|
           flash[:notice] = t("created.success", scope: "decidim.amendments")
-          redirect_to compare_draft_amend_path(amendment)
+          redirect_to preview_draft_amend_path(amendment)
         end
 
         on(:invalid) do
@@ -44,15 +44,6 @@ module Decidim
       end
     end
 
-    def compare_draft
-      enforce_permission_to :create, :amendment, current_component: amendable.component
-
-      if similar_emendations.empty?
-        flash[:notice] = t("no_similars_found", scope: "decidim.amendments.compare_draft")
-        redirect_to edit_draft_amend_path(amendment)
-      end
-    end
-
     def edit_draft
       enforce_permission_to :create, :amendment, current_component: amendable.component
 
@@ -219,10 +210,6 @@ module Decidim
       raise ActionController::RoutingError, "Not Found" unless amendment.draft? && amender == current_user
     end
 
-    def similar_emendations
-      @similar_emendations ||= Decidim::SimilarEmendations.for(amendment)
-    end
-
     def breadcrumb_item
       {
         label: t("decidim.amendments.name"),

data/app/controllers/decidim/application_controller.rb

@@ -6,6 +6,7 @@ module Decidim
     include Browser::ActionController
 
     include NeedsOrganization
+    include NeedsRtlDirection
     include LocaleSwitcher
     include UseOrganizationTimeZone
     include NeedsPermission
@@ -23,7 +24,6 @@ module Decidim
     include DisableRedirectionToExternalHost
     include NeedsPasswordChange
     include LinkedResourceReference
-    include ActiveStorage::SetCurrent
 
     helper Decidim::MetaTagsHelper
     helper Decidim::DecidimFormHelper

data/app/controllers/decidim/components/base_controller.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Components
     # Controller from which all component engines inherit from. It is in charge of
-    # setting the appropiate layout, including necessary helpers, and overall
+    # setting the appropriate layout, including necessary helpers, and overall
     # fooling the engine into thinking it is isolated.
     class BaseController < Decidim::ApplicationController
       include Settings

data/app/controllers/decidim/devise/invitations_controller.rb

@@ -2,7 +2,7 @@
 
 module Decidim
   module Devise
-    # This controller customizes the behaviour of Devise::Invitiable.
+    # This controller customizes the behaviour of Devise::Invitable.
     class InvitationsController < ::Devise::InvitationsController
       include Decidim::DeviseControllers
       include NeedsTosAccepted

data/app/controllers/decidim/devise/passwords_controller.rb

@@ -25,8 +25,8 @@ module Decidim
         self.resource = current_user
         @send_path = apply_password_path
 
-        @form = Decidim::PasswordForm.from_params(params["user"])
-        Decidim::UpdatePassword.call(current_user, @form) do
+        @form = Decidim::PasswordForm.from_params(params["user"]).with_context(current_user:)
+        Decidim::UpdatePassword.call(@form) do
           on(:ok) do
             flash[:notice] = t("passwords.update.success", scope: "decidim")
             bypass_sign_in(current_user)

data/app/controllers/decidim/doorkeeper/credentials_controller.rb

@@ -28,7 +28,7 @@ module Decidim
       end
 
       def avatar_url
-        avatar_url = current_resource_owner.attached_uploader(:avatar).url
+        avatar_url = current_resource_owner.attached_uploader(:avatar).url(host: current_resource_owner.organization.host)
         return unless avatar_url
 
         unless %r{^https?://}.match? avatar_url

data/app/controllers/decidim/endorsements_controller.rb

@@ -48,7 +48,7 @@ module Decidim
       render :identities, layout: false
     end
 
-    # should be pubic in order to be visibe in NeedsPermission#permissions_context
+    # should be pubic in order to be visible in NeedsPermission#permissions_context
     def current_component
       resource.component
     end

data/app/controllers/decidim/favicon_controller.rb

@@ -4,7 +4,7 @@ module Decidim
   # We serve a custom /favicon.ico route for Decidim in order to show the
   # correct favicons also when browsing e.g. PDF documents originating from the
   # same domain displayed inline in the browser. For those special files
-  # displayed in the browser, the favicon could be otherwise incorrectly sohwn
+  # displayed in the browser, the favicon could be otherwise incorrectly shown
   # or cached.
   #
   # This custom route ensures that the favicons are always correctly served for

data/app/controllers/decidim/follows_controller.rb

@@ -10,7 +10,7 @@ module Decidim
       @form = form(Decidim::FollowForm).from_params(params)
       enforce_permission_to :delete, :follow, follow: @form.follow
 
-      DeleteFollow.call(@form, current_user) do
+      DeleteFollow.call(@form) do
         on(:ok) do
           render :update_button
         end
@@ -25,7 +25,7 @@ module Decidim
       @form = form(Decidim::FollowForm).from_params(params)
       enforce_permission_to :create, :follow
 
-      CreateFollow.call(@form, current_user) do
+      CreateFollow.call(@form) do
         on(:ok) do
           render :update_button
         end

data/app/controllers/decidim/group_invites_controller.rb

@@ -34,16 +34,29 @@ module Decidim
 
     def update
       enforce_permission_to :accept, :user_group_invitations
-
       AcceptGroupInvitation.call(inviting_user_group, current_user) do
         on(:ok) do
-          flash[:notice] = t("group_invites.accept.success", scope: "decidim")
-          redirect_to profile_groups_path(current_user.nickname)
+          respond_to do |format|
+            format.json do
+              render json: { message: t("group_invites.accept.success", scope: "decidim") }
+            end
+            format.all do
+              flash[:notice] = t("group_invites.accept.success", scope: "decidim")
+              redirect_to profile_groups_path(current_user.nickname)
+            end
+          end
         end
 
         on(:invalid) do
-          flash[:alert] = t("group_invites.accept.error", scope: "decidim")
-          redirect_to profile_groups_path(current_user.nickname)
+          respond_to do |format|
+            format.json do
+              render json: { message: t("group_invites.accept.error", scope: "decidim") }, status: :unprocessable_entity
+            end
+            format.all do
+              flash[:alert] = t("group_invites.accept.error", scope: "decidim")
+              redirect_to profile_groups_path(current_user.nickname)
+            end
+          end
         end
       end
     end
@@ -53,13 +66,27 @@ module Decidim
 
       RejectGroupInvitation.call(inviting_user_group, current_user) do
         on(:ok) do
-          flash[:notice] = t("group_invites.reject.success", scope: "decidim")
-          redirect_to profile_groups_path(current_user.nickname)
+          respond_to do |format|
+            format.json do
+              render json: { message: t("group_invites.reject.success", scope: "decidim") }
+            end
+            format.all do
+              flash[:notice] = t("group_invites.reject.success", scope: "decidim")
+              redirect_to profile_groups_path(current_user.nickname)
+            end
+          end
         end
 
         on(:invalid) do
-          flash[:alert] = t("group_invites.reject.error", scope: "decidim")
-          redirect_to profile_groups_path(current_user.nickname)
+          respond_to do |format|
+            format.json do
+              render json: { message: t("group_invites.reject.error", scope: "decidim") }, status: :unprocessable_entity
+            end
+            format.all do
+              flash[:alert] = t("group_invites.reject.error", scope: "decidim")
+              redirect_to profile_groups_path(current_user.nickname)
+            end
+          end
         end
       end
     end

data/app/controllers/decidim/links_controller.rb

@@ -41,7 +41,7 @@ module Decidim
     end
 
     def escape_url(external_url)
-      before_fragment, fragment = URI.decode_www_form_component(external_url).split("#", 2)
+      before_fragment, fragment = external_url.split("#", 2)
       escaped_before_fragment = URI::Parser.new.escape(before_fragment)
 
       if fragment

data/app/controllers/decidim/newsletters_opt_in_controller.rb

@@ -23,7 +23,7 @@ module Decidim
 
     def check_current_user_with_token
       unless current_user.newsletter_token == params[:token]
-        flash[:alert] = t("newsletters_opt_in.unathorized", scope: "decidim")
+        flash[:alert] = t("newsletters_opt_in.unauthorized", scope: "decidim")
         redirect_to decidim.root_path(host: current_organization)
       end
     end

data/app/controllers/decidim/notifications_settings_controller.rb

@@ -14,7 +14,7 @@ module Decidim
       enforce_permission_to(:update, :user, current_user:)
       @notifications_settings = form(NotificationsSettingsForm).from_params(params)
 
-      UpdateNotificationsSettings.call(current_user, @notifications_settings) do
+      UpdateNotificationsSettings.call(@notifications_settings) do
         on(:ok) do
           flash.now[:notice] = t("notifications_settings.update.success", scope: "decidim")
         end

data/app/controllers/decidim/profiles_controller.rb

@@ -27,10 +27,6 @@ module Decidim
       redirect_to profile_activity_path(nickname: params[:nickname])
     end
 
-    def tooltip
-      render json: { data: cell("decidim/author", profile_holder.presenter).profile_minicard }
-    end
-
     def following
       @content_cell = "decidim/following"
       @title_key = "following"

data/app/controllers/decidim/report_users_controller.rb

@@ -12,7 +12,7 @@ module Decidim
 
       @form = form(Decidim::ReportForm).from_params(params)
 
-      CreateUserReport.call(@form, reportable, current_user) do
+      CreateUserReport.call(@form, reportable) do
         on(:ok) do
           flash[:notice] = I18n.t("decidim.reports.create.success")
           if @form.block?

data/app/controllers/decidim/reports_controller.rb

@@ -13,7 +13,7 @@ module Decidim
 
       @form = form(Decidim::ReportForm).from_params(params, can_hide: reportable.try(:can_be_administered_by?, current_user))
 
-      CreateReport.call(@form, reportable, current_user) do
+      CreateReport.call(@form, reportable) do
         on(:ok) do
           flash[:notice] = I18n.t("decidim.reports.create.success")
           redirect_back fallback_location: root_path

data/app/controllers/decidim/user_interests_controller.rb

@@ -14,7 +14,7 @@ module Decidim
       enforce_permission_to(:update, :user, current_user:)
       @user_interests = form(UserInterestsForm).from_params(params)
 
-      UpdateUserInterests.call(current_user, @user_interests) do
+      UpdateUserInterests.call(@user_interests) do
         on(:ok) do
           flash.keep[:notice] = t("user_interests.update.success", scope: "decidim")
         end

data/app/events/decidim/change_nickname_event.rb

@@ -2,7 +2,6 @@
 
 module Decidim
   class ChangeNicknameEvent < Decidim::Events::SimpleEvent
-    include Decidim::Events::NotificationEvent
     delegate :organization, to: :user, prefix: false
     delegate :url_helpers, to: "Decidim::Core::Engine.routes"
 

data/app/events/decidim/invited_to_group_event.rb

@@ -29,5 +29,34 @@ module Decidim
     def user_group_name
       extra["user_group_name"]
     end
+
+    def membership_id
+      extra["membership_id"]
+    end
+
+    def invitation
+      @invitation ||= UserGroupMembership.find_by(user:, id: membership_id, role: "invited")
+    end
+
+    def action_cell
+      "decidim/notification_actions/buttons" if invitation
+    end
+
+    def action_data
+      [
+        {
+          url: url_helpers.group_invite_path(user_group_nickname, membership_id, format: :json),
+          icon: "check-line",
+          method: "patch",
+          i18n_label: "decidim.group_invites.accept_invitation"
+        },
+        {
+          url: url_helpers.group_invite_path(user_group_nickname, membership_id, format: :json),
+          icon: "close-circle-line",
+          method: "delete",
+          i18n_label: "decidim.group_invites.reject_invitation"
+        }
+      ]
+    end
   end
 end

data/app/events/decidim/welcome_notification_event.rb

@@ -1,9 +1,7 @@
 # frozen_string_literal: true
 
 module Decidim
-  class WelcomeNotificationEvent < Decidim::Events::BaseEvent
-    include Decidim::Events::EmailEvent
-    include Decidim::Events::NotificationEvent
+  class WelcomeNotificationEvent < Decidim::Events::SimpleEvent
     include TranslationsHelper
 
     delegate :organization, to: :user, prefix: false
@@ -46,7 +44,7 @@ module Decidim
     def interpolate(template)
       template
         .gsub("{{name}}", user.name)
-        .gsub("{{organization}}", organization.name)
+        .gsub("{{organization}}", organization_name(organization))
         .gsub("{{help_url}}", url_helpers.pages_url(host: organization.host))
         .gsub("{{badges_url}}", url_helpers.gamification_badges_url(host: organization.host))
         .html_safe

data/app/forms/decidim/form.rb

@@ -11,5 +11,9 @@ module Decidim
              to: :context, prefix: false, allow_nil: true
 
     delegate :available_locales, to: :current_organization, allow_nil: true
+
+    alias organization current_organization
+    alias component current_component
+    alias participatory_space current_participatory_space
   end
 end

data/app/forms/decidim/omniauth_registration_form.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Decidim
-  # A form object used to fisnish signup from omniauth data
+  # A form object used to finish signup from omniauth data
   class OmniauthRegistrationForm < Form
     mimic :user
 

data/app/forms/decidim/upload_validation_form.rb

@@ -10,7 +10,7 @@ module Decidim
     # Property is named as attribute in upload modal and passthru validator, but
     # it cannot be named as attribute here.
     attribute :property, String
-    attribute :blob, Decidim::Attributes::Blob
+    attribute :blob, String
     attribute :form_class, String
 
     validates :resource_class, presence: true
@@ -73,7 +73,7 @@ module Decidim
     # This class provides ability to interpret the attachment context based on
     # the details available within the context of this class. Normally the
     # attachment context would be defined by the record to which the attachment
-    # are added to, e.g. proposals (participant contenxt) or participatory
+    # are added to, e.g. proposals (participant context) or participatory
     # processes (admin context). Unfortunately this information is not available
     # when the parameters are passed to the upload validation.
     class AttachmentContextProxy

data/app/helpers/concerns/decidim/flash_helper_extensions.rb

@@ -43,7 +43,7 @@ module Decidim
       #
       # @param value [String] - The flash message.
       # @param alert_class [String] - The foundation class of the alert message.
-      # @param closable [Boolean] - Wether the close icon is added.
+      # @param closable [Boolean] - Whether the close icon is added.
       #
       # @return [String] the HTML with the alert box
       def alert_box(value, alert_class, closable, opts = {})

data/app/helpers/decidim/amendments_helper.rb

@@ -5,7 +5,7 @@ module Decidim
   module AmendmentsHelper
     include RichTextEditorHelper
 
-    TOTAL_STEPS = 4
+    TOTAL_STEPS = 2
 
     # Renders the state of an emendation
     #
@@ -130,14 +130,10 @@ module Decidim
 
     def current_step
       @current_step ||= case params[:action].to_sym
-                        when :new, :create
+                        when :new, :create, :edit_draft, :update_draft, :destroy_draft
                           1
-                        when :compare_draft
-                          2
-                        when :edit_draft, :update_draft, :destroy_draft
-                          3
                         when :preview_draft, :publish_draft
-                          4
+                          2
                         end
     end
 
@@ -147,10 +143,6 @@ module Decidim
             when 1
               :new
             when 2
-              :compare_draft
-            when 3
-              :edit_draft
-            when 4
               :preview_draft
             end
 
@@ -162,9 +154,7 @@ module Decidim
       case current_step
       when 1
         Decidim::ResourceLocatorPresenter.new(amendable).path
-      when 3
-        Decidim::Core::Engine.routes.url_helpers.compare_draft_amend_path(amendable.amendment)
-      when 4
+      when 2
         Decidim::Core::Engine.routes.url_helpers.edit_draft_amend_path(amendable.amendment)
       end
     end

data/app/helpers/decidim/application_helper.rb

@@ -105,5 +105,13 @@ module Decidim
     def prevent_timeout_seconds
       0
     end
+
+    def text_initials(name)
+      name.split(/[\s.]+/).map(&:chr).slice(0, 2).join.upcase
+    end
+
+    def add_body_classes(*class_names)
+      content_for :body_class, class_names.map { |class_name| " #{class_name.strip}" }.join
+    end
   end
 end

data/app/helpers/decidim/authorization_form_helper.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 module Decidim
-  # A heper to expose an easy way to add authorization forms in a view.
+  # A helper to expose an easy way to add authorization forms in a view.
   module AuthorizationFormHelper
     # Creates a ew authorization form in a view, accepts the same arguments as
     # `form_for`.
     #
-    # record  - The record to use in the form, it shoulde be a descendant of
+    # record  - The record to use in the form, it should be a descendant of
     # AuthorizationHandler.
     # options - An optional hash with options to pass wo the form builder.
     # block   - A block with the content of the form.