decidim-core 0.27.3 → 0.27.5

data/config/locales/uk.yml

@@ -171,8 +171,6 @@ uk:
           invalid_field: "Для поля %{field} значення %{value} неприпустиме."
           ok: Гаразд
           title: Недостатньо прав через відсутність підтвердження особи
-      show:
-        close_modal: Закрити віконце
     collapsible_list:
       see_less: "(читати менше)"
       see_more: "(читати далі)"
@@ -223,7 +221,6 @@ uk:
           already_have_an_account?: Вже маєте обліковий запис?
           newsletter: Отримувати іноді розсилання зі свіжими новинами
           newsletter_title: Дозвіл на зв'язок
-          sign_in: Увійти
           sign_up: Зареєструватися
           sign_up_as:
             legend: Зареєструватися як
@@ -240,7 +237,6 @@ uk:
         newsletter_modal:
           buttons:
             check: Поставити галочку та продовжити
-            close_modal: Закрити віконце
           notice: |-
             <p>Привіт, ви впевнені, що не хочете отримувати розсилання новин?<br> Будь ласка, ще раз розгляньте можливість поставити нижче галочку щодо розсилання.<br> Для нас дуже важливо, щоб ви могли іноді отримувати електронні листи з важливими оголошеннями. Ви завжди можете змінити це на сторінці налаштувань сповіщень.</p> <p>Якщо ви не поставите цю галочку, ви можете прогледіти доречні відомості щодо свіжих можливостей участі в роботі майданчика.<br> Якщо ви все ж бажаєте уникнути розсилань, ми прекрасно розуміємо таке ваше рішення.</p> <p>Дякуємо, що прочитали це!</p>
           title: Сповіщення про новини
@@ -267,8 +263,6 @@ uk:
       show:
         download_data: Завантажити дані
         request_data: Подати запит на дані
-    endorsable:
-      endorsements: Проявів підтримки
     errors:
       internal_server_error:
         title: При з'єднанні з нашим сервером сталася помилка
@@ -341,9 +335,9 @@ uk:
       default_image: Зображення за замовчуванням
       errors:
         error: У цьому полі є помилка.
-    links:
-      warning:
-        close_modal: Закрити віконце
+      upload:
+        labels:
+          save: Зберегти
     log:
       base_presenter:
         create: "%{user_name} створив %{resource_name}"
@@ -383,7 +377,6 @@ uk:
         create:
           error: Бесіда не розпочата. Спробуйте ще раз пізніше
         index:
-          close: Закрити віконце
           no_conversations: У вас ще немає бесід
         reply:
           send: Надіслати
@@ -440,7 +433,6 @@ uk:
           proposals: Пропозиції
           proposals_explanation: Вносьте нові пропозиції, підтримуйте існуючі та сприяйте змінам, які ви хочете бачити.
         footer_sub_hero:
-          footer_sub_hero_body: Створімо більш відкрите, прозоре та співпрацююче суспільство.<br /> Долучайтесь, беріть участь та приймайте рішення.
           footer_sub_hero_headline: Ласкаво просимо до майданчику співучасті %{organization}!
           register: Зареєструватися
         hero:
@@ -458,7 +450,6 @@ uk:
           legend: Погодитися з умовами участі
         refuse:
           modal_btn_exit: Я перегляну це пізніше
-          modal_close: Закрити віконце
           modal_title: Ви дійсно відмовляєтеся від оновлених "Умов участі"?
         required_review:
           alert: Ми оновили наші "Умови участі", будь ласка, перегляньте їх.
@@ -502,10 +493,8 @@ uk:
       term_input_placeholder: Шукати
     searches:
       filters_small_view:
-        close_modal: Закрити віконце
         filter: Відібрати
         filter_by: 'Відібрати за ознакою:'
-        unfold: Розгорнути
     shared:
       embed_modal:
         close_window: Закрити вікно
@@ -536,6 +525,8 @@ uk:
     statistics:
       comments_count: Коментарів
     user_conversations:
+      index:
+        close: Закрити
       update:
         error: Повідомлення не надіслано. Спробуйте ще раз пізніше
     version_author:

data/config/locales/zh-CN.yml

@@ -386,7 +386,6 @@ zh-CN:
           already_have_an_account?: 已经有一个帐户？
           newsletter: 接收附带相关信息的时事通讯
           newsletter_title: 联系人权限
-          sign_in: 登录
           sign_up: 注册
           sign_up_as:
             legend: 注册为
@@ -432,7 +431,6 @@ zh-CN:
         download_data: 下载数据
         request_data: 请求数据
     endorsable:
-      endorsements: 授权
       endorsements_count: 批准次数
     endorsement_buttons_cell:
       already_endorsed: 已批准
@@ -635,6 +633,10 @@ zh-CN:
       file_validation:
         allowed_file_extensions: '允许的文件扩展名: %{extensions}'
         max_file_size: '最大文件大小: %{megabytes}MB'
+      upload:
+        labels:
+          cancel: 取消
+          save: 保存
     gamification:
       all_badges_link: 查看所有可用的徽章。
       badges:
@@ -959,7 +961,6 @@ zh-CN:
           proposals: 建议
           proposals_explanation: 提出建议、支持现有建议并促进你想要看到的更改。
         footer_sub_hero:
-          footer_sub_hero_body: 让我们建设一个更加开放、透明和协作的社会。<br /> 加入、参与和决定。
           footer_sub_hero_headline: 欢迎使用 %{organization} 参与式平台。
           register: 注册
         hero:
@@ -1334,8 +1335,6 @@ zh-CN:
       updated: 您的帐户已成功更新。
     sessions:
       already_signed_out: 已成功注销。
-      new:
-        sign_in: 登录
       signed_in: 登录成功。
       signed_out: 已成功注销。
     shared:
@@ -1344,7 +1343,6 @@ zh-CN:
         didn_t_receive_confirmation_instructions: 没有收到确认说明？
         didn_t_receive_unlock_instructions: 没有收到解锁指令？
         forgot_your_password: 忘记密码？
-        sign_in: 登录
         sign_in_with_provider: 使用 %{provider} 登录
         sign_up: 注册
       minimum_password_length:
@@ -1385,6 +1383,10 @@ zh-CN:
     timestamp_error_message: 对不起，这太快了！请重新提交。
   layouts:
     decidim:
+      data_consent:
+        details:
+          columns:
+            type: 类型
       edit_link:
         edit: 编辑
       footer:
@@ -1410,8 +1412,9 @@ zh-CN:
         facebook: "%{organization} 在 Facebook"
         github: "%{organization} 在 GitHub"
         instagram: "%{organization} 在 Instagram"
-        twitter: "%{organization} 在 Twitter"
         youtube: "%{organization} 在 YouTube"
+      upload_modal:
+        close_modal: 关闭模式
       user_menu:
         account: '用户帐户: %{name}'
         admin_dashboard: 管理员控制面板

data/config/locales/zh-TW.yml

@@ -89,6 +89,20 @@ zh-TW:
       about_x_months:
         other: "one\n大約 1 月\n\nOther\n大約 %{count} 月"
       half_a_minute: 半分鐘
+      less_than_x_minutes:
+        other: 少於 %{count} 分鐘
+      less_than_x_seconds:
+        other: "< 1秒\n< %{count} 秒"
+      x_days:
+        other: "1天\n%{count} 天"
+      x_hours:
+        other: "{count} 小時"
+      x_minutes:
+        other: "%{count} 分鐘"
+      x_months:
+        other: "%{count} 個月"
+      x_seconds:
+        other: "%{count} 秒"
   decidim:
     accessibility:
       external_link: 外部鏈接
@@ -505,7 +519,6 @@ zh-TW:
           newsletter: 偶爾收到包含相關信息的時事通訊。
           newsletter_title: 聯絡人權限
           nickname_help: 在 %{organization} 中的別名。只能包含字母、數字、'-' 和 '_'。
-          sign_in: 登入
           sign_up: 註冊登記
           sign_up_as:
             legend: 註冊登記為
