decidim-core 0.28.5 → 0.29.0.rc1

data/lib/decidim/paddable.rb

@@ -59,7 +59,7 @@ module Decidim
         @pad_id ||= [reference, token].join("-").slice(0, 50)
       end
 
-      # compatibilize with old versions if no salt available (less secure)
+      # compatibility with old versions if no salt available (less secure)
       def token
         if defined?(salt) && salt.present?
           tokenizer = Decidim::Tokenizer.new(salt:)

data/lib/decidim/participable.rb

@@ -31,7 +31,7 @@ module Decidim
 
       def mounted_params
         {
-          host: organization.host,
+          :host => organization.host,
           "#{underscored_name}_slug".to_sym => slug
         }
       end

data/lib/decidim/participatory_space_manifest.rb

@@ -98,7 +98,7 @@ module Decidim
     #
     # Returns nothing.
     def seed!
-      print "Creating seeds for the #{name} space...\n" unless Rails.env.test?
+      print "Creating seeds for the #{name} space...\n" unless Rails.env.test? # rubocop:disable Rails/Output
       @seeds&.call
     end
 

data/lib/decidim/participatory_space_user.rb

@@ -37,10 +37,6 @@ 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,6 +10,22 @@ 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"
@@ -46,6 +62,16 @@ 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/resourceable.rb

@@ -111,7 +111,7 @@ module Decidim
       end
 
       # Check only the resource visibility not its hierarchy.
-      # This method is intended to be overriden by classes that include this module and have the
+      # This method is intended to be overridden by classes that include this module and have the
       # need to impose its own visibility rules.
       #
       # @return If the resource is also Publicable checks if the resource is published, otherwise returns true by default.

data/lib/decidim/search_resource_fields_mapper.rb

@@ -13,8 +13,8 @@ module Decidim
     #
     # @param declared_fields: A Hash with the mappings between a SearchableResource attributes and
     #    any given model. Mapped fields are:
-    # - scope_id: The field where the scope is setted in the model, if any.
-    # - participatory_space: The field where the ParticipatorySpace is setted in the model.
+    # - scope_id: The field where the scope is set in the model, if any.
+    # - participatory_space: The field where the ParticipatorySpace is set in the model.
     # - datetime: The field that describes where in time the model is placed.
     # - A, B, C, D: Weighted text fields.
     #

data/lib/decidim/searchable.rb

@@ -74,7 +74,7 @@ module Decidim
       after_destroy do |searchable|
         remove_from_index(searchable) if self.class.search_resource_fields_mapper
       end
-      # after_create and after_update callbacks are dynamically setted in `searchable_fields` method.
+      # after_create and after_update callbacks are dynamically set in `searchable_fields` method.
 
       # Public: after_create callback to index the model as a SearchableResource, if configured so.
       #
@@ -178,8 +178,8 @@ module Decidim
       # }
       #
       # `conditions` must be a Hash that only accepts a boolean or a Proc that will be evaluated on runtime and returns a boolean for the following keys:
-      # - index_on_create: Whether to index, or not, the current searchabe when it is created. Defaults to true.
-      # - index_on_update: Whether to index, or not, the current searchabe when it is updated. Defaults to true.
+      # - index_on_create: Whether to index, or not, the current searchable when it is created. Defaults to true.
+      # - index_on_update: Whether to index, or not, the current searchable when it is updated. Defaults to true.
       #
       def searchable_fields(declared_fields, conditions = {})
         @search_resource_indexable_fields = SearchResourceFieldsMapper.new(declared_fields)

data/lib/decidim/seeds.rb

@@ -17,6 +17,27 @@ module Decidim
       @admin_user ||= Decidim::User.find_by(organization:, email: "admin@example.org")
     end
 
+    def find_or_initialize_user_by(email:)
+      user = Decidim::User.find_or_initialize_by(email:)
+      user.update!(
+        name: ::Faker::Name.name,
+        nickname: ::Faker::Twitter.unique.screen_name,
+        password: "decidim123456789",
+        organization:,
+        confirmed_at: Time.current,
+        locale: I18n.default_locale,
+        personal_url: ::Faker::Internet.url,
+        about: ::Faker::Lorem.paragraph(sentence_count: 2),
+        avatar: random_avatar,
+        accepted_tos_version: organization.tos_version + 1.hour,
+        newsletter_notifications_at: Time.current,
+        tos_agreement: true,
+        password_updated_at: Time.current
+      )
+
+      user
+    end
+
     def random_scope(participatory_space:)
       if participatory_space.scope
         scopes = participatory_space.scope.descendants
