decidim-meetings 0.13.1 → 0.14.1

data/app/forms/decidim/meetings/admin/meeting_registration_invite_form.rb

@@ -8,9 +8,16 @@ module Decidim
       class MeetingRegistrationInviteForm < Form
         attribute :name, String
         attribute :email, String
+        attribute :user_id, Integer
+        attribute :existing_user, Boolean, default: false
 
-        validates :name, presence: true
-        validates :email, presence: true, 'valid_email_2/email': { disposable: true }
+        validates :name, presence: true, unless: proc { |object| object.existing_user }
+        validates :email, presence: true, 'valid_email_2/email': { disposable: true }, unless: proc { |object| object.existing_user }
+        validates :user, presence: true, if: proc { |object| object.existing_user }
+
+        def user
+          @user ||= current_organization.users.find_by(id: user_id)
+        end
       end
     end
   end

data/app/forms/decidim/meetings/admin/validate_registration_code_form.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Meetings
+    module Admin
+      # This class holds a Form to validate registration codes from Decidim's admin panel.
+      class ValidateRegistrationCodeForm < Decidim::Form
+        attribute :code, String
+
+        validates :code, presence: true
+        validate :registration_exists
+
+        def registration
+          @registration ||= meeting.registrations.find_by(code: code, validated_at: nil)
+        end
+
+        private
+
+        def meeting
+          @meeting ||= context[:meeting]
+        end
+
+        def registration_exists
+          return unless registration.nil?
+          errors.add(
+            :code,
+            I18n.t("registrations.validate_registration_code.invalid", scope: "decidim.meetings.admin")
+          )
+        end
+      end
+    end
+  end
+end

data/app/mailers/decidim/meetings/registration_mailer.rb

@@ -11,10 +11,11 @@ module Decidim
       helper Decidim::ResourceHelper
       helper Decidim::TranslationsHelper
 
-      def confirmation(user, meeting)
+      def confirmation(user, meeting, registration)
         with_user(user) do
           @user = user
           @meeting = meeting
+          @registration = registration
           @organization = @meeting.organization
           @locator = Decidim::ResourceLocatorPresenter.new(@meeting)
 

data/app/models/decidim/meetings/invite.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Meetings
+    # The data store for an Invite in the Decidim::Meetings component.
+    class Invite < Meetings::ApplicationRecord
+      include Decidim::Traceable
+      include Decidim::Loggable
+      include Decidim::DataPortability
+
+      belongs_to :meeting, foreign_key: "decidim_meeting_id", class_name: "Decidim::Meetings::Meeting"
+      belongs_to :user, foreign_key: "decidim_user_id", class_name: "Decidim::User"
+
+      validates :user, uniqueness: { scope: :meeting }
+
+      def self.export_serializer
+        Decidim::Meetings::DataPortabilityInviteSerializer
+      end
+
+      def self.log_presenter_class_for(_log)
+        Decidim::Meetings::AdminLog::InvitePresenter
+      end
+
+      def self.user_collection(user)
+        where(decidim_user_id: user.id)
+      end
+
+      def accept!
+        update!(accepted_at: Time.current, rejected_at: nil)
+      end
+
+      def reject!
+        update!(rejected_at: Time.current, accepted_at: nil)
+      end
+      alias decline! reject!
+    end
+  end
+end

data/app/models/decidim/meetings/meeting.rb

@@ -20,6 +20,7 @@ module Decidim
 
       belongs_to :organizer, foreign_key: "organizer_id", class_name: "Decidim::User", optional: true
       has_many :registrations, class_name: "Decidim::Meetings::Registration", foreign_key: "decidim_meeting_id", dependent: :destroy
+      has_many :invites, class_name: "Decidim::Meetings::Invite", foreign_key: "decidim_meeting_id", dependent: :destroy
       has_one :minutes, class_name: "Decidim::Meetings::Minutes", foreign_key: "decidim_meeting_id", dependent: :destroy
       has_one :agenda, class_name: "Decidim::Meetings::Agenda", foreign_key: "decidim_meeting_id", dependent: :destroy
 
@@ -85,6 +86,11 @@ module Decidim
         commentable? && !component.current_settings.comments_blocked
       end
 
+      # Public: Overrides the `allow_resource_permissions?` Resourceable concern method.
+      def allow_resource_permissions?
+        component.settings.resources_permissions_enabled
+      end
+
       # Public: Overrides the `comments_have_alignment?` Commentable concern method.
       def comments_have_alignment?
         true

data/app/models/decidim/meetings/registration.rb

