decidim-conferences 0.24.3 → 0.25.0.rc4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 52bc4e0c26ddf75a681b0bb366e8a4c73aa73f65ce09f695360df54e72b08442
-  data.tar.gz: 716b357dc0c52c4758ddec7c5e6d375841294510cd1ca8745829f560ed23512f
+  metadata.gz: ee1804c9f6703c0cb245a2dae893c5c1bb631d70932cb7d9143822f37a9cc4aa
+  data.tar.gz: 88107bfa43fd1dc97572d16b4e6dcacdd04c72211ce8aa38880509298b123a05
 SHA512:
-  metadata.gz: e5c276d6086e5e4904385b79217a7ce0cee89a45c36cb756ac261b7c4dfa2262a74b1fd1efd6711bd8e14c458854dfe94b0de683cb95d4eb2812257d41bce279
-  data.tar.gz: a3afd154a79137fdabbe0cb2d55a230611a25df81ed424ee89bfb437804ac048490f4c419cce482ef9e3a35a02e4067185075bbf588a25a8a9cafdef2fdcae0d
+  metadata.gz: 9968aa877a530ae392f63f4c5364fc1d1a121763a4bd9030ad3b27b315f08c9c5a7df3e9926a7b293c1f6254d5a846e921ae3d402bfdbe4956bd01482f76fe7a
+  data.tar.gz: 50a5346c96ba2d2a1f2516ae46d26750ea1157a3aec498faefbd249b7c38f6c9dccb21c68aa8d62fdbb2cd2e32ebeab5d03065003c62e7a1b7b9578dfce7a40a

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

@@ -27,7 +27,7 @@ module Decidim
       end
 
       def resource_image_path
-        model.hero_image.url
+        model.attached_uploader(:hero_image).path
       end
 
       def statuses

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

@@ -1,7 +1,7 @@
 <div class="column conference-speaker js-conference">
   <div class="speaker-img-wrapper">
     <div class="speaker-img">
-      <%= image_tag avatar.url, alt: "speaker-image" %>
+      <%= image_tag avatar_path, alt: "speaker-image" %>
     </div>
   </div>
   <div class="speaker-data">
@@ -26,7 +26,7 @@
       <div class="column medium-6">
         <div class="speaker-img-wrapper">
           <div class="speaker-img">
-            <%= image_tag avatar.url, alt: "speaker-image" %>
+            <%= image_tag avatar_path, alt: "speaker-image" %>
           </div>
         </div>
         <div class="speaker-data">

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

@@ -39,10 +39,10 @@ module Decidim
         speakers.map { |speaker| present(speaker) }
       end
 
-      def avatar
-        return model.user.avatar if model.user.present?
+      def avatar_path
+        return Decidim::UserPresenter.new(model.user).avatar_url if model.user.present?
 
-        model.avatar
+        model.attached_uploader(:avatar).path
       end
 
       def has_profile?

data/app/cells/decidim/conferences/content_blocks/highlighted_conferences/show.erb

@@ -8,7 +8,7 @@
           <div class="column">
             <%= link_to decidim_conferences.conference_path(conference), class: "card card--conference card--mini" do %>
               <div aria-hidden="true" class="card__image-top"
-                style="background-image:url(<%= conference.hero_image.url %>)"></div>
+                style="background-image:url(<%= conference.attached_uploader(:hero_image).path %>)"></div>
               <div class="card__content">
                 <span class="card__title card__link"><%= translated_attribute conference.title %></span>
               </div>

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

@@ -17,9 +17,9 @@ module Decidim
       end
 
       def logo
-        return unless model.logo.presence
+        return unless model.logo.attached?
 
-        "<div class='card p-m flex--cc'> #{image_tag model.logo.medium.url, alt: "logo"} </div>"
+        "<div class='card p-m flex--cc'> #{image_tag model.attached_uploader(:logo).path(variant: :medium), alt: "logo"} </div>"
       end
     end
   end

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

@@ -26,6 +26,7 @@ module Decidim
 
           Conference.transaction do
             copy_conference
+            copy_conference_attachments
             copy_conference_categories if @form.copy_categories?
             copy_conference_components if @form.copy_components?
           end
@@ -47,8 +48,6 @@ module Decidim
             description: @conference.description,
             short_description: @conference.short_description,
             location: @conference.location,
-            hero_image: @conference.hero_image,
-            banner_image: @conference.banner_image,
             promoted: @conference.promoted,
             scopes_enabled: @conference.scopes_enabled,
             scope: @conference.scope,
@@ -58,6 +57,14 @@ module Decidim
           )
         end
 