@@ -86,10 +107,51 @@ module Decidim
       )
     end
 
+    def create_report!(reportable:, current_user:)
+      moderation = Moderation.find_or_create_by!(reportable:, participatory_space: reportable.participatory_space)
+
+      Decidim::Report.create!(
+        moderation:,
+        user: current_user,
+        locale: I18n.locale,
+        reason: "spam",
+        details: "From the seeds"
+      )
+    rescue ActiveRecord::RecordInvalid
+      # Ignore in case we have an error in the report creation.
+      # Most likely is a "Validation failed: User has already been taken"
+    end
+
+    def hide_report!(reportable:)
+      moderation = Moderation.find_or_create_by!(reportable:, participatory_space: reportable.participatory_space)
+      moderation.update!(hidden_at: Time.zone.now)
+    end
+
+    def create_user_report!(reportable:, current_user:)
+      moderation = UserModeration.find_or_create_by!(user: reportable)
+
+      UserReport.create!(
+        moderation:,
+        user: current_user,
+        reason: "spam",
+        details: "From the seeds"
+      )
+    end
+
     def seed_components_manifests!(participatory_space:)
       Decidim.component_manifests.each do |manifest|
         manifest.seed!(participatory_space.reload)
       end
     end
+
+    def random_avatar
+      file_number = format("%03d", rand(1...100))
+
+      create_blob!(seeds_file: "avatars/#{file_number}.jpg", filename: "#{file_number}.jpg", content_type: "image/jpeg")
+    end
+
+    def create_follow!(user, followable)
+      Decidim::Follow.create!(followable:, user:)
+    end
   end
 end

data/lib/decidim/settings_manifest.rb

@@ -116,8 +116,6 @@ 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/stats_registry.rb

@@ -40,7 +40,7 @@ module Decidim
     # Public: Returns a number returned by executing the corresponding block.
     #
     # name - The name of the stat
-    # context - An arbritrary context object to compute the result.
+    # context - An arbitrary context object to compute the result.
     # start_at - A date to filter resources created after it
     # end_at - A date to filter resources created before it.
     #
@@ -54,7 +54,7 @@ module Decidim
 
     # Public: Resolves every stat with the given context and return an enumerator
     #
-    # context - An arbritrary context object to compute the result.
+    # context - An arbitrary context object to compute the result.
     # start_at - A date to filter resources created after it
     # end_at - A date to filter resources created before it
     #

data/lib/decidim/translatable_attributes.rb

@@ -17,7 +17,7 @@ module Decidim
       # type - The attribute's type
       # options - Options to send to the form class when defining the attribute.
       # block - An optional block to be called for each attribute name and
-      # locale. The block will receive two arguments, one with the attriubte
+      # locale. The block will receive two arguments, one with the attribute
       # name and another with the locale.
       #
       #
@@ -34,15 +34,12 @@ module Decidim
       #   end
       #
       # Returns nothing.
-      # rubocop:disable Metrics/CyclomaticComplexity, Metrics/PerceivedComplexity
-      def translatable_attribute(name, type, **options)
+      def translatable_attribute(name, type, **)
         attribute(name, { String => Object }, default: {})
 
         locales.each do |locale|
           attribute_name = "#{name}_#{locale}".gsub("-", "__")
-          attribute attribute_name, type, **options
-
-          value_type = attribute_types[attribute_name.to_s]
+          attribute(attribute_name, type, **)
 
           define_method attribute_name do
             field = public_send(name) || {}
@@ -54,6 +51,7 @@ 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
 
@@ -62,15 +60,12 @@ 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/decidim/translatable_resource.rb

@@ -19,7 +19,7 @@ module Decidim
           method_name = "#{field}="
 
           # We are overriding the attribute setter method so that we can reuse the
-          # machine translations. This is to fix a bug encoutered when updating
+          # machine translations. This is to fix a bug encountered when updating
           # the resource from a FormObject. The FormObject on the `#create`
           # action in the controller does not have the machine translations loaded,
           # so they are effectively lost whenever a resource is updated.

