decidim-core 0.27.8 → 0.28.0.rc4

data/lib/decidim/api/types/attachment_type.rb

@@ -5,9 +5,11 @@ module Decidim
     class AttachmentType < Decidim::Api::Types::BaseObject
       description "A file attachment"
 
+      field :title, Decidim::Core::TranslatedFieldType, "The title of this attachment.", null: false
+      field :description, Decidim::Core::TranslatedFieldType, "The description of this attachment.", null: false
       field :url, GraphQL::Types::String, "The url of this attachment", null: false
       field :type, GraphQL::Types::String, "The type of this attachment", method: :file_type, null: false
-      field :thumbnail, GraphQL::Types::String, "A thumbnail of this attachment, if it's an image.", method: :thumbnail_url, null: true
+      field :thumbnail, GraphQL::Types::String, "A thumbnail of this attachment, if it is an image.", method: :thumbnail_url, null: true
     end
   end
 end

data/lib/decidim/api/types/participatory_space_manifest_type.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    class ParticipatorySpaceManifestType < Decidim::Api::Types::BaseObject
+      description "A participatory manifest"
+
+      field :name, GraphQL::Types::String, "The name of the manifest", null: false
+      field :human_name, Decidim::Core::QuantifiableTranslatedFieldType, "The human readable name for the manifest", null: false
+
+      def human_name
+        {
+          single: object.human_name,
+          plural: object.human_name(count: 2)
+        }
+      end
+    end
+  end
+end

data/lib/decidim/api/types/quantifiable_translated_field_type.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    class QuantifiableTranslatedFieldType < Decidim::Api::Types::BaseObject
+      description "A quantifiable translated field with singular and plural formats"
+
+      field :single, Decidim::Core::TranslatedFieldType, "The singular format.", null: false
+      field :plural, Decidim::Core::TranslatedFieldType, "The plural format.", null: false
+    end
+  end
+end

data/lib/decidim/api/types/translated_field_type.rb

@@ -28,7 +28,7 @@ module Decidim
         translations = display_translations
         translations = translations.slice(*locales) unless locales.empty?
 
-        translations.map { |locale, text| OpenStruct.new(locale: locale, text: text, machine_translated: defined_translations[locale].blank?) }
+        translations.map { |locale, text| OpenStruct.new(locale:, text:, machine_translated: defined_translations[locale].blank?) }
       end
 
       private

data/lib/decidim/asset_router/pipeline.rb

@@ -5,7 +5,7 @@ module Decidim
     # The pipeline asset router provides global access to the asset routers for
     # assets that are precompiled through the Rails assets pipeline (or webpack)
     # and stored locally or in a remote CDN. This handles the configuration
-    # options for the asset routes so that they don't have to be always manually
+    # options for the asset routes so that they do not have to be always manually
     # created.
     class Pipeline
       # Initializes the router.

data/lib/decidim/asset_router/storage.rb

@@ -19,17 +19,13 @@ module Decidim
       #
       # @param options The options for the URL that are the normal route options
       #   Rails route helpers accept
-      # rubocop:disable Style/CaseLikeIf
       def url(**options)
         if asset.is_a? ActiveStorage::Attached
           routes.rails_blob_url(asset.blob, **default_options.merge(options))
-        elsif asset.is_a? ActiveStorage::Blob
-          routes.rails_blob_url(asset, **default_options.merge(options))
         else
-          routes.rails_representation_url(asset, **default_options.merge(options))
+          representation_url(**options)
         end
       end
-      # rubocop:enable Style/CaseLikeIf
 
       private
 
@@ -81,6 +77,25 @@ module Decidim
           host: Rails.application.secrets.dig(:storage, :cdn_host)
         }.compact
       end
