decidim-core 0.31.7 → 0.32.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cdd2ee47aebc30f3404fc92bd9b5954c1dfae6cdadbbd19d28d495f7939ce3d8
-  data.tar.gz: 150625ae419f2acc7f6c8068ef743c7249cee604a36a14965917e940ec5b393a
+  metadata.gz: caaaf7bbddd64f7bb5d4341c6ebeab59b35dd0f69326b136d11958165e513801
+  data.tar.gz: e18600f3eec00c3fbf76b58e90a34bf4888a114f7dc5a9d35896540d30a7a20f
 SHA512:
-  metadata.gz: c8eb46226c485aa8e26c65b6a35213d73ba3928a47f4e8262816b30be044252cbc4ecc8be8bf70377e0787e5f50eb2d848fde571c26ddb0f33ee62068a3e9d0d
-  data.tar.gz: 012df3885eb89cc74df00a64f599cae1918d4ab9d40675b1444958dbf6ba07694a3770cdc236af3628856c9e97dfc5b809fbed1ecb475fbbb93952a4067d72ee
+  metadata.gz: 847ec9bd0ae30f358fd5330e3bf61cd34adedca669986b5082e391473e3f66df4e7eab20bac8ad4478827fd66b2d3e94e46559ef4138cf289d5a71caf60e904b
+  data.tar.gz: b832cfbd8827d809a1792ff0f7a6688f2282e3f2bfc8975bf821175465333ade87ef0e2fd92bf9ac3512f54e40d31872d2f35a28424b8549de717e2f645141b3

data/README.md

@@ -1,24 +1,14 @@
-# Decidim
+# decidim-core
 
 Core functionality in Decidim. Every single decidim functionality depends on this gem.
 
 ## Usage
 
-You will be using indirectly on any decidim application.
+This module will be using indirectly on any decidim application.
 
 ## Installation
 
-Add `decidim` to your `Gemfile` and you will be using it:
-
-```ruby
-gem 'decidim'
-```
-
-And then execute:
-
-```bash
-bundle
-```
+This is on the default Decidim installation so you should not change anything to use this feature.
 
 ## Users
 

data/app/cells/decidim/amendable/amend_button_card/show.erb

@@ -1,5 +1,5 @@
-<li role="presentation" class="dropdown__item">
-  <%= action_authorized_link_to :amend, new_amend_path, resource: model, data: { "redirect_url" => new_amend_path }, id: "amend-button", class: "dropdown__button", role: "menuitem" do %>
+<li role="menuitem" class="dropdown__item">
+  <%= action_authorized_link_to :amend, new_amend_path, resource: model, data: { "redirect_url" => new_amend_path }, id: "amend-button", class: "dropdown__button" do %>
     <span><%= t("button", scope: "decidim.amendments.amendable", model_name: nil) %></span>
     <%= icon "pencil-line" %>
   <% end %>

data/app/cells/decidim/amendable/amend_button_card_cell.rb

@@ -7,7 +7,7 @@ module Decidim
       def show
         return unless amendments_enabled? && model.amendable?
         return unless current_component.current_settings.amendment_creation_enabled
-        return unless can_participate_in_private_space?
+        return unless can_participate_in_restricted_space?
 
         render
       end

data/app/cells/decidim/author_cell.rb

@@ -31,6 +31,10 @@ module Decidim
       render unless current_user == model
     end
 
+    def perform_caching?
+      true
+    end
+
     def raw_model
       model.try(:__getobj__) || model
     end

data/app/cells/decidim/card_metadata_cell.rb

@@ -29,7 +29,7 @@ module Decidim
       return unless show_space?
 
       {
-        text: translated_attribute(participatory_space.title),
+        text: decidim_escape_translated(participatory_space.title),
         icon: resource_type_icon_key(participatory_space.class),
         url: Decidim::ResourceLocatorPresenter.new(participatory_space).path
       }

data/app/cells/decidim/comments_button_cell.rb

@@ -23,9 +23,7 @@ module Decidim
     end
 
     def path
-      return "#add-comment-anchor" if current_user
-
-      nil
+      "#add-comment-anchor"
     end
 
     def text
@@ -39,11 +37,5 @@ module Decidim
     def button_classes
       "button button__sm button__transparent-secondary add-comment-mobile"
     end
-
-    def html_options
-      return {} if current_user
-
-      { data: { "dialog-open": "loginModal" } }
-    end
   end
 end

