decidim-meetings 0.25.0.rc4 → 0.26.0.rc1

data/app/forms/decidim/meetings/meeting_form.rb

@@ -3,55 +3,49 @@
 module Decidim
   module Meetings
     # This class holds a Form to create/update meetings for Participants and UserGroups.
-    class MeetingForm < Decidim::Form
+    class MeetingForm < ::Decidim::Meetings::BaseMeetingForm
       attribute :title, String
       attribute :description, String
       attribute :location, String
       attribute :location_hints, String
 
-      attribute :address, String
-      attribute :latitude, Float
-      attribute :longitude, Float
-      attribute :start_time, Decidim::Attributes::TimeWithZone
-      attribute :end_time, Decidim::Attributes::TimeWithZone
       attribute :decidim_scope_id, Integer
       attribute :decidim_category_id, Integer
       attribute :user_group_id, Integer
-      attribute :online_meeting_url, String
-      attribute :type_of_meeting, String
       attribute :registration_type, String
       attribute :registrations_enabled, Boolean, default: false
       attribute :registration_url, String
       attribute :available_slots, Integer, default: 0
       attribute :registration_terms, String
-      attribute :show_embedded_iframe, Boolean, default: false
+      attribute :iframe_embed_type, String, default: "none"
+      attribute :iframe_access_level, String
 
+      validates :iframe_embed_type, inclusion: { in: Decidim::Meetings::Meeting.participants_iframe_embed_types }
       validates :title, presence: true
       validates :description, presence: true
       validates :type_of_meeting, presence: true
       validates :location, presence: true, if: ->(form) { form.in_person_meeting? || form.hybrid_meeting? }
-      validates :address, presence: true, if: ->(form) { form.needs_address? }
-      validates :address, geocoding: true, if: ->(form) { form.has_address? && !form.geocoded? && form.needs_address? }
       validates :online_meeting_url, presence: true, url: true, if: ->(form) { form.online_meeting? || form.hybrid_meeting? }
       validates :registration_type, presence: true
       validates :available_slots, numericality: { greater_than_or_equal_to: 0 }, presence: true, if: ->(form) { form.on_this_platform? }
       validates :registration_terms, presence: true, if: ->(form) { form.on_this_platform? }
       validates :registration_url, presence: true, url: true, if: ->(form) { form.on_different_platform? }
-      validates :start_time, presence: true, date: { before: :end_time }
-      validates :end_time, presence: true, date: { after: :start_time }
-
-      validates :current_component, presence: true
       validates :category, presence: true, if: ->(form) { form.decidim_category_id.present? }
       validates :scope, presence: true, if: ->(form) { form.decidim_scope_id.present? }
       validates :decidim_scope_id, scope_belongs_to_component: true, if: ->(form) { form.decidim_scope_id.present? }
       validates :clean_type_of_meeting, presence: true
+      validates(
+        :iframe_access_level,
+        inclusion: { in: Decidim::Meetings::Meeting.iframe_access_levels },
+        if: ->(form) { %w(embed_in_meeting_page open_in_new_tab).include?(form.iframe_embed_type) }
+      )
       validate :embeddable_meeting_url
 
       delegate :categories, to: :current_component
 
       def map_model(model)
         self.decidim_category_id = model.categorization.decidim_category_id if model.categorization
-        presenter = MeetingPresenter.new(model)
+        presenter = MeetingEditionPresenter.new(model)
         self.title = presenter.title(all_locales: false)
         self.description = presenter.description(all_locales: false)
         self.location = presenter.location(all_locales: false)
@@ -82,34 +76,6 @@ module Decidim
         @category ||= categories.find_by(id: decidim_category_id)
       end
 
-      def geocoding_enabled?
-        Decidim::Map.available?(:geocoding)
-      end
-
-      def has_address?
-        geocoding_enabled? && address.present?
-      end
-
-      def needs_address?
-        in_person_meeting? || hybrid_meeting?
-      end
-
-      def geocoded?
-        latitude.present? && longitude.present?
-      end
-
-      def online_meeting?
-        type_of_meeting == "online"
-      end
-
-      def in_person_meeting?
-        type_of_meeting == "in_person"
-      end
-
-      def hybrid_meeting?
-        type_of_meeting == "hybrid"
-      end
-
       def clean_type_of_meeting
         type_of_meeting.presence
       end
@@ -123,6 +89,24 @@ module Decidim
         end
       end
 