+
+      # Converts the variation URLs last part to the correct file extension in
+      # case the variation has a different format than the original image.
+      #
+      # @return [String] The converted representation URL
+      def representation_url(**options)
+        representation_url = routes.rails_representation_url(asset, **default_options.merge(options))
+        variation = asset.try(:variation)
+        return representation_url unless variation
+
+        format = variation.try(:format)
+        return representation_url unless format
+
+        original_ext = File.extname(asset.blob.filename.to_s)
+        return representation_url if original_ext == ".#{format}"
+
+        basename = File.basename(asset.blob.filename.to_s, original_ext)
+        representation_url.sub(/#{basename}\.#{original_ext.tr(".", "")}$/, "#{basename}.#{format}")
+      end
     end
   end
 end

data/lib/decidim/assets/tailwind/instance.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+require "ostruct"
+require "erb"
+
+module Decidim
+  module Assets
+    module Tailwind
+      class Instance
+        def write_runtime_configuration
+          evaluated_template = ERB.new(tailwind_configuration_template).result(tailwind_variables.instance_eval { binding })
+
+          File.write(File.join(app_path, "tailwind.config.js"), evaluated_template)
+        end
+
+        private
+
+        def tailwind_variables
+          # The directories where Tailwind should search for templates are the installed
+          # decidim gems and the current directory (which is the Rails app)
+          decidim_gems = Bundler.load.specs.select { |spec| spec.name =~ /^decidim-/ }.map(&:full_gem_path)
+          content_directories = decidim_gems.push(".")
+
+          # The variable expected by tailwind is a Javascript array of strings
+          # The directory globbing with the star is done in Ruby because it was causing an infinite loop
+          # when processed by Tailwind
+          content_directories_as_array_of_strings = content_directories.map { |content_directory| "'#{content_directory}'" }.join(",")
+
+          OpenStruct.new(tailwind_content_directories: content_directories_as_array_of_strings)
+        end
+
+        def tailwind_configuration_template
+          File.read(configuration_template_path)
+        end
+
+        def configuration_template_path
+          @configuration_template_path ||= begin
+            original_path = File.expand_path("tailwind.config.js.erb", __dir__)
+            alternate_path = File.expand_path("config/tailwind.config.js.erb", app_path)
+
+            File.exist?(alternate_path) ? alternate_path : original_path
+          end
+        end
+
+        def app_path
+          @app_path ||=
+            if defined?(Rails)
+              Rails.application.root
+            else
+              # This is used when Rails is not available from the webpacker binstubs
+              File.expand_path(".", Dir.pwd)
+            end
+        end
+      end
+    end
+  end
+end

data/lib/decidim/assets/tailwind/tailwind.config.js.erb

@@ -0,0 +1,80 @@
+// This file is automatically generated.
+// Do not edit this file, it will be overwritten
+//
+// If you want to modify it, follow instructions from
+// https://docs.decidim.org/en/develop/customize/styles
+//
+
+const { inherit, current, transparent, white } = require("tailwindcss/colors")
+
+const withOpacity =
+  (variable) =>
+  ({ opacityValue }) =>
+    opacityValue === undefined
+      ? `rgb(var(${variable}))`
+      : `rgb(var(${variable}) / ${opacityValue})`;
+
+module.exports = {
+  // This content is generated automatically by decidim:webpacker:install task, it
+  // should not be updated manually.
+  // The array must contain all the decidim modules active in the application
+  content: [<%= tailwind_content_directories %>].flatMap(directory => [
+    `${directory}/app/views/**/*.html.erb`,
+    `${directory}/app/cells/**/*.{rb,erb}`,
+    `${directory}/app/helpers/**/*.rb`,
+    `${directory}/app/packs/**/*.js`,
+    `${directory}/lib/**/*.rb`
+  ]),
+  // Comment out the next line to disable purging the tailwind styles
+  // safelist: [{ pattern: /.*/ }],
+  theme: {
+    colors: {
+      inherit,
+      current,
+      transparent,
+      white,
+      primary: withOpacity("--primary-rgb"),
+      secondary: withOpacity("--secondary-rgb"),
+      tertiary: withOpacity("--tertiary-rgb"),
+      success: withOpacity("--success-rgb"),
+      alert: withOpacity("--alert-rgb"),
+      warning: withOpacity("--warning-rgb"),
+      black: "#020203",
+      gray: {
+        DEFAULT: "#6B7280CC", // 80% opacity
+        2: "#3E4C5C",
+        3: "#E1E5EF",
+        4: "#242424",
+        5: "#F6F8FA"
+      },
+      background: {
+        DEFAULT: "#F3F4F7",
+        2: "#FAFBFC",
+        3: "#EFEFEF",
+        4: "#E4EEFF99" // 60% opacity
+      }
+    },
+    container: {
+      center: true,
+      padding: {
+        DEFAULT: "1rem",
+        lg: "4rem"
+      }
+    },
+    fontFamily: {
+      sans: ["Source Sans Pro", "ui-sans-serif", "system-ui", "sans-serif"]
+    },
+    fontSize: {
+      xs: ["13px", "16px"],
+      sm: ["14px", "18px"],
+      md: ["16px", "20px"],
+      lg: ["18px", "23px"],
+      xl: ["20px", "25px"],
+      "2xl": ["24px", "30px"],
+      "3xl": ["32px", "40px"],
+      "4xl": ["36px", "45px"],
+      "5xl": ["48px", "60px"]
+    }
+  },
+  plugins: [require("@tailwindcss/typography")]
+}

data/lib/decidim/assets/tailwind.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Assets
+    module Tailwind
+      autoload :Instance, "decidim/assets/tailwind/instance"
+
+      def write_runtime_configuration
+        instance = ::Decidim::Assets::Tailwind::Instance.new
+        instance.write_runtime_configuration
+      end
+
+      module_function :write_runtime_configuration
+    end
+  end
+end

data/lib/decidim/assets.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Assets
+    autoload :Tailwind, "decidim/assets/tailwind"
+  end
+end

data/lib/decidim/attribute_encryptor.rb

@@ -19,12 +19,10 @@ module Decidim
     end
 
     def self.cryptor
-      @cryptor ||= begin
-        key = ActiveSupport::KeyGenerator.new("attribute").generate_key(
-          Rails.application.secrets.secret_key_base, ActiveSupport::MessageEncryptor.key_len
-        )
-        ActiveSupport::MessageEncryptor.new(key)
-      end
+      key = ActiveSupport::KeyGenerator.new("attribute").generate_key(
+        Rails.application.secrets.secret_key_base, ActiveSupport::MessageEncryptor.key_len
+      )
+      ActiveSupport::MessageEncryptor.new(key)
     end
   end
 end

data/lib/decidim/attribute_object/nested_validator.rb

@@ -7,7 +7,7 @@ module Decidim
         value = value.values if value.is_a?(Hash)
         return unless Array(value).reject { |r| valid_object?(r) }.any?
 
-        record.errors.add(attribute, :invalid, **options.merge(value: value))
+        record.errors.add(attribute, :invalid, **options.merge(value:))
       end
 
       private

data/lib/decidim/attribute_object/type_resolver.rb

@@ -12,8 +12,8 @@ module Decidim
         case type
         when Symbol, ActiveModel::Type::Value
           {
-            type: type,
-            options: options
+            type:,
+            options:
           }
         when Class
           resolve_class(type, **options)
@@ -56,7 +56,7 @@ module Decidim
 
         {
           type: :hash,
-          options: options.merge(key_type: key_type, value_type: val_type)
+          options: options.merge(key_type:, value_type: val_type)
         }
       end
 
@@ -77,7 +77,7 @@ module Decidim
           # https://github.com/rails/rails/tree/main/activemodel/lib/active_model/type
           {
             type: type_symbol,
-            options: options
+            options:
           }
         else
           {

data/lib/decidim/authorable.rb

@@ -53,8 +53,8 @@ module Decidim
         other_author == author || (other_author.respond_to?(:user_groups) && other_author.user_groups.include?(user_group))
       end
 
-      # Returns the normalized author, whether it's a user group or a user. Ideally this should be
-      # the *author* method, but it's pending a refactor.
+      # Returns the normalized author, whether it is a user group or a user. Ideally this should be
+      # the *author* method, but it is pending a refactor.
       #
       # Returns an Author, a UserGroup or nil.
       def normalized_author

data/lib/decidim/block_registry.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+module Decidim
+  # This class acts as a registry for any code blocks with names and should be
+  # used as a singleton through the `Decidim` core module. This allows
+  # registering code blocks for different use cases, for example for the
+  # authorization transfers.
+  class BlockRegistry
+    # Provides access to the registered blocks with their names.
+    #
+    # @return [Hash<Symbol => Proc>] A hash of the currently registered blocks
+    #   with their keys as the hash keys and the blocks as the hash values.
+    def registrations
+      @registrations ||= {}
+    end
+
+    # Register a code block with a name and the handler block.
+    #
+    # @param name [Symbol] The key for the block.
+    # @yield The block to be registered for the provided key.
+    # @return [Proc] The registered block itself.
+    def register(name, &block)
+      return unless block_given?
+
+      registrations[name] = block
+    end
+
+    # @overload unregister(name)
+    #   Unregister a single registered handler with the provided name.
+    #   @param name [Symbol] The name of the registered block.
+    #   @return [Proc] The originally registered block for the given key.
+    # @overload unregister(*names)
+    #   Unregister registered handlers with the provided names.
+    #   @param names [Array<Symbol>] The names of the registered blocks to be
+    #     unregistered.
+    #   @return [Array<Proc>] An array of the originally registered blocks.
+    def unregister(*names)
+      blocks = names.map do |name|
+        registrations.delete(name)
+      end
+      return blocks.first if names.length == 1
+
+      blocks
+    end
+  end
+end

data/lib/decidim/carrier_wave_migrator_service.rb

@@ -29,14 +29,6 @@ module Decidim
           [Decidim::Conferences::Partner, "logo", Decidim::Cw::Conferences::PartnerLogoUploader, "logo"]
         ]
       end,
-      "Decidim::Consultation" => lambda do
-        [
-          [Decidim::Consultations::Question, "hero_image", Decidim::Cw::HeroImageUploader, "hero_image"],
-          [Decidim::Consultations::Question, "banner_image", Decidim::Cw::BannerImageUploader, "banner_image"],
-          [Decidim::Consultation, "banner_image", Decidim::Cw::BannerImageUploader, "banner_image"],
-          [Decidim::Consultation, "introductory_image", Decidim::Cw::BannerImageUploader, "introductory_image"]
-        ]
-      end,
       "Decidim::Votings::Voting" => lambda do
         [
           [Decidim::Votings::Voting, "banner_image", Decidim::Cw::BannerImageUploader, "banner_image"],
@@ -84,17 +76,17 @@ module Decidim
             cw_checksum = downloaded_file_checksum(file)
             as_checksum = destination.blob.checksum
 
-            logger.info "#{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"}" \
-                        " Migrated Decidim::ContentBlock##{block.id} attachment #{image_config[:name]}" \
-                        " from CW attribute #{image_config[:name]} to AS" \
-                        " in Decidim::ContentBlockAttachment##{destination.record.id} file attribute"
+            logger.info "#{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"} " \
+                        "Migrated Decidim::ContentBlock##{block.id} attachment #{image_config[:name]} " \
+                        "from CW attribute #{image_config[:name]} to AS " \
+                        "in Decidim::ContentBlockAttachment##{destination.record.id} file attribute"
           else