+        def copy_conference_attachments
+          [:hero_image, :banner_image].each do |attribute|
+            next unless @conference.attached_uploader(attribute).attached?
+
+            @copied_conference.send(attribute).attach(@conference.send(attribute).blob)
+          end
+        end
+
         def copy_conference_categories
           @conference.categories.each do |category|
             category_copied = category.dup

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

@@ -6,6 +6,8 @@ module Decidim
       # A command with all the business logic when creating a new conference
       # speaker in the system.
       class CreateConferenceSpeaker < Rectify::Command
+        include ::Decidim::AttachmentAttributesMethods
+
         # Public: Initializes the command.
         #
         # form - A form object with the params.
@@ -55,8 +57,6 @@ module Decidim
             :full_name,
             :twitter_handle,
             :personal_url,
-            :avatar,
-            :remove_avatar,
             :position,
             :affiliation,
             :short_bio
@@ -64,6 +64,8 @@ module Decidim
             decidim_conference_id: conference.id,
             conference: conference,
             user: form.user
+          ).merge(
+            attachment_attributes(:avatar)
           )
           conference_speaker = conference.speakers.build
           conference_speaker.conference = conference

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

@@ -6,6 +6,8 @@ module Decidim
       # A command with all the business logic when creating a new participatory
       # conference in the system.
       class UpdateConference < Rectify::Command
+        include ::Decidim::AttachmentAttributesMethods
+
         # Public: Initializes the command.
         #
         # conference - the Conference to update
@@ -85,16 +87,9 @@ module Decidim
             scopes_enabled: form.scopes_enabled,
             scope: form.scope,
             show_statistics: form.show_statistics
-          }.merge(uploader_attributes)
-        end
-
-        def uploader_attributes
-          {
-            hero_image: form.hero_image,
-            remove_hero_image: form.remove_hero_image,
-            banner_image: form.banner_image,
-            remove_banner_image: form.remove_banner_image
-          }.delete_if { |_k, val| val.is_a?(Decidim::ApplicationUploader) }
+          }.merge(
+            attachment_attributes(:hero_image, :banner_image)
+          )
         end
 
         def send_notification_registrations_enabled

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

@@ -6,12 +6,13 @@ module Decidim
       # A command with all the business logic when updating a conference
       # speaker in the system.
       class UpdateConferenceSpeaker < Rectify::Command
+        include ::Decidim::AttachmentAttributesMethods
+
         # Public: Initializes the command.
         #
         # form - A form object with the params.
         # conference_speaker - The ConferenceSpeaker to update
         def initialize(form, conference_speaker)
-          form.avatar = conference_speaker.avatar if form.avatar.blank?
           @form = form
           @conference_speaker = conference_speaker
         end
@@ -62,14 +63,9 @@ module Decidim
             :short_bio
           ).merge(
             user: form.user
-          ).merge(uploader_attributes)
-        end
-
-        def uploader_attributes
-          {
-            avatar: form.avatar,
-            remove_avatar: form.remove_avatar
-          }.delete_if { |_k, val| val.is_a?(Decidim::ApplicationUploader) }
+          ).merge(
+            attachment_attributes(:avatar)
+          )
         end
 
         def update_conference_speaker!

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

@@ -35,8 +35,8 @@ module Decidim
         attr_reader :form, :conference
 
         def update_conference_diploma
-          @conference.main_logo = @form.main_logo
-          @conference.signature = @form.signature
+          @conference.main_logo = @form.main_logo if @form.main_logo.present?
+          @conference.signature = @form.signature if @form.signature.present?
           @conference.signature_name = @form.signature_name
           @conference.sign_date = @form.sign_date
 

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

@@ -6,13 +6,13 @@ module Decidim
       # A command with all the business logic when updating a conference
       # partner in the system.
       class UpdatePartner < Rectify::Command
+        include ::Decidim::AttachmentAttributesMethods
+
         # Public: Initializes the command.
         #
         # form - A form object with the params.
         # conference_partner - The ConferencePartner to update
         def initialize(form, conference_partner)
-          form.logo = conference_partner.logo if form.logo.blank?
-
           @form = form
           @conference_partner = conference_partner
         end
@@ -54,9 +54,9 @@ module Decidim
             :name,
             :weight,
             :partner_type,
-            :link,
-            :logo,
-            :remove_logo
+            :link
+          ).merge(
+            attachment_attributes(:logo)
           )
         end
 

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_dependency "decidim/admin/application_controller"
-
 module Decidim
   module Conferences
     module Admin

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require_dependency "decidim/admin/application_controller"
-
 module Decidim
   module Conferences
     module Admin

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

@@ -41,9 +41,9 @@ module Decidim
         end
 
         def edit