@@ -1194,7 +1207,6 @@ zh-TW:
           proposals: 提案
           proposals_explanation: 提出建議、支持現有建議並推動您希望看到的變化。
         footer_sub_hero:
-          footer_sub_hero_body: 一起建立更加開放、透明和協作的社會。<br /> 加入、參與和做出決策。
           footer_sub_hero_headline: 歡迎來到 %{organization} 的參與平台。
           register: 註冊
         hero:
@@ -1439,8 +1451,8 @@ zh-TW:
         conversations_info: 對話是私密的，沒有其他人可以看到。可以與平台上註冊的任何參與者或群組開始對話。
         from: 來自
         last_message: 最後訊息
-        modal_title: 新對話
-        new_conversation: 新對話
+        modal_title: 新的對話
+        new_conversation: 新的對話
         next: 下一個
         no_conversations_warning: 目前還沒有任何對話。
         no_results: 無結果
@@ -1624,8 +1636,6 @@ zh-TW:
       updated: 您的帳號已成功升級。
     sessions:
       already_signed_out: 已成功登出。
-      new:
-        sign_in: 登入
       signed_in: 已成功登入。
       signed_out: 已成功登出。
     shared:
@@ -1634,7 +1644,6 @@ zh-TW:
         didn_t_receive_confirmation_instructions: 沒有收到確認的說明嗎？
         didn_t_receive_unlock_instructions: 沒有收到解鎖說明？
         forgot_your_password: 忘記密碼了嗎？
-        sign_in: 登入
         sign_in_with_provider: 用%{provider} 登錄
         sign_up: 註冊登記
       minimum_password_length:
@@ -1686,7 +1695,7 @@ zh-TW:
             description: 說明
             name: 名稱
             service: 服務
-            type: 種類
+            type: 類別
           items:
             _session_id:
               description: 允許網站在使用者在網頁之間移動時記住使用者。
@@ -1759,7 +1768,6 @@ zh-TW:
         facebook: "%{organization} 在 Facebook"
         github: "%{organization} 在 GitHub"
         instagram: "%{organization} 在 Instagram"
-        twitter: "%{organization} 在 Twitter"
         youtube: "%{organization} 在 YouTube"
       timeout_modal:
         body: 您已經閒置了 %{minutes} 分鐘。如果您繼續閒置，為了您的安全，您將被自動登出。

data/db/migrate/20181030090144_destroy_deleted_users_follows.rb

@@ -10,7 +10,7 @@ class DestroyDeletedUsersFollows < ActiveRecord::Migration[5.2]
   end
 
   def change
-    deleted_users = Decidim::User.where.not(deleted_at: nil).pluck(:id)
+    deleted_users = User.where.not(deleted_at: nil).pluck(:id)
     Follow.where(decidim_followable_type: "Decidim::UserBaseEntity", decidim_followable_id: deleted_users).destroy_all
     Follow.where(decidim_user_id: deleted_users).destroy_all
   end

data/db/migrate/20181204110723_remove_following_users_count_from_users.rb

@@ -1,6 +1,15 @@
 # frozen_string_literal: true
 
 class RemoveFollowingUsersCountFromUsers < ActiveRecord::Migration[5.2]
+  class UserBaseEntity < ApplicationRecord
+    self.table_name = :decidim_users
+    self.inheritance_column = nil # disable the default inheritance
+  end
+
+  class Follow < ApplicationRecord
+    self.table_name = :decidim_follows
+  end
+
   def up
     remove_column :decidim_users, :following_users_count
   end
@@ -8,8 +17,8 @@ class RemoveFollowingUsersCountFromUsers < ActiveRecord::Migration[5.2]
   def down
     add_column :decidim_users, :following_users_count, :integer, null: false, default: 0
 
-    Decidim::UserBaseEntity.find_each do |entity|
-      following_users_count = Decidim::Follow.where(decidim_user_id: entity.id, decidim_followable_type: ["Decidim::UserBaseEntity", "Decidim::User", "Decidim::UserGroup"]).count
+    UserBaseEntity.find_each do |entity|
+      following_users_count = Follow.where(decidim_user_id: entity.id, decidim_followable_type: ["Decidim::UserBaseEntity", "Decidim::User", "Decidim::UserGroup"]).count
       entity.following_users_count = following_users_count
       entity.save
     end

