decidim-core 0.31.2 → 0.32.1

data/lib/decidim/api/interfaces/referable_interface.rb

@@ -5,6 +5,7 @@ module Decidim
     # This interface represents an object that have a reference field.
     module ReferableInterface
       include Decidim::Api::Types::BaseInterface
+
       description "An interface that can be used in display reference methods"
 
       field :reference, GraphQL::Types::String, "The reference for this record", null: true

data/lib/decidim/api/interfaces/scopable_interface.rb

@@ -5,6 +5,7 @@ module Decidim
     # This interface represents a scopable object.
     module ScopableInterface
       include Decidim::Api::Types::BaseInterface
+
       description "An interface that can be used in scopable objects."
 
       field :scope, Decidim::Core::ScopeApiType, "The object's scope", null: true

data/lib/decidim/api/interfaces/taxonomizable_interface.rb

@@ -5,6 +5,7 @@ module Decidim
     # This interface represents a categorizable object.
     module TaxonomizableInterface
       include Decidim::Api::Types::BaseInterface
+
       description "An interface that can be used in taxonomizable objects."
 
       field :taxonomies, [Decidim::Core::TaxonomyType], "The object's taxonomies", null: true

data/lib/decidim/api/interfaces/timestamps_interface.rb

@@ -5,6 +5,7 @@ module Decidim
     # This interface represents an object with standard create_at and updated_at timestamps.
     module TimestampsInterface
       include Decidim::Api::Types::BaseInterface
+
       description "An interface that can be used in objects with created_at and updated_at attributes"
 
       field :created_at, Decidim::Core::DateTimeType, description: "The date and time this object was created", null: true

data/lib/decidim/api/interfaces/traceable_interface.rb

@@ -5,6 +5,7 @@ module Decidim
     # This interface represents an traceable object.
     module TraceableInterface
       include Decidim::Api::Types::BaseInterface
+
       description "An interface that can be used in objects with traceability (versions)"
 
       field :versions, [Decidim::Core::TraceVersionType, { null: true }], "This object's versions", null: false

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

@@ -2,13 +2,15 @@
 
 module Decidim
   module Core
+    # This type represents an attachment collection
     class AttachmentCollectionType < Decidim::Api::Types::BaseObject
       description "A file attachment collection"
 
-      field :description, Decidim::Core::TranslatedFieldType, "The description of this AttachmentCollection.", null: false
-      field :name, Decidim::Core::TranslatedFieldType, "The name of this AttachmentCollection.", null: false
-
       field :attachments, [Decidim::Core::AttachmentType, { null: true }], "This object's attachments", null: false
+      field :description, Decidim::Core::TranslatedFieldType, "The description of this Attachment Collection", null: false
+      field :id, GraphQL::Types::ID, "Internal ID of this Attachment Collection", null: false
+      field :name, Decidim::Core::TranslatedFieldType, "The name of this Attachment Collection", null: false
+      field :weight, GraphQL::Types::Int, "The weight of this Attachment Collection", null: true
     end
   end
 end

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

@@ -2,14 +2,21 @@
 
 module Decidim
   module Core
+    # This type represents an attachment
     class AttachmentType < Decidim::Api::Types::BaseObject
       description "A file attachment"
+      implements Decidim::Core::TimestampsInterface
 
-      field :description, Decidim::Core::TranslatedFieldType, "The description of this attachment.", null: false
-      field :thumbnail, GraphQL::Types::String, "A thumbnail of this attachment, if it is an image.", method: :thumbnail_url, null: true
-      field :title, Decidim::Core::TranslatedFieldType, "The title of this attachment.", null: false
-      field :type, GraphQL::Types::String, "The type of this attachment", method: :file_type, null: false
+      field :content_type, GraphQL::Types::String, "The content type of this attachment (could be 'text/uri-list', 'image/jpeg', or any allowed content types)", null: true
+      field :description, Decidim::Core::TranslatedFieldType, "The description of this attachment", null: false
+      field :file_size, GraphQL::Types::String, "The file size of this attachment", null: true
+      field :id, GraphQL::Types::ID, "Internal ID of this attachment", null: false
+      field :link, GraphQL::Types::String, "The attached link of this attachment", null: true
+      field :thumbnail, GraphQL::Types::String, "A thumbnail of this attachment, if it is an image", method: :thumbnail_url, null: true
+      field :title, Decidim::Core::TranslatedFieldType, "The title of this attachment", null: false
+      field :type, GraphQL::Types::String, "The type of this attachment (could be 'link', 'jpeg', 'pdf' or any other allowed extensions)", method: :file_type, null: false
       field :url, GraphQL::Types::String, "The url of this attachment", null: false