@@ -10,6 +10,10 @@ module Decidim
       belongs_to :user, foreign_key: "decidim_user_id", class_name: "Decidim::User"
 
       validates :user, uniqueness: { scope: :meeting }
+      validates :code, uniqueness: { allow_blank: true, scope: :meeting }
+      validates :code, presence: true, on: :create
+
+      before_validation :generate_code, on: :create
 
       def self.user_collection(user)
         where(decidim_user_id: user.id)
@@ -18,6 +22,20 @@ module Decidim
       def self.export_serializer
         Decidim::Meetings::DataPortabilityRegistrationSerializer
       end
+
+      private
+
+      def generate_code
+        self[:code] ||= calculate_registration_code
+      end
+
+      # Calculates a unique code for the model using the class
+      # provided by the configuration and scoped to the meeting.
+      #
+      # Returns a String.
+      def calculate_registration_code
+        Decidim::Meetings::Registrations.code_generator.generate(self)
+      end
     end
   end
 end

data/app/permissions/decidim/meetings/admin/permissions.rb

@@ -33,9 +33,9 @@ module Decidim
           return unless permission_action.subject == :meeting
 
           case permission_action.action
-          when :close, :copy, :destroy, :export_registrations, :update
+          when :close, :copy, :destroy, :export_registrations, :update, :read_invites
             toggle_allow(meeting.present?)
-          when :invite_user
+          when :invite_attendee
             toggle_allow(meeting.present? && meeting.registrations_enabled?)
           when :create
             allow!

data/app/permissions/decidim/meetings/permissions.rb

@@ -17,6 +17,8 @@ module Decidim
           toggle_allow(can_join_meeting?)
         when :leave
           toggle_allow(can_leave_meeting?)
+        when :decline_invitation
+          toggle_allow(can_decline_invitation?)
         end
 
         permission_action
@@ -30,12 +32,17 @@ module Decidim
 
       def can_join_meeting?
         meeting.can_be_joined_by?(user) &&
-          authorized?(:join)
+          authorized?(:join, resource: meeting)
       end
 
       def can_leave_meeting?
         meeting.registrations_enabled?
       end
+
+      def can_decline_invitation?
+        meeting.registrations_enabled? &&
+          meeting.invites.where(user: user).exists?
+      end
     end
   end
 end

data/app/presenters/decidim/meetings/admin_log/invite_presenter.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Meetings
+    module AdminLog
+      # This class holds the logic to present a `Decidim::Meetings::Invite`
+      # for the `AdminLog` log.
+      #
+      # Usage should be automatic and you shouldn't need to call this class
+      # directly, but here's an example:
+      #
+      #    action_log = Decidim::ActionLog.last
+      #    view_helpers # => this comes from the views
+      #    InvitePresenter.new(action_log, view_helpers).present
+      class InvitePresenter < Decidim::Log::BasePresenter
+        private
+
+        def action_string
+          case action
+          when "create", "delete", "update"
+            "decidim.meetings.admin_log.invite.#{action}"
+          else
+            super
+          end
+        end
+
+        def i18n_params
+          super.merge(
+            attendee_name: action_log.resource.user.name
+          )
+        end
+      end
+    end
+  end
+end

data/app/presenters/decidim/meetings/invite_presenter.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Meetings
+    #
+    # Decorator for meeting invites
+    #
+    class InvitePresenter < SimpleDelegator
+      def status
+        return I18n.t("accepted", scope: "decidim.meetings.models.invite.status", at: I18n.l(accepted_at, format: :decidim_short)) if accepted_at.present?
+        return I18n.t("rejected", scope: "decidim.meetings.models.invite.status", at: I18n.l(rejected_at, format: :decidim_short)) if rejected_at.present?
+        return I18n.t("sent", scope: "decidim.meetings.models.invite.status") if sent_at.present?
+
+        "-"
+      end
+
+      def status_html_class
+        return "success" if accepted_at.present?
+        return "danger" if rejected_at.present?
+        return "warning" if sent_at.present?
+
+        ""
+      end
+    end
+  end
+end