data/app/cells/decidim/content_blocks/announcement_settings_form/show.erb

@@ -0,0 +1,5 @@
+<% form.fields_for :settings, form.object.settings do |settings_fields| %>
+  <div class="row column">
+    <%= settings_fields.translated :editor, :announcement, label:, required: true %>
+  </div>
+<% end %>

data/app/cells/decidim/content_blocks/announcement_settings_form_cell.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+module Decidim
+  module ContentBlocks
+    class AnnouncementSettingsFormCell < Decidim::ViewModel
+      alias form model
+
+      def content_block
+        options[:content_block]
+      end
+
+      def label
+        I18n.t("decidim.content_blocks.announcement.body")
+      end
+    end
+  end
+end

data/app/cells/decidim/content_blocks/cta_settings_form/show.erb

@@ -4,6 +4,6 @@
   <%= settings_fields.translated :text_field, :description, label: t("decidim.content_blocks.cta_settings_form.description") %>
 <% end %>
 
-<% form.fields_for :images, content_block.images_container do |images_fields| %>
+<% form.fields_for :images, form.object.images do |images_fields| %>
   <%= images_fields.upload :background_image, label: t("decidim.content_blocks.cta_settings_form.background_image") %>
 <% end %>

data/app/cells/decidim/content_blocks/hero_settings_form/show.erb

@@ -5,7 +5,7 @@
 
   <div class="row column" data-controller="slug">
     <%= settings_fields.translated :text_field, :cta_button_path, label: t(".cta_button_path") %>
-    <p class="help-text"><%= t(".cta_button_path_help", url: decidim_form_slug_url("", translated_attribute(form.object.settings["cta_button_path"]))) %></p>
+    <p class="help-text"><%= t(".cta_button_path_help", url: decidim_form_slug_url("", translated_attribute(form.object.settings.cta_button_path))) %></p>
   </div>
 
   <div class="row column">
@@ -14,7 +14,7 @@
   </div>
 <% end %>
 
-<% form.fields_for :images, content_block.images_container do |images_fields| %>
+<% form.fields_for :images, form.object.images do |images_fields| %>
   <div class="row column">
     <%= images_fields.upload :background_image, label: t(".background_image"), button_class: "button button__sm button__transparent-secondary" %>
   </div>

data/app/cells/decidim/content_blocks/highlighted_content_banner_cell.rb

@@ -32,7 +32,7 @@ module Decidim
       private
 
       def render?
-        required_keys = [:title, :action_button_title, :action_button_subtitle, :action_button_url]
+        required_keys = [:title, :short_description, :action_button_title, :action_button_subtitle, :action_button_url]
         required_keys.all? { |key| model.settings.public_send(key).present? }
       end
     end

data/app/cells/decidim/content_blocks/highlighted_content_banner_settings_form/show.erb

@@ -20,7 +20,7 @@
   </div>
 <% end %>
 
-<% form.fields_for :images, content_block.images_container do |images_fields| %>
+<% form.fields_for :images, form.object.images do |images_fields| %>
   <div class="row column">
     <%= images_fields.upload :background_image, label: t(".background_image"), button_class: "button button__sm button__transparent-secondary" %>
   </div>

data/app/cells/decidim/content_blocks/highlighted_participatory_spaces_cell.rb

@@ -28,7 +28,7 @@ module Decidim
       private
 
       def cache_hash
-        [I18n.locale, model.cache_key_with_version, highlighted_spaces.map(&:cache_key_with_version)].join(Decidim.cache_key_separator)
+        [I18n.locale, highlighted_spaces.map(&:cache_key_with_version)].join(Decidim.cache_key_separator)
       end
 
       def section_class

data/app/cells/decidim/content_blocks/how_to_participate/show.erb

@@ -37,7 +37,7 @@
       </div>
     </div>
   </div>
-  <%= link_to decidim.pages_path, class: "button button__sm md:button__lg button__secondary mt-6" do %>
+  <%= link_to decidim.pages_path(), class: "button button__sm md:button__lg button__secondary mt-6" do %>
     <%= t("more_info", scope: "decidim.pages.home.extended", resource_name: current_organization_name) %>
     <%= icon "arrow-right-line", class: "fill-current" %>
   <% end %>

data/app/cells/decidim/content_blocks/html_cell.rb

