decidim-core 0.7.4 → 0.8.0

data/app/commands/decidim/messaging/reply_to_conversation.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Messaging
+    # A command with all the business logic for replying to a conversation
+    class ReplyToConversation < Rectify::Command
+      # Public: Initializes the command.
+      #
+      # conversation - The conversation to be updated.
+      # form - A form object with the params.
+      def initialize(conversation, form)
+        @conversation = conversation
+        @form = form
+      end
+
+      # Executes the command. Broadcasts these events:
+      #
+      # - :ok when everything is valid, together with the message.
+      # - :invalid if the form wasn't valid and we couldn't proceed.
+      #
+      # Returns nothing.
+      def call
+        return broadcast(:invalid) if form.invalid?
+
+        if message.save
+          notify_interlocutors
+
+          broadcast(:ok, message)
+        else
+          broadcast(:invalid)
+        end
+      end
+
+      private
+
+      def sender
+        form.current_user
+      end
+
+      def message
+        @message ||= conversation.add_message(sender: sender, body: form.body)
+      end
+
+      def notify_interlocutors
+        conversation.interlocutors(sender).each do |recipient|
+          if conversation.unread_count(recipient) == 1
+            ConversationMailer.new_message(sender, recipient, conversation).deliver_later
+          end
+        end
+      end
+
+      attr_reader :conversation, :form
+    end
+  end
+end

data/app/commands/decidim/messaging/start_conversation.rb

@@ -0,0 +1,55 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Messaging
+    # A command with all the business logic for replying to a conversation
+    class StartConversation < Rectify::Command
+      # Public: Initializes the command.
+      #
+      # form - A conversation form
+      def initialize(form)
+        @form = form
+      end
+
+      # Executes the command. Broadcasts these events:
+      #
+      # - :ok when everything is valid, together with the message.
+      # - :invalid if the form wasn't valid and we couldn't proceed.
+      #
+      # Returns nothing.
+      def call
+        return broadcast(:invalid) if form.invalid?
+
+        if conversation.save
+          notify_interlocutors
+
+          broadcast(:ok, conversation)
+        else
+          broadcast(:invalid)
+        end
+      end
+
+      private
+
+      def conversation
+        @conversation ||= Conversation.start(
+          originator: originator,
+          interlocutors: [form.recipient],
+          body: form.body
+        )
+      end
+
+      def notify_interlocutors
+        conversation.interlocutors(originator).each do |recipient|
+          ConversationMailer.new_conversation(originator, recipient, conversation).deliver_later
+        end
+      end
+
+      def originator
+        form.current_user
+      end
+
+      attr_reader :form
+    end
+  end
+end

data/app/controllers/concerns/decidim/action_authorization.rb

@@ -16,13 +16,12 @@ module Decidim
     #                feature's permissions panel.
     # redirect_url - Url to be redirected to when the authorization is finished.
     def authorize_action!(action_name, redirect_url: nil)
-      @action_authorizations ||= {}
-      status = @action_authorizations[action_name] = action_authorization(action_name)
+      status = action_authorization(action_name)
 
       return if status.ok?
       raise Unauthorized if status.code == :invalid
 
-      redirect_to authorize_action_path_from_status(status, redirect_url)
+      redirect_to authorize_action_path(action_name, redirect_url: redirect_url)
     end
 
     # Public: Returns the authorization object for an authorization.
@@ -31,7 +30,9 @@ module Decidim
     #
     # Returns an ActionAuthorizer::AuthorizationStatus
     def action_authorization(action_name)
-      _action_authorizer(action_name).authorize
+      @action_authorizations ||= {}
+
+      @action_authorizations[action_name] = _action_authorizer(action_name).authorize
     end
 
     # Public: Returns the authorization path for a failed authorization with
@@ -44,24 +45,7 @@ module Decidim
     def authorize_action_path(action_name, redirect_url: nil)
       redirect_url ||= request.path
 
-      authorize_action_path_from_status(
-        action_authorization(action_name),
-        redirect_url: redirect_url
-      )
-    end
-
-    # Public: Returns the authorization path for a failed authorization with
-    # the populated redirect url.
-    #
-    # status       - The status after an authorization check.
-    # redirect_url - The url to redirect to when finished.
-    #
-    # Returns a String.
-    def authorize_action_path_from_status(status, redirect_url: nil)
-      decidim.new_authorization_path(
-        handler: status.handler_name,
-        redirect_url: redirect_url
-      )
+      action_authorization(action_name).current_path(redirect_url: redirect_url)
     end
 
     def _action_authorizer(action_name)