data/lib/decidim/view_hooks.rb

@@ -87,7 +87,7 @@ module Decidim
     # Internal class to encapsulate each view registered on a view hook.
     class ViewHook
       # priority - a Number (Integer|Float) to sort the block.
-      # block - a block that will bbe rendered in a view context.
+      # block - a block that will be rendered in a view context.
       def initialize(priority, block)
         @priority = priority
         @block = block

data/lib/decidim/view_model.rb

@@ -17,8 +17,12 @@ module Decidim
     include Decidim::MarkupHelper
     include Decidim::LayoutHelper
     include Decidim::SanitizeHelper
+    include Decidim::ApplicationHelper
+    include Decidim::IconHelper
 
+    delegate :helper_method, to: :controller
     delegate :current_organization, to: :controller
+    delegate_missing_to :view_context
 
     cache :show, if: :perform_caching?, expires_in: :cache_expiry_time do
       cache_hash
@@ -87,7 +91,7 @@ module Decidim
     end
 
     def cache_expiry_time
-      Decidim.cache_expiry_time
+      nil
     end
 
     def decidim

data/lib/decidim/webpacker/esbuild.config.js

@@ -0,0 +1,14 @@
+// eslint-disable-next-line no-undef
+module.exports = {
+  options: {
+    tsconfigRaw: JSON.stringify({}),
+    target: "es2015"
+  },
+  bundle: true,
+  sourcemap: true,
+  minify: true,
+  outdir: "public",
+  outExtension: {
+    ".css": ".scss"
+  }
+};

data/lib/decidim/webpacker/shakapacker.yml

@@ -22,12 +22,12 @@ default: &default
   cache_manifest: false
 
   # Select loader to use, available options are 'babel' (default), 'swc' or 'esbuild'
-  webpack_loader: 'babel'
+  webpack_loader: 'esbuild'
 
   # Set to true to enable check for matching versions of shakapacker gem and NPM package - will raise an error if there is a mismatch or wildcard versioning is used
   ensure_consistent_versioning: true
 
-  # Select whether the compiler will use SHA digest ('digest' option) or most most recent modified timestamp ('mtime') to determine freshness
+  # Select whether the compiler will use SHA digest ('digest' option) or most recent modified timestamp ('mtime') to determine freshness
   compiler_strategy: digest
 
   # Select whether the compiler will always use a content hash and not just in production

data/lib/decidim/webpacker/tsconfig.json

@@ -0,0 +1,11 @@
+{
+  "compilerOptions": {
+    "strict": true,                                      /* Enable all strict type-checking options. */
+    "removeComments": true,                              /* Disable emitting comments. */
+    "allowJs": true,                                     /* Allow JavaScript files to be a part of your program. Use the `checkJS` option to get errors from these files. */
+    "module": "commonjs",                                /* Specify what module code is generated. */
+    "incremental": true,                                 /* Enable incremental compilation */
+    "target": "es6",                                     /* Set the JavaScript language version for emitted JavaScript and include compatible library declarations. */
+    "skipLibCheck": true,                                /* Skip type checking all .d.ts files. */
+  }
+}

data/lib/decidim/webpacker/webpack/custom.js

@@ -1,7 +1,7 @@
 /* eslint-disable */
 const { config } = require("shakapacker");
 const { InjectManifest } = require("workbox-webpack-plugin");
-const TerserPlugin = require('terser-webpack-plugin')
+const { EsbuildPlugin } = require("esbuild-loader");
 
 module.exports = {
   module: {
@@ -15,8 +15,8 @@ module.exports = {
       },
       {
         test: /\.(js|jsx)$/,
-        exclude: /node_modules\/(?!tributejs)/,
-        loader: "babel-loader"
+        exclude: /node_modules\//,
+        loader: "esbuild-loader"
       },
       {
         test: /\.(graphql|gql)$/,
@@ -37,14 +37,11 @@ module.exports = {
         }
       },
       {
-        test: [
-          /\.md$/,
-          /\.odt$/,
-        ],
+        test: [/\.md$/, /\.odt$/],
         exclude: [/\.(js|mjs|jsx|ts|tsx)$/],
-        type: 'asset/resource',
+        type: "asset/resource",
         generator: {
-          filename: 'media/documents/[hash][ext][query]'
+          filename: "media/documents/[hash][ext][query]"
         }
       },
       // Overwrite webpacker files rule to amend the filename output
@@ -68,9 +65,9 @@ module.exports = {
           /\.svg$/
         ],
         exclude: [/\.(js|mjs|jsx|ts|tsx)$/],
-        type: 'asset/resource',
+        type: "asset/resource",
         generator: {
-          filename: 'media/images/[name]-[hash][ext][query]'
+          filename: "media/images/[name]-[hash][ext][query]"
         }
       }
     ]