@@ -8,7 +8,7 @@ module Decidim
       end
 
       def html_content
-        decidim_sanitize_editor_admin(translated_attribute(model.settings.html_content))
+        translated_attribute(model.settings.html_content).html_safe
       end
     end
   end

data/app/cells/decidim/content_blocks/participatory_space_announcement_cell.rb

@@ -3,8 +3,6 @@
 module Decidim
   module ContentBlocks
     class ParticipatorySpaceAnnouncementCell < BaseCell
-      delegate :announcement, to: :resource
-
       def show
         return if announcement_cell.blank_content?
 
@@ -12,7 +10,7 @@ module Decidim
       end
 
       def announcement_cell
-        @announcement_cell ||= cell("decidim/announcement", announcement)
+        @announcement_cell ||= cell("decidim/announcement", model.settings.announcement)
       end
     end
   end

data/app/cells/decidim/content_blocks/participatory_space_hero_cell.rb

@@ -27,12 +27,8 @@ module Decidim
         decidim_escape_translated(resource.subtitle)
       end
 
-      # If it is called from the landing page content block, use the background image defined there
-      # Else, use the banner image defined in the space (for assemblies)
       def image_path
-        return model.images_container.attached_uploader(:background_image).url if model.respond_to?(:images_container)
-
-        attached_uploader(:banner_image).url
+        model.images_container.attached_uploader(:background_image).url
       end
 
       def has_cta?

data/app/cells/decidim/content_blocks/participatory_space_hero_settings_form/show.erb

@@ -8,7 +8,7 @@
   </div>
 <% end %>
 
-<% form.fields_for :images, content_block.images_container do |images_fields| %>
+<% form.fields_for :images, form.object.images do |images_fields| %>
   <div class="row column">
     <%= images_fields.upload :background_image, label: t("decidim.content_blocks.cta_settings_form.background_image"), button_class: "button button__sm button__transparent-secondary" %>
   </div>

data/app/cells/decidim/content_blocks/static_page/section_cell.rb

@@ -5,7 +5,7 @@ module Decidim
     module StaticPage
       class SectionCell < Decidim::ViewModel
         def content
-          decidim_sanitize_editor_admin(translated_attribute(model.settings.content))
+          translated_attribute(model.settings.content).html_safe
         end
       end
     end

data/app/cells/decidim/content_blocks/static_page/section_settings_form/show.erb

@@ -1,5 +1,3 @@
 <% form.fields_for :settings, form.object.settings do |settings_fields| %>
-  <div class="row column">
-    <%= settings_fields.translated :editor, :content, label: %>
-  </div>
+  <%= settings_fields.translated :editor, :content, label: %>
 <% end %>

data/app/cells/decidim/content_blocks/static_page/summary_cell.rb

@@ -5,7 +5,7 @@ module Decidim
     module StaticPage
       class SummaryCell < Decidim::ViewModel
         def content
-          decidim_sanitize_editor_admin(translated_attribute(model.settings.summary))
+          translated_attribute(model.settings.summary).html_safe
         end
       end
     end

data/app/cells/decidim/content_blocks/static_page/summary_settings_form/show.erb

@@ -1,5 +1,3 @@
 <% form.fields_for :settings, form.object.settings do |settings_fields| %>
-  <div class="row column">
-    <%= settings_fields.translated :editor, :summary, label: %>
-  </div>
+  <%= settings_fields.translated :editor, :summary, label: %>
 <% end %>

data/app/cells/decidim/content_blocks/static_page/two_pane_section_cell.rb

@@ -5,11 +5,11 @@ module Decidim
     module StaticPage
       class TwoPaneSectionCell < Decidim::ViewModel
         def left_column
-          decidim_sanitize_editor_admin(translated_attribute(model.settings.left_column))
+          translated_attribute(model.settings.left_column).html_safe
         end
 
         def right_column
-          decidim_sanitize_editor_admin(translated_attribute(model.settings.right_column))
+          translated_attribute(model.settings.right_column).html_safe
         end
       end
     end

data/app/cells/decidim/content_blocks/static_page/two_pane_section_settings_form/show.erb

@@ -1,6 +1,4 @@
 <% form.fields_for :settings, form.object.settings do |settings_fields| %>
-  <div class="row column">
-    <%= settings_fields.translated :editor, :left_column, label: label_left_column %>
-    <%= settings_fields.translated :editor, :right_column, label: label_right_column %>
-  </div>
+  <%= settings_fields.translated :editor, :left_column, label: label_left_column %>
+  <%= settings_fields.translated :editor, :right_column, label: label_right_column %>
 <% end %>

data/app/cells/decidim/data_consent/category.erb

@@ -13,12 +13,12 @@
       <%= icon "close-line", class: "cookies__category-toggle-icon" %>
     </label>
 
-    <h3 id="accordion-title-<%= category[:slug] %>" class="cookies__category-trigger-title">
-      <%= category[:title] %>
-    </h3>
-
     <div id="accordion-trigger-<%= category[:slug] %>" role="group" data-controls="accordion-panel-<%= category[:slug] %>" aria-labelledby="accordion-title-<%= category[:slug] %>">
-      <span aria-hidden="true">
+      <h3 id="accordion-title-<%= category[:slug] %>" class="cookies__category-trigger-title">
+        <%= category[:title] %>
+      </h3>
+
+      <span>
         <%= icon "arrow-down-s-line", class: "cookies__category-trigger-arrow" %>
         <%= icon "arrow-up-s-line", class: "cookies__category-trigger-arrow" %>
       </span>
@@ -30,26 +30,39 @@
     <p><%= category[:description] %></p>
 
     <% if category[:items].present? %>
-    <table>
-      <thead>
-        <tr>
-          <th><%= t("layouts.decidim.data_consent.details.columns.type") %></th>
-          <th><%= t("layouts.decidim.data_consent.details.columns.name") %></th>
-          <th><%= t("layouts.decidim.data_consent.details.columns.service") %></th>
-          <th><%= t("layouts.decidim.data_consent.details.columns.description") %></th>
-        </tr>
-      </thead>
-      <tbody>
-        <% category[:items].each do |item| %>
-          <tr>
-            <td><%= t("layouts.decidim.data_consent.details.types.#{item[:type]}") %></td>
-            <td><%= item[:name] %></td>
-            <td><%= t("layouts.decidim.data_consent.details.items.#{item[:name]}.service") %></td>
-            <td><%= t("layouts.decidim.data_consent.details.items.#{item[:name]}.description") %></td>
-          </tr>
-        <% end %>
-      </tbody>
-    </table>
+    <div>
+      <div class="cookies__category-panel__tr">
+        <div class="cookies__category-panel__th">
+          <%= t("layouts.decidim.data_consent.details.columns.type") %>
+        </div>
+        <div class="cookies__category-panel__th">
+          <%= t("layouts.decidim.data_consent.details.columns.name") %>
+        </div>
+        <div class="cookies__category-panel__th">
+          <%= t("layouts.decidim.data_consent.details.columns.service") %>
+        </div>
+        <div class="cookies__category-panel__th">
+          <%= t("layouts.decidim.data_consent.details.columns.description") %>
+        </div>
+      </div>
+
+      <% category[:items].each do |item| %>
+        <div class="cookies__category-panel__tr">
+          <div class="cookies__category-panel__td">
+            <%= t("layouts.decidim.data_consent.details.types.#{item[:type]}") %>
+          </div>
+          <div class="cookies__category-panel__td">
+            <%= item[:name] %>
+          </div>
+          <div class="cookies__category-panel__td">
+            <%= t("layouts.decidim.data_consent.details.items.#{item[:name]}.service") %>
+          </div>
+          <div class="cookies__category-panel__td">
+            <%= t("layouts.decidim.data_consent.details.items.#{item[:name]}.description") %>
+          </div>
+        </div>
+      <% end %>
+    </div>
     <% end %>
   </div>
 </div>

data/app/cells/decidim/footer_topics/show.erb

@@ -1,5 +1,8 @@
-<ul>
-  <% topics.each do |topic| %>
-    <%= topic_item(topic, class: "font-semibold underline") %>
-  <% end %>
-</ul>
+<nav role="navigation" aria-label="<%= t("layouts.decidim.footer.help") %>">
+  <h2 class="h4 mb-4"><%= t("layouts.decidim.footer.help") %></h2>
+  <ul class="space-y-4 break-inside-avoid">
+    <% topics.each do |topic| %>
+      <%= topic_item(topic, class: "font-semibold underline") %>
+    <% end %>
+  </ul>
+</nav>