+      def iframe_access_level_select
+        Decidim::Meetings::Meeting.iframe_access_levels.map do |level, _value|
+          [
+            I18n.t("iframe_access_level.#{level}", scope: "decidim.meetings"),
+            level
+          ]
+        end
+      end
+
+      def iframe_embed_type_select
+        Decidim::Meetings::Meeting.participants_iframe_embed_types.map do |type, _value|
+          [
+            I18n.t("iframe_embed_type.#{type}", scope: "decidim.meetings"),
+            type
+          ]
+        end
+      end
+
       def on_this_platform?
         registration_type == "on_this_platform"
       end
@@ -145,9 +129,9 @@ module Decidim
       end
 
       def embeddable_meeting_url
-        if online_meeting_url.present? && show_embedded_iframe
+        if online_meeting_url.present? && %w(embed_in_meeting_page open_in_live_event_page).include?(iframe_embed_type)
           embedder_service = Decidim::Meetings::MeetingIframeEmbedder.new(online_meeting_url)
-          errors.add(:show_embedded_iframe, :not_embeddable) unless embedder_service.embeddable?
+          errors.add(:iframe_embed_type, :not_embeddable) unless embedder_service.embeddable?
         end
       end
     end

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

@@ -8,10 +8,10 @@ module Decidim
       include PaginateHelper
       include Decidim::MapHelper
       include Decidim::Meetings::MapHelper
-      include Decidim::Meetings::MeetingsHelper
       include Decidim::Comments::CommentsHelper
       include Decidim::SanitizeHelper
       include Decidim::CheckBoxesTreeHelper
+      include Decidim::RichTextEditorHelper
 
       def filter_origin_values
         origin_values = []
@@ -57,6 +57,18 @@ module Decidim
           ["my_meetings", t("decidim.meetings.meetings.filters.my_meetings")]
         ]
       end
+
+      # If the meeting is official or the rich text editor is enabled on the
+      # frontend, the meeting body is considered as safe content; that's unless
+      # the meeting comes from a collaborative_draft or a participatory_text.
+      def safe_content?
+        rich_text_editor_in_public_views? || @meeting.official?
+      end
+
+      # If the content is safe, HTML tags are sanitized, otherwise, they are stripped.
+      def render_meeting_body(meeting)
+        render_sanitized_content(meeting, :description)
+      end
     end
   end
 end

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

@@ -0,0 +1,150 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Meetings
+    # Custom helpers, scoped to the meetings engine.
+    #
+    module Directory
+      module ApplicationHelper
+        include PaginateHelper
+        include Decidim::MapHelper
+        include Decidim::Meetings::MapHelper
+        include Decidim::Meetings::MeetingsHelper
+        include Decidim::Comments::CommentsHelper
+        include Decidim::SanitizeHelper
+        include Decidim::CheckBoxesTreeHelper
+
+        def filter_type_values
+          type_values = []
+          Decidim::Meetings::Meeting::TYPE_OF_MEETING.each do |type|
+            type_values << TreePoint.new(type, t("decidim.meetings.meetings.filters.type_values.#{type}"))
+          end
+
+          TreeNode.new(
+            TreePoint.new("", t("decidim.meetings.meetings.filters.type_values.all")),
+            type_values
+          )
+        end
+
+        def filter_date_values
+          TreeNode.new(
+            TreePoint.new("", t("decidim.meetings.meetings.filters.date_values.all")),
+            [
+              TreePoint.new("upcoming", t("decidim.meetings.meetings.filters.date_values.upcoming")),
+              TreePoint.new("past", t("decidim.meetings.meetings.filters.date_values.past"))
+            ]
+          )
+        end
+
+        def directory_filter_scopes_values
+          main_scopes = current_organization.scopes.top_level
+
+          scopes_values = main_scopes.includes(:scope_type, :children).flat_map do |scope|
+            TreeNode.new(
+              TreePoint.new(scope.id.to_s, translated_attribute(scope.name, current_organization)),
+              scope_children_to_tree(scope)
+            )
+          end
+
+          scopes_values.prepend(TreePoint.new("global", t("decidim.scopes.global")))
+
+          TreeNode.new(
+            TreePoint.new("", t("decidim.meetings.application_helper.filter_scope_values.all")),
+            scopes_values
+          )
+        end
+
+        def scope_children_to_tree(scope)
+          return unless scope.children.any?
+
+          scope.children.includes(:scope_type, :children).flat_map do |child|
+            TreeNode.new(
+              TreePoint.new(child.id.to_s, translated_attribute(child.name, current_organization)),
+              scope_children_to_tree(child)
+            )
+          end
+        end
+
+        def directory_meeting_spaces_values
+          participatory_spaces = current_organization.public_participatory_spaces
+
+          spaces = participatory_spaces.collect(&:model_name).uniq.map do |participatory_space|
+            TreePoint.new(participatory_space.name.underscore, participatory_space.human(count: 2))
+          end
+
+          TreeNode.new(
+            TreePoint.new("", t("decidim.meetings.application_helper.filter_meeting_space_values.all")),
+            spaces
+          )
+        end
+
+        def directory_filter_categories_values
+          participatory_spaces = current_organization.public_participatory_spaces
+          list_of_ps = participatory_spaces.flat_map do |current_participatory_space|
+            next unless current_participatory_space.respond_to?(:categories)
+
+            sorted_main_categories = current_participatory_space.categories.first_class.includes(:subcategories).sort_by do |category|
+              [category.weight, translated_attribute(category.name, current_organization)]
+            end
+
+            categories_values = categories_values(sorted_main_categories)
+
+            next if categories_values.empty?
+
+            key_point = current_participatory_space.class.name.gsub("::", "__") + current_participatory_space.id.to_s
+
+            TreeNode.new(
+              TreePoint.new(key_point, translated_attribute(current_participatory_space.title, current_organization)),
+              categories_values
+            )
+          end
+
+          list_of_ps.compact!
+          TreeNode.new(
+            TreePoint.new("", t("decidim.meetings.application_helper.filter_category_values.all")),
+            list_of_ps
+          )
+        end
+
+        def directory_filter_origin_values
+          origin_values = []
+          origin_values << TreePoint.new("official", t("decidim.meetings.meetings.filters.origin_values.official"))
+          origin_values << TreePoint.new("citizens", t("decidim.meetings.meetings.filters.origin_values.citizens"))
+          origin_values << TreePoint.new("user_group", t("decidim.meetings.meetings.filters.origin_values.user_groups")) if current_organization.user_groups_enabled?
+
+          TreeNode.new(
+            TreePoint.new("", t("decidim.meetings.meetings.filters.origin_values.all")),
+            origin_values
+          )
+        end
+
+        # Options to filter meetings by activity.
+        def activity_filter_values
+          [
+            ["all", t("decidim.meetings.meetings.filters.all")],
+            ["my_meetings", t("decidim.meetings.meetings.filters.my_meetings")]
+          ]
+        end
+
+        protected
+
+        def categories_values(sorted_main_categories)
+          sorted_main_categories.flat_map do |category|
+            sorted_descendant_categories = category.descendants.includes(:subcategories).sort_by do |subcategory|
+              [subcategory.weight, translated_attribute(subcategory.name, current_organization)]
+            end
+
+            subcategories = sorted_descendant_categories.flat_map do |subcategory|
+              TreePoint.new(subcategory.id.to_s, translated_attribute(subcategory.name, current_organization))
+            end
+
+            TreeNode.new(
+              TreePoint.new(category.id.to_s, translated_attribute(category.name, current_organization)),
+              subcategories
+            )
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -9,8 +9,11 @@ module Decidim
       include Decidim::Meetings::MeetingsHelper
       include Decidim::SanitizeHelper
       include Decidim::Meetings::Engine.routes.url_helpers
+
+      delegate :title, :state, :withdrawn?, to: :model
+
       def description
-        decidim_sanitize meeting_description(model)
+        decidim_sanitize_editor meeting_description(model)
       end
     end
   end

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

@@ -5,6 +5,7 @@ module Decidim
     # Custom helpers used in meetings views
     module MeetingsHelper
       include Decidim::ApplicationHelper
+      include Decidim::Meetings::ApplicationHelper
       include Decidim::TranslationsHelper
       include Decidim::ResourceHelper
 
@@ -33,6 +34,8 @@ module Decidim
           "alert"
         when "transparent"
           "secondary"
+        when "withdraw"
+          "alert"
         end
       end
 

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

@@ -14,6 +14,7 @@ module Decidim
       include Decidim::HasCategory
       include Decidim::Followable
       include Decidim::Comments::CommentableWithComponent
+      include Decidim::Comments::HasAvailabilityAttributes
       include Decidim::Searchable
       include Decidim::Traceable
       include Decidim::Loggable
@@ -44,6 +45,9 @@ module Decidim
         source: :user
       )
 