data/app/queries/decidim/meetings/admin/invites.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Meetings
+    module Admin
+      # A class used to find the Invites by their status status.
+      class Invites < Rectify::Query
+        # Syntactic sugar to initialize the class and return the queried objects.
+        #
+        # invites - the initial Invites relation that needs to be filtered.
+        # query - query to filter invites
+        # status - invite status to be used as a filter
+        def self.for(invites, query = nil, status = nil)
+          new(invites, query, status).query
+        end
+
+        # Initializes the class.
+        #
+        # invites - the initial Invites relation that need to be filtered
+        # query - query to filter invites
+        # status - invite status to be used as a filter
+        def initialize(invites, query = nil, status = nil)
+          @invites = invites
+          @query = query
+          @status = status
+        end
+
+        # List the invites by the different filters.
+        def query
+          @invites = filter_by_search(@invites)
+          @invites = filter_by_status(@invites)
+          @invites
+        end
+
+        private
+
+        def filter_by_search(invites)
+          return invites if @query.blank?
+          invites.joins(:user).where(
+            User.arel_table[:name].lower.matches("%#{@query}%").or(User.arel_table[:email].lower.matches("%#{@query}%"))
+          )
+        end
+
+        def filter_by_status(invites)
+          case @status
+          when "accepted"
+            invites.where.not(accepted_at: nil)
+          when "rejected"
+            invites.where.not(rejected_at: nil)
+          when "sent"
+            invites.where.not(sent_at: nil).where(accepted_at: nil, rejected_at: nil)
+          else
+            invites
+          end
+        end
+      end
+    end
+  end
+end

data/app/serializers/decidim/meetings/data_portability_invite_serializer.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Meetings
+    class DataPortabilityInviteSerializer < Decidim::Exporters::Serializer
+      # Serializes an invite for data portability
+      def serialize
+        {
+          id: resource.id,
+          sent_at: resource.sent_at,
+          accepted_at: resource.accepted_at,
+          rejected_at: resource.rejected_at,
+          user: {
+            name: resource.user.name,
+            email: resource.user.email
+          },
+          meeting: {
+            title: resource.meeting.title,
+            description: resource.meeting.description,
+            start_time: resource.meeting.start_time,
+            end_time: resource.meeting.end_time,
+            address: resource.meeting.address,
+            location: resource.meeting.location,
+            location_hints: resource.meeting.location_hints,
+            reference: resource.meeting.reference,
+            attendees_count: resource.meeting.attendees_count,
+            attending_organizations: resource.meeting.attending_organizations,
+            closed_at: resource.meeting.closed_at,
+            closing_report: resource.meeting.closing_report
+          }
+        }
+      end
+    end
+  end
+end

data/app/serializers/decidim/meetings/data_portability_registration_serializer.rb