@@ -81,32 +78,13 @@ module.exports = {
       crypto: false
     }
   },
-  // https://github.com/rails/webpacker/issues/2932
-  // As Decidim uses multiple packs, we need to enforce a single runtime, to prevent duplication
   optimization: {
     minimizer: [
-      new TerserPlugin({
-        parallel: Number.parseInt(process.env.SHAKAPACKER_PARALLEL, 10) || true,
-        terserOptions: {
-          parse: {
-            // Let terser parse ecma 8 code but always output
-            // ES5 compliant code for older browsers
-            ecma: 8
-          },
-          compress: {
-            ecma: 5,
-            warnings: false,
-            comparisons: false
-          },
-          mangle: { safari10: true },
-          output: {
-            ecma: 5,
-            comments: false,
-            ascii_only: true
-          }
-        }
+      new EsbuildPlugin({
+        target: "es2015",
+        css: true
       })
-    ].filter(Boolean)
+    ]
   },
   entry: config.entrypoints,
   plugins: [
@@ -122,4 +100,4 @@ module.exports = {
       swDest: "../sw.js"
     })
   ]
-}
+};

data/lib/devise/models/decidim_newsletterable.rb

@@ -2,7 +2,7 @@
 
 module Devise
   module Models
-    # Newsletterable adds all needed validations to acomplish GDPR standards
+    # Newsletterable adds all needed validations to accomplish GDPR standards
     # in newsletter notification mailing
     #
     # Validates that all users with notifications activated have Opt-in in

data/lib/tasks/decidim_download_your_data_tasks.rake

@@ -32,7 +32,7 @@ namespace :decidim do
           else
             puts " #{index}/#{users_count} !! [#{user_id}] DELETING USER"
             log.info " #{index}/#{users_count} !! [#{user_id}] DELETING USER"
-            Decidim::DestroyAccount.call(user, Decidim::DeleteAccountForm.from_params({})) # Delete user!
+            Decidim::DestroyAccount.call(Decidim::DeleteAccountForm.from_params({}).with_context({ current_user: user })) # Delete user!
           end
         else
           puts " #{index}/#{users_count} - [#{user_id}] User not found"
@@ -55,7 +55,7 @@ namespace :decidim do
     end
   end
 
