decidim-conferences 0.17.2 → 0.20.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6a050b26126fa1de7d61e8b46c5cfd833ca3e177c5a06eefc9a494a405a5ed7a
-  data.tar.gz: b59b6a60da46b12fcf24f9d0ef26661c459e40cfdfa0fd260e315289d401c6ab
+  metadata.gz: 5fe396d63f9e335c79c6ef5d3c2e7220226e03e6797e184cb60a681b30c79d6b
+  data.tar.gz: 8ee8b6a31e1f5f2cccc59de336da8f8f62f4943c8144d9571243727dbeff4300
 SHA512:
-  metadata.gz: f0c1e67e26f000a95eb0c029d72763c8f3930d3af2e3a67781d80050207c74f838424f6a071b999a42c8fb09560e3cb41903d942733fd280c75a6e2dbecb4ad8
-  data.tar.gz: 2b0886ad963ac3bb4596925cf9a9d52243259ae72f2f389c482164a994cc6a31700d668a33c273cd77f317ca066b9ffa282c84b953b7a4caaa84d822bd85f2cc
+  metadata.gz: dee2e08fa8880ee4c54af3b7da1291adde063e006377b0472481cc515396eb632f8bb78667e714ce0062a99c74a1d7f0d9a7afaac8ce62a129588ce8af4287cd
+  data.tar.gz: 6dbb9a34018713b7d5d19891a03f5a7e5cd32dc2f1deb219828ec1c502e14197be5499b9c61947f707008d09f8be70554c5de207fc6361c48ab0c3edcdf70a5a

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

@@ -0,0 +1,3 @@
+.conference--date{
+  cursor: default;
+}

data/app/cells/decidim/conferences/conference_address/show.erb

@@ -1,7 +1,7 @@
 <div class="mb-s">
   <% if model.respond_to? :location %>
     <div>
-      <strong><%= translated_attribute model.location %></strong><br />
+      <strong><%= translated_attribute model.location %></strong><br>
     </div>
   <% end %>
   <div class="text-medium"><%= model.address %></div>

data/app/cells/decidim/conferences/conference_m/footer.erb

@@ -1,7 +1,7 @@
 <div class="card__footer card__footer--spaces">
   <div class="card__support">
     <%= link_to(
-      t("layouts.decidim.conferences.conference.take_part"),
+      t(model.cta_button_text_key, scope: "layouts.decidim.conferences.conference"),
       resource_path,
       class: "card__button button button--sc secondary light small"
     ) %>

data/app/cells/decidim/conferences/conference_speaker/show.erb

@@ -67,14 +67,14 @@
       <% end %>
     </div>
     <% if meetings.present? %>
-      <hr class="reset mt-none mb-s" />
+      <hr class="reset mt-none mb-s">
       <div class="row">
         <div class="column medium-12">
           <div class="bio-acts">Speaking at</div>
 
           <ul class="list-reset">
             <% meetings.each do |meeting| %>
-              <%= meeting_title meeting %> <br />
+              <%= meeting_title meeting %> <br>
             <% end %>
           </ul>
         </div>

data/app/cells/decidim/conferences/conference_speaker_cell.rb

@@ -41,6 +41,7 @@ module Decidim
 
       def avatar
         return model.user.avatar if model.user.present?
+
         model.avatar
       end
 
@@ -58,17 +59,20 @@ module Decidim
 
       def short_bio
         return unless model.short_bio.presence
+
         translated_attribute model.short_bio
       end
 
       def twitter_handle
         return unless model.twitter_handle.presence
-        link_to t(".go_to_twitter"), "https://twitter.com/#{model.twitter_handle}", target: "_blank"
+
+        link_to t(".go_to_twitter"), "https://twitter.com/#{model.twitter_handle}", target: "_blank", rel: "noopener"
       end
 
       def personal_url
         return unless model.personal_url.presence || (model.user.presence && model.user.personal_url.presence)
-        link_to model.personal_url || model.user.personal_url, target: "_blank", class: "card-link" do
+
+        link_to model.personal_url || model.user.personal_url, target: "_blank", class: "card-link", rel: "noopener" do
           "#{icon "external-link"}" " #{t(".personal_website")}"
         end
       end

data/app/cells/decidim/conferences/linked_participatory_spaces_cell.rb