@@ -7,6 +7,7 @@ module Decidim
       def serialize
         {
           id: resource.id,
+          code: resource.code,
           user: {
             name: resource.user.name,
             email: resource.user.email

data/app/serializers/decidim/meetings/registration_serializer.rb

@@ -7,6 +7,7 @@ module Decidim
       def serialize
         {
           id: resource.id,
+          code: resource.code,
           user: {
             name: resource.user.name,
             email: resource.user.email

data/app/views/decidim/meetings/admin/invite_join_meeting_mailer/invite.html.erb

@@ -4,4 +4,5 @@
   <%= t ".invited_you_to_join_a_meeting", invited_by: @invited_by.name, application: @user.organization.name %>
 </p>
 
+<p><%= link_to t(".decline", meeting_title: translated_attribute(@meeting.title)),routes.decline_invitation_meeting_registration_path(meeting_id: @meeting, participatory_space_id: @meeting.component.participatory_space) %>
 <p><%= link_to t(".join", meeting_title: translated_attribute(@meeting.title)),routes.meeting_registration_url(meeting_id: @meeting, participatory_space_id: @meeting.component.participatory_space) %>

data/app/views/decidim/meetings/admin/invites/_form.html.erb

@@ -1,7 +1,36 @@
-<div class="row column">
-  <%= form.text_field :name %>
-</div>
+<div class="attendee-fields">
+  <div class="row column">
+    <fieldset class="check-radio-collection">
+      <legend><%= t(".attendee_type") %></legend>
+      <%= form.collection_radio_buttons(:existing_user, [[t(".non_user"), false], [t(".existing_user"), true]], :last, :first) %>
+    </fieldset>
+  </div>
+
+  <div class="text-warning attendee-fields--new-user">
+    <p><%= t(".invite_explanation") %></p>
+  </div>
+
+  <div class="grid-x grid-margin-x">
+    <div class="auto cell attendee-fields--new-user">
+      <%= form.text_field :name, disabled: disabled %>
+    </div>
+    <div class="auto cell attendee-fields--new-user">
+      <%= form.text_field :email, disabled: disabled %>
+    </div>
 
-<div class="row column">
-  <%= form.text_field :email %>
+    <div class="auto cell attendee-fields--user-picker">
+        <% prompt_options = { url: decidim_admin.users_organization_url, placeholder: t(".select_user") } %>
+        <%= form.autocomplete_select(:user_id, form.object.user.presence, { multiple: false, class: "autocomplete-field--results-inline" }, prompt_options) do |user|
+          { value: user.id, label: "#{user.name} (@#{user.nickname})" }
+        end %>
+    </div>
+
+    <div class="shrink cell">
+      <div class="text-center mt-sm">
+        <%= form.submit t(".invite"), disabled: disabled %>
+      </div>
+    </div>
+  </div>
 </div>
+
+<%= javascript_include_tag "decidim/meetings/admin/registrations_invite_form" %>

data/app/views/decidim/meetings/admin/invites/index.html.erb

@@ -0,0 +1,99 @@
+<div class="card">
+  <div class="card-divider">
+    <h2 class="card-title"><%= t(".invite_attendee") %></h2>
+  </div>
+  <div class="card-section">
+    <%= decidim_form_for(@form, url: meeting_registrations_invites_path, method: :post, html: { class: "form new_meeting_registration_invite" }) do |f| %>
+      <% disable_form = !allowed_to?(:invite_attendee, :meeting, meeting: meeting) %>
+
+      <%= render partial: "form", object: f, locals: { disabled: disable_form } %>
+
+      <% unless meeting.registrations_enabled? %>
+        <div class="text-alert">
+          <p><%= t(".registrations_disabled") %></p>
+        </div>
+      <% end %>
+    <% end %>
+  </div>
+</div>
+
+<div class="filters grid-x mt-m">
+  <div class="medium-7">
+    <span class="dropdown-menu-inverted_label"><%= t(".filter_by") %> :</span>
+    <ul class="dropdown menu dropdown-inverted" data-dropdown-menu data-click-open="true" data-close-on-click-inside="false">
+      <li class="is-dropdown-submenu-parent">
+        <a href="#">
+          <% if @status.present? %>
+            <%= t(".filter.#{@status}") %>
+          <% else %>
+            <%= t(".filter.all") %>
+          <% end %>
+        </a>
+        <ul class="menu is-dropdown-submenu">
+          <li><%= link_to t(".filter.sent"), url_for(status: "sent", q: @query) %></li>
+          <li><%= link_to t(".filter.accepted"), url_for(status: "accepted", q: @query) %></li>
+          <li><%= link_to t(".filter.rejected"), url_for(status: "rejected", q: @query) %></li>
+          <li><%= link_to t(".filter.all"), url_for(q: @query) %></li>
+        </ul>
+      </li>
+    </ul>
+  </div>
+  <div class="medium-5">
+    <%= form_tag "", method: :get do %>
+      <div class="filters__search">
+        <div class="input-group">
+          <%= search_field_tag :q, @query,label: false, class: "input-group-field", placeholder: t(".search") %>
+          <%= hidden_field_tag :state, @state %>
+          <div class="input-group-button">
+            <button type="submit" class="button button--muted">
+              <%= icon "magnifying-glass", aria_label: t(".search") %>
+            </button>
+          </div>
+        </div>
+      </div>
+    <% end %>
+  </div>
+</div>
+
+<div class="card" id="meeting-invites">
+  <div class="card-divider">
+    <h2 class="card-title"><%= title %> - <%= t(".invites") %></h2>
+  </div>
+
+  <div class="card-section">
+    <div class="table-scroll">
+      <table class="table-list">
+        <thead>
+          <tr>
+            <th><%= t("models.invite.fields.name", scope: "decidim.meetings") %></th>
+            <th><%= t("models.invite.fields.email", scope: "decidim.meetings") %></th>
+            <th><%= t("models.invite.fields.sent_at", scope: "decidim.meetings") %></th>
+            <th><%= t("models.invite.fields.status", scope: "decidim.meetings") %></th>
+          </tr>
+        </thead>
+        <tbody>
+          <% invites.each do |invite| %>
+            <% presenter = Decidim::Meetings::InvitePresenter.new(invite) %>
+            <tr data-id="<%= invite.id %>">
+              <td>
+                <%= invite.user.name %>
+              </td>
+              <td>
+                <%= invite.user.email %>
+              </td>
+              <td>
+                <% if invite.sent_at %>
+                  <%= l invite.sent_at, format: :long %>
+                <% end %>
+              </td>
+              <td class="<%= presenter.status_html_class %>">
+                <%= presenter.status %>
+              </td>
+            </tr>
+          <% end %>
+        </tbody>
+      </table>
+      <%= paginate invites, theme: "decidim" %>
+    </div>
+  </div>
+</div>