-  desc "Check and notify users to update their newsletter notifications settings"
+  desc "Checks and notifies users to update their newsletter notifications settings"
   task check_users_newsletter_opt_in: :environment do
     print %(
 > This will send an email to all the users that have marked the newsletter by default. This should only be run if you were using Decidim before v0.11

data/lib/tasks/decidim_locales_tasks.rake

@@ -21,7 +21,7 @@ namespace :decidim do
       end
     end
 
-    desc "Rebuild the search index"
+    desc "Rebuilds the search index"
     task rebuild_search: :environment do
       Decidim::SearchableResource.destroy_all
       total = Decidim::Searchable.searchable_resources.count

data/lib/tasks/decidim_mailers_tasks.rake

@@ -2,12 +2,12 @@
 
 namespace :decidim do
   namespace :mailers do
-    desc "Task to send the notification digest email with the daily report"
+    desc "Sends the notification digest email with the daily report"
     task notifications_digest_daily: :environment do
       notifications_digest(:daily)
     end
 
-    desc "Task to send the notification digest email with the weekly report"
+    desc "Sends the notification digest email with the weekly report"
     task notifications_digest_weekly: :environment do
       notifications_digest(:weekly)
     end

data/lib/tasks/decidim_metrics_tasks.rake

@@ -7,7 +7,7 @@ namespace :decidim do
     # Get all metrics entities and execute its own rake task.
     # It admits a date-string parameter, in a 'YYYY-MM-DD' format from
     # today to all past dates
-    desc "Execute all metrics calculation methods"
+    desc "Calculates all metrics"
     task :all, [:day] => :environment do |_task, args|
       Decidim::Organization.find_each do |organization|
         Decidim.metrics_registry.all.each do |metric_manifest|
@@ -21,7 +21,7 @@ namespace :decidim do
     # Execute metric calculations for just one metric
     # It need a metric name and permits a date-string parameter, in a 'YYYY-MM-DD' format from
     # today to all past dates
-    desc "Execute one metric calculation method"
+    desc "Calculates one metric"
     task :one, [:metric, :day] => :environment do |_task, args|
       next if args.metric.blank?
 
@@ -31,7 +31,7 @@ namespace :decidim do
       end
     end
 
-    desc "Rebuild calculations from specific day"
+    desc "Rebuilds calculations from specific day"
     task :rebuild, [:metric, :day] => :environment do |_task, args|
       metric = args.metric
       day = args.day
@@ -47,10 +47,10 @@ namespace :decidim do
           Decidim::Organization.find_each do |organization|
             if metric
               m_manifest = Decidim.metrics_registry.for(metric)
-              log_info "[#{organization.name}]: rebuilding metric [#{metric}] for day [#{current_day}]"
+              log_info "[#{organization.name[organization.default_locale.to_s]}]: rebuilding metric [#{metric}] for day [#{current_day}]"
               call_metric_job(m_manifest, organization, current_day)
             else
-              log_info "[#{organization.name}]: rebuilding all metrics for day [#{current_day}]"
+              log_info "[#{organization.name[organization.default_locale.to_s]}]: rebuilding all metrics for day [#{current_day}]"
               Decidim.metrics_registry.all.each do |metric_manifest|
                 call_metric_job(metric_manifest, organization, current_day)
               end
@@ -63,7 +63,7 @@ namespace :decidim do
       end
     end
 
-    desc "Show available metrics"
+    desc "Shows the available metrics"
     task list: :environment do
       puts Decidim.metrics_registry.all.pluck :metric_name
     end

data/lib/tasks/decidim_procfile.rake

@@ -4,7 +4,7 @@ require "thor"
 
 namespace :decidim do
   namespace :procfile do
-    desc "Generates a dummy app for testing in external installations"
+    desc "Generates a script for starting the development app server"
     task :install do
       actions :create_file, "Procfile.dev", <<~RUBY
         web: bin/rails server -b 0.0.0.0 -p 3000
@@ -13,6 +13,12 @@ namespace :decidim do
 
       actions :create_file, "bin/dev", %(#!/usr/bin/env sh
 
+set -e
+
+bundle check || bundle install --jobs 20 --retry 5
+
+bin/rails decidim:upgrade db:migrate
+
 if ! gem list foreman -i --silent; then
   echo "Installing foreman..."
   gem install foreman
@@ -29,8 +35,8 @@ exec foreman start -f Procfile.dev "$@")
       include Thor::Actions
     end
 
-    def actions(*args)
-      Actions.new.send(*args)
+    def actions(*)
+      Actions.new.send(*)
     end
   end
 end

data/lib/tasks/decidim_tasks.rake

@@ -1,16 +1,18 @@
 # frozen_string_literal: true
 
 namespace :decidim do
-  desc "Install migrations from Decidim to the app."
+  desc "Performs upgrade tasks (migrations, node, docs )."
   task upgrade: [
     :choose_target_plugins,
     :"decidim:upgrade_app",
     :"railties:install:migrations",
-    :"decidim:webpacker:upgrade",
+    :"decidim:upgrade:webpacker",
     :"decidim_api:generate_docs"
   ]
 
-  desc "Setup environment so that only decidim migrations are installed."
+  task update: [:upgrade]
+
+  desc "Sets up environment so that only decidim migrations are installed."
   task :choose_target_plugins do
     ENV["FROM"] = %w(
       decidim
@@ -22,7 +24,6 @@ namespace :decidim do
       decidim_comments
       decidim_conferences
       decidim_debates
-      decidim_elections
       decidim_forms
       decidim_initiatives
       decidim_meetings