decidim-proposals 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0e402f0c7815c844f71feab64eb6c0ba24eb0212
-  data.tar.gz: 6272d3cf287ba6e1bd302cf89165ead266c41fe5
+  metadata.gz: 119d43dd9b091b7b6f894a98cf5b1a5b0f8e5b38
+  data.tar.gz: 60787db2fd895e03b62fed6f3f69c69db1feb23c
 SHA512:
-  metadata.gz: 17d7d9569a92625edd185177e230107c81c41d6209fd6448180aa662bb939fa4169cc05838638b089ed3c9b73ee017fd5029e05ec098c4e5329944f461a94387
-  data.tar.gz: 8f91779210c7b26b0b133e0b9e941a9966088adfcd4c7ff99f389b1542073a57bc9380cd7244334a925a75ddf556f517576bb5fd047a1d89b0e68e1616fee80f
+  metadata.gz: 94c3bcb27db06bfe1010caa03994bab8601107a5b1b05fe18cb2d1bf04b3e3e6e5a72a7ab9615c12d45c1b29237e0fa1c14933035c8b8d4cf1bcd4c32e3727be
+  data.tar.gz: fd598e7e6ae7842ffb504f6d2fa600e447ef22681347a6489b31ee7fdc44a628e3355d5d36ce40061d19f61bb537a8139e95e6304e3eadcb0f3d71baf31a651c

data/app/controllers/decidim/proposals/proposals_controller.rb

@@ -8,6 +8,7 @@ module Decidim
       include FormFactory
       include FilterResource
       include Orderable
+      include Paginable
 
       helper_method :geocoded_proposals
 
@@ -30,7 +31,7 @@ module Decidim
                              []
                            end
 
-        @proposals = @proposals.page(params[:page]).per(12)
+        @proposals = paginate(@proposals)
         @proposals = reorder(@proposals)
       end
 

data/app/forms/decidim/proposals/admin/proposal_answer_form.rb

@@ -11,7 +11,7 @@ module Decidim
         translatable_attribute :answer, String
         attribute :state, String
 
-        validates :state, presence: true, inclusion: { in: %w(accepted rejected) }
+        validates :state, presence: true, inclusion: { in: %w(accepted rejected evaluating) }
         validates :answer, translatable_presence: true, if: ->(form) { form.state == "rejected" }
       end
     end

data/app/helpers/decidim/proposals/application_helper.rb

@@ -17,15 +17,7 @@ module Decidim
       #
       # Returns a String.
       def humanize_proposal_state(state)
-        value = if state == "accepted"
-                  "accepted"
-                elsif state == "rejected"
-                  "rejected"
-                else
-                  "not_answered"
-                end
-
-        I18n.t(value, scope: "decidim.proposals.answers")
+        I18n.t(state, scope: "decidim.proposals.answers", default: :not_answered)
       end
 
       # Public: The css class applied based on the proposal state.
@@ -34,14 +26,34 @@ module Decidim
       #
       # Returns a String.
       def proposal_state_css_class(state)
-        if state == "accepted"
+        case state
+        when "accepted"
           "text-success"
-        elsif state == "rejected"
+        when "rejected"
           "text-alert"
+        when "evaluating"
+          "text-info"
         else
           "text-warning"
         end
       end
+
+      # Public: The css class applied based on the proposal state to
+      #         the proposal badge.
+      #
+      # state - The String state of the proposal.
+      #
+      # Returns a String.
+      def proposal_state_badge_css_class(state)
+        case state
+        when "accepted"
+          "success"
+        when "rejected"
+          "warning"
+        when "evaluating"
+          "secondary"
+        end
+      end
     end
   end
 end

data/app/models/decidim/proposals/abilities/{admin_user.rb → admin_ability.rb}

@@ -5,16 +5,9 @@ module Decidim
     module Abilities
       # Defines the abilities related to proposals for a logged in admin user.
       # Intended to be used with `cancancan`.
-      class AdminUser
-        include CanCan::Ability
-
-        attr_reader :user, :context
-
-        def initialize(user, context)
-          return unless user && user.role?(:admin)
-
-          @user = user
-          @context = context
+      class AdminAbility < Decidim::Abilities::AdminAbility
+        def define_abilities
+          super
 
           can :manage, Proposal
           can :unreport, Proposal
@@ -26,11 +19,11 @@ module Decidim
         private
 
         def current_settings