data/app/cells/decidim/{participatory_space_private_user_cell.rb → member_cell.rb}

@@ -2,7 +2,7 @@
 
 module Decidim
   # This cell renders the card for an instance of an Assembly member
-  class ParticipatorySpacePrivateUserCell < Decidim::ViewModel
+  class MemberCell < Decidim::ViewModel
     property :name
     property :role
     property :nickname

data/app/cells/decidim/onboarding_action_message_cell.rb

@@ -8,6 +8,7 @@ module Decidim
   #
   class OnboardingActionMessageCell < Decidim::ViewModel
     include ActiveLinkTo
+    include Decidim::ActiveLinkToHelper
 
     alias user model
 

data/app/cells/decidim/profile/details.erb

@@ -20,4 +20,4 @@
   </div>
 </div>
 <p class="profile__description"><%= description %></p>
-<%= render_hook(:user_profile_bottom) %>
+<%= cell "decidim/profile_member_of", model %>

data/app/cells/decidim/profile_cell.rb

@@ -7,6 +7,7 @@ module Decidim
     include Decidim::AriaSelectedLinkToHelper
     include Decidim::ViewHooksHelper
     include ActiveLinkTo
+    include Decidim::ActiveLinkToHelper
 
     delegate :current_organization, :current_user, to: :controller
     delegate :avatar_url, :nickname, :personal_url, :followers_count, :users_followings, :officialized_as, to: :presented_profile

data/app/cells/decidim/profile_member_of/show.erb

@@ -0,0 +1,13 @@
+<div class="profile__description">
+  <div class="font-semibold"><%= t("member_of", scope: "decidim.participatory_space.pages.user_profile.member_of") %></div>
+  <% shown = false %>
+  <% sorted_types.each do |type| %>
+    <% spaces = grouped_members[type].map(&:participatory_space).sort_by { |s| translated_attribute(s.title) } %>
+    <% spaces.each do |space| %>
+      <%= "&nbsp;&#183;&nbsp;".html_safe if shown %>
+      <% path = Decidim::EngineRouter.main_proxy(space).public_send("#{type.demodulize.underscore}_path", space) %>
+      <%= link_to decidim_escape_translated(space.title), path, class: "hover:underline" %>
+      <% shown = true %>
+    <% end %>
+  <% end %>
+</div>

data/app/cells/decidim/profile_member_of_cell.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Decidim
+  class ProfileMemberOfCell < Decidim::ViewModel
+    include Decidim::Core::Engine.routes.url_helpers
+
+    def members
+      @members ||= Decidim::ParticipatorySpace::Member.where(user:)
+                                                      .includes(:participatory_space)
+                                                      .published
+                                                      .select { |m| m.participatory_space.present? && m.participatory_space.published? }
+                                                      .sort_by { |m| translated_attribute(m.participatory_space.title) }
+    end
+
+    def show
+      return unless members.any?
+
+      render
+    end
+
+    private
+
+    def user
+      model
+    end
+
+    def grouped_members
+      @grouped_members ||= members.group_by(&:participatory_space_type)
+    end
+
+    def sorted_types
+      @sorted_types ||= grouped_members.keys.sort_by { |k| k.split("::").last.downcase }
+    end
+  end
+end

data/app/cells/decidim/resource_types_filter/show.erb

@@ -10,8 +10,8 @@
   </button>
   <ul id="dropdown-menu-resource">
     <% resource_types.each do |resource_type| %>
-      <li role="presentation">
-        <%= link_to filter_url(resource_type[0]), class: "filter#{" is-active" if filter_param == resource_type[0]}", role: "menuitem" do %>
+      <li role="menuitem">
+        <%= link_to filter_url(resource_type[0]), class: "filter#{" is-active" if filter_param == resource_type[0]}" do %>
           <span class="sr-only"><%= resource_type[1] %></span>
           <%= text_with_resource_icon(*resource_type) %>
         <% end %>

data/app/cells/decidim/share_widget/qr_code_modal.erb

@@ -2,7 +2,7 @@
   <div data-dialog-container>
     <div data-dialog-title>
       <%= icon "qr-code-line" %>
-      <h2 id="dialog-title-qr-code-dialog" tabindex="-1"><%= t("decidim.share_widget.qr_code") %></h2>
+      <h2 tabindex="-1"><%= t("decidim.share_widget.qr_code") %></h2>
     </div>
     <div class="content">
       <div>

