decidim-admin 0.0.3 → 0.0.5

data/app/controllers/decidim/admin/participatory_processes_controller.rb

@@ -24,9 +24,9 @@ module Decidim
         @form = form(ParticipatoryProcessForm).from_params(params)
 
         CreateParticipatoryProcess.call(@form) do
-          on(:ok) do
+          on(:ok) do |participatory_process|
             flash[:notice] = I18n.t("participatory_processes.create.success", scope: "decidim.admin")
-            redirect_to participatory_processes_path
+            redirect_to participatory_process_path(participatory_process)
           end
 
           on(:invalid) do
@@ -48,9 +48,9 @@ module Decidim
         @form = form(ParticipatoryProcessForm).from_params(params)
 
         UpdateParticipatoryProcess.call(@participatory_process, @form) do
-          on(:ok) do
+          on(:ok) do |participatory_process|
             flash[:notice] = I18n.t("participatory_processes.update.success", scope: "decidim.admin")
-            redirect_to participatory_processes_path
+            redirect_to participatory_process_path(participatory_process)
           end
 
           on(:invalid) do

data/app/controllers/decidim/admin/users_controller.rb

@@ -23,7 +23,9 @@ module Decidim
           organization: current_organization,
           invitation_instructions: "invite_admin",
           roles: %w(admin),
-          invited_by: current_user
+          invited_by: current_user,
+          comments_notifications: true,
+          replies_notifications: true
         }
 
         @form = form(InviteAdminForm).from_params(params.merge(default_params))

data/app/forms/decidim/admin/newsletter_form.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+module Decidim
+  module Admin
+    # The form that validates the data to construct a valid Newsletter.
+    class NewsletterForm < Decidim::Form
+      mimic :newsletter
+
+      include TranslatableAttributes
+
+      translatable_attribute :subject, String
+      translatable_attribute :body, String
+      validates :subject, :body, translatable_presence: true
+    end
+  end
+end

data/app/forms/decidim/admin/organization_form.rb

@@ -11,10 +11,17 @@ module Decidim
 
       attribute :name, String
       attribute :twitter_handler, String
+      attribute :facebook_handler, String
+      attribute :instagram_handler, String
+      attribute :youtube_handler, String
+      attribute :github_handler, String
       attribute :default_locale, String
       attribute :homepage_image
       attribute :logo
       attribute :favicon
+      attribute :official_url
+      attribute :official_img_header
+      attribute :official_img_footer
       attribute :show_statistics
 
       translatable_attribute :description, String
@@ -24,6 +31,9 @@ module Decidim
       validates :default_locale, presence: true
       validates :default_locale, inclusion: { in: :available_locales }
 
+      validates :official_img_header, file_size: { less_than_or_equal_to: ->(_record) { Decidim.maximum_attachment_size } }, file_content_type: { allow: ["image/jpeg", "image/png"] }
+      validates :official_img_footer, file_size: { less_than_or_equal_to: ->(_record) { Decidim.maximum_attachment_size } }, file_content_type: { allow: ["image/jpeg", "image/png"] }
+
       private
 
       def available_locales

data/app/forms/decidim/admin/participatory_process_form.rb

@@ -11,13 +11,16 @@ module Decidim
       translatable_attribute :subtitle, String
       translatable_attribute :description, String
       translatable_attribute :short_description, String
-      translatable_attribute :domain, String
       translatable_attribute :scope, String
+      translatable_attribute :developer_group, String
+      translatable_attribute :local_area, String
+      translatable_attribute :target, String
+      translatable_attribute :participatory_scope, String
+      translatable_attribute :participatory_structure, String
 
       mimic :participatory_process
 
       attribute :end_date, Date
-      attribute :developer_group, String
       attribute :slug, String
       attribute :hashtag, String
       attribute :promoted, Boolean
@@ -29,10 +32,10 @@ module Decidim
 
       validate :slug, :slug_uniqueness
 
