decidim-core 0.28.1 → 0.29.0.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 800e096eb86537caa7de18236fe0becf73b1c30ee38e4631888025b52e831f3a
-  data.tar.gz: 9f05dfffcd55fcc9d4e76163a4b98c5b7e851bbdbcfe317f9a9566cb73a142d9
+  metadata.gz: b9d7e02680508906160916e60c9df702cacea39927e796c1a5e8c7b25a598e98
+  data.tar.gz: acaa0d17b9911bff8465bcc41e7617b239d3cba7db74a96d166247edd71c0952
 SHA512:
-  metadata.gz: 97568f03aab3b39a4b75e98421a5b9e43f30f8f1c71b548478f0ee2f73c6303981643948289f9483d9cfbe71af47b872bf3e3c49d39009270e9e3ef72e55bf70
-  data.tar.gz: 8fa7ecb28d0d2e8097124330aced622ac52fc96d6cc8d6039e629c897d29af8f79366e8992107a9d8b89c88280c723b667b345ccbc643d2326303f376abd243a
+  metadata.gz: 9a7d5cb4b7dd42ed7950df79a9fc55da0f00f3eca5732eb496828d8295ca47c0b3cfd60ecb7821d2496a256a4c5df452faacffbae4a84bb1824d52d8b51c2206
+  data.tar.gz: 1ae409446d015bba3a91cb9bf7da8b91777730e98345166f16993e9b9ae72d4137af5135a82f4b817c5193854fbd75188918b4800b44b10cee552b7bfab625df

data/README.md