+      field :weight, GraphQL::Types::Int, "The weight of this attachment", null: true
     end
   end
 end

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

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    class BadgeScoreType < Decidim::Api::Types::BaseObject
+      description "An user badge score"
+
+      field :description, GraphQL::Types::String, "The description of this badge", null: false
+      field :image, GraphQL::Types::String, "The image of this badge", null: false
+      field :level, GraphQL::Types::Int, "The level of this badge", null: false
+      field :name, GraphQL::Types::String, "The name of this badge", null: false, method: :badge_name
+      field :score, GraphQL::Types::Int, "The score of this badge", null: false
+
+      delegate :level, :score, to: :status
+      delegate :image, :description, to: :manifest
+
+      private
+
+      def manifest
+        @manifest ||= status.badge
+      end
+
+      def status
+        @status ||= Decidim::Gamification.status_for(object.user, object.badge_name)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    # This type represents a content moderation record
+    class ModerationType < Decidim::Api::Types::BaseObject
+      description "A moderation detail"
+
+      implements Decidim::Core::TimestampsInterface
+
+      field :hidden_at, Decidim::Core::DateTimeType, "The date and time when the resource was hidden", null: true
+      field :id, GraphQL::Types::ID, "The ID of the moderation", null: false
+      field :report_count, GraphQL::Types::Int, "The number of reports of this resource", null: true
+      field :reported_content, GraphQL::Types::String, "The content that has been reported", null: true
+      field :reported_url, GraphQL::Types::String, "The URL of the resource that was reported", null: true
+      field :reports, [Decidim::Core::ReportableType, { null: true }], "The reports of this resource", null: true
+
+      def reported_url
+        return "" unless object.reportable.respond_to?(:reported_content_url)
+
+        object.reportable.reported_content_url
+      end
+    end
+  end
+end

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    class ParticipantDetailsType < Decidim::Api::Types::BaseObject
+      description "details of a participant"
+
+      field :email, GraphQL::Types::String, "The user's email", null: false
+      field :name, GraphQL::Types::String, "The user's name", null: false
+      field :nickname, GraphQL::Types::String, "The user's nickname", null: false
+
+      def self.authorized?(object, context)
+        super && allowed_to?(:read, :admin_dashboard, object, context, scope: :admin)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    # This type represents a content report
+    class ReportableType < Decidim::Api::Types::BaseObject
+      description "A report object"
+
+      implements Decidim::Core::TimestampsInterface
+
+      field :details, GraphQL::Types::String, "The details of this report", null: false
+      field :id, GraphQL::Types::ID, "The ID of this reportable", null: false
+      field :locale, GraphQL::Types::String, "The locale of the reportable", null: false
+      field :reason, GraphQL::Types::String, "The reason of this report", null: false
+      field :user, UserType, "The author of this report", null: true
+    end
+  end
+end

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

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    # This type represents an user report
+    class ReportableUserType < Decidim::Api::Types::BaseObject
+      description "An user report"
+
+      implements Decidim::Core::TimestampsInterface
+
+      field :details, GraphQL::Types::String, "The details of this report", null: false
+      field :id, GraphQL::Types::ID, "The ID of this report", null: false
+      field :reason, GraphQL::Types::String, "The reason of this report", null: false
+      field :user, UserType, "The author of this report", null: true
+    end
+  end
+end

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

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    # This type represents a topic's page
+    class StaticPageTopicType < Decidim::Api::Types::BaseObject
+      description "The current organization static page topics"
+
+      field :description, Decidim::Core::TranslatedFieldType, "The description of this static page topic", null: false
+      field :id, GraphQL::Types::ID, "Internal ID of this static page topic", null: false
+      field :show_in_footer, GraphQL::Types::Boolean, "Whether this static page topic should be shown in the footer", null: false
+      field :static_pages, [Decidim::Core::StaticPageType, { null: true }], "The static pages associated to this static page topic", null: true, method: :pages
+      field :title, Decidim::Core::TranslatedFieldType, "The title of this static page topic", null: false
+    end
+  end
+end

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

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    # This type represents a page
+    class StaticPageType < Decidim::Api::Types::BaseObject
+      description "The current organization static pages"
+
+      implements Decidim::Core::TimestampsInterface
+
+      field :content, Decidim::Core::TranslatedFieldType, "The content of this page", null: true
+      field :id, GraphQL::Types::ID, "The id of this page", null: false
+      field :title, Decidim::Core::TranslatedFieldType, "The title of this page", null: false
+      field :topic, Decidim::Core::StaticPageTopicType, "The topic of this page", null: true
+      field :url, GraphQL::Types::String, "The URL of this page", null: false
+
+      def url
+        Decidim::EngineRouter.new("decidim", { host: object.organization.host }).page_url(object)
+      end
+    end
+  end
+end

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