-          @conference_speaker = collection.find(params[:id])
-          enforce_permission_to :update, :conference_speaker, speaker: @conference_speaker
-          @form = form(ConferenceSpeakerForm).from_model(@conference_speaker)
+          @item = collection.find(params[:id])
+          enforce_permission_to :update, :conference_speaker, speaker: @item
+          @form = form(ConferenceSpeakerForm).from_model(@item)
         end
 
         def update

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

@@ -82,11 +82,7 @@ module Decidim
         end
 
         def conference_params
-          {
-            id: params[:slug],
-            hero_image: current_conference.hero_image,
-            banner_image: current_conference.banner_image
-          }.merge(params[:conference].to_unsafe_h)
+          { id: params[:slug] }.merge(params[:conference].to_unsafe_h)
         end
       end
     end

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

@@ -16,7 +16,7 @@ module Decidim
         def update
           enforce_permission_to :update, :conference, conference: current_conference
 
-          @form = form(DiplomaForm).from_params(diploma_params).with_context(conference: current_conference)
+          @form = form(DiplomaForm).from_params(diploma_params).with_context(conference: current_conference, current_organization: current_organization)
 
           UpdateDiploma.call(@form, current_conference) do
             on(:ok) do
@@ -51,9 +51,7 @@ module Decidim
 
         def diploma_params
           {
-            id: params[:slug],
-            signature: current_conference.signature,
-            main_logo: current_conference.main_logo
+            id: params[:slug]
           }.merge(params[:conference].to_unsafe_h)
         end
       end

data/app/forms/decidim/conferences/admin/conference_form.rb

@@ -25,9 +25,9 @@ module Decidim
         attribute :scopes_enabled, Boolean
         attribute :scope_id, Integer
         attribute :hero_image
-        attribute :remove_hero_image
+        attribute :remove_hero_image, Boolean, default: false
         attribute :banner_image
-        attribute :remove_banner_image
+        attribute :remove_banner_image, Boolean, default: false
         attribute :show_statistics, Boolean
         attribute :start_date, Decidim::Attributes::LocalizedDate
         attribute :end_date, Decidim::Attributes::LocalizedDate

data/app/forms/decidim/conferences/admin/conference_speaker_form.rb

@@ -18,7 +18,7 @@ module Decidim
         attribute :twitter_handle, String
         attribute :personal_url, String
         attribute :avatar
-        attribute :remove_avatar
+        attribute :remove_avatar, Boolean, default: false
         attribute :user_id, Integer
         attribute :existing_user, Boolean, default: false
         attribute :conference_meeting_ids, Array[Integer]

data/app/forms/decidim/conferences/admin/partner_form.rb

@@ -14,10 +14,10 @@ module Decidim
         attribute :partner_type, String
         attribute :weight, Integer, default: 0
         attribute :logo
-        attribute :remove_logo
+        attribute :remove_logo, Boolean, default: false
 
         validates :name, :partner_type, presence: true, if: ->(form) { form.logo.present? }
