decidim-initiatives 0.23.2 → 0.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3e781acff60b29ae1a2008b0684335bd0208a09eb0bef5c9129f65e3f3d534f0
-  data.tar.gz: 2dab76e1b6b9e9b004246f1c22cdba69babc3885f3ad8836a05813ef600df3e6
+  metadata.gz: 557c6b43a7cda759d2aefa2b9893893e336ccef62cc3533f19df7902ade4694b
+  data.tar.gz: fa9a227e8937d5f8b42ee55505ef04b471f85630b3a265ddd655feab095c7d48
 SHA512:
-  metadata.gz: 5938e314ad1f71ea0a8f00baf6cfa9402eecb4cb5bde7a3a3d92d1d0a3c4bc7bf3b1e8c3d19e666683a861f262c33d5b6240144ccb258d130cc6521efe9b92e9
-  data.tar.gz: db5387c0893ccf865301f1a2f815d3925537be13608d6beb6ef7e2d99b14666da371f06bacbf088026585e207374ad47331122a7dc71c2cfc8226d13e4e51dcc
+  metadata.gz: b835b100231867c4ee8a0dbbe93fbecf67f98bc032dca5319d02ae5aacc32b5aef64a7cbe044263825d9fcb09909a8ba11e51e740a8ebfb608de0f88872b2e03
+  data.tar.gz: 2987d8c74edf38e2e5be4e103fe667ae3d153d591f13e88ab011d6e9e8f0a42e5642aeacf629181083df616e9251bf91ba241b56ec9c948532a5b40e63f618d3

data/README.md

@@ -53,7 +53,7 @@ Just use the following line:
 Decidim::Initiatives.do_not_require_authorization = true
 ```
 
-All the settings and their default values which can be overriden can be found in the file [`lib/decidim/initiatives.rb`](https://github.com/decidim/decidim/blob/master/decidim-initiatives/lib/decidim/initiatives.rb).
+All the settings and their default values which can be overriden can be found in the file [`lib/decidim/initiatives.rb`](https://github.com/decidim/decidim/blob/develop/decidim-initiatives/lib/decidim/initiatives.rb).
 
 For example, you can also change the minimum number of required committee members to 1 (default is 2) by adding this line:
 ```

data/app/assets/stylesheet/decidim/initiatives/initiatives.scss

@@ -57,3 +57,9 @@
     margin-bottom: 1.5rem;
   }
 }
+
+fieldset{
+  border: 1px solid #e8e8e8;
+  padding: .5em;
+  margin-bottom: 1em;
+}

data/app/assets/stylesheet/decidim/initiatives/print-initiative.css.scss