-          context.fetch(:current_settings, nil)
+          @context.fetch(:current_settings, nil)
         end
 
         def feature_settings
-          context.fetch(:feature_settings, nil)
+          @context.fetch(:feature_settings, nil)
         end
 
         def can_create_proposal?

data/app/models/decidim/proposals/abilities/{current_user.rb → current_user_ability.rb}

@@ -5,7 +5,7 @@ module Decidim
     module Abilities
       # Defines the abilities related to proposals for a logged in user.
       # Intended to be used with `cancancan`.
-      class CurrentUser
+      class CurrentUserAbility
         include CanCan::Ability
 
         attr_reader :user, :context
@@ -27,7 +27,6 @@ module Decidim
           can :create, Proposal if authorized?(:create) && creation_enabled?
 
           can :report, Proposal
-          can :report, Decidim::Comments::Comment
         end
 
         private

data/app/models/decidim/proposals/abilities/{process_admin_user.rb → participatory_process_admin_ability.rb}

@@ -5,22 +5,14 @@ module Decidim
     module Abilities
       # Defines the abilities related to proposals for a logged in process admin user.
       # Intended to be used with `cancancan`.
-      class ProcessAdminUser
-        include CanCan::Ability
+      class ParticipatoryProcessAdminAbility < Decidim::Abilities::ParticipatoryProcessAdminAbility
+        def define_participatory_process_abilities
+          super
 
-        attr_reader :user, :context
-
-        def initialize(user, context)
-          return unless user && !user.role?(:admin)
-
-          @user = user
-          @context = context
-
-          can :manage, Proposal do |proposal|
-            participatory_processes.include?(proposal.feature.participatory_process)
+          can [:manage, :unreport, :hide], Proposal do |proposal|
+            can_manage_process?(proposal.feature.participatory_process)
           end
-          can :unreport, Proposal
-          can :hide, Proposal
+
           cannot :create, Proposal unless can_create_proposal?
           cannot :update, Proposal unless can_update_proposal?
         end
@@ -28,31 +20,27 @@ module Decidim
         private
 
         def current_settings
-          context.fetch(:current_settings, nil)
+          @context.fetch(:current_settings, nil)
         end
 
         def feature_settings
-          context.fetch(:feature_settings, nil)
+          @context.fetch(:feature_settings, nil)
         end
 
         def current_feature
-          context.fetch(:current_feature, nil)
+          @context.fetch(:current_feature, nil)
         end
 
         def can_create_proposal?
           current_settings.try(:creation_enabled?) &&
             feature_settings.try(:official_proposals_enabled) &&
-            participatory_processes.include?(current_feature.try(:participatory_process))
+            can_manage_process?(current_feature.try(:participatory_process))
         end
 
         def can_update_proposal?
           current_settings.try(:proposal_answering_enabled) &&
             feature_settings.try(:proposal_answering_enabled)
         end
-
-        def participatory_processes
-          @participatory_processes ||= Decidim::Admin::ManageableParticipatoryProcessesForUser.for(@user)
-        end
       end
     end
   end

data/app/models/decidim/proposals/abilities/participatory_process_moderator_ability.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Proposals
+    module Abilities
+      # Defines the abilities related to proposals for a logged in process admin user.
+      # Intended to be used with `cancancan`.
+      class ParticipatoryProcessModeratorAbility < Decidim::Abilities::ParticipatoryProcessModeratorAbility
+        def define_participatory_process_abilities
+          super
+
+          can [:unreport, :hide], Proposal do |proposal|
+            can_manage_process?(proposal.feature.participatory_process)
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -23,6 +23,7 @@ module Decidim
 
       scope :accepted, -> { where(state: "accepted") }
       scope :rejected, -> { where(state: "rejected") }
+      scope :evaluating, -> { where(state: "evaluating") }
 
       def self.order_randomly(seed)
         transaction do
@@ -67,6 +68,13 @@ module Decidim
         state == "rejected"
       end
 
+      # Public: Checks if the organization has marked the proposal as evaluating it.
+      #
+      # Returns Boolean.
+      def evaluating?
+        state == "evaluating"
+      end
+
       # Public: Overrides the `commentable?` Commentable concern method.
       def commentable?
         feature.settings.comments_enabled?

data/app/services/decidim/proposals/proposal_search.rb

@@ -47,10 +47,13 @@ module Decidim
 
       # Handle the state filter
       def search_state
-        if state == "accepted"
+        case state
+        when "accepted"
           query.accepted
-        elsif state == "rejected"
+        when "rejected"
           query.rejected
+        when "evaluating"
+          query.evaluating
         else # Assume 'all'
           query
         end

data/app/views/decidim/proposals/admin/proposal_answers/edit.html.erb

@@ -6,7 +6,7 @@
 
     <div class="card-section">
       <div class="row column">
-        <%= f.collection_radio_buttons :state, [["accepted", t('.accepted')], ["rejected", t('.rejected')]], :first, :last, prompt: true %>
+        <%= f.collection_radio_buttons :state, [["accepted", t('.accepted')], ["rejected", t('.rejected')], ["evaluating", t('.evaluating')]], :first, :last, prompt: true %>
       </div>
 
       <div class="row column">

data/app/views/decidim/proposals/proposals/_filters.html.erb

@@ -17,7 +17,7 @@
   <% end %>
 
   <% if feature_settings.proposal_answering_enabled && current_settings.proposal_answering_enabled %>
-    <%= form.collection_radio_buttons :state, [["all", t('.all')], ["accepted", t('.accepted')], ["rejected", t('.rejected')]], :first, :last, legend_title: t('.state') %>
+    <%= form.collection_radio_buttons :state, [["all", t('.all')], ["accepted", t('.accepted')], ["rejected", t('.rejected')], ["evaluating", t('.evaluating')]], :first, :last, legend_title: t('.state') %>
   <% end %>
 
   <% if linked_classes_for(Decidim::Proposals::Proposal).any? %>

data/app/views/decidim/proposals/proposals/_proposal_badge.html.erb

@@ -1,3 +1,3 @@
 <% if proposal.answered? %>
-  <span class="<%= (proposal.accepted? ? 'success' : 'warning') %> label proposal-status"><%= humanize_proposal_state proposal.state %></span>
+  <span class="<%= proposal_state_badge_css_class(proposal.state) %> label proposal-status"><%= humanize_proposal_state proposal.state %></span>
 <% end %>

data/app/views/decidim/proposals/proposals/_proposals.html.erb

@@ -1,4 +1,13 @@
-<%= order_selector available_orders, i18n_scope: "decidim.proposals.proposals.orders" %>
+<div class="collection-sort-controls row small-up-1 medium-up-3 card-grid">
+  <div class="column">
+    <%= order_selector available_orders, i18n_scope: "decidim.proposals.proposals.orders" %>
+  </div>
+
+  <div class="column">
+    <%= render partial: "decidim/shared/results_per_page" %>
+  </div>
+</div>
+
 <div class="row small-up-1 medium-up-2 card-grid">
   <%= render @proposals %>
 </div>

data/config/locales/ca.yml

@@ -48,6 +48,7 @@ ca:
           edit:
             accepted: Acceptada
             answer_proposal: Resposta
+            evaluating: En avaluació
             rejected: Rebutjada
             title: Respondre a la proposta %{title}
         proposals:
@@ -66,6 +67,7 @@ ca:
             title: Crear proposta
       answers:
         accepted: Acceptada
+        evaluating: En avaluació
         not_answered: No resposta
         rejected: Rebutjada
       create:
@@ -97,6 +99,7 @@ ca:
           category: Categoria
           category_prompt: Selecciona una categoria
           citizenship: Ciutadania
+          evaluating: En avaluació
           official: Oficial
           origin: Origen
           rejected: Rebutjades

data/config/locales/en.yml

@@ -50,6 +50,7 @@ en:
           edit:
             accepted: Accepted
             answer_proposal: Answer
+            evaluating: Evaluating
             rejected: Rejected
             title: Answer for proposal %{title}
         proposals:
@@ -68,6 +69,7 @@ en:
             title: Create proposal
       answers:
         accepted: Accepted
+        evaluating: Evaluating
         not_answered: Not answered
         rejected: Rejected
       create:
@@ -99,6 +101,7 @@ en:
           category: Category
           category_prompt: Select a category
           citizenship: Citizenship
+          evaluating: Evaluating
           official: Official
           origin: Origin
           rejected: Rejected

data/config/locales/es.yml

@@ -48,6 +48,7 @@ es:
           edit:
             accepted: Aceptada
             answer_proposal: Respuesta
+            evaluating: En evaluación
             rejected: Rechazada
             title: Responder a la propuesta %{title}
         proposals:
@@ -66,6 +67,7 @@ es:
             title: Crear propuesta
       answers:
         accepted: Aceptadas
+        evaluating: En evaluación
         not_answered: No contestada
         rejected: Rechazada
       create:
@@ -97,6 +99,7 @@ es:
           category: Categoría
           category_prompt: Selecciona una categoría
           citizenship: Ciudadanía
+          evaluating: En evaluación
           official: Oficial
           origin: Origen
           rejected: Rechazadas

data/lib/decidim/proposals/admin_engine.rb

@@ -18,8 +18,11 @@ module Decidim
 
       initializer "decidim_proposals.inject_abilities_to_user" do |_app|
         Decidim.configure do |config|
-          config.admin_abilities += ["Decidim::Proposals::Abilities::AdminUser"]
-          config.admin_abilities += ["Decidim::Proposals::Abilities::ProcessAdminUser"]
+          config.admin_abilities += [
+            "Decidim::Proposals::Abilities::AdminAbility",
+            "Decidim::Proposals::Abilities::ParticipatoryProcessAdminAbility",
+            "Decidim::Proposals::Abilities::ParticipatoryProcessModeratorAbility"
+          ]
         end
       end
 

data/lib/decidim/proposals/engine.rb

@@ -25,7 +25,7 @@ module Decidim
 
       initializer "decidim_proposals.inject_abilities_to_user" do |_app|
         Decidim.configure do |config|
-          config.abilities += ["Decidim::Proposals::Abilities::CurrentUser"]
+          config.abilities += ["Decidim::Proposals::Abilities::CurrentUserAbility"]
         end
       end
     end

data/lib/decidim/proposals/feature.rb

@@ -114,6 +114,10 @@ Decidim.register_feature(:proposals) do |feature|
           proposal.answered_at = Time.current
           proposal.answer = Decidim::Faker::Localized.sentence(10)
           proposal.save!
+        elsif n > 6
+          proposal.state = "evaluating"
+          proposal.answered_at = Time.current
+          proposal.save!
         end
 
         rand(3).times do |m|

data/lib/decidim/proposals/proposal_serializer.rb

@@ -21,6 +21,10 @@ module Decidim
             id: @proposal.category.try(:id),
             name: @proposal.category.try(:name)
           },
+          scope: {
+            id: @proposal.scope.try(:id),
+            name: @proposal.scope.try(:name)
+          },
           title: @proposal.title,
           body: @proposal.body,
           votes: @proposal.proposal_votes_count,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: decidim-proposals
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Josep Jaume Rey Peroy
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-12 00:00:00.000000000 Z
+date: 2017-08-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: decidim-core
@@ -18,28 +18,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
 - !ruby/object:Gem::Dependency
   name: decidim-comments
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
 - !ruby/object:Gem::Dependency
   name: rectify
   requirement: !ruby/object:Gem::Requirement
@@ -88,56 +88,56 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
 - !ruby/object:Gem::Dependency
   name: decidim-meetings
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
 - !ruby/object:Gem::Dependency
   name: decidim-results
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
 - !ruby/object:Gem::Dependency
   name: decidim-budgets
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.4.0
+        version: 0.4.1
 description: A proposals component for decidim's participatory processes.
 email:
 - josepjaume@gmail.com
@@ -175,9 +175,10 @@ files:
 - app/helpers/decidim/proposals/application_helper.rb
 - app/helpers/decidim/proposals/map_helper.rb
 - app/helpers/decidim/proposals/proposal_votes_helper.rb
-- app/models/decidim/proposals/abilities/admin_user.rb
-- app/models/decidim/proposals/abilities/current_user.rb
-- app/models/decidim/proposals/abilities/process_admin_user.rb
+- app/models/decidim/proposals/abilities/admin_ability.rb
+- app/models/decidim/proposals/abilities/current_user_ability.rb
+- app/models/decidim/proposals/abilities/participatory_process_admin_ability.rb
+- app/models/decidim/proposals/abilities/participatory_process_moderator_ability.rb
 - app/models/decidim/proposals/application_record.rb
 - app/models/decidim/proposals/proposal.rb
 - app/models/decidim/proposals/proposal_vote.rb