data/app/controllers/concerns/decidim/user_profile.rb

@@ -17,7 +17,7 @@ module Decidim
       helper Decidim::UserProfileHelper
       layout "layouts/decidim/user_profile"
 
-      helper_method :available_authorization_handlers,
+      helper_method :available_verification_workflows,
                     :user_groups
 
       before_action :current_user
@@ -26,8 +26,10 @@ module Decidim
 
     # Public: Available authorization handlers in order to conditionally
     # show the menu element.
-    def available_authorization_handlers
-      current_organization.available_authorizations.map(&:constantize)
+    def available_verification_workflows
+      Verifications::Adapter.from_collection(
+        current_organization.available_authorizations & Decidim.authorization_workflows.map(&:name)
+      )
     end
   end
 end

data/app/controllers/decidim/application_controller.rb

@@ -18,6 +18,7 @@ module Decidim
     helper Decidim::AriaSelectedLinkToHelper
     helper Decidim::MenuHelper
     helper Decidim::FeaturePathHelper
+    helper Decidim::ViewHooksHelper
 
     protect_from_forgery with: :exception, prepend: true
     after_action :add_vary_header

data/app/controllers/decidim/devise/omniauth_registrations_controller.rb

@@ -45,14 +45,14 @@ module Decidim
 
       def after_sign_in_path_for(user)
         if first_login_and_not_authorized?(user)
-          authorizations_path
+          decidim_verifications.authorizations_path
         else
           super
         end
       end
 
       def first_login_and_not_authorized?(user)
-        user.is_a?(User) && user.sign_in_count == 1 && Decidim.authorization_handlers.any?
+        user.is_a?(User) && user.sign_in_count == 1 && Decidim::Verifications.workflows.any?
       end
 
       def action_missing(action_name)

data/app/controllers/decidim/devise/sessions_controller.rb

@@ -7,9 +7,11 @@ module Decidim
       include Decidim::DeviseControllers
 
       def after_sign_in_path_for(user)
-        return first_login_authorizations_path if first_login_and_not_authorized?(user) &&
-                                                  !user.admin?
-        super
+        if first_login_and_not_authorized?(user) && !user.admin?
+          decidim_verifications.first_login_authorizations_path
+        else
+          super
+        end
       end
 
       def first_login_and_not_authorized?(user)

data/app/controllers/decidim/features/base_controller.rb

@@ -18,6 +18,7 @@ module Decidim
       helper Decidim::ScopesHelper
       helper Decidim::ActionAuthorizationHelper
       helper Decidim::AttachmentsHelper
+      helper Decidim::SanitizeHelper
 
       helper_method :current_feature,
                     :current_participatory_space,

data/app/controllers/decidim/messaging/conversations_controller.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Messaging
+    # The controller to handle the user's conversations.
+    class ConversationsController < Decidim::ApplicationController
+      include FormFactory
+
+      helper Decidim::DatetimeHelper
+
+      before_action :authenticate_user!
+
+      helper_method :username_list, :conversation
+
+      def new
+        authorize! :create, Conversation
+
+        @form = form(ConversationForm).from_params(params)
+      end
+
+      def create
+        authorize! :create, Conversation
+
+        @form = form(ConversationForm).from_params(params)
+
+        StartConversation.call(@form) do
+          on(:ok) do |conversation|
+            render action: :create, locals: {
+              conversation: conversation,
+              form: MessageForm.new
+            }
+          end
+
+          on(:invalid) do
+            render json: { error: I18n.t("messaging.conversations.create.error", scope: "decidim") }, status: 422
+          end
+        end
+      end
+
+      def index
+        authorize! :index, Conversation
+
+        @conversations = UserConversations.for(current_user)
+      end
+
+      def show
+        authorize! :show, conversation
+
+        @conversation.mark_as_read(current_user)
+
+        @form = MessageForm.new
+      end
+
+      def update
+        authorize! :update, conversation
+
+        @form = form(MessageForm).from_params(params)
+
+        ReplyToConversation.call(conversation, @form) do
+          on(:ok) do |message|
+            render action: :update, locals: { message: message }
+          end
+
+          on(:invalid) do
+            render json: { error: I18n.t("messaging.conversations.update.error", scope: "decidim") }, status: 422
+          end
+        end
+      end
+
+      private
+
+      def conversation
+        @conversation ||= Conversation.find(params[:id])
+      end
+
+      def username_list(users)
+        users.pluck(:name).join(", ")
+      end
+    end
+  end
+end

data/app/controllers/decidim/pages_controller.rb