@@ -6,10 +6,12 @@ module Decidim
       description "A taxonomy that can be applied to other resources."
 
       field :children, [Decidim::Core::TaxonomyType], "The children of this taxonomy.", null: false
+      field :children_count, GraphQL::Types::Int, "The number of children of this taxonomy."
       field :id, GraphQL::Types::ID, "The id of this taxonomy.", null: false
       field :is_root, Boolean, "Whether this taxonomy is a root taxonomy (root taxonomies have no parents).", null: false, method: :root?
       field :name, Decidim::Core::TranslatedFieldType, "The name of this taxonomy.", null: false
       field :parent, Decidim::Core::TaxonomyType, "The parent of this taxonomy.", null: true
+      field :taxonomizations_count, GraphQL::Types::Int, "The number of taxonomizations of this taxonomy."
     end
   end
 end

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

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Core
+    # This type represents a user moderation
+    class UserModerationType < Decidim::Api::Types::BaseObject
+      description "A moderated user detail"
+
+      implements Decidim::Core::TimestampsInterface
+
+      field :about, GraphQL::Types::String, "The about data of this user", null: true
+      field :block_reasons, GraphQL::Types::String, "The reasons why the user was blocked", null: true
+      field :blocked_at, Decidim::Core::DateTimeType, "The date at which the user was blocked", null: true
+      field :blocking_user, UserType, "The user who blocked the user", null: true
+      field :id, GraphQL::Types::ID, "The ID of the moderation", null: false
+      field :reports, [Decidim::Core::ReportableUserType, { null: true }], "The reports for this user", null: true
+      field :user_id, GraphQL::Types::ID, "The ID of this user'", null: false, method: :decidim_user_id
+
+      def about
+        object.user.presenter.about
+      end
+
+      def block_reasons
+        object.blocking.justification
+      end
+
+      def blocked_at
+        object.user.blocked_at
+      end
+
+      def blocking_user
+        object.blocking.blocking_user
+      end
+    end
+  end
+end

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

@@ -7,18 +7,25 @@ module Decidim
       description "A user"
 
       implements Decidim::Core::AuthorInterface
+      implements Decidim::Core::FollowableInterface
+      implements Decidim::Core::TimestampsInterface
 
+      field :about, GraphQL::Types::String, "The user's about data", null: true
       field :avatar_url, GraphQL::Types::String, "The user's avatar url", null: false
       field :badge, GraphQL::Types::String, "A badge for the user", null: false
+      field :badges, [Decidim::Core::BadgeScoreType, { null: true }], "The user's badges", null: true
       field :deleted, GraphQL::Types::Boolean, "Whether the user's account has been deleted or not", null: false
       field :direct_messages_enabled, GraphQL::Types::String,
             null: false,
             description: ["If the user making the request is logged in, it will return whether this recipient accepts a conversation or not.",
                           " It will return false for non-logged requests."].join