-      validates :hero_image, file_size: { less_than_or_equal_to: lambda { |_record| Decidim.maximum_attachment_size } }, file_content_type: { allow: ['image/jpeg', 'image/png'] }
-      validates :banner_image, file_size: { less_than_or_equal_to:lambda { |_record| Decidim.maximum_attachment_size } }, file_content_type: { allow: ['image/jpeg', 'image/png'] }
+      validates :hero_image, file_size: { less_than_or_equal_to: ->(_record) { Decidim.maximum_attachment_size } }, file_content_type: { allow: ["image/jpeg", "image/png"] }
+      validates :banner_image, file_size: { less_than_or_equal_to: ->(_record) { Decidim.maximum_attachment_size } }, file_content_type: { allow: ["image/jpeg", "image/png"] }
 
-       private
+      private
 
       def slug_uniqueness
         return unless current_organization.participatory_processes.where(slug: slug).where.not(id: id).any?

data/app/forms/decidim/admin/permission_form.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+module Decidim
+  module Admin
+    # This form handles permissions for a particular action in the admin panel.
+    class PermissionForm < Form
+      attribute :authorization_handler_name, String
+      attribute :options, String
+
+      validate :sanitize
+      validate :options_is_valid_json
+
+      private
+
+      def sanitize
+        self.authorization_handler_name = nil if authorization_handler_name.blank?
+        self.options = nil if authorization_handler_name.blank?
+        self.options = nil if options.blank?
+      end
+
+      def options_is_valid_json
+        return unless options
+
+        result = JSON.parse(options)
+        errors.add(:options, :invalid_json) unless result.is_a?(Hash)
+        result
+      rescue JSON::ParserError
+        errors.add(:options, :invalid_json)
+      end
+    end
+  end
+end

data/app/forms/decidim/admin/permissions_form.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+module Decidim
+  module Admin
+    # This form handles a set of forms related to handling permissions
+    # in the admin panel.
+    class PermissionsForm < Form
+      mimic :feature_permissions
+
+      attribute :permissions, Hash[String => PermissionForm]
+
+      def valid?
+        super && permissions.values.all?(&:valid?)
+      end
+    end
+  end
+end

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

@@ -8,6 +8,7 @@ module Decidim
       include Decidim::TranslationsHelper
       include Decidim::HumanizeBooleansHelper
       include Decidim::AriaSelectedLinkToHelper
+      include Decidim::MetaTagsHelper
 
       def title
         current_organization.name

data/app/jobs/decidim/admin/newsletter_delivery_job.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+module Decidim
+  module Admin
+    # Custom ApplicationJob scoped to the admin panel.
+    #
+    class NewsletterDeliveryJob < ApplicationJob
+      queue_as :newsletter
+
+      def perform(user, newsletter)
+        NewsletterMailer.newsletter(user, newsletter).deliver_now
+
+        newsletter.with_lock do
+          newsletter.increment!(:total_deliveries)
+        end
+      end
+    end
+  end
+end

data/app/jobs/decidim/admin/newsletter_job.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+module Decidim
+  module Admin
+    # Custom ApplicationJob scoped to the admin panel.
+    #
+    class NewsletterJob < ApplicationJob
+      queue_as :newsletter
+
+      def perform(newsletter)
+        newsletter.with_lock do
+          raise "Newsletter already sent" if newsletter.sent?
+
+          newsletter.update_attributes!(
+            sent_at: Time.current,
+            total_recipients: recipients.count,
+            total_deliveries: 0
+          )
+        end
+
+        recipients.find_each do |user|
+          NewsletterDeliveryJob.perform_later(user, newsletter)
+        end
+      end
+
+      private
+
+      def recipients
+        @recipients ||= User.where(newsletter_notifications: true).where.not(email: nil)
+      end
+    end
+  end
+end

data/app/models/decidim/admin/abilities/admin_user.rb

@@ -15,9 +15,11 @@ module Decidim
           can :manage, Category
           can :manage, ParticipatoryProcessUserRole
           can [:create, :update, :index, :new, :read], StaticPage
+
           can [:update_slug, :destroy], [StaticPage, StaticPageForm] do |page|
             !StaticPage.default?(page.slug)
           end
+
           can [:read, :update], Decidim::Organization do |organization|
             organization == user.organization
           end
@@ -27,7 +29,9 @@ module Decidim
           can :manage, :admin_users
           can :manage, Attachment
           can :manage, Scope
+          can :manage, Newsletter
           can [:create, :index, :new, :read, :invite], User
+
           can [:destroy], [User] do |user_to_destroy|
             user != user_to_destroy
           end