data/db/migrate/20181214101250_add_notification_types_to_users.rb

@@ -1,10 +1,15 @@
 # frozen_string_literal: true
 
 class AddNotificationTypesToUsers < ActiveRecord::Migration[5.2]
+  class UserBaseEntity < ApplicationRecord
+    self.table_name = :decidim_users
+    self.inheritance_column = nil # disable the default inheritance
+  end
+
   def change
     add_column :decidim_users, :notification_types, :string, default: "all"
     # rubocop:disable Rails/SkipsModelValidations
-    Decidim::UserBaseEntity.update_all(notification_types: "all")
+    UserBaseEntity.update_all(notification_types: "all")
     # rubocop:enable Rails/SkipsModelValidations
 
     change_column_null :decidim_users, :notification_types, false

data/db/migrate/20190412131728_fix_user_names.rb

@@ -1,6 +1,13 @@
 # frozen_string_literal: true
 
 class FixUserNames < ActiveRecord::Migration[5.2]
+  class UserBaseEntity < ApplicationRecord
+    include Decidim::Nicknamizable
+
+    self.table_name = :decidim_users
+    self.inheritance_column = nil # disable the default inheritance
+  end
+
   def change
     # Comes from Decidim::User specs
     weird_characters =
@@ -8,7 +15,7 @@ class FixUserNames < ActiveRecord::Migration[5.2]
     characters_to_remove = "<>?%&^*\#@()[]=+:;\"{}\\|/"
 
     weird_characters.each do |character|
-      Decidim::UserBaseEntity.where(deleted_at: nil).where("name like '%#{character}%' escape '\' OR nickname like '%#{character}%' escape '\'").find_each do |entity|
+      UserBaseEntity.where(deleted_at: nil).where("name like '%#{character}%' escape '\' OR nickname like '%#{character}%' escape '\'").find_each do |entity|
         Rails.logger.debug { "detected character: #{character}" }
         Rails.logger.debug { "UserBaseEntity ID: #{entity.id}" }
         Rails.logger.debug { "#{entity.name} => #{entity.name.delete(characters_to_remove).strip}" }
@@ -16,7 +23,7 @@ class FixUserNames < ActiveRecord::Migration[5.2]
 
         entity.name = entity.name.delete(characters_to_remove).strip
         sanitized_nickname = entity.nickname.delete(characters_to_remove).strip
-        entity.nickname = Decidim::UserBaseEntity.nicknamize(sanitized_nickname, organization: entity.organization)
+        entity.nickname = UserBaseEntity.nicknamize(sanitized_nickname, organization: entity.organization)
         entity.save(validate: false)
       end
     end

data/db/migrate/20200211173227_add_direct_message_types_to_users.rb

@@ -1,10 +1,15 @@
 # frozen_string_literal: true
 
 class AddDirectMessageTypesToUsers < ActiveRecord::Migration[5.2]
+  class UserBaseEntity < ApplicationRecord
+    self.table_name = :decidim_users
+    self.inheritance_column = nil # disable the default inheritance
+  end
+
   def change
     add_column :decidim_users, :direct_message_types, :string, default: "all"
     # rubocop:disable Rails/SkipsModelValidations
-    Decidim::UserBaseEntity.update_all(direct_message_types: "all")
+    UserBaseEntity.update_all(direct_message_types: "all")
     # rubocop:enable Rails/SkipsModelValidations
 
     change_column_null :decidim_users, :direct_message_types, false

data/db/migrate/20210302150803_invalidate_all_sessions_for_deleted_users.rb

@@ -1,10 +1,17 @@
 # frozen_string_literal: true
 
 class InvalidateAllSessionsForDeletedUsers < ActiveRecord::Migration[5.2]