-        validates :logo, presence: true, unless: ->(form) { form.logo.present? }
+        validates :logo, presence: true, unless: :persisted?
         validates :logo, passthru: {
           to: Decidim::Conferences::Partner,
           with: {

data/app/models/decidim/conference.rb

@@ -55,17 +55,17 @@ module Decidim
     validates :slug, uniqueness: { scope: :organization }
     validates :slug, presence: true, format: { with: Decidim::Conference.slug_format }
 
-    validates_upload :hero_image
-    mount_uploader :hero_image, Decidim::HeroImageUploader
+    has_one_attached :hero_image
+    validates_upload :hero_image, uploader: Decidim::HeroImageUploader
 
-    validates_upload :banner_image
-    mount_uploader :banner_image, Decidim::HomepageImageUploader
+    has_one_attached :banner_image
+    validates_upload :banner_image, uploader: Decidim::HomepageImageUploader
 
-    validates_upload :main_logo
-    mount_uploader :main_logo, Decidim::Conferences::DiplomaUploader
+    has_one_attached :main_logo
+    validates_upload :main_logo, uploader: Decidim::Conferences::DiplomaUploader
 
-    validates_upload :signature
-    mount_uploader :signature, Decidim::Conferences::DiplomaUploader
+    has_one_attached :signature
+    validates_upload :signature, uploader: Decidim::Conferences::DiplomaUploader
 
     searchable_fields({
                         scope_id: :decidim_scope_id,

data/app/models/decidim/conference_speaker.rb

@@ -18,8 +18,8 @@ module Decidim
 
     default_scope { order(full_name: :asc, created_at: :asc) }
 
-    validates_avatar
-    mount_uploader :avatar, Decidim::AvatarUploader
+    has_one_attached :avatar
+    validates_avatar :avatar, uploader: Decidim::AvatarUploader
 
     delegate :organization, to: :conference
 

data/app/models/decidim/conferences/partner.rb

@@ -14,8 +14,8 @@ module Decidim
 
       default_scope { order(partner_type: :desc, weight: :asc) }
 
-      validates_avatar :logo
-      mount_uploader :logo, Decidim::Conferences::PartnerLogoUploader
+      has_one_attached :logo
+      validates_avatar :logo, uploader: Decidim::Conferences::PartnerLogoUploader
 
       delegate :organization, to: :conference
 

data/app/packs/entrypoints/decidim_conferences_admin.js

@@ -0,0 +1,7 @@
+import "src/decidim/conferences/admin/conferences"
+import "src/decidim/conferences/admin/conference_speakers"
+import "src/decidim/conferences/admin/conference_invite_form"
+import "src/decidim/slug_form"
+
+// Images
+require.context("../images", true)

data/app/{assets/javascripts/decidim/conferences/admin/conference_invite_form.es6 → packs/src/decidim/conferences/admin/conference_invite_form.js}

@@ -1,6 +1,6 @@
-((exports) => {
-  const { createFieldDependentInputs } = exports.DecidimAdmin;
+import createFieldDependentInputs from "src/decidim/admin/field_dependent_inputs.component"
 
+$(() => {
   const $attendeeType = $('[name="conference_registration_invite[existing_user]"');
 
   createFieldDependentInputs({
@@ -22,4 +22,4 @@
       return $("#conference_registration_invite_existing_user_true").is(":checked")
     }
   });
-})(window);
+})

data/app/{assets/javascripts/decidim/conferences/admin/conference_speakers.js.es6 → packs/src/decidim/conferences/admin/conference_speakers.js}

@@ -1,6 +1,6 @@
-((exports) => {
-  const { createFieldDependentInputs } = exports.DecidimAdmin;
+import createFieldDependentInputs from "src/decidim/admin/field_dependent_inputs.component"
 
+$(() => {
   const $conferenceSpeakerType = $("#conference_speaker_existing_user");
 
   createFieldDependentInputs({
@@ -23,4 +23,4 @@
     }
   });
 
-})(window);
+})

data/app/packs/src/decidim/conferences/admin/conferences.js

@@ -0,0 +1,27 @@
+$(() => {
+  const $conferenceScopeEnabled = $("#conference_scopes_enabled");
+  const $conferenceScopeId = $("#conference_scope_id");
+  const $form = $(".edit_conference, .new_conference");
+
+  if ($form.length > 0) {
+    $conferenceScopeEnabled.on("change", (event) => {
+      const checked = event.target.checked;
+      window.theDataPicker.enabled($conferenceScopeId, checked);
+    })
+    window.theDataPicker.enabled($conferenceScopeId, $conferenceScopeEnabled.prop("checked"));
+
+    const $registrationsEnabled = $form.find("#conference_registrations_enabled");
+    const $availableSlots = $form.find("#conference_available_slots");
+    const toggleDisabledFields = () => {
+      const enabled = $registrationsEnabled.prop("checked");
+      $availableSlots.attr("disabled", !enabled);
+
+      $form.find("#conference_registrations_terms .editor-container").each((idx, node) => {
+        const quill = Quill.find(node);
+        quill.enable(enabled);
+      })
+    };
+    $registrationsEnabled.on("change", toggleDisabledFields);
+    toggleDisabledFields();
+  }
+});

data/app/presenters/decidim/conferences/conference_stats_presenter.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Conferences
     # A presenter to render statistics in the homepage.
-    class ConferenceStatsPresenter < Rectify::Presenter
+    class ConferenceStatsPresenter < Decidim::StatsPresenter
       attribute :conference, Decidim::Conference
       include IconHelper
 
@@ -11,39 +11,25 @@ module Decidim
       def highlighted
         highlighted_stats = component_stats(priority: StatsRegistry::HIGH_PRIORITY)
         highlighted_stats.concat(component_stats(priority: StatsRegistry::MEDIUM_PRIORITY))
+        highlighted_stats.concat(comments_stats(:conferences))
         highlighted_stats = highlighted_stats.reject(&:empty?)
         highlighted_stats = highlighted_stats.reject { |_manifest, _name, data| data.zero? }
-        grouped_highlighted_stats = highlighted_stats.group_by { |stats| stats.first.name }
+        grouped_highlighted_stats = highlighted_stats.group_by(&:first)
 
-        safe_join(
-          grouped_highlighted_stats.map do |_manifest_name, stats|
-            content_tag :div, class: "process_stats-item" do
-              safe_join(
-                stats.each_with_index.map do |stat, index|
-                  render_stats_data(stat[0], stat[1], stat[2], index)
-                end
-              )
-            end
-          end
-        )
+        statistics(grouped_highlighted_stats)
       end
 
       private
 
       def component_stats(conditions)
         Decidim.component_manifests.map do |component_manifest|
-          component_manifest.stats.filter(conditions).with_context(published_components).map { |name, data| [component_manifest, name, data] }.flatten
+          component_manifest.stats
+                            .filter(conditions)
+                            .with_context(published_components)
+                            .map { |name, data| [component_manifest.name, name, data] }.flatten
         end
       end
 
-      def render_stats_data(component_manifest, name, data, index)
-        safe_join([
-                    index.zero? ? manifest_icon(component_manifest, role: "img", "aria-hidden": true) : " /&nbsp".html_safe,
-                    content_tag(:span, "#{number_with_delimiter(data)} " + I18n.t(name, scope: "decidim.conferences.statistics"),
-                                class: "#{name} process_stats-text")
-                  ])
-      end
-
       def published_components
         @published_components ||= Component.where(participatory_space: conference).published
       end

data/app/queries/decidim/conferences/admin/admin_users.rb

@@ -3,7 +3,7 @@
 module Decidim
   module Conferences
     module Admin
-      # A class used to find the admins for an conference.
+      # A class used to find the admins for an conference or an organization conferences.
       class AdminUsers < Rectify::Query
         # Syntactic sugar to initialize the class and return the queried objects.
         #
@@ -12,33 +12,46 @@ module Decidim
           new(conference).query
         end
 
+        # Syntactic sugar to initialize the class and return the queried objects.
+        #
+        # organization - an organization that needs to find its conference admins
+        def self.for_organization(organization)
+          new(nil, organization).query
+        end
+
         # Initializes the class.
         #
         # conference - an conference that needs to find its conference admins
-        def initialize(conference)
+        # organization - an organization that needs to find its conference admins
+        def initialize(conference, organization = nil)
           @conference = conference
+          @organization = conference&.organization || organization
         end
 
         # Finds organization admins and the users with role admin for the given conference.
         #
         # Returns an ActiveRecord::Relation.
         def query
-          Decidim::User.where(id: organization_admins).or(conference_user_admins)
+          organization.admins.or(conferences_user_admins)
         end
 
         private
 
-        attr_reader :conference
+        attr_reader :conference, :organization
 
-        def organization_admins
-          conference.organization.admins
+        def conferences_user_admins
+          Decidim::User.where(
+            id: Decidim::ConferenceUserRole.where(conference: conferences, role: :admin)
+                                           .select(:decidim_user_id)
+          )
         end
 
-        def conference_user_admins
-          conference_user_admin_ids = Decidim::ConferenceUserRole
-                                      .where(conference: conference, role: :admin)
-                                      .pluck(:decidim_user_id)
-          Decidim::User.where(id: conference_user_admin_ids)
+        def conferences
+          if conference
+            [conference]
+          else
+            Decidim::Conference.where(organization: organization)
+          end
         end
       end
     end

data/app/uploaders/decidim/conferences/diploma_uploader.rb

@@ -4,8 +4,10 @@ module Decidim
   module Conferences
     # This class deals with uploading the conference partner logo.
     class DiplomaUploader < ImageUploader
-      version :thumb do
-        process resize_to_fit: [275, 90]
+      set_variants do
+        {
+          thumb: { resize_to_fit: [275, 90] }
+        }
       end
     end
   end

data/app/uploaders/decidim/conferences/partner_logo_uploader.rb

@@ -4,11 +4,11 @@ module Decidim
   module Conferences
     # This class deals with uploading the conference partner logo.
     class PartnerLogoUploader < ImageUploader
-      version :thumb do
-        process resize_to_fit: [200, 55]
-      end
-      version :medium do
-        process resize_to_fit: [600, 160]
+      set_variants do
+        {
+          thumb: { resize_to_fit: [200, 55] },
+          medium: { resize_to_fit: [600, 160] }
+        }
       end
     end
   end

data/app/uploaders/decidim/cw/conferences/diploma_uploader.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Decidim::Cw
+  module Conferences
+    # This class deals with uploading the conference partner logo.
+    class DiplomaUploader < Decidim::Cw::ImageUploader
+      set_variants do
+        {
+          thumb: { resize_to_fit: [275, 90] }
+        }
+      end
+    end
+  end
+end