data/app/models/decidim/admin/abilities/base.rb

@@ -12,6 +12,10 @@ module Decidim
         def initialize(user, context)
           merge ::Decidim::Ability.new(user, context)
           merge ParticipatoryProcessAdmin.new(user, context)
+
+          Decidim.admin_abilities.each do |ability|
+            merge ability.constantize.new(user, context)
+          end
         end
       end
     end

data/app/models/decidim/admin/abilities/collaborator_user.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+module Decidim
+  module Admin
+    module Abilities
+      # Defines the abilities for a collaborator user in the admin
+      # section. Intended to be used with `cancancan`.
+      class CollaboratorUser
+        include CanCan::Ability
+
+        def initialize(user, _context)
+          return unless user && user.role?(:collaborator)
+
+          can :manage, :admin_dashboard
+          can :preview, ParticipatoryProcess
+        end
+      end
+    end
+  end
+end

data/app/models/decidim/admin/abilities/participatory_process_admin.rb

@@ -14,7 +14,7 @@ module Decidim
         def initialize(user, _context)
           @user = user
 
-          return unless user && !user.role?(:admin)
+          return unless user && !user.role?(:admin) && !user.role?(:collaborator)
 
           can :read, :admin_dashboard do
             participatory_processes.any?

data/app/models/decidim/admin/participatory_process_user_role.rb

@@ -8,7 +8,7 @@ module Decidim
       belongs_to :participatory_process, foreign_key: "decidim_participatory_process_id", class_name: Decidim::ParticipatoryProcess
 
       ROLES = %w(admin).freeze
-      validates :role, inclusion: { in: ROLES }
+      validates :role, inclusion: { in: ROLES }, uniqueness: { scope: [:user, :participatory_process] }
     end
   end
 end

data/app/queries/decidim/admin/manageable_participatory_processes_for_user.rb

@@ -22,7 +22,7 @@ module Decidim
       #
       # Returns an ActiveRecord::Relation.
       def query
-        return user.organization.participatory_processes if user.role?(:admin)
+        return user.organization.participatory_processes if user.role?(:admin) || user.role?(:collaborator)
 
         ParticipatoryProcess.where(id: process_ids)
       end

data/app/views/decidim/admin/attachments/edit.html.erb

@@ -1,6 +1,6 @@
 <h3><%= t ".title" %></h3>
 
-<%= form_for(@form, url: url_for([@attachment.attached_to, @attachment])) do |f| %>
+<%= decidim_form_for(@form, url: url_for([@attachment.attached_to, @attachment])) do |f| %>
   <%= render partial: 'decidim/admin/attachments/form', object: f %>
 
   <div class="actions">

data/app/views/decidim/admin/attachments/new.html.erb

@@ -1,6 +1,6 @@
 <h3><%= t ".title" %></h3>
 
-<%= form_for(@form, url: [attached_to, @form]) do |f| %>
+<%= decidim_form_for(@form, url: [attached_to, @form]) do |f| %>
   <%= render partial: 'decidim/admin/attachments/form', object: f %>
 
   <div class="actions">

data/app/views/decidim/admin/categories/edit.html.erb

@@ -1,6 +1,6 @@
 <h3><%= t ".title" %></h3>
 
-<%= form_for(@form, url: participatory_process_category_path(@category.participatory_process, @category)) do |f| %>
+<%= decidim_form_for(@form, url: participatory_process_category_path(@category.participatory_process, @category)) do |f| %>
   <%= render partial: 'form', object: f %>
 
   <div class="actions">

data/app/views/decidim/admin/categories/new.html.erb

@@ -1,6 +1,6 @@
 <h3><%= t ".title" %></h3>
 
-<%= form_for(@form, url: participatory_process_categories_path(participatory_process)) do |f| %>
+<%= decidim_form_for(@form, url: participatory_process_categories_path(participatory_process)) do |f| %>
   <%= render partial: 'form', object: f %>
 
   <div class="actions">

data/app/views/decidim/admin/dashboard/show.html.erb

@@ -1,3 +1,3 @@
-<% content_for :title do %>
+<% provide :title do %>
   <h2><%= t("decidim.admin.titles.dashboard") %></h2>
 <% end %>

data/app/views/decidim/admin/feature_permissions/edit.html.erb