-            logger.info "[SKIP] Pending migration of Decidim::ContentBlock##{block.id} attachment #{image_config[:name]}" \
-                        " from CW attribute #{image_config[:name]} to AS" \
+            logger.info "[SKIP] Pending migration of Decidim::ContentBlock##{block.id} attachment #{image_config[:name]} " \
+                        "from CW attribute #{image_config[:name]} to AS" \
           end
         rescue StandardError
-          logger.info "[ERROR] Exception checking Decidim::ContentBlock##{block.id} attachment #{image_config[:name]}" \
-                      " from CW attribute #{image_config[:name]} to AS file attribute in a Decidim::ContentBlockAttachment instance #{$ERROR_INFO}"
+          logger.info "[ERROR] Exception checking Decidim::ContentBlock##{block.id} attachment #{image_config[:name]} " \
+                      "from CW attribute #{image_config[:name]} to AS file attribute in a Decidim::ContentBlockAttachment instance #{$ERROR_INFO}"
         end
       end
     end
@@ -107,9 +99,9 @@ module Decidim
           destination = block.images_container.send(image_config[:name])
 
           if destination.attached?
-            logger.info "[SKIP] Migrated Decidim::ContentBlock##{block.id} attachment #{image_config[:name]}" \
-                        " from CW attribute #{image_config[:name]} to AS" \
-                        " in Decidim::ContentBlockAttachment##{destination.record.id} file attribute"
+            logger.info "[SKIP] Migrated Decidim::ContentBlock##{block.id} attachment #{image_config[:name]} " \
+                        "from CW attribute #{image_config[:name]} to AS " \
+                        "in Decidim::ContentBlockAttachment##{destination.record.id} file attribute"
             next
           end
 