data/app/cells/decidim/tags_cell.rb

@@ -25,7 +25,7 @@ module Decidim
       @taxonomies ||= model.taxonomies.map do |taxonomy|
         {
           name: decidim_sanitize_translated(taxonomy.name),
-          url: resource_locator(model).index(filter: { "with_any_taxonomies[#{taxonomy.root_taxonomy.id}]" => [taxonomy.id.to_s] })
+          url: resource_locator(model).index(filter: { "with_any_taxonomies" => { taxonomy.root_taxonomy.id => [taxonomy.id.to_s] } })
         }
       end
     end

data/app/cells/decidim/upload_modal_cell.rb

@@ -177,11 +177,6 @@ module Decidim
 
     def file_attachment_path(attachment)
       return unless attachment
-
-      if attachment.respond_to?(:record) && attachment.record.is_a?(Decidim::Authorization) && attachment.name.to_s == "verification_attachment"
-        return decidim.private_download_path(Decidim::PrivateDownload.for(attachment.record, attachment_name: attachment.name).token)
-      end
-
       return Rails.application.routes.url_helpers.rails_blob_url(attachment, only_path: true) if attachment.is_a? ActiveStorage::Blob
 
       if attachment.try(:attached?)

data/app/cells/decidim/versions_list/show.erb

@@ -1,3 +1,28 @@
 <% reverse_ordered_versions.each_with_index do |version, index| %>
   <%= cell("decidim/versions_list_item", version, index:, versioned_resource:, version_path:, i18n_scope:, total:) %>
 <% end %>
+
+<% if versioned_resource.class.include?(Decidim::Fingerprintable) %>
+  <% fingerprint_id = dom_id(versioned_resource, :fingerprint_dialog) %>
+  <section class="mt-4 ml-2">
+    <%= content_tag :button, t("decidim.fingerprint.check"), class: "button button__lg button__text-secondary font-normal text-sm underline", data: { dialog_open: fingerprint_id } %>
+  </section>
+  <%= decidim_modal id: fingerprint_id, class: "fingerprint-modal" do %>
+    <div data-dialog-container>
+      <%= icon "fingerprint-line" %>
+      <h2 id="dialog-title-<%= fingerprint_id %>" tabindex="-1" data-dialog-title><%= t "decidim.fingerprint.title" %></h2>
+      <div class="fingerprint-modal__container">
+        <p id="dialog-desc-<%= fingerprint_id %>"><%= t "decidim.fingerprint.explanation" %></p>
+        <div>
+          <span class="fingerprint-modal__span"><%= t "decidim.fingerprint.value" %>:</span>
+          <code class="fingerprint-modal__code"><%= decidim_html_escape versioned_resource.fingerprint.value %></code>
+        </div>
+        <div>
+          <span class="fingerprint-modal__span"><%= t "decidim.fingerprint.source" %>:</span>
+          <code class="fingerprint-modal__code"><%= decidim_html_escape versioned_resource.fingerprint.source %></code>
+        </div>
+        <p><%= t("decidim.fingerprint.replicate_help", online_calculator_link: link_to(t("decidim.fingerprint.online_calculator_name"), "https://www.md5calc.com/sha256", target: "_blank", rel: "noopener")).html_safe %></p>
+      </div>
+    </div>
+  <% end %>
+<% end %>

data/app/commands/decidim/create_follow.rb

@@ -20,6 +20,7 @@ module Decidim
     # Returns nothing.
     def call
       return broadcast(:invalid) if form.invalid?
+      return broadcast(:invalid) unless current_user.is_a?(Decidim::User)
 
       create_follow!
       increment_score
@@ -32,13 +33,10 @@ module Decidim
     attr_reader :follow, :form
 
     def create_follow!
-      @follow = Follow.find_by(
+      @follow = Follow.where(
         followable: form.followable,
         user: current_user
-      ) || Follow.create!(
-        followable: form.followable,
-        user: current_user
-      )
+      ).first_or_create!
     end
 
     def increment_score

data/app/commands/decidim/create_user_report.rb

@@ -58,7 +58,7 @@ module Decidim
       current_user.organization.admins.each do |admin|
         next unless admin.email_on_moderations
 
-        Decidim::UserReportJob.perform_later(admin, report)
+        UserReportMailer.notify(admin, report).deliver_later
       end
     end