decidim-meetings 0.28.1 → 0.28.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2938d2f32497611f571869b77a695164b51a47bcac475980697e0fb05b5ee681
-  data.tar.gz: 7eeb90d0095cfd435bc7e1a27bfebff6d9d48eacfb98fa3fcf12fa7f72d6a46e
+  metadata.gz: d0085a8f2f8df80b300246b11a3248dc8b9891e11699a49ec6beb81e65bf2e93
+  data.tar.gz: 4d5b5be680be38f726f4f12a19bba8c0da23feba1e2bb878d68015791770fded
 SHA512:
-  metadata.gz: 4bd4972aefef8559a1b546210dd95d71d1db90e64406ab796ad6314a3d92a8c6dfbde876394fe68b62194a492f1a4374687b1ae50325cd131e37ef600c5e0076
-  data.tar.gz: 9d2d4b400fb87da7ee84d7d8ef1c39e78a8882be1b53465b72c76adbf551af0590fabb312a458235fcf953a5b4f0e3d6b7191e4c8a0a38a0aae2489e92ccaf9d
+  metadata.gz: d9639caeb5db1426d878a2243a06a8a3e7b999d96cc491aedf76a3b60d45ddbf5362bb4fdc18d46e245d6658e360eac3b6b46e27761fc5555d7008d20ee19c8d
+  data.tar.gz: e84aa50842bb51384579aea8438e00a1872efffd830791601ead617503dbb19592450407447b98b6c146695e285254c247ea03f8cdbe57d821a7f90e67aa13ca

data/app/cells/decidim/meetings/join_meeting_button/registration_modal.erb

@@ -8,7 +8,7 @@
         <div class="form__wrapper">
           <%= cell("decidim/represent_user_group", form) %>
           <%= cell("decidim/public_participation", form) %>
-          <span class="meeting__registration-modal__terms"><%= registration_terms_text %></span>
+          <div class="meeting__registration-modal__terms"><%= registration_terms_text %></div>
         </div>
       </div>
     </div>

data/app/cells/decidim/meetings/meeting_l/image.erb

@@ -1,5 +1,5 @@
 <time class="card__calendar" datetime="<%= meeting.start_time.iso8601 %>">
-  <div class="card__calendar-month"><%= l(meeting.start_time, format: "%b") %></div>
-  <div class="card__calendar-day"><%= l(meeting.start_time, format: "%d") %></div>
-  <div class="card__calendar-year"><%= l(meeting.start_time, format: "%Y") %></div>
+  <span class="card__calendar-month"><%= l(meeting.start_time, format: "%b") %></span>
+  <span class="card__calendar-day"><%= l(meeting.start_time, format: "%d") %></span>
+  <span class="card__calendar-year"><%= l(meeting.start_time, format: "%Y") %></span>
 </time>

data/app/cells/decidim/meetings/meeting_l_cell.rb

@@ -11,10 +11,6 @@ module Decidim
 
       alias meeting model
 
-      def has_image?
-        true
-      end
-
       def extra_class
         "card__calendar-list__reset"
       end

data/app/helpers/decidim/meetings/application_helper.rb

@@ -59,7 +59,11 @@ module Decidim
 
       # If the content is safe, HTML tags are sanitized, otherwise, they are stripped.
       def render_meeting_body(meeting)
-        sanitized = render_sanitized_content(meeting, :description)
+        render_meeting_sanitize_field(meeting, :description)
+      end
+
+      def render_meeting_sanitize_field(meeting, field)
+        sanitized = render_sanitized_content(meeting, field)
         if safe_content?
           Decidim::ContentProcessor.render_without_format(sanitized).html_safe
         else

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

@@ -170,7 +170,7 @@ module Decidim
       end
 
       def can_be_joined_by?(user)
-        !closed? && registrations_enabled? && can_participate?(user)
+        !started? && registrations_enabled? && can_participate?(user)
       end
 
       def can_register_invitation?(user)
@@ -182,6 +182,10 @@ module Decidim
         closed_at.present?
       end
 
+      def started?
+        start_time < Time.current
+      end
+
       def past?
         end_time < Time.current
       end

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

@@ -15,10 +15,6 @@ module Decidim
       class MeetingPresenter < Decidim::Log::BasePresenter
         private
 