@@ -123,26 +115,26 @@ module Decidim
           file = cw_file(attachment)
           destination.attach(
             io: file,
-            content_type: content_type,
-            filename: filename
+            content_type:,
+            filename:
           )
           destination.record.save if destination.record.new_record?
 
           cw_checksum = downloaded_file_checksum(file)
           as_checksum = destination.blob.checksum
 
-          logger.info "[OK] Migrated - #{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"}" \
-                      " Decidim::ContentBlock##{block.id} attachment #{image_config[:name]}" \
-                      " from CW attribute #{image_config[:name]} to AS" \
-                      " in Decidim::ContentBlockAttachment##{destination.record.id} file attribute" \
+          logger.info "[OK] Migrated - #{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"} " \
+                      "Decidim::ContentBlock##{block.id} attachment #{image_config[:name]} " \
+                      "from CW attribute #{image_config[:name]} to AS " \
+                      "in Decidim::ContentBlockAttachment##{destination.record.id} file attribute" \
 
           routes_mappings << { instance: "Decidim::ContentBlock##{block.id}",
                                attachment_origin_attribute: image_config[:name],
-                               origin_path: origin_path,
+                               origin_path:,
                                destination_path: Rails.application.routes.url_helpers.rails_blob_url(destination.blob, only_path: true) }
         rescue StandardError