+      field :followers_count, GraphQL::Types::Int, "The number of users following this user", null: true
+      field :following_count, GraphQL::Types::Int, "The number of users this user is following", null: true
       field :id, GraphQL::Types::ID, "The user's id", null: false
       field :name, GraphQL::Types::String, "The user's name", null: false
       field :nickname, GraphQL::Types::String, "The user's nickname", null: false
       field :organization_name, Decidim::Core::TranslatedFieldType, "The user's organization name", null: false
+      field :personal_url, GraphQL::Types::String, "The user's personal url", null: true
       field :profile_path, GraphQL::Types::String, "The user's profile url", null: false
 
       def nickname
@@ -49,6 +56,12 @@ module Decidim
         object.presenter.badge
       end
 
+      def badges
+        return [] unless object.organization.badges_enabled?
+
+        Decidim::Gamification::BadgeScore.where(user_id: object.id)
+      end
+
       def self.authorized?(object, context)
         super && object.confirmed? && !object.blocked? && !object.deleted?
       end

data/lib/decidim/asset_router/pipeline.rb

@@ -22,7 +22,7 @@ module Decidim
 
       # Generates the correct URL to the asset with the provided options.
       #
-      # @param options [Hash] The options for the URL that are the normal route
+      # Accepts the options for the URL that are the normal route
       #   options Rails route helpers accept
       # @return [String] The full URL to the asset or when host cannot be
       #   resolved, the asset path.

data/lib/decidim/asset_router/storage.rb

@@ -26,7 +26,7 @@ module Decidim
     class Storage
       # Initializes the router.
       #
-      # @param [ActiveStorage::Attached, ActiveStorage::Blob] The asset to route
+      # @param asset [ActiveStorage::Attached, ActiveStorage::Blob] The asset to route
       #   to
       def initialize(asset)
         @asset = asset
@@ -41,7 +41,7 @@ module Decidim
 
       # Generates the correct URL to the asset with the provided options.
       #
-      # @param options The options for the URL that are the normal route options
+      # Accepts any options for the URL that are the normal route options
       #   Rails route helpers accept
       # @return [String] The URL of the asset
       def url(**options)
@@ -115,8 +115,8 @@ module Decidim
       # already when the logic below is unnecessary. This logic is needed e.g.
       # for serializers where the request context is not available.
       #
-      # @param record The record for which to check the organization
-      # @param opts Options for building the URL
+      # @param record [Object] The record for which to check the organization
+      # @param opts [Hash] Options for building the URL
       # @return [void]
       def ensure_current_host(record, **opts)
         return if asset_url_available?
@@ -141,7 +141,7 @@ module Decidim
 
       # Determines the organization for the passed record.
       #
-      # @param record The record for which to fetch the organization
+      # @param record [Object] The record for which to fetch the organization
       # @return [Decidim::Organization, nil] The organization for the record or
       #   `nil` if the organization cannot be determined
       def organization_for(record)
@@ -154,8 +154,7 @@ module Decidim
 
       # Returns the URL for the given blob object.
       #
-      # @param blob The blob object
-      # @param options Options for building the URL
+      # @param options [Hash] Options for building the URL
       # @return [String, nil] The URL to the blob object or `nil` if the blob is
       #   not defined.
       def blob_url(**options)
@@ -207,7 +206,8 @@ module Decidim
       # because once the image is stored at the storage service, it already has
       # the correct file extension.
       #
-      # @param options The options for building the URL
+      # Accepts any options for the URL that are the normal route options
+      # Rails route helpers accept
       # @return [String, nil] The converted representation URL or `nil` if the
       #   asset is not defined.
       def rails_representation_url(**)
@@ -235,7 +235,7 @@ module Decidim
       # variant has to be served through the service's own representation URL
       # causing it to be processed and stored at the storage service.
       #
-      # @param options The options for building the URL
+      # @param options [Hash] The options for building the URL
       # @return [String, nil] The variant URL at the storage service or `nil` if
       #   the variant has not been processed yet and does not yet exist at the
       #   storage service or `nil` when the asset is not defined

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

