decidim-conferences 0.18.0 → 0.20.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e0e842238aaaceb5245a50065ab55bded2a36ea31fa14c54dc8631f223ff75d3
-  data.tar.gz: c47d206b41f4c12721743942fdb7efe0e519f3c4fb4a8737f9a9a0cfc1bdc015
+  metadata.gz: 073cb7b12f74449977cfec9e2d7adb06068471c034cd41bfd4d7bfb18a34e178
+  data.tar.gz: 5f10e62f54d3a8511f565ef53c7b2d50ab49f1e75da0df5f89f35056efb36284
 SHA512:
-  metadata.gz: 1fb525774a7fa599adf5f6b7cb736cd9b1ee11a5ef7d7668a0a72ed26511b9424f862f3c9da1e89bfc6660608bc3f3c720f2b5d095cc379438b7f8cb546d7d5d
-  data.tar.gz: 0a16252c6a48fdab476be5217fac2966c009fa5387479e2818f788e028a5be8ee1105037dad3a999815156160e167a7a1f04776855fd70f2071a92957187eecd
+  metadata.gz: 5ab470eaa1b17bbd6cee13c6238fe492abd41f2888728dde9c650c7ac595b5af7852532740626726b457ca81933519c4673fcb66c81295d303ca2a65cb221145
+  data.tar.gz: 3a7469d214456fb69eb48eeeb0afe2656df021d630dba27dc97beac8eafd751960f047dcbcbb6816f4f2db20ae795a48295fd356c51c16ff32814984619eec53

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_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/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/conference_program_controller.rb

@@ -11,7 +11,8 @@ module Decidim
       helper_method :collection, :conference, :meeting_days, :meeting_component
 
       def show
-        raise ActionController::RoutingError, "No meetings for this conference " if meetings.empty?
+        raise ActionController::RoutingError, "No meetings for this conference " if meetings.blank?
+
         enforce_permission_to :list, :program
         redirect_to decidim_conferences.conference_path(current_participatory_space) unless current_user_can_visit_space?
       end
@@ -20,11 +21,13 @@ module Decidim
 
       def meeting_component
         return if params[:id].blank?
+
         @meeting_component ||= current_participatory_space.components.where(manifest_name: "meetings").find_by(id: params[:id])
       end
 
       def meetings
         return unless meeting_component.published? || !meeting_component.presence
+
         @meetings ||= Decidim::Meetings::Meeting.where(component: meeting_component).visible_meeting_for(current_user).order(:start_time)
       end
 

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

@@ -65,6 +65,7 @@ module Decidim
       def redirect_after_path
         referer = request.headers["Referer"]
         return redirect_to(conference_path(conference)) if referer =~ /invitation_token/
+
         redirect_back fallback_location: conference_path(conference)
       end
     end

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

@@ -11,6 +11,7 @@ module Decidim
 
       def index
         raise ActionController::RoutingError, "No speakers for this conference " if speakers.empty?
+
         enforce_permission_to :list, :speakers
         redirect_to decidim_conferences.conference_path(current_participatory_space) unless current_user_can_visit_space?
       end

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

@@ -13,6 +13,7 @@ module Decidim
 
       def index
         raise ActionController::RoutingError, "No media_links for this conference " if media_links.empty? && current_participatory_space.attachments.empty?
+
         enforce_permission_to :list, :media_links
         redirect_to decidim_conferences.conference_path(current_participatory_space) unless current_user_can_visit_space?
       end

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

@@ -11,6 +11,7 @@ module Decidim
 
       def index
         raise ActionController::RoutingError, "No registration types for this conference " if registration_types.empty? && current_participatory_space.registrations_enabled.empty?
+
         enforce_permission_to :list, :registration_types
         redirect_to decidim_conferences.conference_path(current_participatory_space) unless current_user_can_visit_space?
       end