@@ -20,22 +20,28 @@ module Decidim
 
       def conference_participatory_processes
         return unless Decidim.participatory_space_manifests.map(&:name).include?(:participatory_processes)
+
         processes = model.linked_participatory_space_resources(:participatory_processes, "included_participatory_processes")
         return unless processes.any?
+
         processes
       end
 
       def conference_assemblies
         return unless Decidim.participatory_space_manifests.map(&:name).include?(:assemblies)
+
         assemblies = model.linked_participatory_space_resources(:assemblies, "included_assemblies")
         return unless assemblies.any?
+
         assemblies
       end
 
       def conference_consultations
         return unless Decidim.participatory_space_manifests.map(&:name).include?(:consultations)
+
         consultations = model.linked_participatory_space_resources(:consultations, "included_consultations")
         return unless consultations.any?
+
         consultations
       end
 

data/app/cells/decidim/conferences/partner_cell.rb

@@ -12,11 +12,13 @@ module Decidim
 
       def name
         return unless model.name.presence
+
         "<div class='text-medium'> #{model.name} </div>"
       end
 
       def logo
         return unless model.logo.presence
+
         "<div class='card p-m flex--cc'> #{image_tag model.logo.medium.url} </div>"
       end
     end

data/app/cells/decidim/conferences/photos_list_cell.rb

@@ -10,6 +10,7 @@ module Decidim
 
       def show
         return unless model.any?
+
         render
       end
     end

data/app/cells/decidim/conferences/registration_type_cell.rb

@@ -27,6 +27,7 @@ module Decidim
 
       def price
         return I18n.t("free", scope: "decidim.conferences.conference.show") if model.price.blank?
+
         number_to_currency(model.price, locale: I18n.locale, unit: Decidim.currency_unit)
       end
 
@@ -44,6 +45,7 @@ module Decidim
 
       def i18n_join_text
         return I18n.t("registration", scope: "decidim.conferences.conference.show") if conference.has_available_slots?
+
         I18n.t("no_slots_available", scope: "decidim.conferences.conference.show")
       end
     end

data/app/commands/decidim/conferences/admin/confirm_conference_registration.rb

@@ -20,6 +20,7 @@ module Decidim
         def call
           @conference_registration.with_lock do
             return broadcast(:invalid) unless can_join_conference?
+
             confirm_registration
             send_email_confirmation
             send_notification_confirmation

data/app/commands/decidim/conferences/admin/create_conference.rb

@@ -29,7 +29,6 @@ module Decidim
             link_consultations
 
             broadcast(:ok, conference)
-            send_notification
           else
             form.errors.add(:hero_image, conference.errors[:hero_image]) if conference.errors.include? :hero_image
             form.errors.add(:banner_image, conference.errors[:banner_image]) if conference.errors.include? :banner_image
@@ -81,15 +80,6 @@ module Decidim
           end
         end
 
-        def send_notification
-          Decidim::EventsManager.publish(
-            event: "decidim.events.conferences.registrations_enabled",
-            event_class: Decidim::Conferences::ConferenceRegistrationsEnabledEvent,
-            resource: conference,
-            followers: conference.followers
-          )
-        end
-
         def participatory_processes
           @participatory_processes ||= conference.participatory_space_sibling_scope(:participatory_processes).where(id: @form.participatory_processes_ids)
         end

data/app/commands/decidim/conferences/admin/create_conference_admin.rb

@@ -91,6 +91,7 @@ module Decidim
 
         def invitation_instructions
           return "invite_admin" if form.role == "admin"
+
           "invite_collaborator"
         end
 

data/app/commands/decidim/conferences/admin/publish_conference.rb

@@ -28,11 +28,23 @@ module Decidim
           end
 
           broadcast(:ok)
+          send_notification
         end
 
         private
 
         attr_reader :conference, :current_user
+
+        def send_notification
+          return unless conference.registrations_enabled?
+
+          Decidim::EventsManager.publish(
+            event: "decidim.events.conferences.registrations_enabled",
+            event_class: Decidim::Conferences::ConferenceRegistrationsEnabledEvent,
+            resource: conference,
+            followers: conference.followers
+          )
+        end
       end
     end
   end

data/app/commands/decidim/conferences/admin/update_conference.rb