@@ -1,4 +1,10 @@
 @media all{
+  .printable{
+    width: 80%;
+    margin-left: auto;
+    margin-right: auto;
+  }
+
   .initiative-letterhead{
     font-size: 14pt;
     font-weight: bold;
@@ -104,6 +110,14 @@
 }
 
 @media print{
+  .process-header{
+    display: none;
+  }
+
+  .main-nav{
+    display: none;
+  }
+
   .title-bar{
     display: none;
   }
@@ -119,4 +133,12 @@
   .page-break{
     page-break-before: auto;
   }
+
+  .main-footer{
+    display: none;
+  }
+
+  .mini-footer{
+    display: none;
+  }
 }

data/app/cells/decidim/initiatives/content_blocks/highlighted_initiatives/show.erb

@@ -5,16 +5,12 @@
       <div class="row small-up-1 smallmedium-up-2 mediumlarge-up-3 large-up-4 card-grid">
         <% highlighted_initiatives.each do |initiative| %>
           <div class="column">
-            <div class="card card--initiative card--mini">
-              <%= link_to decidim_initiatives.initiative_path(initiative), class: "card__link" do %>
-                <span class="show-for-sr"><%= translated_attribute initiative.title %></span>
-              <% end %>
+            <%= link_to decidim_initiatives.initiative_path(initiative), class: "card card--initiative card--mini" do %>
+              <span class="show-for-sr"><%= translated_attribute initiative.title %></span>
               <div class="card__content">
-                <%= link_to decidim_initiatives.initiative_path(initiative), class: "card__link" do %>
-                  <span class="card__title"><%= decidim_html_escape(translated_attribute(initiative.title)) %></span>
-                <% end %>
+                <span class="card__title card__link"><%= decidim_html_escape(translated_attribute(initiative.title)) %></span>
               </div>
-            </div>
+            <% end %>
           </div>
         <% end %>
       </div>

data/app/cells/decidim/initiatives/content_blocks/highlighted_initiatives_cell.rb

@@ -16,9 +16,13 @@ module Decidim
           model.settings.max_results
         end
 
+        def order
+          model.settings.order
+        end
+
         def highlighted_initiatives
           @highlighted_initiatives ||= OrganizationPrioritizedInitiatives
-                                       .new(current_organization)
+                                       .new(current_organization, order)
                                        .query
                                        .limit(max_results)
         end

data/app/cells/decidim/initiatives/content_blocks/highlighted_initiatives_settings_form/show.erb

@@ -1,3 +1,4 @@
 <% form.fields_for :settings, form.object.settings do |settings_fields| %>
-  <%= settings_fields.select :max_results, [4, 8, 12], prompt: "", label: label %>
+  <%= settings_fields.select :max_results, [4, 8, 12], prompt: "", label: max_results_label %>
+  <%= settings_fields.select :order, order_select, prompt: "", label: order_label %>
 <% end %>

data/app/cells/decidim/initiatives/content_blocks/highlighted_initiatives_settings_form_cell.rb

@@ -10,9 +10,20 @@ module Decidim
           options[:content_block]
         end
 
-        def label
+        def max_results_label
           I18n.t("decidim.initiatives.admin.content_blocks.highlighted_initiatives.max_results")
         end
+
+        def order_label
+          I18n.t("decidim.initiatives.admin.content_blocks.highlighted_initiatives.order.label")
+        end
+
+        def order_select
+          [
+            [I18n.t("decidim.initiatives.admin.content_blocks.highlighted_initiatives.order.default"), "default"],
+            [I18n.t("decidim.initiatives.admin.content_blocks.highlighted_initiatives.order.most_recent"), "most_recent"]
+          ]
+        end
       end
     end
   end

data/app/cells/decidim/initiatives/initiative_m/tags.erb

@@ -1,5 +1,5 @@
 <% if model.hashtag.present? %>
-  <%= link_to_if(
+  <%= link_to(
     "##{hashtag}",
     "https://twitter.com/hashtag/#{hashtag}",
     class: "card__text--category",

data/app/commands/decidim/initiatives/admin/update_initiative.rb

@@ -80,9 +80,9 @@ module Decidim
           attrs[:state] = form.state if form.state
           attrs[:decidim_area_id] = form.area_id
 
-          if initiative.published?
-            @notify_extended = true if form.signature_end_date != initiative.signature_end_date &&
-                                       form.signature_end_date > initiative.signature_end_date
+          if initiative.published? && form.signature_end_date != initiative.signature_end_date &&
+             form.signature_end_date > initiative.signature_end_date
+            @notify_extended = true
           end
         end
 

data/app/commands/decidim/initiatives/admin/update_initiative_answer.rb

@@ -53,9 +53,9 @@ module Decidim
             attrs[:signature_start_date] = form.signature_start_date
             attrs[:signature_end_date] = form.signature_end_date
 
-            if initiative.published?
-              @notify_extended = true if form.signature_end_date != initiative.signature_end_date &&
-                                         form.signature_end_date > initiative.signature_end_date
+            if initiative.published? && form.signature_end_date != initiative.signature_end_date &&
+               form.signature_end_date > initiative.signature_end_date
+              @notify_extended = true
             end
           end
 

data/app/commands/decidim/initiatives/approve_membership_request.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Initiatives
+    # A command with all the business logic that creates a new initiative.
+    class ApproveMembershipRequest < Rectify::Command
+      # Public: Initializes the command.
+      #
+      # membership_request - A pending committee member
+      def initialize(membership_request)
+        @membership_request = membership_request
+      end
+
+      # Executes the command. Broadcasts these events:
+      #
+      # - :ok when everything is valid.
+      #
+      # Returns nothing.
+      def call
+        @membership_request.accepted!
+        notify_applicant
+
+        broadcast(:ok, @membership_request)
+      end
+
+      private
+
+      def notify_applicant
+        Decidim::EventsManager.publish(
+          event: "decidim.events.initiatives.approve_membership_request",
+          event_class: Decidim::Initiatives::ApproveMembershipRequestEvent,
+          resource: @membership_request.initiative,
+          affected_users: [@membership_request.user],
+          force_send: true,
+          extra: { author: @membership_request.initiative.author }
+        )
+      end
+    end
+  end
+end

data/app/commands/decidim/initiatives/create_initiative.rb

@@ -5,7 +5,7 @@ module Decidim
     # A command with all the business logic that creates a new initiative.
     class CreateInitiative < Rectify::Command
       include CurrentLocale
-      include AttachmentMethods
+      include ::Decidim::MultipleAttachmentsMethods
 
       # Public: Initializes the command.
       #
@@ -26,8 +26,8 @@ module Decidim
         return broadcast(:invalid) if form.invalid?
 
         if process_attachments?
-          build_attachment
-          return broadcast(:invalid) if attachment_invalid?
+          build_attachments
+          return broadcast(:invalid) if attachments_invalid?
         end
 
         initiative = create_initiative
@@ -51,7 +51,8 @@ module Decidim
         initiative.transaction do
           initiative.save!
           @attached_to = initiative
-          create_attachment if process_attachments?
+          create_attachments if process_attachments?
+
           create_components_for(initiative)
           send_notification(initiative)
           add_author_as_follower(initiative)

data/app/commands/decidim/initiatives/revoke_membership_request.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Initiatives
+    # A command with all the business logic that creates a new initiative.
+    class RevokeMembershipRequest < Rectify::Command
+      # Public: Initializes the command.
+      #
+      # membership_request - A pending committee member
+      def initialize(membership_request)
+        @membership_request = membership_request
+      end
+
+      # Executes the command. Broadcasts these events:
+      #
+      # - :ok when everything is valid.
+      #
+      # Returns nothing.
+      def call
+        @membership_request.rejected!
+        notify_applicant
+
+        broadcast(:ok, @membership_request)
+      end
+
+      private
+
+      def notify_applicant
+        Decidim::EventsManager.publish(
+          event: "decidim.events.initiatives.revoke_membership_request",
+          event_class: Decidim::Initiatives::RevokeMembershipRequestEvent,
+          resource: @membership_request.initiative,
+          affected_users: [@membership_request.user],
+          force_send: true,
+          extra: { author: @membership_request.initiative.author }
+        )
+      end
+    end
+  end
+end

data/app/commands/decidim/initiatives/send_initiative_to_technical_validation.rb

@@ -0,0 +1,57 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Initiatives
+    # A command with all the business logic that sends an
+    # existing initiative to technical validation.
+    class SendInitiativeToTechnicalValidation < Rectify::Command
+      # Public: Initializes the command.
+      #
+      # initiative - Decidim::Initiative
+      # current_user - the user performing the action
+      def initialize(initiative, current_user)
+        @initiative = initiative
+        @current_user = current_user
+      end
+
+      # Executes the command. Broadcasts these events:
+      #
+      # - :ok when everything is valid.
+      # - :invalid if the form wasn't valid and we couldn't proceed.
+      #
+      # Returns nothing.
+      def call
+        @initiative = Decidim.traceability.perform_action!(
+          :send_to_technical_validation,
+          initiative,
+          current_user
+        ) do
+          initiative.validating!
+          initiative
+        end
+
+        notify_admins
+
+        broadcast(:ok, initiative)
+      end
+
+      private
+
+      attr_reader :initiative, :current_user
+
+      def notify_admins
+        affected_users = Decidim::User.org_admins_except_me(current_user).all
+
+        data = {
+          event: "decidim.events.initiatives.initiative_sent_to_technical_validation",
+          event_class: Decidim::Initiatives::InitiativeSentToTechnicalValidationEvent,
+          resource: initiative,
+          affected_users: affected_users,
+          force_send: true
+        }
+
+        Decidim::EventsManager.publish(data)
+      end
+    end
+  end
+end

data/app/commands/decidim/initiatives/spawn_committee_request.rb

@@ -26,6 +26,7 @@ module Decidim
         request = create_request
 
         if request.persisted?
+          notify_author
           broadcast(:ok, request)
         else
           broadcast(:invalid, request)
@@ -47,6 +48,23 @@ module Decidim
         request.save
         request
       end
+
+      def notify_author
+        return if initiative.author == current_user
+
+        Decidim::EventsManager.publish(
+          event: "decidim.events.initiatives.spawn_committee_request",
+          event_class: Decidim::Initiatives::SpawnCommitteeRequestEvent,
+          resource: initiative,
+          affected_users: [initiative.author],
+          force_send: true,
+          extra: { applicant: current_user }
+        )
+      end
+
+      def initiative
+        @initiative ||= Decidim::Initiative.find(form.initiative_id)
+      end
     end
   end
 end

data/app/commands/decidim/initiatives/update_initiative.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Initiatives
+    # A command with all the business logic that updates an
+    # existing initiative.
+    class UpdateInitiative < Rectify::Command
+      include ::Decidim::MultipleAttachmentsMethods
+      include CurrentLocale
+
+      # Public: Initializes the command.
+      #
+      # initiative - Decidim::Initiative
+      # form       - A form object with the params.
+      def initialize(initiative, form, current_user)
+        @form = form
+        @initiative = initiative
+        @current_user = current_user
+        @attached_to = initiative
+      end
+
+      # Executes the command. Broadcasts these events:
+      #
+      # - :ok when everything is valid.
+      # - :invalid if the form wasn't valid and we couldn't proceed.
+      #
+      # Returns nothing.
+      def call
+        return broadcast(:invalid) if form.invalid?
+
+        if process_attachments?
+          @initiative.attachments.where(id: form.documents).destroy_all
+
+          build_attachments
+          return broadcast(:invalid) if attachments_invalid?
+        end
+        @initiative = Decidim.traceability.update!(
+          initiative,
+          current_user,
+          attributes
+        )
+
+        create_attachments if process_attachments?
+        broadcast(:ok, initiative)
+      rescue ActiveRecord::RecordInvalid
+        broadcast(:invalid, initiative)
+      end
+
+      private
+
+      attr_reader :form, :initiative, :current_user
+
+      def attributes
+        attrs = {
+          title: { current_locale => form.title },
+          description: { current_locale => form.description },
+          hashtag: form.hashtag
+        }
+
+        if form.signature_type_updatable?
+          attrs[:signature_type] = form.signature_type
+          attrs[:scoped_type_id] = form.scoped_type_id if form.scoped_type_id
+        end
+
+        if initiative.created?
+          attrs[:signature_end_date] = form.signature_end_date if initiative.custom_signature_end_date_enabled?
+          attrs[:decidim_area_id] = form.area_id if initiative.area_enabled?
+        end
+
+        attrs
+      end
+    end
+  end
+end