-          logger.info "[ERROR] Exception migrating Decidim::ContentBlock##{block.id} attachment #{image_config[:name]}" \
-                      " from CW attribute #{image_config[:name]} to AS file attribute in a Decidim::ContentBlockAttachment instance #{$ERROR_INFO}"
+          logger.info "[ERROR] Exception migrating Decidim::ContentBlock##{block.id} attachment #{image_config[:name]} " \
+                      "from CW attribute #{image_config[:name]} to AS file attribute in a Decidim::ContentBlockAttachment instance #{$ERROR_INFO}"
         end
       end
     end
@@ -169,20 +161,20 @@ module Decidim
         file = cw_file(attachment)
         copy.send(as_attribute).attach(
           io: file,
-          content_type: content_type,
-          filename: filename
+          content_type:,
+          filename:
         )
 
         cw_checksum = downloaded_file_checksum(file)
         as_checksum = copy.send(as_attribute).blob.checksum
 
-        logger.info "[OK] Migrated - #{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"}" \
-                    " #{klass}##{item.id} from CW attribute #{cw_attribute} to AS #{as_attribute} attribute"
+        logger.info "[OK] Migrated - #{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"} " \
+                    "#{klass}##{item.id} from CW attribute #{cw_attribute} to AS #{as_attribute} attribute"
 
         routes_mappings << {
           instance: "#{klass}##{item.id}",
           attachment_origin_attribute: cw_attribute.to_s,
-          origin_path: origin_path,
+          origin_path:,
           destination_path: Rails.application.routes.url_helpers.rails_blob_url(copy.send(as_attribute).blob, only_path: true)
         }
       rescue StandardError
@@ -207,9 +199,9 @@ module Decidim
           cw_checksum = downloaded_file_checksum(file)
           as_checksum = copy.send(as_attribute).blob.checksum
 
-          logger.info "#{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"}" \
-                      " Migrated #{klass}##{item.id} from CW attribute #{cw_attribute}" \
-                      " to AS #{as_attribute} attribute"
+          logger.info "#{cw_checksum == as_checksum ? "[OK] Checksum identical:" : "[KO] Checksum different:"} " \
+                      "Migrated #{klass}##{item.id} from CW attribute #{cw_attribute} " \
+                      "to AS #{as_attribute} attribute"
         else
           logger.info "[SKIP] Pending migration of #{klass}##{item.id} from CW attribute #{cw_attribute} to AS #{as_attribute} attribute"
         end

data/lib/decidim/coauthorable.rb

@@ -75,7 +75,7 @@ module Decidim
 
       # Returns all the authors of a coauthorable.
       #
-      # We need to do it this ways since the authors are polymorphic, and we can't have a has_many through relation
+      # We need to do it this ways since the authors are polymorphic, and we cannot have a has_many through relation
       # with polymorphic objects.
       def authors
         return @authors if defined?(@authors)