-  def up
-    Decidim::User.reset_column_information
+  class User < ApplicationRecord
+    self.table_name = "decidim_users"
+    self.inheritance_column = nil # disable the default inheritance
+
+    default_scope { where(type: "Decidim::User") }
+  end
 
-    Decidim::User.where.not(deleted_at: nil).find_each(&:invalidate_all_sessions!)
+  def up
+    User.where.not(deleted_at: nil).find_each do |user|
+      user.update!(session_token: SecureRandom.hex)
+    end
   end
 
   def down; end

data/db/migrate/20210310120640_add_followable_counter_cache_to_users.rb

@@ -1,14 +1,24 @@
 # frozen_string_literal: true
 
 class AddFollowableCounterCacheToUsers < ActiveRecord::Migration[5.2]
+  class Follow < ApplicationRecord
+    self.table_name = "decidim_follows"
+  end
+
+  class User < ApplicationRecord
+    self.table_name = "decidim_users"
+    self.inheritance_column = nil # disable the default inheritance
+
+    default_scope { where(type: "Decidim::User") }
+  end
+
   def change
     add_column :decidim_users, :follows_count, :integer, null: false, default: 0, index: true
 
     reversible do |dir|
       dir.up do
-        Decidim::User.reset_column_information
-        Decidim::User.find_each do |record|
-          record.class.reset_counters(record.id, :follows)
+        User.find_each do |record|
+          record.update!(follows_count: Follow.where(decidim_user_id: record.id).count)
         end
       end
     end

data/db/migrate/20231027142329_change_default_value_for_decidim_endorsements.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class ChangeDefaultValueForDecidimEndorsements < ActiveRecord::Migration[6.1]
+  def up
+    change_column_default :decidim_endorsements, :decidim_user_group_id, 0
+  end
+
+  def down
+    change_column_default :decidim_endorsements, :decidim_user_group_id, nil
+  end
+end

data/db/seeds.rb

@@ -37,6 +37,7 @@ if !Rails.env.production? || ENV.fetch("SEED", nil)
       port: ENV.fetch("SMTP_PORT", nil) || ENV.fetch("DECIDIM_SMTP_PORT", "25")
     },
     host: ENV.fetch("DECIDIM_HOST", "localhost"),
+    secondary_hosts: ENV.fetch("DECIDIM_HOST", "localhost") == "localhost" ? ["0.0.0.0", "127.0.0.1"] : nil,
     external_domain_whitelist: ["decidim.org", "github.com"],
     description: Decidim::Faker::Localized.wrapped("<p>", "</p>") do
       Decidim::Faker::Localized.sentence(word_count: 15)
@@ -125,7 +126,8 @@ if !Rails.env.production? || ENV.fetch("SEED", nil)
     tos_agreement: true,
     personal_url: Faker::Internet.url,
     about: Faker::Lorem.paragraph(sentence_count: 2),
-    accepted_tos_version: organization.tos_version,
+    accepted_tos_version: organization.tos_version + 1.hour,
+    password_updated_at: Time.current,
     admin_terms_accepted_at: Time.current
   }
   admin_hash.merge!(password: "decidim123456789", password_confirmation: "decidim123456789") if admin.encrypted_password.blank?
@@ -143,7 +145,7 @@ if !Rails.env.production? || ENV.fetch("SEED", nil)
       tos_agreement: true,
       personal_url: Faker::Internet.url,
       about: Faker::Lorem.paragraph(sentence_count: 2),
-      accepted_tos_version: organization.tos_version
+      accepted_tos_version: organization.tos_version + 1.hour
     )
   end
 
@@ -162,7 +164,7 @@ if !Rails.env.production? || ENV.fetch("SEED", nil)
     tos_agreement: true,
     personal_url: Faker::Internet.url,
     about: Faker::Lorem.paragraph(sentence_count: 2),
-    accepted_tos_version: organization.tos_version
+    accepted_tos_version: organization.tos_version + 1.hour
   )
 
   locked_user.lock_access!

data/lib/decidim/core/test/factories.rb