@@ -11,8 +11,9 @@ module Decidim
 
     authorize_resource :public_pages, class: false
     delegate :page, to: :page_finder
-    helper_method :page, :promoted_participatory_processes, :highlighted_participatory_processes, :stats
+    helper_method :page, :stats
     helper CtaButtonHelper
+    helper Decidim::SanitizeHelper
 
     def index
       @pages = current_organization.static_pages.all.to_a.sort do |a, b|
@@ -24,16 +25,6 @@ module Decidim
       @page_finder ||= Decidim::PageFinder.new(params[:id], current_organization)
     end
 
-    def promoted_participatory_processes
-      @promoted_participatory_processes ||=
-        ParticipatoryProcesses::OrganizationPrioritizedParticipatoryProcesses.new(current_organization) | ParticipatoryProcesses::PromotedParticipatoryProcesses.new
-    end
-
-    def highlighted_participatory_processes
-      @highlighted_participatory_processes ||=
-        ParticipatoryProcesses::OrganizationPublishedParticipatoryProcesses.new(current_organization) | ParticipatoryProcesses::HighlightedParticipatoryProcesses.new
-    end
-
     private
 
     def stats

data/app/forms/decidim/messaging/conversation_form.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Messaging
+    # A form object to be used when users want to follow a followable resource.
+    class ConversationForm < Decidim::Form
+      mimic :conversation
+
+      attribute :body, String
+      attribute :recipient_id, Integer
+
+      validates :body, :recipient, presence: true
+
+      def recipient
+        Decidim::User.find_by(id: recipient_id)
+      end
+    end
+  end
+end

data/app/forms/decidim/messaging/message_form.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Messaging
+    # A form object to be used when sending a message
+    class MessageForm < Decidim::Form
+      mimic :message
+
+      attribute :body, String
+
+      validates :body, presence: true
+    end
+  end
+end

data/app/forms/translatable_presence_validator.rb

@@ -5,24 +5,21 @@
 #
 #   validates :my_i18n_field, translatable_presence: true
 #
-# This will automatically check for presence for each of the
-# `available_locales` of the form object (or the `available_locales` of the
-# form's organization) for the given field.
+# This will automatically check for presence for the default locale of the form
+# object (or the `default_locale` of the form's organization) for the given field.
 class TranslatablePresenceValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, _value)
-    available_locales_for(record).each do |locale|
-      translated_attr = "#{attribute}_#{locale}"
-      record.errors.add(translated_attr, :blank) if record.send(translated_attr).blank?
-    end
+    translated_attr = "#{attribute}_#{default_locale_for(record)}"
+    record.errors.add(translated_attr, :blank) if record.send(translated_attr).blank?
   end
 
   private
 
-  def available_locales_for(record)
-    return record.available_locales if record.respond_to?(:available_locales)
+  def default_locale_for(record)
+    return record.default_locale if record.respond_to?(:default_locale)
 
     if record.current_organization
-      record.current_organization.available_locales
+      record.current_organization.default_locale
     else
       record.errors.add(:current_organization, :blank)
       []

data/app/helpers/decidim/authorization_form_helper.rb

@@ -16,7 +16,7 @@ module Decidim
       default_options = {
         builder: AuthorizationFormBuilder,
         as: "authorization_handler",
-        url: authorizations_path
+        url: decidim_verifications.authorizations_path
       }
 
       options = default_options.merge(options)

data/app/helpers/decidim/cta_button_helper.rb

@@ -4,7 +4,7 @@ module Decidim
   # A Helper to render the Call To Action button.
   module CtaButtonHelper
     # Renders the Call To Action button. Link and text can be configured
-    # per organizationn.
+    # per organization.
     def cta_button
       button_text = translated_attribute(current_organization.cta_button_text).presence || t("pages.home.hero.participate")
 

data/app/helpers/decidim/datetime_helper.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require "active_support/core_ext/date_time/calculations"
+
+module Decidim
+  # A Helper to render dates
+  module DatetimeHelper
+    # Renders a date in the simplest possible format.
+    def simple_date(datetime)
+      current_datetime = Time.zone.now
+
+      if datetime > current_datetime.beginning_of_day
+        I18n.l(datetime, format: :time_of_day)
+      elsif datetime > current_datetime.beginning_of_week
+        I18n.l(datetime, format: :day_of_week)
+      elsif datetime > current_datetime.beginning_of_year
+        I18n.l(datetime, format: :day_of_month)
+      else
+        I18n.l(datetime, format: :day_of_year)
+      end
+    end
+  end
+end