@@ -109,7 +109,7 @@ module Decidim
       #
       # author - the author to check for authorship
       def authored_by?(author)
-        coauthorships.exists?(author: author)
+        coauthorships.exists?(author:)
       end
 
       # Returns the identities for the authors, whether they are user groups, users or others.
@@ -159,9 +159,9 @@ module Decidim
         user_group = extra_attributes[:user_group]
 
         return if coauthorships.exists?(decidim_author_id: author.id, decidim_author_type: author.class.base_class.name) && user_group.blank?
-        return if user_group && coauthorships.exists?(user_group: user_group)
+        return if user_group && coauthorships.exists?(user_group:)
 
-        coauthorship_attributes = extra_attributes.merge(author: author)
+        coauthorship_attributes = extra_attributes.merge(author:)
 
         if persisted?
           coauthorships.create!(coauthorship_attributes)

data/lib/decidim/command.rb

@@ -8,13 +8,14 @@
 module Decidim
   class Command
     include ::Wisper::Publisher
+    delegate :locale, to: :I18n
 
-    def self.call(*args, **kwargs, &block)
+    def self.call(*args, **kwargs, &)
       event_recorder = Decidim::EventRecorder.new
 
       command = new(*args, **kwargs)
       command.subscribe(event_recorder)
-      command.evaluate(&block) if block_given?
+      command.evaluate(&) if block_given?
       command.call
 
       event_recorder.events
@@ -25,8 +26,8 @@ module Decidim
       instance_eval(&block)
     end
 
-    def transaction(&block)
-      ActiveRecord::Base.transaction(&block) if block_given?
+    def transaction(&)
+      ActiveRecord::Base.transaction(&) if block_given?
     end
 
     def method_missing(method_name, ...)
@@ -40,5 +41,23 @@ module Decidim
     def respond_to_missing?(method_name, include_private = false)
       @caller.respond_to?(method_name, include_private)
     end
+
+    def with_events(with_transaction: false, &block)
+      ActiveSupport::Notifications.publish("#{event_namespace}:before", **event_arguments)
+
+      with_transaction ? transaction(&block) : yield
+
+      ActiveSupport::Notifications.publish("#{event_namespace}:after", **event_arguments)
+    end
+
+    protected
+
+    def event_arguments
+      raise "#{self.class} must implement #event_arguments"
+    end
+
+    def event_namespace
+      @event_namespace ||= self.class.name.to_s.underscore.parameterize(separator: ".")
+    end
   end
 end

data/lib/decidim/component_manifest.rb

@@ -8,7 +8,7 @@ module Decidim
   # This class handles all the logic associated to configuring a component
   # associated to a participatory process.
   #
-  # It's normally not used directly but through the API exposed through
+  # It is normally not used directly but through the API exposed through
   # `Decidim.register_component`.
   class ComponentManifest
     include ActiveModel::Model
@@ -44,6 +44,9 @@ module Decidim
     # engine's assets path.
     attribute :icon, String
 
+    # A String with the component's icon key.
+    attribute :icon_key, String
+
     # Actions are used to validate permissions of a component against particular
     # authorizations or potentially other authorization rules.
     #
@@ -51,7 +54,7 @@ module Decidim
     # on meetings.
     #
     # A Component can expose as many actions as it wants and the admin panel will
-    # generate a UI to handle them. There's a set of controller helpers available
+    # generate a UI to handle them. There is a set of controller helpers available
     # as well that allows checking for those permissions.
     attribute :actions, Array[String]
 
@@ -224,8 +227,8 @@ module Decidim
     # block - A block that receive the components to filter out the stat.
     #
     # Returns nothing.
-    def register_stat(name, options = {}, &block)
-      stats.register(name, options, &block)
+    def register_stat(name, options = {}, &)
+      stats.register(name, options, &)
     end
 
     # Public: Finds the permission class from its name, using the

data/lib/decidim/components/namer.rb

@@ -23,7 +23,7 @@ module Decidim
 
       def i18n_name
         locales.inject({}) do |names, locale|
-          names.update(locale => I18n.t("decidim.components.#{component_name}.name", locale: locale))
+          names.update(locale => I18n.t("decidim.components.#{component_name}.name", locale:))
         end
       end