+      enum iframe_access_level: [:all, :signed_in, :registered], _prefix: true
+      enum iframe_embed_type: [:none, :embed_in_meeting_page, :open_in_live_event_page, :open_in_new_tab], _prefix: true
+
       component_manifest_name "meetings"
 
       validates :title, presence: true
@@ -53,6 +57,8 @@ module Decidim
       scope :published, -> { where.not(published_at: nil) }
       scope :past, -> { where(arel_table[:end_time].lteq(Time.current)) }
       scope :upcoming, -> { where(arel_table[:end_time].gteq(Time.current)) }
+      scope :withdrawn, -> { where(state: "withdrawn") }
+      scope :except_withdrawn, -> { where.not(state: "withdrawn").or(where(state: nil)) }
 
       scope :visible_meeting_for, lambda { |user|
         (all.published.distinct if user&.admin?) ||
@@ -95,7 +101,7 @@ module Decidim
               "
             end
 
-            where(query, false, true, user.id, user.id, *user_role_queries.compact.map { user.id }).published.distinct
+            where(Arel.sql(query).to_s, false, true, user.id, user.id, *user_role_queries.compact.map { user.id }).published.distinct
           else
             published.visible
           end
@@ -121,6 +127,10 @@ module Decidim
       # we create a salt for the meeting only on new meetings to prevent changing old IDs for existing (Ether)PADs
       before_create :set_default_salt
 
+      def self.participants_iframe_embed_types
+        iframe_embed_types.except(:open_in_live_event_page)
+      end
+
       # Return registrations of a particular meeting made by users representing a group
       def user_group_registrations
         registrations.where.not(decidim_user_group_id: nil)
@@ -153,6 +163,10 @@ module Decidim
         end_time < Time.current
       end
 
+      def emendation?
+        false
+      end
+
       def has_available_slots?
         return true if available_slots.zero?
 
@@ -171,6 +185,11 @@ module Decidim
         component.settings.maps_enabled?
       end
 
+      # Public: Overrides the `accepts_new_comments?` CommentableWithComponent concern method.
+      def accepts_new_comments?
+        commentable? && !component.current_settings.comments_blocked && comments_allowed?
+      end
+
       # Public: Overrides the `allow_resource_permissions?` Resourceable concern method.
       def allow_resource_permissions?
         component.settings.resources_permissions_enabled
@@ -199,6 +218,17 @@ module Decidim
         Decidim::Meetings::Meeting.visible_meeting_for(user).exists?(id: id)
       end
 
+      def iframe_access_level_allowed_for_user?(user)
+        case iframe_access_level
+        when "all"
+          true
+        when "signed_in"
+          user.present?
+        else
+          has_registration_for?(user)
+        end
+      end
+
       # Return the duration of the meeting in minutes
       def meeting_duration
         @meeting_duration ||= ((end_time - start_time) / 1.minute).abs
@@ -210,6 +240,21 @@ module Decidim
         !private_meeting? || transparent?
       end
 
+      # Public: Checks if the author has withdrawn the meeting.
+      #
+      # Returns Boolean.
+      def withdrawn?
+        state == "withdrawn"
+      end
+
+      # Checks whether the user can withdraw the given meeting.
+      #
+      # user - the user to check for withdrawability.
+      # past meetings cannot be withdrawn
+      def withdrawable_by?(user)
+        user && !withdrawn? && !past? && authored_by?(user)
+      end
+
       # Overwrites method from Paddable to add custom rules in order to know
       # when to display a pad or not.
       def pad_is_visible?
@@ -280,6 +325,13 @@ module Decidim
         !!attendees_count && attendees_count.positive?
       end
 
+      def live?
+        start_time &&
+          end_time &&
+          Time.current >= (start_time - 10.minutes) &&
+          Time.current <= end_time
+      end
+
       def self.sort_by_translated_title_asc
         field = Arel::Nodes::InfixOperation.new("->>", arel_table[:title], Arel::Nodes.build_quoted(I18n.locale))
         order(Arel::Nodes::InfixOperation.new("", field, Arel.sql("ASC")))

data/app/packs/src/decidim/meetings/admin/meetings_form.js

@@ -2,6 +2,7 @@ import AutoButtonsByPositionComponent from "src/decidim/admin/auto_buttons_by_po
 import AutoLabelByPositionComponent from "src/decidim/admin/auto_label_by_position.component"
 import createSortList from "src/decidim/admin/sort_list.component"
 import createDynamicFields from "src/decidim/admin/dynamic_fields.component"
+import createFieldDependentInputs from "src/decidim/admin/field_dependent_inputs.component"
 import attachGeocoding from "src/decidim/geocoding/attach_input"
 
 $(() => {
@@ -118,15 +119,14 @@ $(() => {
     });
 
     $meetingRegistrationType.trigger("change");
-  }
 
-  const $meetingForm = $(".meetings_form");
-  if ($meetingForm.length > 0) {
-    const $meetingTypeOfMeeting = $meetingForm.find("#meeting_type_of_meeting");
-    const $meetingOnlineFields = $meetingForm.find(".field[data-meeting-type='online']");
-    const $meetingInPersonFields = $meetingForm.find(".field[data-meeting-type='in_person']");
+    const $meetingTypeOfMeeting = $form.find("#meeting_type_of_meeting");
+    const $meetingOnlineFields = $form.find(".field[data-meeting-type='online']");
+    const $meetingInPersonFields = $form.find(".field[data-meeting-type='in_person']");
+    const $meetingOnlineAccessLevelFields = $form.find(".field[data-meeting-type='online-access-level']");
+    const $meetingIframeEmbedType = $form.find("#meeting_iframe_embed_type");
 
-    const toggleDependsOnSelect = ($target, $showDiv, type) => {
+    const toggleTypeDependsOnSelect = ($target, $showDiv, type) => {
       const value = $target.val();
       if (value === "hybrid") {
         $showDiv.show();
@@ -140,11 +140,27 @@ $(() => {
 
     $meetingTypeOfMeeting.on("change", (ev) => {
       const $target = $(ev.target);
-      toggleDependsOnSelect($target, $meetingOnlineFields, "online");
-      toggleDependsOnSelect($target, $meetingInPersonFields, "in_person");
+      const embedTypeValue = $("#meeting_iframe_embed_type select").val();
+
+      toggleTypeDependsOnSelect($target, $meetingOnlineFields, "online");
+      toggleTypeDependsOnSelect($target, $meetingInPersonFields, "in_person");
+      if (embedTypeValue === "none") {
+        $meetingOnlineAccessLevelFields.hide();
+      } else {
+        toggleTypeDependsOnSelect($target, $meetingOnlineAccessLevelFields, "online");
+      }
     });
 
-    toggleDependsOnSelect($meetingTypeOfMeeting, $meetingOnlineFields, "online");
-    toggleDependsOnSelect($meetingTypeOfMeeting, $meetingInPersonFields, "in_person");
+    toggleTypeDependsOnSelect($meetingTypeOfMeeting, $meetingOnlineFields, "online");
+    toggleTypeDependsOnSelect($meetingTypeOfMeeting, $meetingInPersonFields, "in_person");
+    createFieldDependentInputs({
+      controllerField: $meetingIframeEmbedType,
+      wrapperSelector: ".iframe-fields",
+      dependentFieldsSelector: ".iframe-fields--access-level",
+      dependentInputSelector: "input",
+      enablingCondition: ($field) => {
+        return $field.find("select").val() !== "none"
+      }
+    });
   }
 })

data/app/packs/src/decidim/meetings/meetings_form.js

@@ -1,4 +1,5 @@
 import attachGeocoding from "src/decidim/geocoding/attach_input"
+import createFieldDependentInputs from "src/decidim/admin/field_dependent_inputs.component"
 
 $(() => {
   // Adds the latitude/longitude inputs after the geocoding is done
@@ -12,6 +13,7 @@ $(() => {
     const $meetingTypeOfMeeting = $form.find("#meeting_type_of_meeting");
     const $meetingOnlineFields = $form.find(".field[data-meeting-type='online']");
     const $meetingInPersonFields = $form.find(".field[data-meeting-type='in_person']");
+    const $meetingOnlineAccessLevelFields = $form.find(".field[data-meeting-type='online-access-level']");
 
     const toggleDependsOnSelect = ($target, $showDiv, type) => {
       const value = $target.val();
@@ -27,8 +29,15 @@ $(() => {
 
     $meetingTypeOfMeeting.on("change", (ev) => {
       const $target = $(ev.target);
+      const embedTypeValue = $("#meeting_iframe_embed_type").val();
+
       toggleDependsOnSelect($target, $meetingOnlineFields, "online");
       toggleDependsOnSelect($target, $meetingInPersonFields, "in_person");
+      if (embedTypeValue === "none") {
+        $meetingOnlineAccessLevelFields.hide();
+      } else {
+        toggleDependsOnSelect($target, $meetingOnlineAccessLevelFields, "online");
+      }
     });
 
     toggleDependsOnSelect($meetingTypeOfMeeting, $meetingOnlineFields, "online");
@@ -50,5 +59,15 @@ $(() => {
     toggleDependsOnSelect($meetingRegistrationType, $meetingAvailableSlots, "on_this_platform");
     toggleDependsOnSelect($meetingRegistrationType, $meetingRegistrationTerms, "on_this_platform");
     toggleDependsOnSelect($meetingRegistrationType, $meetingRegistrationUrl, "on_different_platform");
+
+    createFieldDependentInputs({
+      controllerField: $("#meeting_iframe_embed_type"),
+      wrapperSelector: ".iframe-fields",
+      dependentFieldsSelector: ".iframe-fields--access-level",
+      dependentInputSelector: "input",
+      enablingCondition: ($field) => {
+        return $field.val() !== "none"
+      }
+    });
   }
 });

data/app/packs/src/decidim/meetings/meetings_polls.js

@@ -5,37 +5,42 @@ $(() => {
   // Mount polls component for users
   const $container = $("[data-decidim-meetings-poll]");
   const $counter = $("#visible-questions-count");
-  const poll = new MeetingsPollComponent($container, $container.data("decidim-meetings-poll"), $counter);
 
-  $(".meeting-polls__action-list").on("click", (event) => {
-    event.preventDefault();
+  if ($container.length) {
+    const poll = new MeetingsPollComponent($container, $container.data("decidim-meetings-poll"), $counter);
 
-    if (poll.isMounted()) {
-      $(event.target).removeClass(OPEN_CLASS);
-      $container.removeClass(OPEN_CLASS);
-      poll.unmountComponent();
-    } else {
-      $(event.target).addClass(OPEN_CLASS);
-      $container.addClass(OPEN_CLASS);
-      poll.mountComponent();
-    }
-  });
+    $(".meeting-polls__action-list").on("click", (event) => {
+      event.preventDefault();
+
+      if (poll.isMounted()) {
+        $(event.target).removeClass(OPEN_CLASS);
+        $container.removeClass(OPEN_CLASS);
+        poll.unmountComponent();
+      } else {
+        $(event.target).addClass(OPEN_CLASS);
+        $container.addClass(OPEN_CLASS);
+        poll.mountComponent();
+      }
+    });
+  }
 
   // Mount polls component for admins
   const $adminContainer = $("[data-decidim-admin-meetings-poll]");
-  const adminPoll = new MeetingsPollComponent($adminContainer, $adminContainer.data("decidim-admin-meetings-poll"));
 
-  $(".meeting-polls__action-administrate").on("click", (event) => {
-    event.preventDefault();
+  if ($adminContainer.length) {
+    const adminPoll = new MeetingsPollComponent($adminContainer, $adminContainer.data("decidim-admin-meetings-poll"));
+    $(".meeting-polls__action-administrate").on("click", (event) => {
+      event.preventDefault();
 
-    if (adminPoll.isMounted()) {
-      $(event.target).removeClass(OPEN_CLASS);
-      $adminContainer.removeClass(OPEN_CLASS);
-      adminPoll.unmountComponent();
-    } else {
-      $(event.target).addClass(OPEN_CLASS);
-      $adminContainer.addClass(OPEN_CLASS);
-      adminPoll.mountComponent();
-    }
-  });
+      if (adminPoll.isMounted()) {
+        $(event.target).removeClass(OPEN_CLASS);
+        $adminContainer.removeClass(OPEN_CLASS);
+        adminPoll.unmountComponent();
+      } else {
+        $(event.target).addClass(OPEN_CLASS);
+        $adminContainer.addClass(OPEN_CLASS);
+        adminPoll.mountComponent();
+      }
+    });
+  }
 });

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

@@ -33,6 +33,8 @@ module Decidim
             toggle_allow(can_create_meetings?)
           when :update
             toggle_allow(can_update_meeting?)
+          when :withdraw
+            toggle_allow(can_withdraw_meeting?)
           when :close
             toggle_allow(can_close_meeting?)
           when :register
@@ -79,10 +81,16 @@ module Decidim
           !meeting.closed?
       end
 
+      def can_withdraw_meeting?
+        component_settings&.creation_enabled_for_participants? &&
+          meeting.authored_by?(user) &&
+          !meeting.withdrawn? &&
+          !meeting.past?
+      end
+
       def can_close_meeting?
         component_settings&.creation_enabled_for_participants? &&
           meeting.authored_by?(user) &&
-          !meeting.closed? &&
           meeting.past?
       end