-        def resource_presenter
-          @resource_presenter ||= Decidim::Meetings::Log::ResourcePresenter.new(action_log.resource, h, action_log.extra["resource"])
-        end
-
         def diff_fields_mapping
           {
             address: :string,

data/app/presenters/decidim/meetings/admin_log/value_types/meeting_title_description_presenter.rb

@@ -10,7 +10,7 @@ module Decidim
           def present
             return unless value
 
-            renderer = Decidim::ContentRenderers::HashtagRenderer.new(h.translated_attribute(value))
+            renderer = Decidim::ContentRenderers::HashtagRenderer.new(h.decidim_escape_translated(value))
             renderer.render(links: false).html_safe
           end
         end

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

@@ -64,13 +64,10 @@ module Decidim
         end
       end
 
-      def closing_report(links: false, all_locales: false)
+      def closing_report(links: false, extras: false, strip_tags: false, all_locales: false)
         return unless meeting
 
-        handle_locales(meeting.closing_report, all_locales) do |content|
-          renderer = Decidim::ContentRenderers::HashtagRenderer.new(sanitized(content))
-          renderer.render(links:).html_safe
-        end
+        content_handle_locale(meeting.closing_report, all_locales, extras, links, strip_tags)
       end
 
       def registration_email_custom_content(links: false, all_locales: false)

data/app/services/decidim/meetings/close_meeting_reminder_generator.rb

@@ -38,11 +38,9 @@ module Decidim
 
       def send_reminders(component)
         intervals = Array(reminder_manifest.settings.attributes[:reminder_times].default)
-        space_admins = Decidim::ParticipatoryProcessUserRole.where(decidim_participatory_process_id: component.participatory_space_id, role: "admin").collect(&:user)
-        space_admins = (global_admins + space_admins).uniq
         intervals.each do |interval|
           finder_query(component.id, interval).find_each do |meeting|
-            authors = meeting.official? ? space_admins : [meeting.author]
+            authors = meeting.official? ? space_admins(component) : [meeting.author]
             authors.each do |author|
               send_notif = author.notification_settings.fetch("close_meeting_reminder", "1")
               next unless send_notif == "1"
@@ -60,8 +58,16 @@ module Decidim
         end
       end
 
-      def global_admins
-        @global_admins ||= Decidim::User.where(admin: true).all
+      def space_admins(component)
+        @space_admins ||= begin
+          space_admins = if component.participatory_space.respond_to?(:user_roles)
+                           component.participatory_space.user_roles(:admin).collect(&:user)
+                         else
+                           []
+                         end
+          global_admins = component.organization.admins
+          (global_admins + space_admins).uniq
+        end
       end
     end
   end

data/app/views/decidim/meetings/admin/registrations/edit.html.erb

@@ -12,7 +12,9 @@
         <div class="dropdown-pane" id="export-dropdown" data-dropdown data-auto-focus="true" data-close-on-click="true">
           <ul class="vertical menu add-components">
             <% %w(CSV JSON Excel).each do |format| %>
-              <li class="exports--format--<%= format.downcase %> exports--registrations"><%= link_to t("decidim.admin.exports.export_as", name: t("decidim.#{current_component.manifest.name}.admin.exports.registrations"), export_format: format), export_meeting_registrations_path(meeting_id: meeting, format:) %></li>
+              <%= link_to export_meeting_registrations_path(meeting_id: meeting, format:) do %>
+                <li class="exports--format--<%= format.downcase %> exports--registrations"><%= t("decidim.admin.exports.export_as", name: t("decidim.#{current_component.manifest.name}.admin.exports.registrations"), export_format: format) %></li>
+              <% end %>
             <% end %>
           </ul>
         </div>

data/app/views/decidim/meetings/meetings/_meeting_aside.html.erb

@@ -32,7 +32,6 @@
             data: { confirm: t("withdraw_confirmation_html", scope: "decidim.meetings.meetings.show") }
           ) do %>
           <span><%= t("withdraw_meeting", scope:"decidim.meetings.meetings.show") %></span>
-          <%= icon "pencil-line" %>
       <% end %>
     <% end %>
   </section>

data/app/views/decidim/meetings/meetings/_meeting_minutes.html.erb

@@ -5,7 +5,7 @@
   </div>
 
   <div class="meeting__agenda-item">
-    <div class="meeting__agenda-item__description editor-content mb-4"><%= decidim_sanitize_editor translated_attribute meeting.closing_report %></div>
+    <div class="meeting__agenda-item__description editor-content mb-4"><%= render_meeting_sanitize_field(meeting, :closing_report) %></div>
 
     <% if meeting.video_url.presence %>
       <div class="flex items-center gap-2 mt-2">

data/config/locales/ar.yml

@@ -396,7 +396,6 @@ ar:
         new_meeting: 'اجتماع جديد:'
       layouts:
         live_event:
-          administrate: إدارة
           close: إغلاق
           questions: الأسئلة
       mailer: