decidim-core 0.28.4 → 0.28.5

data/lib/decidim/core/test/shared_examples/comments_examples.rb

@@ -613,7 +613,7 @@ shared_examples "comments" do
     end
 
     context "when the user has verified organizations" do
-      let(:user_group) { create(:user_group, :verified) }
+      let(:user_group) { create(:user_group, :verified, organization:) }
       let(:content) { "This is a new comment" }
 
       before do
@@ -741,6 +741,20 @@ shared_examples "comments" do
               expect(page).to have_content("Edited")
             end
           end
+
+          it "has only one edit modal" do
+            expect(page).to have_css("#editCommentModal#{comment.id}", visible: :hidden, count: 1)
+            3.times do |index|
+              sleep 2
+              within "#comment_#{comment.id}" do
+                page.find("[id^='dropdown-trigger']").click
+                click_on "Edit"
+              end
+              fill_in "edit_comment_#{comment.id}", with: " This comment has been edited #{1 + index} times"
+              click_on "Send"
+            end
+            expect(page).to have_css("#editCommentModal#{comment.id}", visible: :all, count: 1)
+          end
         end
       end
     end
@@ -1025,7 +1039,6 @@ shared_examples "comments blocked" do
   end
 
   context "when authenticated" do
-    let!(:organization) { create(:organization) }
     let!(:user) { create(:user, :confirmed, organization:) }
     let!(:comments) { create_list(:comment, 3, commentable:) }
 

data/lib/decidim/core/version.rb

@@ -4,7 +4,7 @@ module Decidim
   # This holds the decidim-core version.
   module Core
     def self.version
-      "0.28.4"
+      "0.28.5"
     end
   end
 end

data/lib/decidim/diffy_extension.rb

@@ -13,6 +13,24 @@ module Decidim
         str = wrap_lines(@diff.map { |line| wrap_line(line) })
         ActionView::Base.new(ActionView::LookupContext.new(nil), {}, nil).sanitize(str, tags: TAGS)
       end
+
+      private
+
+      def wrap_line(line)
+        cleaned = clean_line(line)
+        case line
+        when /^(---|\+\+\+|\\\\)/
+          "    <li class=\"diff-comment\"><div>#{line.chomp}</div></li>"
+        when /^\+/
+          "    <li class=\"ins\"><ins>#{cleaned}</ins></li>"
+        when /^-/
+          "    <li class=\"del\"><del>#{cleaned}</del></li>"
+        when /^ /
+          "    <li class=\"unchanged\"><div>#{cleaned}</div></li>"
+        when /^@@/
+          "    <li class=\"diff-block-info\"><div>#{line.chomp}</div></li>"
+        end
+      end
     end
 
     # Adding a new method to Diffy::Format so we can pass the

data/lib/decidim/form_builder.rb

@@ -390,7 +390,7 @@ module Decidim
     # options      - A Hash with options to build the field. See upload method for
     # more detailed information.
     def attachment(attribute, options = {})
-      object_attachment = object.attachment.present?
+      object_attachment = object.respond_to?(:attachment) && object.attachment.present?
       record = object_attachment ? object.attachment : object
       options = {
         titled: options[:multiple],

data/lib/decidim/map/autocomplete.rb

@@ -42,6 +42,7 @@ module Decidim
           @autocomplete_utility ||= Decidim::Map.autocomplete(
             organization: @template.current_organization
           )
+
           return text_field(attribute, options) unless @autocomplete_utility
 
           # Decidim::Map::Autocomplete::Builder

data/lib/decidim/participatory_space_user.rb

@@ -37,6 +37,10 @@ module Decidim
         raise "Not implemented"
       end
 
+      def self.order_by_name
+        includes(:user).order("decidim_users.name ASC")
+      end
+
       private
 
       # Private: check if the process and the user have the same organization

data/lib/decidim/query_extensions.rb

@@ -10,22 +10,6 @@ module Decidim
     #
     # Returns nothing.
     def self.included(type)
-      type.field :participatory_processes,
-                 [Decidim::ParticipatoryProcesses::ParticipatoryProcessType],
-                 null: true,
-                 description: "Lists all participatory_processes" do
-        argument :filter, Decidim::ParticipatoryProcesses::ParticipatoryProcessInputFilter, "This argument lets you filter the results", required: false
-        argument :order, Decidim::ParticipatoryProcesses::ParticipatoryProcessInputSort, "This argument lets you order the results", required: false
-      end
-
-      type.field :participatory_process,
-                 Decidim::ParticipatoryProcesses::ParticipatoryProcessType,
-                 null: true,
-                 description: "Finds a participatory_process" do
-        argument :id, GraphQL::Types::ID, "The ID of the participatory space", required: false
-        argument :slug, String, "The slug of the participatory process", required: false
-      end
-
       type.field :component, Decidim::Core::ComponentInterface, null: true do
         description "Lists the components this space contains."
         argument :id, GraphQL::Types::ID, required: true, description: "The ID of the component to be found"
@@ -62,16 +46,6 @@ module Decidim
       end
     end
 
-    def participatory_processes(filter: {}, order: {})
-      manifest = Decidim.participatory_space_manifests.select { |m| m.name == :participatory_processes }.first
-      Decidim::Core::ParticipatorySpaceListBase.new(manifest:).call(object, { filter:, order: }, context)
-    end
-
-    def participatory_process(id: nil, slug: nil)
-      manifest = Decidim.participatory_space_manifests.select { |m| m.name == :participatory_processes }.first
-      Decidim::Core::ParticipatorySpaceFinderBase.new(manifest:).call(object, { id:, slug: }, context)
-    end
-
     def component(id: {})
       component = Decidim::Component.published.find_by(id:)
       component&.organization == context[:current_organization] ? component : nil

data/lib/decidim/settings_manifest.rb

@@ -116,6 +116,8 @@ module Decidim
       validates :type, inclusion: { in: TYPES.keys }
 
       def type_class
+        return Decidim::Attributes::RichText if type == :text && editor == true
+
         TYPES[type][:klass]
       end
 

data/lib/decidim/translatable_attributes.rb

@@ -34,6 +34,7 @@ module Decidim
       #   end
       #
       # Returns nothing.
+      # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
       def translatable_attribute(name, type, **options)
         attribute(name, { String => Object }, default: {})
 
@@ -41,6 +42,8 @@ module Decidim
           attribute_name = "#{name}_#{locale}".gsub("-", "__")
           attribute attribute_name, type, **options
 
+          value_type = attribute_types[attribute_name.to_s]
+
           define_method attribute_name do
             field = public_send(name) || {}
             value =
@@ -51,7 +54,6 @@ module Decidim
                 # changed and the old value is still stored against the record.
                 field
               end
-            value_type = self.class.attribute_types[attribute_name.to_s]
             value_type ? value_type.cast(value) : value
           end
 
@@ -60,12 +62,15 @@ module Decidim
             final = super(value)
             return unless final # Do not set the `nil` values for the parent hash
 
+            final = value_type.serialize(final) if value_type
+
             public_send("#{name}=", field.merge(locale => final))
           end
 
           yield(attribute_name, locale) if block_given?
         end
       end
+      # rubocop:enable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
 
       def locales
         Decidim.available_locales

data/lib/tasks/upgrade/decidim_fix_categorization.rake

@@ -8,9 +8,18 @@ namespace :decidim do
         :"decidim:upgrade:clean:searchable_resources",
         :"decidim:upgrade:clean:notifications",
         :"decidim:upgrade:clean:follows",
-        :"decidim:upgrade:clean:action_logs"
+        :"decidim:upgrade:clean:action_logs",
+        :"decidim:upgrade:clean:clean_deleted_users",
+        :"decidim:upgrade:clean:fix_blocked_user_notification"
       ]
 
+      desc "Remove data from deleted users"
+      task clean_deleted_users: :environment do
+        logger = Logger.new($stdout)
+        logger.info("=== Removing extra data from deleted users")
+        Decidim::User.where.not(deleted_at: nil).update_all(personal_url: "", about: "", notifications_sending_frequency: "none") # rubocop:disable Rails/SkipsModelValidations
+      end
+
       desc "Removes any action logs belonging to invalid resources"
       task :action_logs, [] => :environment do
         puts "=== Deleting Action logs\n"
@@ -100,15 +109,32 @@ namespace :decidim do
         end
         puts "===== Deleted #{invalid} invalid resources\n"
       end
-    end
 