@@ -23,6 +23,7 @@ module Decidim
         # Returns nothing.
         def call
           return broadcast(:invalid) if form.invalid?
+
           update_conference
           link_participatory_processes
           link_assemblies
@@ -99,7 +100,9 @@ module Decidim
         end
 
         def should_notify_followers_registrations_enabled?
-          @conference.previous_changes["registrations_enabled"].present? && @conference.registrations_enabled?
+          @conference.previous_changes["registrations_enabled"].present? &&
+            @conference.registrations_enabled? &&
+            @conference.published?
         end
 
         def send_notification_update_conference
@@ -112,7 +115,8 @@ module Decidim
         end
 
         def should_notify_followers_update_conference?
-          important_attributes.any? { |attr| @conference.previous_changes[attr].present? }
+          important_attributes.any? { |attr| @conference.previous_changes[attr].present? } &&
+            @conference.published?
         end
 
         def important_attributes

data/app/commands/decidim/conferences/admin/update_diploma.rb

@@ -20,6 +20,7 @@ module Decidim
         def call
           @conference.with_lock do
             return broadcast(:invalid) if form.invalid?
+
             update_conference_diploma
             Decidim.traceability.perform_action!(:update_diploma, @conference, form.current_user) do
               @conference

data/app/commands/decidim/conferences/join_conference.rb

@@ -22,6 +22,7 @@ module Decidim
       def call
         conference.with_lock do
           return broadcast(:invalid) unless can_join_conference?
+
           create_registration
           create_meetings_registrations
           accept_invitation
@@ -77,6 +78,7 @@ module Decidim
       def notify_admin_over_percentage
         return send_notification_over(0.5) if occupied_slots_over?(0.5)
         return send_notification_over(0.8) if occupied_slots_over?(0.8)
+
         send_notification_over(1.0) if occupied_slots_over?(1.0)
       end
 

data/app/commands/decidim/conferences/leave_conference.rb

@@ -23,6 +23,7 @@ module Decidim
         @conference.with_lock do
           return broadcast(:invalid) unless registration
           return broadcast(:invalid) unless meetings_registrations
+
           destroy_registration
           destroy_meeting_registration
         end

data/app/controllers/decidim/conferences/admin/application_controller.rb

@@ -5,6 +5,10 @@ module Decidim
     module Admin
       # The main admin application controller for conferences
       class ApplicationController < Decidim::Admin::ApplicationController
+        register_permissions(::Decidim::Conferences::Admin::ApplicationController,
+                             Decidim::Conferences::Permissions,
+                             Decidim::Admin::Permissions)
+
         private
 
         def permissions_context
@@ -14,10 +18,7 @@ module Decidim
         end
 
         def permission_class_chain
-          [
-            Decidim::Conferences::Permissions,
-            Decidim::Admin::Permissions
-          ]
+          ::Decidim.permissions_registry.chain_for(::Decidim::Conferences::Admin::ApplicationController)
         end
       end
     end

data/app/controllers/decidim/conferences/admin/concerns/conference_admin.rb

@@ -12,6 +12,11 @@ module Decidim
         module ConferenceAdmin
           extend ActiveSupport::Concern
 
+          RegistersPermissions
+            .register_permissions(::Decidim::Conferences::Admin::Concerns::ConferenceAdmin,
+                                  ::Decidim::Conferences::Permissions,
+                                  ::Decidim::Admin::Permissions)
+
           included do
             include Decidim::Admin::ParticipatorySpaceAdminContext
             participatory_space_admin_layout
@@ -35,10 +40,7 @@ module Decidim
             end
 
             def permission_class_chain
-              [
-                Decidim::Conferences::Permissions,
-                Decidim::Admin::Permissions
-              ]
+              PermissionsRegistry.chain_for(ConferenceAdmin)
             end
           end
         end

data/app/controllers/decidim/conferences/admin/conference_registrations_controller.rb

@@ -9,13 +9,14 @@ module Decidim
       #
       class ConferenceRegistrationsController < Decidim::Conferences::Admin::ApplicationController
         include Concerns::ConferenceAdmin
+        include Decidim::Paginable
 
         helper_method :conference
 
         def index
           enforce_permission_to :read_conference_registrations, :conference, conference: conference
 
