decidim-proposals 0.29.3 → 0.30.0.rc1

data/app/models/decidim/proposals/proposal.rb

@@ -7,6 +7,7 @@ module Decidim
       include Decidim::Resourceable
       include Decidim::Coauthorable
       include Decidim::HasComponent
+      include Decidim::Taxonomizable
       include Decidim::ScopableResource
       include Decidim::HasReference
       include Decidim::HasCategory
@@ -28,6 +29,7 @@ module Decidim
       include Decidim::TranslatableResource
       include Decidim::TranslatableAttributes
       include Decidim::FilterableResource
+      include Decidim::SoftDeletable
 
       def assign_state(token)
         proposal_state = Decidim::Proposals::ProposalState.where(component:, token:).first
@@ -301,6 +303,11 @@ module Decidim
         state == "evaluating"
       end
 
+      # Public: Overrides the `reported_content_url` Reportable concern method.
+      def reported_content_url
+        ResourceLocatorPresenter.new(self).url
+      end
+
       # Returns the presenter for this author, to be used in the views.
       # Required by ResourceRenderer.
       def presenter
@@ -398,13 +405,21 @@ module Decidim
       end
 
       def self.ransackable_scopes(_auth_object = nil)
-        [:with_any_origin, :with_any_state, :state_eq, :voted_by, :coauthored_by, :related_to, :with_any_scope, :with_any_category, :valuator_role_ids_has]
+        [:with_any_origin, :with_any_state, :state_eq, :voted_by, :coauthored_by, :related_to, :with_any_taxonomies, :valuator_role_ids_has]
       end
 
       # Create i18n ransackers for :title and :body.
       # Create the :search_text ransacker alias for searching from both of these.
       ransacker_i18n_multi :search_text, [:title, :body]
 