@@ -68,7 +68,7 @@ module.exports = {
       sans: ["Source Sans Pro", "ui-sans-serif", "system-ui", "sans-serif"]
     },
     fontSize: {
-      xs: ["13px", "16px"],
+      xs: ["12px", "16px"],
       sm: ["14px", "18px"],
       md: ["16px", "20px"],
       lg: ["18px", "23px"],
@@ -81,6 +81,9 @@ module.exports = {
     extend: {
       transitionProperty: {
         'top': 'top',
+      },
+      screens: {
+        'xs': '375px'
       }
     }
   },

data/lib/decidim/attachment_attributes.rb

@@ -15,24 +15,39 @@ module Decidim
       #
       # name - The attribute's name
       #
-      #
       # Example:
       #
-      #   attachment_attribute :photos
+      #   attachments_attribute :photos
       #   # This will create two attributes of the following types:
       #   #   attribute :photos, Array[Integer]
       #   #   attribute :add_photos, Array
-      #   # In addition, it will generate the getter method for the attribute
-      #   # returning an array of the Decidim::Attachment records.
+      #   # In addition, it will generate:
+      #   #   - A setter that handles String (JSON/CSV), Integer, or Array inputs.
+      #   #   - A getter that falls back to add_photos when the attribute is blank.
+      #   #   - Private helpers for parsing and extracting IDs.
       #
       # Returns nothing.
-      def attachments_attribute(name)
+      def attachments_attribute(name) # rubocop:disable Metrics/CyclomaticComplexity,Metrics/PerceivedComplexity
         attribute name, Array[Integer]
         attribute :"add_#{name}", Array
 
-        # Define the getter method that fetches the attachment records based on
-        # their types. For Strings and Integers, assumes they are IDs and will
-        # fetch the attachment record matching that ID.
+        # Setter: coerces String (JSON or comma-separated IDs) and bare Integer
+        # values into the expected Array[Integer] format before delegating to super.
+        define_method :"#{name}=" do |value|
+          case value
+          when String
+            parsed = send(:"parse_string_#{name}", value)
+            parsed.any? ? super(parsed) : super(value)
+          when Integer
+            super([value])
+          else
+            super(value)
+          end
+        end
+
+        # Getter: resolves stored integer IDs to Decidim::Attachment records,
+        # caching the result on the instance. Falls back to extracting IDs from
+        # add_<name> when the attribute itself is blank, then resolves those too.
         variable_name = "@#{name}_records"
         define_method name do
           return instance_variable_get(variable_name) if instance_variable_defined?(variable_name)
@@ -40,9 +55,15 @@ module Decidim
           original = @attributes[name.to_s].value_before_type_cast
           return original if original && !original.is_a?(Array)
 
+          ids = if super().blank? && send(:"add_#{name}").present?
+                  send(:"extract_ids_from_add_#{name}")
+                else
+                  super()
+                end
+
           instance_variable_set(
             variable_name,
-            super().map do |attachment|
+            ids.map do |attachment|
               if attachment.is_a?(Integer)
                 Decidim::Attachment.find_by(id: attachment)
               else
@@ -51,6 +72,34 @@ module Decidim
             end.compact
           )
         end
+
+        # Private helpers -------------------------------------------------------
+
+        define_method :"extract_ids_from_add_#{name}" do
+          send(:"add_#{name}")
+            .select { |item| item.is_a?(Hash) && (item[:id].present? || item["id"].present?) }
+            .map { |item| (item[:id] || item["id"]).to_i }
+        end
+
+        define_method :"parse_string_#{name}" do |value|
+          return [] if value.blank?
+
+          send(:"parse_#{name}_ids", value)
+        end
+
+        define_method :"parse_#{name}_ids" do |value|
+          ids = begin
+            Array(JSON.parse(value))
+          rescue JSON::ParserError
+            value.split(",").map(&:strip)
+          end
+
+          ids.map(&:to_i).reject(&:zero?)
+        end
+
+        private :"extract_ids_from_add_#{name}"
+        private :"parse_string_#{name}"
+        private :"parse_#{name}_ids"
       end
     end
   end

data/lib/decidim/attribute_encryptor.rb

@@ -2,11 +2,23 @@
 
 module Decidim
   class AttributeEncryptor
-    def self.encrypt(string)
-      cryptor.encrypt_and_sign(string) if string.present?
+    attr_reader :secret, :hash_digest_class, :secret_key_base, :key_len, :is_retry
+
+    def initialize(secret: "attribute", **options)
+      @secret = secret
+      @hash_digest_class = options.fetch(:hash_digest_class, Rails.application.config.active_support.hash_digest_class)
+      @secret_key_base = options.fetch(:secret_key_base, Rails.application.secret_key_base)
+      @key_len = options.fetch(:key_len, ActiveSupport::MessageEncryptor.key_len)
+      @is_retry = options.fetch(:is_retry, false)
     end
 
-    def self.decrypt(string_encrypted)
+    def encrypt(string)
+      return if string.blank?
+
+      encryptor.encrypt_and_sign(string)
+    end
+
+    def decrypt(string_encrypted)
       return if string_encrypted.blank?
 
       # `ActiveSupport::MessageEncryptor` expects all values passed to the
@@ -15,16 +27,45 @@ module Decidim
       # If something else is passed, just return the value as is.
       return string_encrypted unless string_encrypted.is_a?(String)
 
-      cryptor.decrypt_and_verify(string_encrypted)
+      encryptor.decrypt_and_verify(string_encrypted)
+    rescue ActiveSupport::MessageEncryptor::InvalidMessage => e
+      # Since we have migrated from SHA1 to SHA256, we need to ensure that any encrypted string not migrated is still being decrypted successfully.
+      # There are some resources that are still using SHA1, so we need to retry with the legacy encryptor.
+      # Some of those resources are:
+      #  - Newsletter unsubscribe links (being sent to users via email)
+      raise e if is_retry
+
+      legacy_encryptor.decrypt(string_encrypted)
+    end
+
+    def self.encrypt(string)
+      cryptor.encrypt(string)
+    end
+
+    def self.decrypt(string_encrypted)
+      cryptor.decrypt(string_encrypted)
     end
 
     def self.cryptor
-      @cryptor ||= begin
-        key = ActiveSupport::KeyGenerator.new("attribute", hash_digest_class: OpenSSL::Digest::SHA1).generate_key(
-          Rails.application.secret_key_base, ActiveSupport::MessageEncryptor.key_len
-        )
-        ActiveSupport::MessageEncryptor.new(key)
-      end
+      @cryptor ||= new(secret: "attribute")
+    end
+
+    private
+
+    def legacy_encryptor
+      @legacy_encryptor ||= self.class.new(secret:, hash_digest_class: OpenSSL::Digest::SHA1, is_retry: true)
+    end
+
+    def encryptor
+      @encryptor ||= ActiveSupport::MessageEncryptor.new(key)
+    end
+
+    def key
+      @key ||= key_generator.generate_key(secret_key_base, key_len)
+    end
+
+    def key_generator
+      @key_generator ||= ActiveSupport::KeyGenerator.new(secret, hash_digest_class:)
     end
   end
 end

data/lib/decidim/attribute_object/model.rb

@@ -107,11 +107,6 @@ module Decidim
       # through symbols by calling `obj.attributes[:key]` or
       # `obj.attributes.slice(:key1, :key2)`. In the legacy Virtus models, this
       # returned a hash with symbol keys.
-      #
-      # Deprecated:
-      # Attributes access through symbols is deprecated and may be removed in
-      # future versions. Please refactor all your attributes calls to access the
-      # attributes through string keys.
       def attributes
         super.with_indifferent_access
       end

data/lib/decidim/block_registry.rb

@@ -8,7 +8,7 @@ module Decidim
   class BlockRegistry
     # Provides access to the registered blocks with their names.
     #
-    # @return [Hash<Symbol => Proc>] A hash of the currently registered blocks
+    # @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 ||= {}

data/lib/decidim/coauthorable.rb

@@ -138,7 +138,8 @@ module Decidim
       # Adds a new coauthor to the list of coauthors. The coauthorship is created with
       # current object as coauthorable and `user` param as author.
       #
-      # @param author: The new coauthor.
+      # @param author [Decidim::User] The new coauthor.
+      # @param extra_attributes [Hash{Symbol => Object}]
       # @extra_attrs: Extra
       def add_coauthor(author, extra_attributes = {})
         return if coauthorships.exists?(decidim_author_id: author.id, decidim_author_type: author.class.base_class.name)