@@ -28,7 +28,7 @@ User authentication is set up with [`Devise`](https://github.com/plataformatec/d
 
 Core implements an Amendment feature that can be activated in the components. As of now, it is only implemented in the proposal component.
 
-This feature makes it possible for anyone to edit the text of an amendable resource and create a child resource as an amendment. This child resource may receive support and the author of the amendable resource may accept or reject the amendment (or child proposal). In case of rejection, the author of the rejected emendation may raise the child resource to an independent resource.
+This feature makes it possible for anyone to edit the text of an amendable resource and create a child resource as an amendment. This child resource may receive votes and the author of the amendable resource may accept or reject the amendment (or child proposal). In case of rejection, the author of the rejected emendation may raise the child resource to an independent resource.
 
 ### Key artifacts for Amendments
 

data/app/cells/decidim/activities_cell.rb

@@ -5,7 +5,6 @@ module Decidim
   # each one.
   class ActivitiesCell < Decidim::ViewModel
     include Decidim::CardHelper
-    include Decidim::IconHelper
     include Decidim::Core::Engine.routes.url_helpers
 
     # Since we are rendering each activity separatedly we need to trigger

data/app/cells/decidim/activity_cell.rb

@@ -7,10 +7,6 @@ module Decidim
   # tweak the necessary methods (usually `title` is enough).
   class ActivityCell < Decidim::ViewModel
     include Cell::ViewModel::Partial
-    include Decidim::IconHelper
-    include Decidim::ApplicationHelper
-    include Decidim::SanitizeHelper
-    include ActionView::Helpers::DateHelper
 
     def show
       return unless renderable?
@@ -27,6 +23,8 @@ module Decidim
 
     # The resource linked to the activity.
     def resource
+      return if model.blank?
+
       model.resource_lazy
     end
 
@@ -75,7 +73,7 @@ module Decidim
 
     # The text to show as the link to the resource.
     def resource_link_text
-      decidim_html_escape(translated_attribute(resource.title))
+      decidim_escape_translated(resource.title)
     end
 
     def created_at
@@ -106,7 +104,7 @@ module Decidim
       hash << id_prefix
       hash << I18n.locale.to_s
       hash << model.class.name.underscore
-      hash << model.cache_key_with_version
+      hash << model.cache_key_with_version if model.respond_to?(:cache_key_with_version)
       if (author_cell = author)
         hash.push(Digest::MD5.hexdigest(author_cell.send(:cache_hash)))
       end
@@ -157,7 +155,7 @@ module Decidim
 
     def participatory_space_link
       link_to(
-        decidim_html_escape(translated_attribute(participatory_space.title)),
+        decidim_escape_translated(participatory_space.title),
         resource_locator(participatory_space).path
       )
     end

data/app/cells/decidim/address_cell.rb

@@ -4,8 +4,6 @@ module Decidim
   # This cell renders the address of a meeting.
   class AddressCell < Decidim::ViewModel
     include Cell::ViewModel::Partial
-    include LayoutHelper
-    include Decidim::SanitizeHelper
 
     def show
       return render :online if options[:online]
@@ -22,15 +20,15 @@ module Decidim
     end
 
     def location_hints
-      decidim_sanitize(translated_attribute(model.location_hints))
+      decidim_sanitize_translated(model.location_hints)
     end
 
     def location
-      decidim_sanitize(translated_attribute(model.location))
+      decidim_sanitize_translated(model.location)
     end
 
     def address
-      decidim_sanitize(translated_attribute(model.address))
+      decidim_sanitize_translated(model.address)
     end
 
     def display_start_and_end_time?

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

@@ -4,8 +4,6 @@ module Decidim
   module Amendable
     # This cell renders the button to amend the given resource.
     class AmendButtonCardCell < Decidim::ViewModel
-      delegate :current_user, to: :controller, prefix: false
-
       def model_name
         model.model_name.human
       end

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

@@ -3,9 +3,7 @@
 module Decidim::Amendable
   # This cell renders the list of amendments of a resource.
   class AmendmentsCell < Decidim::ViewModel
-    include Decidim::ApplicationHelper
     include Decidim::CardHelper
-    include Decidim::IconHelper
 
     delegate :amendable?, :visible_emendations_for, to: :model
 

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

@@ -3,10 +3,8 @@
 module Decidim::Amendable
   # This cell renders the callout with information about the state of the emendation
   class AnnouncementCell < Decidim::ViewModel
-    include Decidim::ApplicationHelper
-
     def show
-      cell "decidim/announcement", announcement, callout_class: state_classes
+      cell "decidim/announcement", announcement, callout_class:, callout_styles:
     end
 
     private
@@ -50,17 +48,15 @@ module Decidim::Amendable
       model.amendment.updated_at
     end
 
-    def state_classes
-      case model.state
-      when "accepted"
-        "success"
-      when "rejected", "withdrawn"
-        "alert"
-      when "evaluating"
-        "warning"
-      else
-        "muted"
-      end
+    def callout_class
+      return "muted" if model.state.blank?
+      return "alert" if model.withdrawn?
+    end
+
+    def callout_styles
+      return if model.state.blank? || model.withdrawn?
+
+      model.proposal_state&.css_style
     end
   end
 end

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

@@ -3,8 +3,6 @@
 module Decidim::Amendable
   # This cell renders the action buttons for coauthors to amend the given resource.
   class EmendationActionsCell < Decidim::ViewModel
-    include Decidim::LayoutHelper
-
     delegate :amendment, to: :model
 
     def current_component

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

@@ -4,8 +4,6 @@ module Decidim
   module Amendable
     # This cell renders the button to amend the given resource.
     class PromoteButtonCardCell < Decidim::ViewModel
-      delegate :current_user, to: :controller, prefix: false
-
       def emendation
         @emendation ||= model
       end

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

@@ -14,14 +14,10 @@ module Decidim
       # Returns the current step as Integer.
       def current_step
         @current_step ||= case params[:action].to_sym
-                          when :new, :create
+                          when :new, :create, :edit_draft, :update_draft, :destroy_draft
                             1
-                          when :compare_draft
-                            2
-                          when :edit_draft, :update_draft, :destroy_draft
-                            3
                           when :preview_draft, :publish_draft
-                            4
+                            2
                           end
       end
 
@@ -47,12 +43,10 @@ module Decidim
         {
           view:,
           current_step:,
-          total_steps: 4,
+          total_steps: 2,
           steps: {
             "1": i18n_step(1),
-            "2": i18n_step(2),
-            "3": i18n_step(3),
-            "4": i18n_step(4)
+            "2": i18n_step(2)
           }
         }
       end
@@ -68,7 +62,6 @@ module Decidim
       def wizard_header_options
         {
           wizard_header_title:,
-          wizard_header_similar_resources_count: options[:similar_resources_count],
           wizard_header_help_text:
         }
       end
@@ -86,9 +79,7 @@ module Decidim
         case current_step
         when 1
           Decidim::ResourceLocatorPresenter.new(amendable).path
-        when 3
-          Decidim::Core::Engine.routes.url_helpers.compare_draft_amend_path(amendment)
-        when 4
+        when 2
           Decidim::Core::Engine.routes.url_helpers.edit_draft_amend_path(amendment)
         end
       end
@@ -99,11 +90,7 @@ module Decidim
               when 1
                 :new
               when 2
-                :compare_draft
-              when 3
-                :edit_draft
-              when 4
-                :preview_draft
+                :publish_draft
               end
 
         t("decidim.amendments.#{key}.title")

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

@@ -1,11 +1,11 @@
-<%= content_tag :div, id: options[:id], class: "flash flex-col #{css_class}", data: { announcement: "" }.compact, hidden: options[:hidden] do %>
+<%= content_tag :div, id: options[:id], class: "flash flex-col #{css_class}", style: css_styles, data: { announcement: "" }.compact, hidden: options[:hidden] do %>
   <% if has_title? %>
     <div class="flash__title">
       <%= clean_title %>
     </div>
   <% end %>
 
-  <span class="flash__message">
+  <div class="flash__message">
     <%= text %>
-  </span>
+  </div>
 <% end %>

data/app/cells/decidim/announcement_cell.rb

@@ -19,8 +19,6 @@ module Decidim
   #   - `callout_class`: The Css class to apply
   #
   class AnnouncementCell < Decidim::ViewModel
-    include Decidim::SanitizeHelper
-
     def show
       return if blank_content?
 
@@ -41,6 +39,12 @@ module Decidim
       has_title? ? clean_body : clean_announcement
     end
 
+    def css_styles
+      return unless options[:callout_styles]
+
+      options[:callout_styles]
+    end
+
     def css_class
       return unless options[:callout_class]
 

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

@@ -0,0 +1,3 @@
+<div class="row column">
+  <%= form.upload :file, button_class: "button button__sm button__transparent-secondary" %>
+</div>

data/app/cells/decidim/attachments_file_tab_cell.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module Decidim
+  class AttachmentsFileTabCell < Decidim::ViewModel
+    alias form model
+
+    def show
+      render :show
+    end
+  end
+end

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

@@ -0,0 +1,12 @@
+<div class="row column">
+  <%= form.text_field :link, aria: { label: :url }, type: :text %>
+
+  <div class="upload-modal__text">
+    <span class="sr-only"><%= explanation %></span>
+    <ul>
+      <% help_messages.each do |message| %>
+        <li><%= message %></li>
+      <% end %>
+    </ul>
+  </div>
+</div>

data/app/cells/decidim/attachments_link_tab_cell.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module Decidim
+  class AttachmentsLinkTabCell < Decidim::ViewModel
+    alias form model
+
+    def show
+      render :show
+    end
+
+    private
+
+    def explanation
+      I18n.t("decidim.forms.attachment_link.explanation")
+    end
+
+    def help_messages
+      I18n.t("decidim.forms.attachment_link.help_messages")
+    end
+  end
+end

data/app/cells/decidim/author/flag.erb

@@ -1,6 +1,6 @@
-<button type="button" class="link-alt" data-open="<%= current_user.present? ? "flagModal" : "loginModal" %>" title="<%= t("report", scope: "decidim.shared.flag_modal" ) %>" aria-controls="<%= current_user.present? ? "flagModal" : "loginModal" %>" aria-haspopup="true" tabindex="0">
+<button type="button" data-open="<%= current_user.present? ? "flagModal" : "loginModal" %>" title="<%= t("report", scope: "decidim.shared.flag_modal" ) %>" aria-controls="<%= current_user.present? ? "flagModal" : "loginModal" %>" aria-haspopup="true" tabindex="0">
   <%= icon "flag", aria_hidden: true, class: "icon--small", role: "img", "aria-hidden": true %>
-  <span class="show-for-sr">
+  <span>
     <%= t("report", scope: "decidim.shared.flag_modal" ) %>
   </span>
 </button>

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

@@ -1,16 +1,16 @@
 <% data = has_tooltip? ? { tooltip: render(:profile_minicard).html_safe } : nil %>
-<div class="author" data-author>
-  <%= content_tag :div, class: "author__container#{" is-compact" if layout == :compact}", data: do %>
+<span class="author" data-author>
+  <%= content_tag :span, class: "author__container#{" is-compact" if layout == :compact}", data: do %>
     <% if layout == :compact %>
       <%= render :avatar %>
 
-      <div>
+      <span>
         <%= render :name %>
 
         <% context_actions.each do |action| %>
           <%= render action %>
         <% end %>
-      </div>
+      </span>
     <% elsif layout == :avatar %>
       <%= render :avatar %>
     <% else %>
@@ -24,4 +24,4 @@
       <%= render action %>
     <% end %>
   <% end %>
-</div>
+</span>

data/app/cells/decidim/author_cell.rb

@@ -5,9 +5,7 @@ module Decidim
   # below resource titles to indicate its authorship & such, and is intended
   # for resources that have a single author.
   class AuthorCell < Decidim::ViewModel
-    include LayoutHelper
     include CellsHelper
-    include Decidim::SanitizeHelper
     include ::Devise::Controllers::Helpers
     include ::Devise::Controllers::UrlHelpers
     include Messaging::ConversationHelper

data/app/cells/decidim/authorization_modal_cell.rb

@@ -2,9 +2,6 @@
 
 module Decidim
   class AuthorizationModalCell < Decidim::ViewModel
-    include LayoutHelper
-    include Decidim::MetaTagsHelper
-
     delegate :authorize_action_path, to: :controller
 
     alias authorizations model

data/app/cells/decidim/badge_cell.rb

@@ -2,11 +2,7 @@
 
 module Decidim
   class BadgeCell < Decidim::ViewModel
-    include Decidim::ApplicationHelper
     include Decidim::Core::Engine.routes.url_helpers
-    include Decidim::IconHelper
-
-    delegate :current_user, to: :controller
 
     def small
       render "small"

data/app/cells/decidim/badges_cell.rb

@@ -2,8 +2,6 @@
 
 module Decidim
   class BadgesCell < Decidim::ViewModel
-    include Decidim::ApplicationHelper
-    include Decidim::LayoutHelper
     include Decidim::Core::Engine.routes.url_helpers
 
     def available_badges

data/app/cells/decidim/button_cell.rb

@@ -3,9 +3,6 @@
 module Decidim
   # This cell renders a generic button.
   class ButtonCell < Decidim::ViewModel
-    include LayoutHelper
-    include IconHelper
-    include Decidim::SanitizeHelper
     include Decidim::ResourceHelper
 
     DEFAULT_ATTRIBUTES = {

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

@@ -1,5 +1,5 @@
 <div class="py-4 space-y-2">
-  <a href="" class="text-lg text-secondary font-semibold hover:underline"><%= title %></a>
+  <a href="" class="text-lg text-secondary font-semibold hover:underline"><%= decidim_html_escape(title) %></a>
   <div class="flex items-center divide-x divide-gray-3">
    <% metadata.first(4).each do |item| %>
     <div class="flex items-center gap-1 px-4 lg:px-6 first:pl-0 last:pr-0 max-w-xs">

data/app/cells/decidim/card_cell.rb

@@ -3,12 +3,8 @@
 module Decidim
   # This cell renders the card of the given instance of a Component
   # delegated to the components' cell if specified in the manifest
-  # otherwise a primary cell wil be shown.
+  # otherwise a primary cell will be shown.
   class CardCell < Decidim::ViewModel
-    include Decidim::ApplicationHelper
-    include Decidim::IconHelper
-    include ActionView::Helpers::NumberHelper
-
     ICONS_MAPPING = {
       published_at: "calendar-line",
       votings_count: "check-double-line",

data/app/cells/decidim/card_g_cell.rb

@@ -73,11 +73,11 @@ module Decidim
     end
 
     def title
-      decidim_html_escape(translated_attribute(resource.title))
+      decidim_escape_translated(resource.title)
     end
 
     def alt_title
-      [t("decidim.application.photo.alt"), decidim_html_escape(translated_attribute(resource.title))].join(": ")
+      [t("decidim.application.photo.alt"), decidim_escape_translated(resource.title)].join(": ")
     end
 
     def title_tag

data/app/cells/decidim/card_l/space.erb

@@ -1 +1 @@
-<span class="muted"><%= searchable_resource_human_name(model.participatory_space.class, count: 1) %>:</span>&nbsp;<%= link_to translated_attribute(participatory_space.title), Decidim::ResourceLocatorPresenter.new(participatory_space).path %>
+<span><%= searchable_resource_human_name(model.participatory_space.class, count: 1) %>:</span>&nbsp;<%= link_to translated_attribute(participatory_space.title), Decidim::ResourceLocatorPresenter.new(participatory_space).path %>

data/app/cells/decidim/card_l_cell.rb

@@ -5,9 +5,7 @@ module Decidim
   # so other cells only have to customize a few methods or overwrite views.
   class CardLCell < Decidim::ViewModel
     include Cell::ViewModel::Partial
-    include Decidim::ApplicationHelper
     include Decidim::TooltipHelper
-    include Decidim::SanitizeHelper
     include Decidim::CardHelper
     include Decidim::LayoutHelper
     include Decidim::SearchesHelper
@@ -104,7 +102,7 @@ module Decidim
     end
 
     def title
-      decidim_html_escape(translated_attribute(resource.title))
+      decidim_escape_translated(resource.title)
     end
 
     def title_tag

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

@@ -3,12 +3,12 @@
     <% hook_output = render_hook(item[:hook]) %>
     <% next if hook_output.blank? %>
 
-    <%= content_tag :span, data: item[:data_attributes] do %>
+    <%= content_tag :div, data: item[:data_attributes] do %>
       <%= icon item[:icon] if item[:icon].present? %>
       <%= hook_output %>
     <% end %>
   <% else %>
-    <%= content_tag :span, data: item[:data_attributes] do %>
+    <%= content_tag :div, data: item[:data_attributes] do %>
       <%= icon item[:icon] if item[:icon].present? %>
       <% if item[:text].present? %>
         <%= link_to_if enable_links? && item.has_key?(:url), item[:text], item[:url] %>

data/app/cells/decidim/card_metadata_cell.rb

@@ -4,11 +4,7 @@ module Decidim
   # This cell is used to display metadata inside a list card
   # so other cells only have to customize a few methods or overwrite views.
   class CardMetadataCell < Decidim::ViewModel
-    include Decidim::IconHelper
-    include Decidim::ApplicationHelper
-    include Decidim::SanitizeHelper
     include Decidim::DateRangeHelper
-    include ActionView::Helpers::DateHelper
     include Cell::ViewModel::Partial
     include Decidim::ViewHooksHelper
 
@@ -33,7 +29,7 @@ module Decidim
       return unless show_space?
 
       {
-        text: decidim_html_escape(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/card_s_cell.rb

@@ -4,9 +4,6 @@ module Decidim
   # This cell is used a base for all Search cards. It holds the basic layout
   # so other cells only have to customize a few methods or overwrite views.
   class CardSCell < Decidim::ViewModel
-    include Decidim::ApplicationHelper
-    include Decidim::SanitizeHelper
-
     alias resource model
 
     def show
@@ -24,7 +21,7 @@ module Decidim
     end
 
     def title
-      decidim_html_escape(translated_attribute(resource.title))
+      decidim_escape_translated(resource.title)
     end
 
     def title_tag

data/app/cells/decidim/coauthorships_cell.rb

@@ -7,8 +7,6 @@ module Decidim
   #
   #    cell("decidim/coauthorships", @proposal)
   class CoauthorshipsCell < Decidim::ViewModel
-    include Decidim::ApplicationHelper
-
     def show
       if authorable?
         cell "decidim/author", presenter_for_author(model), has_actions: has_actions?, from: model

data/app/cells/decidim/comments_button_cell.rb

@@ -2,8 +2,6 @@
 
 module Decidim
   class CommentsButtonCell < ButtonCell
-    delegate :current_settings, :component_settings, to: :controller
-
     def show
       if options.has_key?(:display)
         return render if options[:display]

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

@@ -1,7 +1,7 @@
 <section id="footer_sub_hero" class="home__section text-center">
   <div class="flex justify-center mb-12">
     <h2 class="title-decorator">
-      <%= t("decidim.pages.home.footer_sub_hero.footer_sub_hero_headline", organization: current_organization.name) %>
+      <%= t("decidim.pages.home.footer_sub_hero.footer_sub_hero_headline", organization: current_organization_name) %>
     </h2>
   </div>
   <p class="text-lg text-gray-2">

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

@@ -3,9 +3,6 @@
 module Decidim
   module ContentBlocks
     class FooterSubHeroCell < Decidim::ViewModel
-      include Decidim::IconHelper
-
-      delegate :current_user, to: :controller
     end
   end
 end

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

@@ -2,7 +2,7 @@
   <div class="hero">
     <h1 class="hero__title">
       <% if translated_welcome_text.blank? %>
-        <%= t("decidim.pages.home.hero.welcome", organization: current_organization.name) %>
+        <%= t("decidim.pages.home.hero.welcome", organization: current_organization_name) %>
       <% else %>
         <%= decidim_sanitize_admin translated_welcome_text %>
       <% end %>

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

@@ -4,7 +4,6 @@ module Decidim
   module ContentBlocks
     class HeroCell < Decidim::ViewModel
       include Decidim::CtaButtonHelper
-      include Decidim::SanitizeHelper
 
       # Needed so that the `CtaButtonHelper` can work.
       def decidim_participatory_processes

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

@@ -3,8 +3,6 @@
 module Decidim
   module ContentBlocks
     class HighlightedContentBannerCell < Decidim::ViewModel
-      include Decidim::SanitizeHelper
-
       def show
         return unless current_organization.highlighted_content_banner_enabled
 

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

@@ -38,7 +38,7 @@
     </div>
   </div>
   <%= 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: translated_attribute(current_organization.name, current_organization)) %>
+    <%= t("more_info", scope: "decidim.pages.home.extended", resource_name: current_organization_name) %>
     <%= icon "arrow-right-line", class: "fill-current" %>
   <% end %>
 </section>

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

@@ -3,7 +3,6 @@
 module Decidim
   module ContentBlocks
     class HowToParticipateCell < Decidim::ViewModel
-      include Decidim::IconHelper
     end
   end
 end