-          @conference_registrations = Decidim::Conferences::ConferenceRegistration.where(conference: conference).page(params[:page]).per(15)
+          @conference_registrations = paginate(Decidim::Conferences::ConferenceRegistration.where(conference: conference))
         end
 
         def export
@@ -52,6 +53,7 @@ module Decidim
 
         def conference_registration
           return if params[:id].blank?
+
           @conference_registration ||= conference.conference_registrations.find_by(id: params[:id])
         end
       end

data/app/controllers/decidim/conferences/admin/conference_speakers_controller.rb

@@ -8,13 +8,14 @@ module Decidim
       class ConferenceSpeakersController < Decidim::Conferences::Admin::ApplicationController
         helper Decidim::Conferences::Admin::ConferenceSpeakersHelper
         include Concerns::ConferenceAdmin
+        include Decidim::Paginable
 
         def index
           enforce_permission_to :index, :conference_speaker
 
           @query = params[:q]
 
-          @conference_speakers = Decidim::Conferences::Admin::ConferenceSpeakers.for(collection, @query).page(params[:page]).per(15)
+          @conference_speakers = paginate(Decidim::Conferences::Admin::ConferenceSpeakers.for(collection, @query))
         end
 
         def new

data/app/controllers/decidim/conferences/admin/conference_user_roles_controller.rb

@@ -7,10 +7,11 @@ module Decidim
       #
       class ConferenceUserRolesController < Decidim::Conferences::Admin::ApplicationController
         include Concerns::ConferenceAdmin
+        include Decidim::Paginable
 
         def index
           enforce_permission_to :index, :conference_user_role
-          @conference_user_roles = collection
+          @conference_user_roles = paginate(collection)
         end
 
         def new

data/app/controllers/decidim/conferences/admin/conferences_controller.rb

@@ -8,10 +8,11 @@ module Decidim
       class ConferencesController < Decidim::Conferences::Admin::ApplicationController
         helper_method :current_conference, :current_participatory_space
         layout "decidim/admin/conferences"
+        include Decidim::Paginable
 
         def index
           enforce_permission_to :read, :conference_list
-          @conferences = collection
+          @conferences = paginate(collection)
         end
 
         def new

data/app/controllers/decidim/conferences/admin/media_links_controller.rb

@@ -6,11 +6,12 @@ module Decidim
       # Controller that allows managing conference media links.
       class MediaLinksController < Decidim::Conferences::Admin::ApplicationController
         include Concerns::ConferenceAdmin
+        include Decidim::Paginable
 
         def index
           enforce_permission_to :index, :media_link
 
-          @media_links = collection.page(params[:page]).per(15)
+          @media_links = paginate(collection)
         end
 
         def new

data/app/controllers/decidim/conferences/admin/partners_controller.rb

@@ -7,11 +7,12 @@ module Decidim
       #
       class PartnersController < Decidim::Conferences::Admin::ApplicationController
         include Concerns::ConferenceAdmin
+        include Decidim::Paginable
 
         def index
           enforce_permission_to :index, :partner
 
-          @partners = collection.page(params[:page]).per(15)
+          @partners = paginate(collection)
         end
 
         def new

data/app/controllers/decidim/conferences/admin/registration_types_controller.rb

@@ -8,11 +8,12 @@ module Decidim
       class RegistrationTypesController < Decidim::Conferences::Admin::ApplicationController
         include Concerns::ConferenceAdmin
         include Decidim::ApplicationHelper
+        include Decidim::Paginable
 
         def index
           enforce_permission_to :index, :registration_type
 
-          @registration_types = collection.page(params[:page]).per(15)
+          @registration_types = paginate(collection)
         end
 
         def new

data/app/controllers/decidim/conferences/application_controller.rb

@@ -10,6 +10,11 @@ module Decidim
 
       include NeedsPermission
 
+      register_permissions(::Decidim::Conferences::ApplicationController,
+                           Decidim::Conferences::Permissions,
+                           Decidim::Admin::Permissions,
+                           Decidim::Permissions)
+
       private
 
       def permissions_context
@@ -19,11 +24,7 @@ module Decidim
       end
 
       def permission_class_chain
-        [
-          Decidim::Conferences::Permissions,
-          Decidim::Admin::Permissions,
-          Decidim::Permissions
-        ]
+        ::Decidim.permissions_registry.chain_for(::Decidim::Conferences::ApplicationController)
       end
 
       def permission_scope