@@ -0,0 +1,27 @@
+<section id="features">
+  <div class="row">
+    <div class="columns">
+      <h3><%= t(".title") %></h3>
+    </div>
+  </div>
+
+  <%= form_for @permissions_form, url: url_for(action: :update), method: "put" do |form| %>
+    <%= form.fields_for :permissions, form.object do |form| %>
+      <% form.object.permissions.each do |action, permission| %>
+        <%= form.fields_for(action, permission) do |form| %>
+          <fieldset class="card <%=action %>-permission">
+            <div class="card-divider"><%= t("#{feature.manifest.name}.actions.#{action}", scope: "decidim.features") %></div>
+            <div class="card-section">
+              <%= form.select :authorization_handler_name,
+                authorizations.map { |authorization|
+                  [t("#{authorization}.name", scope: "decidim.authorization_handlers"), authorization]
+                }, include_blank: t(".everyone") %>
+              <%= form.text_field :options %>
+            </div>
+          </fieldset>
+        <% end %>
+      <% end %>
+    <% end %>
+    <%= form.submit t(".submit") %>
+  <% end %>
+</section>

data/app/views/decidim/admin/features/_feature.html.erb

@@ -16,8 +16,18 @@
       <% end %>
     <% end %>
 
+    <% if can?(:update, feature) %>
+      <% if feature.published? %>
+        <%= link_to t("actions.unpublish", scope: "decidim.admin"), url_for(action: :unpublish, id: feature, controller: "features"), class: "small button", method: :put %>
+      <% else %>
+        <%= link_to t("actions.publish", scope: "decidim.admin"), url_for(action: :publish, id: feature, controller: "features"), class: "small button", method: :put %>
+      <% end %>
+    <% end %>
+
     <%= link_to t("actions.configure", scope: "decidim.admin"), url_for(action: :edit, id: feature, controller: "features"), class: "small button" if can? :update, feature %>
 
+    <%= link_to t("actions.permissions", scope: "decidim.admin"), edit_participatory_process_feature_permissions_path(feature_id: feature), class: "small button" if can? :update, feature %>
+
     <%= link_to t("actions.destroy", scope: "decidim.admin"), url_for(action: :destroy, id: feature, controller: "features"), method: :delete, class: "small alert button", data: { confirm: t("actions.confirm_destroy", scope: "decidim.admin") } if can? :destroy, feature %>
   </td>
 </tr>

data/app/views/decidim/admin/features/edit.html.erb

@@ -1,6 +1,6 @@
 <h3><%= t ".title", name: t("#{@feature.manifest.name}.name", scope: "decidim.features") %></h3>
 
-<%= form_for([participatory_process, @form], url: participatory_process_feature_path(participatory_process, @feature), method: "put") do |form| %>
+<%= decidim_form_for([participatory_process, @form], url: participatory_process_feature_path(participatory_process, @feature), method: "put") do |form| %>
   <%= render partial: 'form', object: form, locals: { feature: @feature } %>
 
   <div class="actions">

data/app/views/decidim/admin/features/new.html.erb

@@ -1,6 +1,6 @@
 <h3><%= t ".title", name: t("#{manifest.name}.name", scope: "decidim.features") %></h3>
 
-<%= form_for([participatory_process, @form], url: participatory_process_features_path(type: params[:type]), method: "post") do |form| %>
+<%= decidim_form_for([participatory_process, @form], url: participatory_process_features_path(type: params[:type]), method: "post") do |form| %>
   <%= render partial: 'form', object: form, locals: { feature: @feature } %>
 
   <div class="actions">

data/app/views/decidim/admin/newsletters/_form.html.erb

@@ -0,0 +1,5 @@
+<div class="field">
+  <%= form.translated :text_field, :subject %>
+  <%= form.translated :editor, :body %>
+  <%= t('.interpolations_hint') %>
+</div>

data/app/views/decidim/admin/newsletters/edit.html.erb

@@ -0,0 +1,11 @@
+<% provide :title do %>
+  <h2><%= t ".title" %></h2>
+<% end %>
+
+<%= decidim_form_for(@form) do |f| %>
+  <%= render partial: 'form', object: f %>
+
+  <div class="actions">
+    <%= f.submit t(".save_and_preview") %>
+  </div>
+<% end %>