decidim-accountability 0.20.0 → 0.23.1.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9beb43f5bf3c16a46ec170ca79f1e94220465f3139dd4f99aab9d547f385fc99
-  data.tar.gz: 4e06b956040827466179fa7b336fc9f9e02bff131feeefa085714c479ee89745
+  metadata.gz: bb033d61df2ae5da6ff56d0124bf0ace65a2288f76a89cc89795bdd7f52e51bf
+  data.tar.gz: 3ec3d80a20393f5fa4c1c5b330c49037716a8794b2a7e49cde58b088e4e0bcc0
 SHA512:
-  metadata.gz: 913c8837866d582dcd9f9fc3ecc799b7b388accaf3f9f16339b98d36037a41246a0a1c5353ac54ca3df0e0be48b4cad4a744020a4bcc0ecbc4983c0aa32c5a65
-  data.tar.gz: 473097609d8d99e6e90ae81dea70da7c382b347e21e4963db46e770cb1bfef2d468c87ad76d8312e632335882319d8747328dc332d8c9925ecdd21778bab628e
+  metadata.gz: 28ba61333d2aeb51780548372f94ffd7123648c58a1b01f73de84310c169e9d6df01c0c9e4caf5820b8a8a55951113b1a8ac84f26e0f4d4ff5612771f23cae7e
+  data.tar.gz: a21713c1fa2238d9f64851b1ed9afc05d45d850e70e49bda306f072b87af6689ce9995d4e2cbae0a3670c0c4517bdc95ac65b9a7391dd16789a882f7444fa908

data/app/assets/javascripts/decidim/accountability/admin/accountability_admin.js.es6

@@ -1,47 +1,11 @@
 // = require_self
 
