decidim-core 0.30.2 → 0.31.0.rc1

data/app/packs/stylesheets/decidim/editor.scss

@@ -1,6 +1,5 @@
 .editor-props {
   --editor-active-color: #487bda;
-  --editor-border-color: #ccc;
   --editor-disabled-color: #eee;
   --editor-bubble-border-color: #ccc;
   --editor-bubble-shadow-color: #333;
@@ -8,20 +7,16 @@
 
 .editor-suggestions-props {
   --editor-suggestions-border-color: #000;
-  --editor-suggestions-background-color: #fff;
-  --editor-suggestions-selected-highlight-color: #c7c7c7;
+  --editor-suggestions-background-color: #eee;
+  --editor-suggestions-selected-highlight-color: #ccc;
 }
 
 .editor-active {
   @apply outline outline-4 outline-[var(--editor-active-color)];
 }
 
-.editor-border {
-  @apply border-solid border-[#ccc];
-}
-
 .editor-container {
-  @apply editor-props editor-suggestions-props flex flex-col mt-4 border editor-border;
+  @apply editor-props editor-suggestions-props flex flex-col border border-neutral-200 rounded mt-2;
 
   &.editor-disabled {
     .editor-input .ProseMirror {
@@ -30,7 +25,7 @@
   }
 
   .editor-toolbar {
-    @apply py-1.5 px-2.5 border-0 border-b editor-border;
+    @apply py-1.5 px-2.5 border-b border-b-neutral-200;
 
     .editor-toolbar-group {
       @apply inline-block mx-2.5;
@@ -181,7 +176,7 @@
 }
 
 .editor-suggestions {
-  @apply editor-suggestions-props border border-solid border-[var(--editor-suggestions-border-color)] bg-[var(--editor-suggestions-background-color)];
+  @apply editor-suggestions-props border-0 bg-[var(--editor-suggestions-background-color)] max-w-sm drop-shadow-md rounded-md;
 
   &:hover {
     .editor-suggestions-item[data-selected]:not(:hover) {
@@ -190,10 +185,14 @@
   }
 
   .editor-suggestions-item {
-    @apply block w-full text-left py-2.5 px-1.5 border-0 border-t border-solid border-[var(--editor-suggestions-border-color)] rounded-none;
+    @apply block w-full text-left py-[0.375rem] px-1.5 border-0 rounded-none truncate text-sm;
 
     &:first-child {
-      @apply border-t-0;
+      @apply rounded-t-md;
+    }
+
+    &:last-child {
+      @apply rounded-b-md;
     }
 
     &:hover,

data/app/packs/stylesheets/decidim/legacy/email.scss

@@ -1,4 +1,5 @@
-@import "stylesheets/decidim/legacy/variables";
+@use "sass:meta";
+@use "stylesheets/decidim/legacy/variables";
 
 /* Index:
   1 - Foundation settings
@@ -1190,8 +1191,8 @@ table.button {
 table.button table td {
   text-align: left;
   color: #fefefe;
-  background: $primary;
-  border: 2px solid $primary;
+  background: variables.$primary;
+  border: 2px solid variables.$primary;
 }
 
 table.button table td a {
@@ -1201,7 +1202,7 @@ table.button table td a {
   text-decoration: none;
   display: inline-block;
   padding: 8px 16px;
-  border: 0 solid $primary;
+  border: 0 solid variables.$primary;
   border-radius: 3px;
 }
 
@@ -1290,7 +1291,7 @@ table.button.expanded center {
 table.button:hover table td,
 table.button:visited table td,
 table.button:active table td {
-  background: $primary;
+  background: variables.$primary;
   color: #fefefe;
 }
 

data/app/packs/stylesheets/decidim/map.scss

@@ -1,6 +1,8 @@
-@import "leaflet/dist/leaflet";
-@import "leaflet.markercluster/dist/MarkerCluster";
-@import "leaflet.markercluster/dist/MarkerCluster.Default";
+@use "sass:meta";
+
+@include meta.load-css("leaflet/dist/leaflet");
+@include meta.load-css("leaflet.markercluster/dist/MarkerCluster");
+@include meta.load-css("leaflet.markercluster/dist/MarkerCluster.Default");
 
 /* overwrite leaflet and card styles */
 .leaflet-popup-content-wrapper {

data/app/packs/stylesheets/decidim/qr.scss

@@ -0,0 +1,65 @@
+@use "sass:meta";
+@use "tailwindcss/base";
+@use "tailwindcss/components";
+@use "tailwindcss/utilities";
+
+@layer base {
+  @include meta.load-css("stylesheets/decidim/fonts");
+  @include meta.load-css("stylesheets/decidim/typography");
+  // Keep this in separate file as long as we are maintaining Foundation compatibility
+  @include meta.load-css("stylesheets/decidim/rich_text");
+}
+
+body {
+  @apply text-center mt-16 px-8;
+
+  > * {
+    @apply mx-auto w-full;
+  }
+
+  h1 {
+    @apply text-2xl text-black mb-6;
+  }
+
+  h3 {
+    @apply text-lg text-black mt-4;
+  }
+
+  .code-qr {
+    @apply mx-auto w-[80%] md:w-[40%] lg:w-[20%];
+  }
+
+  .card__grid-img {
+    @apply aspect-[21/9] rounded-t overflow-hidden mx-auto w-full lg:w-96 mb-12;
+
+    img {
+      @apply w-full h-full object-cover;
+    }
+  }
+
+  .logo {
+    @apply mt-10;
+
+    a {
+      @apply inline-block;
+
+      img {
+        @apply h-20 w-full;
+      }
+    }
+  }
+}
+
+@media print {
+  body {
+    zoom: 0.75;
+
+    h1 {
+      @apply text-2xl text-black mb-12;
+    }
+
+    .card__grid-img {
+      @apply w-[80%];
+    }
+  }
+}

data/app/permissions/decidim/default_permissions.rb

@@ -4,6 +4,7 @@ module Decidim
   # Default permissions class for all components and spaces. It deauthorizes all
   # actions by any kind of user. Also works as a default implementation so other
   # components can inherit from it and get some convenience methods.
+  # Delegates the admin permission check on :subject and :action to the permissions of each module.
   class DefaultPermissions
     include Decidim::UserRoleChecker
 
@@ -14,6 +15,13 @@ module Decidim
     end
 
     def permissions
+      return permission_action if permission_action.scope != target_scope
+
+      return permission_action unless subject == target_subject
+
+      permission_method = "can_#{action}?"
+      toggle_allow(send(permission_method)) if respond_to?(permission_method, true)
+
       permission_action
     end
 
@@ -21,6 +29,14 @@ module Decidim
 
     attr_reader :user, :permission_action, :context
 
+    delegate :subject, :action, to: :permission_action
+
+    def target_scope = ""
+
+    def target_subject
+      raise NotImplementedError "You need to implement target_subject"
+    end
+
     def disallow!
       permission_action.trace(self.class.name, :disallowed)
       permission_action.disallow!

data/app/permissions/decidim/permissions.rb

@@ -21,9 +21,7 @@ module Decidim
       amend_action?
       notification_action?
       conversation_action?
-      user_group_action?
-      user_group_invitations_action?
-      apply_endorsement_permissions if permission_action.subject == :endorsement
+      apply_like_permissions if permission_action.subject == :like
       show_my_location_button?
 
       permission_action
@@ -127,10 +125,10 @@ module Decidim
       toggle_allow(amendment&.amender == user)
     end
 
-    def apply_endorsement_permissions
-      is_allowed = current_settings.endorsements_enabled &&
-                   !current_settings.endorsements_blocked &&
-                   authorized?(:endorse, resource: context.fetch(:resource, nil))
+    def apply_like_permissions
+      is_allowed = current_settings.likes_enabled &&
+                   !current_settings.likes_blocked &&
+                   authorized?(:like, resource: context.fetch(:resource, nil))
 
       toggle_allow(is_allowed)
     end
@@ -155,25 +153,6 @@ module Decidim
       toggle_allow(conversation&.participating?(interlocutor))
     end
 
-    def user_group_action?
-      return unless permission_action.subject == :user_group
-      return allow! if [:join, :create].include?(permission_action.action)
-
-      user_group = context.fetch(:user_group)
-
-      if permission_action.action == :leave
-        user_can_leave_group = Decidim::UserGroupMembership.where(user:, user_group:).any?
-        return toggle_allow(user_can_leave_group)
-      end
-
-      user_manages_group = Decidim::UserGroups::ManageableUserGroups.for(user).include?(user_group)
-      toggle_allow(user_manages_group) if permission_action.action == :manage
-    end
-
-    def user_group_invitations_action?
-      allow! if permission_action.subject == :user_group_invitations
-    end
-
     def user_can_preview_component?
       context[:share_token].present? && Decidim::ShareToken.use!(token_for: component, token: context[:share_token], user:)
     rescue ActiveRecord::RecordNotFound, StandardError

data/app/presenters/decidim/admin_log/base_user_presenter.rb

@@ -2,8 +2,8 @@
 
 module Decidim
   module AdminLog
-    # This class holds the shared logic for `Decidim::User` and `Decidim::UserGroup`
-    # presenters for the `AdminLog`.
+    # This class holds the shared logic for `Decidim::User`
+    # presenter for the `AdminLog`.
     class BaseUserPresenter < Decidim::Log::BasePresenter
       private
 

data/app/presenters/decidim/admin_log/organization_presenter.rb

@@ -19,7 +19,6 @@ module Decidim
 
         settings_attributes_mapping
           .merge(omnipresent_banner_attributes_mapping)
-          .merge(highlighted_content_banner_attributes_mapping)
           .merge(appearance_attributes_mapping)
           .merge(id_documents_attributes_mapping)
       end
@@ -47,22 +46,8 @@ module Decidim
         }
       end
 
-      def highlighted_content_banner_attributes_mapping
-        {
-          highlighted_content_banner_enabled: :boolean,
-          highlighted_content_banner_action_url: :string,
-          highlighted_content_banner_image: :string,
-          highlighted_content_banner_title: :i18n,
-          highlighted_content_banner_short_description: :i18n,
-          highlighted_content_banner_action_title: :i18n,
-          highlighted_content_banner_action_subtitle: :i18n
-        }
-      end
-
       def appearance_attributes_mapping
         {
-          cta_button_path: :string,
-          cta_button_text: :i18n,
           description: :i18n,
           logo: :string,
           header_snippets: :string,

data/app/presenters/decidim/home_stats_presenter.rb

@@ -2,63 +2,13 @@
 
 module Decidim
   # A presenter to render statistics in the homepage.
-  class HomeStatsPresenter < SimpleDelegator
-    def organization
+  class HomeStatsPresenter < StatsPresenter
+    def scope_entity
       __getobj__.fetch(:organization)
     end
 
-    # Public: Render a collection of primary stats.
-    def highlighted
-      highlighted_stats = Decidim.stats.only([:users_count, :processes_count]).with_context(organization).map { |name, data| [name, data] }
-      highlighted_stats.concat(global_stats(priority: StatsRegistry::HIGH_PRIORITY))
-      highlighted_stats.concat(component_stats(priority: StatsRegistry::HIGH_PRIORITY))
-      highlighted_stats = highlighted_stats.reject(&:empty?)
-      highlighted_stats = highlighted_stats.reject { |_name, data| data.zero? }
-
-      highlighted_stats.map do |name, data|
-        { stat_title: name, stat_number: data }
-      end
-    end
-
-    # Public: Render a collection of stats that are not primary.
-    def not_highlighted
-      not_highlighted_stats = global_stats(priority: StatsRegistry::MEDIUM_PRIORITY)
-      not_highlighted_stats.concat(component_stats(priority: StatsRegistry::MEDIUM_PRIORITY))
-      not_highlighted_stats = not_highlighted_stats.reject(&:empty?)
-      not_highlighted_stats = not_highlighted_stats.reject { |_name, data| data.zero? }
-
-      not_highlighted_stats.map do |name, data|
-        { stat_title: name, stat_number: data }
-      end
-    end
-
-    private
-
-    def global_stats(conditions)
-      Decidim.stats.except([:users_count, :processes_count, :followers_count])
-             .filter(conditions)
-             .with_context(organization)
-             .map { |name, data| [name, data] }
-    end
-
-    def component_stats(conditions)
-      stats = {}
-      Decidim.component_manifests.flat_map do |component|
-        component
-          .stats.except([:votes_count])
-          .filter(conditions)
-          .with_context(published_components)
-          .each do |name, data|
-            stats[name] ||= 0
-            stats[name] += data
-          end
-      end
-
-      stats.to_a
-    end
-
-    def published_components
-      @published_components ||= organization.published_components
+    def collection(priority: StatsRegistry::HIGH_PRIORITY)
+      super
     end
   end
 end

data/app/presenters/decidim/log/base_presenter.rb

@@ -157,7 +157,7 @@ module Decidim
       # Returns an HTML-safe String.
       def present_action_log
         classes = ["logs__log"] + action_log_extra_classes.to_a
-        h.content_tag(:div, id: "accordion-#{h.dom_id(action_log)}", class: classes.join(" "), data: { component: "accordion" }) do
+        h.content_tag(:div, id: "accordion-#{h.dom_id(action_log)}", class: classes.join(" "), data: { controller: "accordion" }) do
           h.concat(present_content)
           h.concat(present_diff)
         end

data/app/presenters/decidim/menu_item_presenter.rb

@@ -13,6 +13,9 @@ module Decidim
     # @option options [String] :element_class
     #         The CSS class to be used for the item
     #
+    # @option options [String] :element_wrapper_class
+    #         The CSS class to be used for the wrapper of the item
+    #
     # @option options [String] :active_class
     #         The CSS class to be used for the active item
     #
@@ -49,17 +52,21 @@ module Decidim
         { aria: { current: "page" } }
       else
         {}
-      end
+      end.merge({ class: link_classes })
     end
 
     def composed_label
       icon_name.present? ? icon(icon_name) + label : label
     end
 
+    def link_classes
+      @options.element_class.nil? ? "" : @options.element_class
+    end
+
     def link_wrapper_classes
-      return @options.element_class unless is_active_link?(url, active)
+      return @options.element_wrapper_class unless is_active_link?(url, active)
 
-      [@options.element_class, active_class].compact.join(" ")
+      [@options.element_wrapper_class, active_class].compact.join(" ")
     end
 
     def active_class

data/app/presenters/decidim/official_author_presenter.rb

@@ -36,9 +36,5 @@ module Decidim
     def can_be_contacted?
       false
     end
-
-    def has_tooltip?
-      false
-    end
   end
 end

data/app/presenters/decidim/participatory_space_private_user_presenter.rb

@@ -31,10 +31,6 @@ module Decidim
       attached_uploader(:non_user_avatar)
     end
 
-    def has_tooltip?
-      false
-    end
-
     def deleted?
       false
     end

data/app/presenters/decidim/resource_locator_presenter.rb

@@ -119,7 +119,18 @@ module Decidim
     def manifest_for(record)
       record.try(:resource_manifest) ||
         record.class.try(:resource_manifest) ||
-        record.class.try(:participatory_space_manifest)
+        record.class.try(:participatory_space_manifest) ||
+        record.to_s
+    end
+
+    def route_name_for(record)
+      manifest = manifest_for(record)
+
+      if manifest.respond_to?(:route_name)
+        manifest.route_name
+      else
+        manifest.to_s
+      end
     end
 
     def component
@@ -130,14 +141,14 @@ module Decidim
       if polymorphic?
         polymorphic_member_route_name
       else
-        manifest_for(target).route_name
+        route_name_for(target)
       end
     end
 
     def polymorphic_member_route_name
       return unless polymorphic?
 
-      resource.map { |record| manifest_for(record).route_name }.join("_")
+      resource.map { |record| route_name_for(record) }.join("_")
     end
 
     def collection_route_name
@@ -149,7 +160,7 @@ module Decidim
 
       parent_resources = {}
       (resource - [target]).each do |parent|
-        parent_resources["#{manifest_for(parent).route_name}_id"] = parent.id
+        parent_resources["#{route_name_for(parent)}_id"] = parent.id unless parent.is_a?(String)
       end
       parent_resources
     end

data/app/presenters/decidim/resource_presenter.rb

@@ -6,12 +6,12 @@ module Decidim
     include Decidim::TranslatableAttributes
     include Decidim::SanitizeHelper
 
-    def title(resource_title, links, html_escape, all_locales, extras: true)
+    def title(resource_title, html_escape, all_locales)
       handle_locales(resource_title, all_locales) do |content|
         content = decidim_html_escape(content) if html_escape
 
-        renderer = Decidim::ContentRenderers::HashtagRenderer.new(content)
-        renderer.render(links:, extras:).html_safe
+        renderer = Decidim::ContentRenderers::BlobRenderer.new(content)
+        renderer.render.html_safe
       end
     end
 
@@ -30,16 +30,16 @@ module Decidim
     end
 
     # Prepares the HTML content for the editors with the correct tags included
-    # to identify the hashtags and mentions.
-    def editor_locales(data, all_locales, extras: true)
+    # to identify the mentions.
+    def editor_locales(data, all_locales)
       handle_locales(data, all_locales) do |content|
         [
-          Decidim::ContentRenderers::HashtagRenderer,
+          Decidim::ContentRenderers::BlobRenderer,
           Decidim::ContentRenderers::UserRenderer,
-          Decidim::ContentRenderers::UserGroupRenderer
+          Decidim::ContentRenderers::MentionResourceRenderer
         ].each do |renderer_class|
           renderer = renderer_class.new(content)
-          content = renderer.render(links: false, editor: true, extras:).html_safe
+          content = renderer.render(editor: true).html_safe
         end
 
         content

data/app/presenters/decidim/stats_presenter.rb

@@ -3,83 +3,62 @@
 module Decidim
   # A general presenter to render statistics in participatory spaces.
   class StatsPresenter < SimpleDelegator
-    def comments_stats(name)
-      comments = Decidim.component_manifests.map do |component_manifest|
-        component_manifest.stats.only([:comments_count])
-                          .filter({ tag: :comments })
-                          .with_context(published_components)
-                          .map { |_name, value| value }.sum
-      end
-      comments_count = comments.inject(0, :+) { |sum, value| sum + value }
-      [[name, :comments_count, comments_count]]
-    end
-
-    def statistics(grouped_stats)
-      statistics = {}
-
-      grouped_stats.each do |_manifest_name, stats|
-        stats.each do |_space_manifest, component_manifest, count|
-          next if count.zero?
-
-          statistics[component_manifest] ||= 0
-          statistics[component_manifest] += count
+    # Public: returns a collection of stats (Hash) for the Participatory Space Home.
+    def collection(priority: StatsRegistry::MEDIUM_PRIORITY)
+      stats = all_stats(priority:).reject(&:empty?)
+      stats = stats.reject { |stat| stat[:data].blank? || stat[:data][0].zero? }
+      stats.each_with_object({}) do |stat, hash|
+        name = stat[:name]
+        if hash[name]
+          stat[:data].each_with_index do |value, idx|
+            hash[name][:data][idx] ||= 0
+            hash[name][:data][idx] += value
+          end
+        else
+          hash[name] = stat
         end
-      end
-      statistics.map { |key, number| { stat_title: key, stat_number: number } }
+      end.values
     end
 
-    # Public: returns a collection of stats (Hash) for the Participatory Space Home.
-    def collection
-      highlighted_stats = participatory_space_participants_stats
-      highlighted_stats.concat(participatory_space_followers_stats(priority: StatsRegistry::HIGH_PRIORITY))
-      highlighted_stats.concat(component_stats(priority: StatsRegistry::HIGH_PRIORITY))
-      highlighted_stats.concat(component_stats(priority: StatsRegistry::MEDIUM_PRIORITY))
-      highlighted_stats.concat(comments_stats(participatory_space_sym))
-      highlighted_stats = highlighted_stats.reject(&:empty?)
-      highlighted_stats = highlighted_stats.reject { |_stat_manifest, _stat_title, stat_number| stat_number.zero? }
-      grouped_highlighted_stats = highlighted_stats.group_by(&:first)
-
-      statistics(grouped_highlighted_stats)
+    def all_stats(conditions)
+      @global_stats ||= {}
+      @global_stats[conditions] ||= global_stats(**conditions).concat(component_stats(**conditions))
     end
 
-    def card_collection
-      card_stats = Decidim.stats.only([:followers_count, :votings_count])
-                          .filter(priority: StatsRegistry::HIGH_PRIORITY)
-                          .with_context(participatory_space)
-                          .map { |stat_title, stat_number| [participatory_space_sym, stat_title, stat_number] }
-
-      statistics(card_stats.group_by(&:first))
-    end
+    # Public: returns the entity where to limit the stats on.
+    # Usually, this is a Participatory Space or an Organization.
+    def scope_entity = raise NotImplementedError
 
     private
 
-    def participatory_space = raise "Not implemented"
-
-    def participatory_space_sym = raise "Not implemented"
-
-    def participatory_space_participants_stats
-      Decidim.stats.only([:participants_count]).with_context(participatory_space)
-             .map { |stat_title, stat_number| [participatory_space_sym, stat_title, stat_number] }
+    def global_stats(conditions)
+      Decidim.stats.filter(**conditions).with_context(scope_entity).map do |stat|
+        stat[:data] = [stat[:data]] unless stat[:data].is_a?(Array)
+        stat
+      end
     end
 
     def component_stats(conditions)
       Decidim.component_manifests.map do |component_manifest|
-        component_manifest.stats.except([:proposals_accepted])
+        component_manifest.stats
                           .filter(conditions)
                           .with_context(published_components)
-                          .map { |stat_title, stat_number| [component_manifest.name, stat_title, stat_number] }.flatten
-      end
-    end
-
-    def participatory_space_followers_stats(conditions)
-      Decidim.stats.only([:followers_count])
-             .filter(conditions)
-             .with_context(participatory_space)
-             .map { |stat_title, stat_number| [participatory_space_sym, stat_title, stat_number] }
+                          .map do |stat|
+                            stat[:data] = [stat[:data]] unless stat[:data].is_a?(Array)
+                            stat
+                          end
+      end.flatten
     end
 
+    # Override if the scope_entity is not a Participatory Space or an Organization.
     def published_components
-      @published_components ||= Component.where(participatory_space:).published
+      @published_components ||= if scope_entity.is_a?(Decidim::Organization)
+                                  scope_entity.published_components
+                                elsif scope_entity.is_a?(Decidim::ParticipatoryProcessGroup)
+                                  Component.where(participatory_space: scope_entity.participatory_processes).published
+                                else
+                                  Component.where(participatory_space: scope_entity).published
+                                end
     end
   end
 end

data/app/presenters/decidim/taxonomy_presenter.rb

@@ -4,11 +4,15 @@ module Decidim
   #
   # Decorator for taxonomies.
   #
-  class TaxonomyPresenter < SimpleDelegator
+  class TaxonomyPresenter < ResourcePresenter
     include Decidim::TranslationsHelper
 
     def translated_name
       @translated_name ||= translated_attribute name
     end
+
+    def title(html_escape: false, all_locales: false)
+      super(name, html_escape, all_locales)
+    end
   end
 end