+      def self.ransackable_attributes(_auth_object = nil)
+        %w(id_string search_text title body is_emendation comments_count proposal_votes_count published_at proposal_notes_count)
+      end
+
+      def self.ransackable_associations(_auth_object = nil)
+        %w(taxonomies proposal_state)
+      end
+
       ransacker :state_published do
         Arel.sql("CASE
           WHEN EXISTS (
@@ -450,7 +465,7 @@ module Decidim
       end
 
       def self.export_serializer
-        Decidim::Proposals::ProposalSerializer
+        Decidim::Proposals::DownloadYourDataProposalSerializer
       end
 
       def self.download_your_data_images(user)
@@ -467,8 +482,18 @@ module Decidim
         return true if draft?
         return true if component.settings.proposal_edit_time == "infinite"
 
-        limit = updated_at + component.settings.proposal_edit_before_minutes.minutes
-        Time.current < limit
+        time_value, time_unit = component.settings.edit_time
+
+        limit_time = case time_unit
+                     when "minutes"
+                       updated_at + time_value.minutes
+                     when "hours"
+                       updated_at + time_value.hours
+                     else
+                       updated_at + time_value.days
+                     end
+
+        Time.current < limit_time
       end
 
       def process_amendment_state_change!
@@ -481,10 +506,51 @@ module Decidim
         end
       end
 
+      def user_has_actions?(user)
+        return false if authors.include?(user)
+        return false if user&.blocked?
+        return false if user&.deleted?
+        return false unless user&.confirmed?
+
+        true
+      end
+
+      def actions_for_comment(comment, current_user)
+        return if comment.commentable != self
+        return unless authors.include?(current_user)
+        return unless user_has_actions?(comment.author)
+
+        if coauthor_invitations_for(comment.author).any?
+          [
+            {
+              label: I18n.t("decidim.proposals.actions.cancel_coauthor_invitation"),
+              url: EngineRouter.main_proxy(component).cancel_proposal_invite_coauthors_path(proposal_id: id, id: comment.author.id),
+              icon: "user-forbid-line",
+              method: :delete,
+              data: { confirm: I18n.t("decidim.proposals.actions.cancel_coauthor_invitation_confirm") }
+            }
+          ]
+        else
+          [
+            {
+              label: I18n.t("decidim.proposals.actions.mark_as_coauthor"),
+              url: EngineRouter.main_proxy(component).proposal_invite_coauthors_path(proposal_id: id, id: comment.author.id),
+              icon: "user-add-line",
+              method: :post,
+              data: { confirm: I18n.t("decidim.proposals.actions.mark_as_coauthor_confirm") }
+            }
+          ]
+        end
+      end
+
+      def coauthor_invitations_for(user)
+        Decidim::Notification.where(event_class: "Decidim::Proposals::CoauthorInvitedEvent", resource: self, user:)
+      end
+
       private
 
       def copied_from_other_component?
-        linked_resources(:proposals, "copied_from_component").any?
+        linked_resources(:proposals, %w(splitted_from_component merged_from_component copied_from_component)).any?
       end
     end
   end

data/app/models/decidim/proposals/proposal_note.rb

@@ -9,12 +9,23 @@ module Decidim
 
       belongs_to :proposal, foreign_key: "decidim_proposal_id", class_name: "Decidim::Proposals::Proposal", counter_cache: true
       belongs_to :author, foreign_key: "decidim_author_id", class_name: "Decidim::User"
+      has_many :replies, foreign_key: "parent_id", class_name: "Decidim::Proposals::ProposalNote", inverse_of: :parent, dependent: :destroy
+      belongs_to :parent, class_name: "Decidim::Proposals::ProposalNote", inverse_of: :replies, optional: true
 
+      scope :not_reply, -> { where(parent_id: nil) }
       default_scope { order(created_at: :asc) }
 
       def self.log_presenter_class_for(_log)
         Decidim::Proposals::AdminLog::ProposalNotePresenter
       end
+
+      def reply?
+        parent.present?
+      end
+
+      def formatted_body
+        Decidim::ContentProcessor.render_without_format(body)
+      end
     end
   end
 end

data/app/models/decidim/proposals/proposal_state.rb

@@ -38,7 +38,7 @@ module Decidim
       protected
 
       def generate_token
-        self.token = ensure_unique_token(translated_attribute(title).parameterize(separator: "_"))
+        self.token = ensure_unique_token(token.presence || translated_attribute(title).parameterize(separator: "_"))
       end
 
       def ensure_unique_token(token)

data/app/packs/entrypoints/decidim_proposals.js

@@ -1,6 +1,7 @@
 import "src/decidim/proposals/utils"
 import "src/decidim/proposals/add_proposal"
 import "src/decidim/proposals/choose_proposals"
+import "src/decidim/proposals/exit_handler"
 
 // Images
 require.context("../images", true)

data/app/packs/entrypoints/decidim_proposals_geocoding.js

@@ -0,0 +1,2 @@
+import "src/decidim/proposals/reverse_geocoding.js"
+import "stylesheets/decidim/proposals/geocoding_addons.scss";

data/app/packs/src/decidim/proposals/admin/proposals.js

@@ -13,13 +13,20 @@ $(() => {
     return $(".table-list [data-published-state=false] .js-check-all-proposal:checked").length
   }
 
+  const selectedProposalsAllowsAnswerCount = function() {
+    return $(".table-list [data-allow-answer=true] .js-check-all-proposal:checked").length
+  }
+
   const selectedProposalsCountUpdate = function() {
     const selectedProposals = selectedProposalsCount();
     const selectedProposalsNotPublishedAnswer = selectedProposalsNotPublishedAnswerCount();
+    const allowAnswerProposals = selectedProposalsAllowsAnswerCount();
+
     if (selectedProposals === 0) {
       $("#js-selected-proposals-count").text("")
       $("#js-assign-proposals-to-valuator-actions").addClass("hide");
       $("#js-unassign-proposals-from-valuator-actions").addClass("hide");
+      $("#js-taxonomy-change-proposals-actions").addClass("hide");
     } else {
       $("#js-selected-proposals-count").text(selectedProposals);
     }
@@ -36,6 +43,13 @@ $(() => {
     } else {
       $('button[data-action="publish-answers"]').parent().hide();
     }
+
+    if (allowAnswerProposals > 0) {
+      $('button[data-action="apply-answer-template"]').parent().show();
+      $("#js-form-apply-answer-template-number").text(allowAnswerProposals);
+    } else {
+      $('button[data-action="apply-answer-template"]').parent().hide();
+    }
   }
 
   const showBulkActionsButton = function() {
@@ -91,7 +105,8 @@ $(() => {
       let action = $(e.target).data("action");
       const panelActions = [
         "assign-proposals-to-valuator",
-        "unassign-proposals-from-valuator"
+        "unassign-proposals-from-valuator",
+        "taxonomy-change-proposals"
       ];
 
       if (!action) {

data/app/packs/src/decidim/proposals/exit_handler.js

@@ -0,0 +1,73 @@
+const allowExitFrom = (el) => {
+  if (el.id === "exit-proposal-notification-link" || el.classList.contains("no-modal")) {
+    return true;
+  }
+
+  return false;
+};
+
+document.addEventListener("DOMContentLoaded", () => {
+  const exitNotification = document.getElementById("exit-proposal-notification");
+  const exitLink = document.getElementById("exit-proposal-notification-link");
+  if (!exitLink) {
+    return;
+  }
+  const defaultExitUrl = exitLink.href;
+  const defaultExitLinkText = exitLink.textContent;
+  const signOutPath = window.Decidim.config.get("sign_out_path");
+  let exitLinkText = defaultExitLinkText;
+
+  if (!exitNotification) {
+    // Do not apply when not inside the voting pipeline
+    return;
+  }
+
+  const openExitNotification = (url, method = null) => {
+    if (method && method !== "get") {
+      exitLink.setAttribute("data-method", method);
+    } else {
+      exitLink.removeAttribute("data-method");
+    }
+
+    exitLink.setAttribute("href", url);
+    exitLink.textContent = exitLinkText;
+    window.Decidim.currentDialogs["exit-proposal-notification"].open();
+  };
+
+  const handleClicks = (link) => {
+    link.addEventListener("click", (event) => {
+      exitLinkText = defaultExitLinkText;
+
+      if (
+        !allowExitFrom(link) &&
+        ((window.Decidim.currentDialogs["exit-proposal-notification"].dialog.querySelector("[data-dialog-container]")).dataset.minimumVotesReached !== "true") &&
+        ((window.Decidim.currentDialogs["exit-proposal-notification"].dialog.querySelector("[data-dialog-container]")).dataset.minimumVotesCount > 0)
+      ) {
+        event.preventDefault();
+        openExitNotification(link.getAttribute("href"), link.dataset.method);
+      }
+    });
+  };
+
+  document.querySelectorAll("a").forEach(handleClicks);
+  // Custom handling for the header sign-out link
+  const signOutLink = document.querySelector(`[href='${signOutPath}']`);
+  if (signOutLink) {
+    signOutLink.addEventListener("click", (event) => {
+      event.preventDefault();
+      event.stopPropagation();
+
+      exitLinkText = signOutLink.textContent;
+      openExitNotification(signOutLink.getAttribute("href"), signOutLink.dataset.method);
+    });
+  }
+
+  // Custom handling for links that open the exit notification dialog
+  const dialogOpenLinks = document.querySelectorAll("a[data-dialog-open='exit-proposal-notification']");
+  dialogOpenLinks.forEach((link) => {
+    link.addEventListener("click", () => {
+      exitLinkText = defaultExitLinkText;
+      openExitNotification(defaultExitUrl);
+    });
+  });
+});

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

@@ -103,11 +103,213 @@
   }
 
   &__grid-text-title {
-    @apply flex justify-between flex-row md:flex-col items-start lg:flex-row;
+    @apply flex justify-between items-start gap-y-2 flex-row md:flex-col lg:flex-row;
   }
 
-  &__list-metadata {
-    @apply md:flex md:flex-row md:items-center md:gap-4;
+  &__list-list {
+    .card__proposals-item {
+      @apply border-l-4 rounded-[10px] md:rounded-md border-[#F5F5F5] hover:border-tertiary focus-visible:border-tertiary flex flex-col md:items-center md:flex-row md:justify-between min-h-0 md:min-h-[127px] lg:min-h-[83px];
+
+      .card__list {
+        @apply border-none md:flex-1 md:items-center mb-2.5 md:mb-0;
+
+        &-metadata {
+          @apply gap-y-1 md:flex md:flex-row md:items-center md:gap-x-2 md:mt-0.5;
+
+          & > div {
+            @apply md:pl-0 md:pr-2;
+          }
+        }
+      }
+
+      .card__proposals-votes {
+        @apply flex md:flex-col lg:flex-row bg-[#F5F5F5] justify-around md:flex-[0.6] lg:flex-[0.8] items-center md:items-stretch lg:items-center lg:min-w-[196px] h-[68px] md:h-auto lg:h-[68px];
+
+        button {
+          @apply bg-[var(--secondary)];
+
+          &[disabled] {
+            @apply opacity-50 text-white border-current cursor-not-allowed;
+          }
+        }
+
+        &-limited {
+          @apply flex flex-col p-0 md:p-2 lg:p-0 md:flex-row md:flex-wrap lg:flex-col lg:flex-nowrap flex-1 md:flex-none lg:flex-1 justify-center;
+
+          .proposals__aside-progress {
+            @apply flex flex-col w-[80%] items-center md:order-1 lg:order-none md:flex-[1_1_100%] lg:flex-none;
+          }
+
+          .progress-bar {
+            &__units {
+              @apply md:order-none;
+            }
+
+            &__number {
+              span {
+                @apply text-sm;
+              }
+            }
+          }
+        }
+
+        &-unlimited {
+          @apply flex flex-row flex-1 lg:flex-1 justify-center md:flex-none;
+
+          .progress-bar__number {
+            span {
+              @apply text-xl;
+            }
+          }
+        }
+
+        .progress-bar {
+          &__number {
+            @apply flex justify-center items-center mx-1;
+
+            span {
+              @apply leading-6 font-semibold #{!important};
+            }
+          }
+
+          &__units {
+            @apply text-sm leading-[22px] mx-1;
+          }
+        }
+
+        &-container {
+          @apply flex-1 md:w-full md:flex-none p-2 lg:p-0 lg:m-3 lg:w-[164px];
+
+          button {
+            @apply text-sm leading-[18px] font-semibold py-1;
+          }
+        }
+
+        &-hidden {
+          @apply md:flex-none;
+
+          .card__proposals-votes-container {
+            @apply md:flex-none md:w-[130px] lg:w-[164px] lg:p-0;
+          }
+        }
+      }
+    }
+  }
+
+  &__grid-grid {
+    @apply md:gap-4;
+
+    .card__proposals-item {
+      @apply flex flex-col justify-between border-solid border-[#D3D5D9] border rounded;
+
+      .card__grid {
+        @apply mb-2.5 md:mb-0 flex-1 justify-between;
+
+        box-shadow: none !important;
+
+        &-text {
+          @apply justify-end lg:gap-y-8;
+        }
+
+        &-metadata {
+          @apply mt-0 md:flex md:flex-row;
+
+          > :first-child {
+            flex: 1 !important;
+          }
+
+          > :not(:first-child) {
+            @apply border-r border-[#D3D5D9];
+          }
+
+          & > div {
+            @apply px-2 md:px-2;
+          }
+        }
+      }
+
+      .card__proposals-votes {
+        @apply flex bg-[#F5F5F5EE] justify-around items-center px-4 h-[68px];
+
+        button {
+          @apply bg-[var(--secondary)];
+
+          &[disabled] {
+            @apply opacity-50 text-white border-current cursor-not-allowed;
+          }
+        }
+
+        &-limited {
+          @apply flex flex-col lg:items-start flex-1 md:mr-4 lg:mr-0;
+
+          .proposals__aside-progress {
+            @apply flex flex-col-reverse w-[80%] items-center;
+          }
+
+          .progress-bar {
+            &__number {
+              @apply lg:w-[80%];
+
+              span {
+                @apply text-sm;
+              }
+            }
+
+            &__units {
+              @apply flex justify-center w-[80%];
+            }
+          }
+        }
+
+        &-unlimited {
+          @apply flex flex-row flex-1 lg:flex-1 md:flex-none md:mr-4 lg:mr-0;
+
+          .progress-bar__number {
+            span {
+              @apply text-xl;
+            }
+          }
+        }
+
+        .progress-bar {
+          &__number {
+            @apply flex justify-center items-center;
+
+            span {
+              @apply leading-6 font-semibold #{!important};
+            }
+          }
+
+          &__units {
+            @apply text-sm leading-[22px] mx-1;
+          }
+        }
+
+        &-container {
+          @apply flex-1 lg:w-[164px] lg:flex-none;
+
+          button {
+            @apply text-sm leading-[18px] font-semibold py-1;
+
+            .already-voted-icon {
+              @apply w-4 h-4;
+            }
+          }
+        }
+
+        &-hidden {
+          @apply md:justify-end;
+
+          .card__proposals-votes-container {
+            @apply md:flex-none md:w-[130px] lg:w-[164px];
+
+            button {
+              @apply md:px-0 lg:px-6;
+            }
+          }
+        }
+      }
+    }
   }
 }
 
@@ -129,3 +331,46 @@
     }
   }
 }
+
+#proposal-voting-rules {
+  [id="dropdown-menu-proposal-voting-rules"] {
+    &[aria-hidden="true"] {
+      @apply hidden;
+    }
+  }
+
+  [data-target="dropdown-menu-proposal-voting-rules"] {
+    @apply w-full flex items-center justify-between gap-2 p-2 first-of-type:[&>svg]:block last-of-type:[&>svg]:hidden;
+
+    & > span {
+      @apply hidden font-semibold text-secondary;
+
+      &:only-of-type,
+      &.is-active {
+        @apply flex items-center gap-2 [&_svg]:fill-current;
+      }
+    }
+
+    > svg {
+      @apply w-6 h-6 flex-none text-secondary fill-current;
+    }
+
+    &[aria-expanded="false"] > svg:last-of-type,
+    &[aria-expanded="true"] > svg:first-of-type {
+      @apply hidden;
+    }
+
+    &[aria-expanded="true"] > svg:last-of-type,
+    &[aria-expanded="false"] > svg:first-of-type {
+      @apply block;
+    }
+  }
+}
+
+.layout-item-complementary {
+  @apply container grid grid-cols-1 lg:grid-cols-12;
+
+  .item_voting_rules {
+    @apply lg:col-start-2 lg:col-span-10;
+  }
+}

data/app/permissions/decidim/proposals/admin/permissions.rb

@@ -30,11 +30,8 @@ module Decidim
           # time limit.
           allow! if permission_action.subject == :proposal && permission_action.action == :edit && admin_edition_is_available?
 
-          # Every user allowed by the space can update the category of the proposal
-          allow! if permission_action.subject == :proposal_category && permission_action.action == :update
-
-          # Every user allowed by the space can update the scope of the proposal
-          allow! if permission_action.subject == :proposal_scope && permission_action.action == :update
+          # Every user allowed by the space can update the taxonomy of the proposal
+          allow! if permission_action.subject == :proposal_taxonomy && permission_action.action == :update
 
           # Every user allowed by the space can import proposals from another_component
           allow! if permission_action.subject == :proposals && permission_action.action == :import

data/app/permissions/decidim/proposals/permissions.rb

@@ -16,6 +16,8 @@ module Decidim
           apply_proposal_permissions(permission_action) unless permission_action.action == :read
         when :collaborative_draft
           apply_collaborative_draft_permissions(permission_action)
+        when :proposal_coauthor_invites
+          apply_proposal_coauthor_invites(permission_action)
         else
           permission_action
         end
@@ -159,6 +161,46 @@ module Decidim
 
         toggle_allow(collaborative_draft.created_by?(user))
       end
+
+      def apply_proposal_coauthor_invites(permission_action)
+        return toggle_allow(false) unless coauthor
+        return toggle_allow(false) unless proposal
+
+        case permission_action.action
+        when :invite
+          toggle_allow(valid_coauthor? && !notification_already_sent?)
+        when :cancel
+          toggle_allow(valid_coauthor? && notification_already_sent?)
+        when :accept, :decline
+          toggle_allow(can_be_coauthor?)
+        end
+      end
+
+      def coauthor
+        context.fetch(:coauthor, nil)
+      end
+
+      def notification_already_sent?
+        @notification_already_sent ||= proposal.coauthor_invitations_for(coauthor).any?
+      end
+
+      def coauthor_in_comments?
+        @coauthor_in_comments ||= proposal.comments.where(author: coauthor).any?
+      end
+
+      def valid_coauthor?
+        return false unless proposal.authors.include?(user)
+        return false unless proposal.user_has_actions?(coauthor)
+
+        coauthor_in_comments?
+      end
+
+      def can_be_coauthor?
+        return false unless user == coauthor
+        return false unless proposal.user_has_actions?(coauthor)
+
+        notification_already_sent?
+      end
     end
   end
 end

data/app/presenters/decidim/proposals/admin_log/proposal_presenter.rb

@@ -27,7 +27,7 @@ module Decidim
 
         def action_string
           case action
-          when "answer", "create", "update", "publish_answer"
+          when "answer", "create", "update", "publish_answer", "soft_delete", "restore"
             "decidim.proposals.admin_log.proposal.#{action}"
           else
             super

data/app/presenters/decidim/proposals/proposal_presenter.rb

@@ -40,7 +40,7 @@ module Decidim
       def title(links: false, extras: true, html_escape: false, all_locales: false)
         return unless proposal
 
-        super proposal.title, links, html_escape, all_locales, extras:
+        super(proposal.title, links, html_escape, all_locales, extras:)
       end
 
       def id_and_title(links: false, extras: true, html_escape: false)

data/app/queries/decidim/proposals/filtered_proposals.rb

@@ -28,8 +28,8 @@ module Decidim
       # by a range of dates.
       def query
         proposals = Decidim::Proposals::Proposal.where(component: @components)
-        proposals = proposals.where("created_at >= ?", @start_at) if @start_at.present?
-        proposals = proposals.where("created_at <= ?", @end_at) if @end_at.present?
+        proposals = proposals.where(created_at: @start_at..) if @start_at.present?
+        proposals = proposals.where(created_at: ..@end_at) if @end_at.present?
         proposals
       end
     end

data/app/queries/decidim/proposals/metrics/accepted_proposals_metric_manage.rb

@@ -18,13 +18,13 @@ module Decidim
           end
           @query = Decidim::Proposals::Proposal.where(component: visible_components_from_spaces(spaces)).joins(:component)
                                                .left_outer_joins(:category)
-          @query = @query.where("decidim_proposals_proposals.created_at <= ?", end_time).accepted
+          @query = @query.where(decidim_proposals_proposals: { created_at: ..end_time }).accepted
           @query = @query.group("decidim_categorizations.id", :participatory_space_type, :participatory_space_id)
           @query
         end
 
         def quantity
-          @quantity ||= query.where("decidim_proposals_proposals.created_at >= ?", start_time).count
+          @quantity ||= query.where(decidim_proposals_proposals: { created_at: start_time.. }).count
         end
       end
     end