-$("#result_decidim_accountability_status_id").change(function () {
-  /* eslint-disable no-invalid-this */
-  const progress = $(this).find(":selected").data("progress")
-  if (progress || progress === 0) {
-    $("#result_progress").val(progress);
-  }
-});
-
 $(function() {
-  $(document).on("open.zf.reveal", "#data_picker-modal", function () {
-    let xhr = null;
-
-    $("#data_picker-autocomplete").autoComplete({
-      minChars: 2,
-      source: function(term, response) {
-        try {
-          xhr.abort();
-        } catch (exception) { xhr = null}
-
-        let url = $("#proposal-picker-choose").attr("href")
-        xhr = $.getJSON(
-          url,
-          { term: term },
-          function(data) { response(data); }
-        );
-      },
-      renderItem: function (item, search) {
-        let sanitizedSearch = search.replace(/[-/\\^$*+?.()|[\]{}]/g, "\\$&");
-        let re = new RegExp(`(${sanitizedSearch.split(" ").join("|")})`, "gi");
-        let title = item[0]
-        let modelId = item[1]
-        let val = `#${modelId}- ${title}`;
-        return `<div class="autocomplete-suggestion" data-model-id="${modelId}" data-val ="${title}">${val.replace(re, "<b>$1</b>")}</div>`;
-      },
-      onSelect: function(event, term, item) {
-        let choose = $("#proposal-picker-choose")
-        let modelId = item.data("modelId")
-        let val = `#${modelId}- ${item.data("val")}`;
-        choose.data("picker-value", modelId)
-        choose.data("picker-text", val)
-        choose.data("picker-choose", "")
-      }
-    })
+  $("#result_decidim_accountability_status_id").change(function () {
+    /* eslint-disable no-invalid-this */
+    const progress = $(this).find(":selected").data("progress")
+    if (progress || progress === 0) {
+      $("#result_progress").val(progress);
+    }
   });
 })

data/app/assets/stylesheets/decidim/accountability/accountability/_categories.scss

@@ -1,5 +1,13 @@
 .accountability{
   .categories{
+    a:hover{
+      text-decoration: underline;
+    }
+
+    a:focus{
+      outline: $anchor-outline-focus;
+    }
+
     .categories--header{
       border-bottom: 1px solid $medium-gray;
       padding-bottom: 1rem;
@@ -10,7 +18,6 @@
       }
 
       span{
-        text-transform: uppercase;
         font-size: 1.15rem;
         color: lighten($dark-gray, 50);
       }

data/app/assets/stylesheets/decidim/accountability/accountability/_lines_breadcrumb.scss

@@ -13,6 +13,14 @@
   a{
     font-weight: 600;
     color: $dark-gray;
+
+    &:hover{
+      text-decoration: underline;
+    }
+
+    &:focus{
+      outline: 2px solid $secondary;
+    }
   }
 
   ul{

data/app/assets/stylesheets/decidim/accountability/accountability/_results.scss

@@ -10,7 +10,6 @@
 
       p{
         font-weight: 600;
-        text-transform: uppercase;
         margin-bottom: .25rem;
         display: inline-block;
         margin-right: .5rem;
@@ -44,7 +43,6 @@
   .scope-filters{
     div{
       color: lighten($dark-gray, 50);
-      text-transform: uppercase;
       margin-bottom: .15rem;
     }
 
@@ -107,7 +105,6 @@
       margin-bottom: .5rem;
 
       .progress-text{
-        text-transform: uppercase;
         color: lighten($dark-gray, 30);
       }
 

data/app/cells/decidim/accountability/highlighted_results_for_component/show.erb

@@ -1,7 +1,7 @@
 <section class="section row collapse highlighted_results">
-  <h4 class="section-heading">
+  <h3 class="section-heading">
     <%= translated_attribute(model.name) %> <a href="<%= main_component_path(model) %>" class="text-small"><%= t("decidim.participatory_spaces.highlighted_results.see_all", count: results_count) %></a>
-  </h4>
+  </h3>
 
   <div class="row">
     <div class="columns accountability">
@@ -16,6 +16,6 @@
   <%= link_to(
     t("decidim.participatory_spaces.highlighted_results.see_all", count: results_count),
     main_component_path(model),
-    class: "button button--sc light secondary button--right"
+    class: "button button--sc hollow button--right"
   ) %>
 </section>

data/app/cells/decidim/accountability/highlighted_results_for_component_cell.rb

@@ -25,7 +25,7 @@ module Decidim
       end
 
       def results_to_render
-        @results_to_render ||= results.limit(4)
+        @results_to_render ||= results.includes(:component, :status).limit(4)
       end
 
       def results_count

data/app/commands/decidim/accountability/admin/create_imported_result.rb

@@ -0,0 +1,106 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Accountability
+    module Admin
+      # This command is executed when the user creates a Result from the admin
+      # panel.
+      class CreateImportedResult < Rectify::Command
+        def initialize(form, parent_id = nil)
+          @form = form
+          @parent_id = parent_id
+        end
+
+        # Creates the result if valid.
+        #
+        # Broadcasts :ok if successful, :invalid otherwise.
+        def call
+          return broadcast(:invalid) if @form.invalid?
+
+          transaction do
+            create_result
+            link_meetings
+            link_proposals
+            link_projects
+            notify_proposal_followers
+          end
+
+          broadcast(:ok)
+        end
+
+        private
+
+        attr_reader :result
+
+        def create_result
+          params = {
+            component: @form.current_component,
+            scope: @form.scope,
+            category: @form.category,
+            parent_id: @parent_id,
+            title: @form.title,
+            description: @form.description,
+            start_date: @form.start_date,
+            end_date: @form.end_date,
+            progress: @form.progress,
+            decidim_accountability_status_id: @form.decidim_accountability_status_id,
+            external_id: @form.external_id.presence,
+            weight: @form.weight
+          }
+
+          @result = Decidim.traceability.create!(
+            Result,
+            @form.current_user,
+            params,
+            visibility: "all"
+          )
+        end
+
+        def proposals
+          @proposals ||= result.sibling_scope(:proposals).where(id: @form.proposal_ids)
+        end
+
+        def projects
+          @projects ||= result.sibling_scope(:projects).where(id: @form.project_ids)
+        end
+
+        def meeting_ids
+          @meeting_ids ||= proposals.flat_map do |proposal|
+            proposal.linked_resources(:meetings, "proposals_from_meeting").pluck(:id)
+          end.uniq
+        end
+
+        def meetings
+          @meetings ||= result.sibling_scope(:meetings).where(id: meeting_ids)
+        end
+
+        def link_proposals
+          result.link_resources(proposals, "included_proposals")
+        end
+
+        def link_projects
+          result.link_resources(projects, "included_projects")
+        end
+
+        def link_meetings
+          result.link_resources(meetings, "meetings_through_proposals")
+        end
+
+        def notify_proposal_followers
+          proposals.each do |proposal|
+            Decidim::EventsManager.publish(
+              event: "decidim.events.accountability.proposal_linked",
+              event_class: Decidim::Accountability::ProposalLinkedEvent,
+              resource: result,
+              affected_users: proposal.notifiable_identities,
+              followers: proposal.followers - proposal.notifiable_identities,
+              extra: {
+                proposal_id: proposal.id
+              }
+            )
+          end
+        end
+      end
+    end
+  end
+end

data/app/commands/decidim/accountability/admin/update_imported_result.rb

@@ -0,0 +1,110 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Accountability
+    module Admin
+      # This command is executed when the user changes a Result from the admin
+      # panel.
+      class UpdateImportedResult < Rectify::Command
+        # Initializes an UpdateResult Command.
+        #
+        # form - The form from which to get the data.
+        # result - The current instance of the result to be updated.
+        def initialize(form, result, parent_id = nil)
+          @form = form
+          @result = result
+          @parent_id = parent_id
+        end
+
+        # Updates the result if valid.
+        #
+        # Broadcasts :ok if successful, :invalid otherwise.
+        def call
+          return broadcast(:invalid) if form.invalid?
+
+          transaction do
+            update_result
+            link_proposals
+            link_meetings
+            link_projects
+            send_notifications if should_notify_followers?
+          end
+
+          broadcast(:ok)
+        end
+
+        private
+
+        attr_reader :result, :form
+
+        def update_result
+          Decidim.traceability.update!(
+            result,
+            form.current_user,
+            scope: @form.scope,
+            category: @form.category,
+            parent_id: @parent_id,
+            title: @form.title,
+            description: @form.description,
+            start_date: @form.start_date,
+            end_date: @form.end_date,
+            progress: @form.progress,
+            decidim_accountability_status_id: @form.decidim_accountability_status_id,
+            external_id: @form.external_id.presence,
+            weight: @form.weight
+          )
+        end
+
+        def proposals
+          @proposals ||= result.sibling_scope(:proposals).where(id: form.proposal_ids)
+        end
+
+        def projects
+          @projects ||= result.sibling_scope(:projects).where(id: form.project_ids)
+        end
+
+        def meeting_ids
+          @meeting_ids ||= proposals.flat_map do |proposal|
+            proposal.linked_resources(:meetings, "proposals_from_meeting").pluck(:id)
+          end.uniq
+        end
+
+        def meetings
+          @meetings ||= result.sibling_scope(:meetings).where(id: meeting_ids)
+        end
+
+        def link_proposals
+          result.link_resources(proposals, "included_proposals")
+        end
+
+        def link_projects
+          result.link_resources(projects, "included_projects")
+        end
+
+        def link_meetings
+          result.link_resources(meetings, "meetings_through_proposals")
+        end
+
+        def send_notifications
+          result.linked_resources(:proposals, "included_proposals").each do |proposal|
+            Decidim::EventsManager.publish(
+              event: "decidim.events.accountability.result_progress_updated",
+              event_class: Decidim::Accountability::ResultProgressUpdatedEvent,
+              resource: result,
+              affected_users: proposal.notifiable_identities,
+              followers: proposal.followers - proposal.notifiable_identities,
+              extra: {
+                progress: result.progress,
+                proposal_id: proposal.id
+              }
+            )
+          end
+        end
+
+        def should_notify_followers?
+          result.previous_changes["progress"].present?
+        end
+      end
+    end
+  end
+end

data/app/controllers/decidim/accountability/admin/import_results_controller.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Accountability
+    module Admin
+      # This controller allows an admin to import results from a csv file for the Accountability component
+      class ImportResultsController < Admin::ApplicationController
+        def new
+          @errors = []
+        end
+
+        def create
+          @csv_file = params[:csv_file]
+          redirect_to(new_import_path) && return if @csv_file.blank?
+
+          Decidim::Accountability::Admin::ImportResultsCSVJob.perform_later(current_user, current_component, @csv_file.read.force_encoding("utf-8").encode("utf-8"))
+
+          flash[:notice] = I18n.t("imports.create.success", scope: "decidim.accountability.admin")
+          redirect_to import_results_path(current_participatory_process, current_component)
+        end
+
+        private
+
+        def current_component
+          @current_component ||= current_participatory_process.components.find(params[:component_id])
+        end
+
+        def current_participatory_process
+          @current_participatory_process ||= ParticipatoryProcess.find_by(slug: params[:participatory_process_slug])
+        end
+      end
+    end
+  end
+end

data/app/controllers/decidim/accountability/admin/results_controller.rb

@@ -7,6 +7,8 @@ module Decidim
       class ResultsController < Admin::ApplicationController
         include Decidim::ApplicationHelper
         include Decidim::SanitizeHelper
+        include Decidim::Proposals::Admin::Picker
+
         helper_method :results, :parent_result, :parent_results, :statuses, :present
 
         def new
@@ -70,26 +72,6 @@ module Decidim
           end
         end
 
-        def proposals
-          respond_to do |format|
-            format.html do
-              render partial: "proposals"
-            end
-            format.json do
-              query = Decidim.find_resource_manifest(:proposals)
-                             .try(:resource_scope, current_component)&.order(title: :asc)
-              term = params[:term]
-              if term&.start_with?("#")
-                term.delete!("#")
-                query = query.where("CAST(id AS TEXT) LIKE ?", "#{term}%")
-              else
-                query = query.where("title ilike ?", "%#{params[:term]}%")
-              end
-              render json: query.all.collect { |p| [decidim_html_escape(present(p).title), p.id] }
-            end
-          end
-        end
-
         private
 
         def results

data/app/controllers/decidim/accountability/results_controller.rb

@@ -5,7 +5,6 @@ module Decidim
     # Exposes the result resource so users can view them
     class ResultsController < Decidim::Accountability::ApplicationController
       include FilterResource
-      helper Decidim::WidgetUrlsHelper
       helper Decidim::TraceabilityHelper
       helper Decidim::Accountability::BreadcrumbHelper