@@ -364,7 +364,8 @@ FactoryBot.define do
     published_at { Time.current }
     settings do
       {
-        dummy_global_translatable_text: generate_localized_title
+        dummy_global_translatable_text: generate_localized_title,
+        comments_max_length: participatory_space.organization.comments_max_length || organization.comments_max_length
       }
     end
 

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

@@ -838,5 +838,29 @@ shared_examples "comments" do
         expect(page).to have_link "#decidim", href: "/search?term=%23decidim"
       end
     end
+
+    describe "export_serializer" do
+      let(:comment) { comments.first }
+
+      it "returns the serializer for the comment" do
+        expect(comment.class.export_serializer).to eq(Decidim::Comments::CommentSerializer)
+      end
+
+      context "with instance" do
+        subject { comment.class.export_serializer.new(comment).serialize }
+
+        it { is_expected.to have_key(:id) }
+        it { is_expected.to have_key(:created_at) }
+        it { is_expected.to have_key(:body) }
+        it { is_expected.to have_key(:locale) }
+        it { is_expected.to have_key(:author) }
+        it { is_expected.to have_key(:alignment) }
+        it { is_expected.to have_key(:depth) }
+        it { is_expected.to have_key(:user_group) }
+        it { is_expected.to have_key(:commentable_id) }
+        it { is_expected.to have_key(:commentable_type) }
+        it { is_expected.to have_key(:root_commentable_url) }
+      end
+    end
   end
 end

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

@@ -18,6 +18,8 @@ shared_examples_for "a 404 page" do
       expect(page).to have_content("The page you're looking for can't be found")
 
       expect(page).to have_http_status(:not_found)
+
+      expect(page).to have_current_path(target_path)
     end
   end
 end

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

@@ -10,7 +10,9 @@ shared_context "with map utility" do
 end
 
 shared_context "with frontend map builder" do
-  subject { described_class.new(template, options) }
+  subject { builder_class.new(template, options) }
+
+  let(:builder_class) { described_class }
 
   let(:template_class) do
     Class.new(ActionView::Base) do
@@ -102,6 +104,7 @@ shared_context "with frontend map elements" do
     # context.
     final_html = html_document
     Rails.application.routes.draw do
+      get "maptiles/:z/:x/:y.png", to: ->(_) { [200, {}, [final_html]] }
       get "test_dynamic_map", to: ->(_) { [200, {}, [final_html]] }
       get "offline", to: ->(_) { [200, {}, [""]] }
     end

data/lib/decidim/core/version.rb

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

data/lib/decidim/core.rb

@@ -118,6 +118,12 @@ module Decidim
   include ActiveSupport::Configurable
   # Loads seeds from all engines.
   def self.seed!
+    # After running the migrations, some records may have loaded their column
+    # caches at different stages of the migration process, so in order to
+    # prevent any "undefined method" errors if these tasks are run
+    # consecutively, reset the column cache before the migrations.
+    reset_all_column_information
+
     # Faker needs to have the `:en` locale in order to work properly, so we
     # must enforce it during the seeds.
     original_locale = I18n.available_locales
@@ -157,6 +163,17 @@ module Decidim
     I18n.available_locales = original_locale
   end
 
+  # Finds all currently loaded Decidim ActiveRecord classes and resets their
+  # column information.
+  def self.reset_all_column_information
+    ActiveRecord::Base.descendants.each do |cls|
+      next if cls.name.nil? # abstract classes registered during tests
+      next if cls.abstract_class? || !cls.name.match?(/^Decidim::/)
+
+      cls.reset_column_information
+    end
+  end
+
   # Exposes a configuration option: The application name String.
   config_accessor :application_name
 
@@ -568,7 +585,7 @@ module Decidim
   #
   # Returns an Array[ComponentManifest].
   def self.component_manifests
-    component_registry.manifests
+    component_registry.manifests.sort_by(&:name)
   end
 
   # Public: Finds all registered participatory space manifest's via the

data/lib/decidim/endorsable.rb

@@ -21,7 +21,7 @@ module Decidim
         if user_group
           endorsements.where(user_group: user_group).any?
         else
-          endorsements.where(author: user, user_group: nil).any?
+          endorsements.where(author: user, user_group: 0).any?
         end
       end
     end

data/lib/decidim/exporters.rb

@@ -9,6 +9,11 @@ module Decidim
     autoload :PDF, "decidim/exporters/pdf"
     autoload :ExportData, "decidim/exporters/export_data"
 
+    # Lock the export formats to one of the available exporters
+    EXPORT_FORMATS = [:JSON, :CSV, :Excel, :PDF, :FormPDF].freeze
+
+    class UnknownFormatError < StandardError; end
+
     # Necessary for the i18n normalizer to locate strings not directly invoked in views:
 
     # i18n-tasks-use t('decidim.admin.exports.formats.JSON')
@@ -19,7 +24,11 @@ module Decidim
     #
     # format - The exporter format as a string. i.e "CSV"
     def self.find_exporter(format)
-      const_get(format)
+      raise UnknownFormatError unless format.respond_to?(:to_sym)
+      raise UnknownFormatError unless EXPORT_FORMATS.include?(format.to_sym)
+      raise UnknownFormatError unless const_defined?(format.to_sym)
+
+      const_get(format.to_sym)
     end
   end
 end

data/lib/decidim/form_builder.rb

@@ -181,7 +181,6 @@ module Decidim
     #                      or 'full' (optional) (default: 'basic')
     #           :lines - The Integer to indicate how many lines should editor have (optional) (default: 10)
     #           :disabled - Whether the editor should be disabled
-    #           :editor_images - Allow attached images (optional) (default: false)
     #
     # Renders a container with both hidden field and editor container
     def editor(name, options = {})
@@ -199,9 +198,12 @@ module Decidim
         template += label(name, label_text + required_for_attribute(name)) if options.fetch(:label, true)
         template += hidden_field(name, hidden_options)
         template += content_tag(:div, nil, class: "editor-container #{"js-hashtags" if hashtaggable}", data: {
-          toolbar: toolbar,
-          disabled: options[:disabled]
-        }.merge(editor_images_options(options)), style: "height: #{lines}rem")
+                                  toolbar: toolbar,
+                                  disabled: options[:disabled],
+                                  editor_images: true,
+                                  upload_images_path: Decidim::Core::Engine.routes.url_helpers.editor_images_path,
+                                  drag_and_drop_help_text: I18n.t("drag_and_drop_help", scope: "decidim.editor_images")
+                                }, style: "height: #{lines}rem")
         template += error_for(name, options) if error?(name)
         template.html_safe
       end
@@ -315,6 +317,7 @@ module Decidim
       template = ""
       template += "<label>#{label_for(attribute) + required_for_attribute(attribute)}</label>" unless options[:label] == false
       template += @template.render("decidim/scopes/scopes_picker_input",
+                                   values_options: { delete_button: true, class: "label primary" },
                                    picker_options: picker_options,
                                    prompt_params: prompt_params,
                                    scopes: scopes,
@@ -926,15 +929,6 @@ module Decidim
       end
     end
 
-    def editor_images_options(options)
-      return {} unless options[:editor_images]
-
-      {
-        upload_images_path: Decidim::Core::Engine.routes.url_helpers.editor_images_path,
-        drag_and_drop_help_text: I18n.t("drag_and_drop_help", scope: "decidim.editor_images")
-      }
-    end
-
     # Private: Determines the correct resource class for validators from the
     # object or its PassthruValidator.
     def resource_class(attribute)

data/lib/decidim/has_category.rb

@@ -26,11 +26,11 @@ module Decidim
         cat_ids = parent_ids.dup
         cat_ids.prepend(nil) if categories.include?("without")
 
-        subquery = includes(:category).where(decidim_categories: { id: cat_ids })
+        subquery = left_outer_joins(:category).where(decidim_categories: { id: cat_ids })
         return subquery if parent_ids.none?
 
         subquery.or(
-          includes(:category).where(decidim_categories: { parent_id: parent_ids })
+          left_outer_joins(:category).where(decidim_categories: { parent_id: parent_ids })
         )
       }