-    desc "Removes orphan categorizations"
-    task fix_orphan_categorizations: :environment do
-      logger = Logger.new($stdout)
-      logger.info("Removing orphan categorizations...")
+      desc "Removes orphan categorizations"
+      task fix_orphan_categorizations: :environment do
+        logger = Logger.new($stdout)
+        logger.info("Removing orphan categorizations...")
+
+        Decidim::Categorization.find_each do |categorization|
+          categorization.destroy if categorization.categorizable.nil?
+        end
+      end
+
+      desc "Update all blocked users notifications_sending_frequency setting"
+      task fix_blocked_user_notification: :environment do
+        logger.info("=== Updating all blocked users notifications_sending_frequency ...")
+        blocked_users = 0
+        Decidim::User.blocked.where.not("notifications_sending_frequency = ?", "none").find_each do |blocked_user|
+          unless blocked_user.notifications_sending_frequency == "none"
+            blocked_user.update(notifications_sending_frequency: "none")
+            blocked_users += 1
+          end
+        end
+        logger.info("===== Updated #{blocked_users} blocked users")
+      end
 
-      Decidim::Categorization.find_each do |categorization|
-        categorization.destroy if categorization.categorizable.nil?
+      def logger
+        @logger ||= Logger.new($stdout)
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: decidim-core
 version: !ruby/object:Gem::Version
-  version: 0.28.4
+  version: 0.28.5
 platform: ruby
 authors:
 - Josep Jaume Rey Peroy
@@ -10,8 +10,22 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-10-02 00:00:00.000000000 Z
+date: 2025-02-12 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: concurrent-ruby
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.2.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 1.2.2
 - !ruby/object:Gem::Dependency
   name: active_link_to
   requirement: !ruby/object:Gem::Requirement
@@ -750,28 +764,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.28.4
+        version: 0.28.5
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.28.4
+        version: 0.28.5
 - !ruby/object:Gem::Dependency
   name: decidim-dev
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.28.4
+        version: 0.28.5
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.28.4
+        version: 0.28.5
 description: Adds core features so other engines can hook into the framework.
 email:
 - josepjaume@gmail.com
@@ -1300,6 +1314,7 @@ files:
 - app/forms/decidim/user_interest_scope_form.rb
 - app/forms/decidim/user_interests_form.rb
 - app/helpers/concerns/decidim/flash_helper_extensions.rb
+- app/helpers/concerns/decidim/user_role_checker.rb
 - app/helpers/decidim/account_helper.rb
 - app/helpers/decidim/action_authorization_helper.rb
 - app/helpers/decidim/amendments_helper.rb
@@ -1650,6 +1665,7 @@ files:
 - app/packs/src/decidim/notifications.js
 - app/packs/src/decidim/password_toggler.js
 - app/packs/src/decidim/remote_modal.js
+- app/packs/src/decidim/remote_tooltips.js
 - app/packs/src/decidim/responsive_horizontal_tabs.js
 - app/packs/src/decidim/results_listing.js
 - app/packs/src/decidim/scroll_to_last_child.js
@@ -2124,6 +2140,8 @@ files:
 - config/locales/ar.yml
 - config/locales/bg-BG.yml
 - config/locales/bg.yml
+- config/locales/bn-BD.yml
+- config/locales/bs-BA.yml
 - config/locales/ca.yml
 - config/locales/cs-CZ.yml
 - config/locales/cs.yml
@@ -2520,6 +2538,7 @@ files:
 - lib/decidim/attributes/localized_date.rb
 - lib/decidim/attributes/model.rb
 - lib/decidim/attributes/object.rb
+- lib/decidim/attributes/rich_text.rb
 - lib/decidim/attributes/symbol.rb
 - lib/decidim/attributes/time_with_zone.rb
 - lib/decidim/authorable.rb
@@ -2539,6 +2558,7 @@ files:
 - lib/decidim/content_blocks/has_related_components.rb
 - lib/decidim/content_parsers.rb
 - lib/decidim/content_parsers/base_parser.rb
+- lib/decidim/content_parsers/blob_parser.rb
 - lib/decidim/content_parsers/hashtag_parser.rb
 - lib/decidim/content_parsers/inline_images_parser.rb
 - lib/decidim/content_parsers/link_parser.rb
@@ -2550,6 +2570,7 @@ files:
 - lib/decidim/content_processor.rb
 - lib/decidim/content_renderers.rb
 - lib/decidim/content_renderers/base_renderer.rb
+- lib/decidim/content_renderers/blob_renderer.rb
 - lib/decidim/content_renderers/hashtag_renderer.rb
 - lib/decidim/content_renderers/link_renderer.rb
 - lib/decidim/content_renderers/resource_renderer.rb
@@ -2851,7 +2872,7 @@ files:
 - lib/tasks/upgrade/decidim_user_moderation.rake
 homepage: https://decidim.org
 licenses:
-- AGPL-3.0
+- AGPL-3.0-or-later
 metadata:
   bug_tracker_uri: https://github.com/decidim/decidim/issues
   documentation_uri: https://docs.decidim.org/