decidim-core 0.28.3 → 0.29.0.rc1

data/app/models/decidim/messaging/message.rb

@@ -34,7 +34,7 @@ module Decidim
 
       #
       # Associates receipts for this message for each of the given users,
-      # including also a receipt for the remitent (sender) of the message.
+      # including also a receipt for the remittant (sender) of the message.
       # Receipts are unread by default, except for the sender's receipt.
       #
       # If the sender is a UserGroup then receipts will be created for its managers
@@ -64,7 +64,7 @@ module Decidim
 
       private
 
-      # returns all posible recipients from a list of users or groups
+      # returns all possible recipients from a list of users or groups
       def all_recipients(recipients)
         users = recipients.flat_map do |recipient|
           recipient.is_a?(UserGroup) ? recipient.managers : recipient

data/app/models/decidim/newsletter.rb

@@ -46,7 +46,7 @@ module Decidim
       extended_data["send_to_participants"]
     end
 
-    def sended_to_partipatory_spaces
+    def sent_to_participatory_spaces
       extended_data["participatory_space_types"]
     end
 
@@ -56,16 +56,16 @@ module Decidim
                     .find_by(scoped_resource_id: id)
     end
 
-    def url(**kwargs)
-      proxy_url(:newsletter_url, id:, **kwargs)
+    def url(**)
+      proxy_url(:newsletter_url, id:, **)
     end
 
-    def notifications_settings_url(**kwargs)
-      proxy_url(__method__, **kwargs)
+    def notifications_settings_url(**)
+      proxy_url(__method__, **)
     end
 
-    def unsubscribe_newsletters_url(**kwargs)
-      proxy_url(__method__, **kwargs)
+    def unsubscribe_newsletters_url(**)
+      proxy_url(__method__, **)
     end
 
     def organization_official_url
@@ -82,10 +82,10 @@ module Decidim
       errors.add(:author, :invalid) unless author.organization == organization
     end
 
-    def proxy_url(method, **kwargs)
+    def proxy_url(method, **)
       return "#" unless sent?
 
-      router.public_send(method, host: organization.host, **kwargs)
+      router.public_send(method, host: organization.host, **)
     end
 
     def router

data/app/models/decidim/notification.rb

@@ -9,12 +9,12 @@ module Decidim
 
     # Daily notifications should contain all notifications within the previous
     # day from the given day.
-    scope :daily, ->(time: Time.now.utc) { where(created_at: (time - 1.day).all_day) }
+    scope :daily, ->(time = Time.now.utc) { where(created_at: (time - 1.day).all_day) }
 
     # Weekly notifications should contain all notifications within the previous
     # week counting from the end of the previous day until the start of the day
     # 1 week ago from the previous day.
-    scope :weekly, lambda { |time: Time.now.utc|
+    scope :weekly, lambda { |time = Time.now.utc|
       end_of_previous_day = (time - 1.day).end_of_day
       where(created_at: (end_of_previous_day - 7.days).beginning_of_day..end_of_previous_day)
     }

data/app/models/decidim/organization.rb

@@ -15,7 +15,7 @@ module Decidim
     SOCIAL_HANDLERS = [:twitter, :facebook, :instagram, :youtube, :github].freeze
     AVAILABLE_MACHINE_TRANSLATION_DISPLAY_PRIORITIES = %w(original translation).freeze
 
-    translatable_fields :description, :cta_button_text, :omnipresent_banner_title, :omnipresent_banner_short_description,
+    translatable_fields :name, :description, :cta_button_text, :omnipresent_banner_title, :omnipresent_banner_short_description,
                         :highlighted_content_banner_title, :highlighted_content_banner_short_description, :highlighted_content_banner_action_title,
                         :highlighted_content_banner_action_subtitle, :welcome_notification_subject, :welcome_notification_body, :id_documents_explanation_text,
                         :admin_terms_of_service_body
@@ -41,7 +41,7 @@ module Decidim
     #  disable: Users cannot register or sign in.
     enum users_registration_mode: [:enabled, :existing, :disabled], _prefix: true
 
-    validates :name, :host, uniqueness: true
+    validates :host, uniqueness: true
     validates :reference_prefix, presence: true
     validates :time_zone, presence: true, time_zone: true
     validates :default_locale, inclusion: { in: :available_locales }
@@ -60,6 +60,27 @@ module Decidim
 
     has_one_attached :open_data_file
 
+    validate :unique_name
+
+    def unique_name
+      base_query = new_record? ? Decidim::Organization.all : Decidim::Organization.where.not(id:).all
+
+      organization_names = []
+
+      base_query.pluck(:name).each do |value|
+        organization_names += value.except("machine_translations").values
+        organization_names += value.fetch("machine_translations", {}).values
+      end
+
+      organization_names = organization_names.map(&:downcase).compact_blank
+
+      name.each do |language, value|
+        next if value.is_a?(Hash)
+
+        errors.add("name_#{language}", :taken) if organization_names.include?(value.downcase)
+      end
+    end
+
     def self.log_presenter_class_for(_log)
       Decidim::AdminLog::OrganizationPresenter
     end

data/app/models/decidim/push_notification_message.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Decidim
-  # A messsage from a conversation that will be sent as a push notification
+  # A message from a conversation that will be sent as a push notification
   class PushNotificationMessage
     class InvalidActionError < StandardError; end
 

data/app/models/decidim/resource_link.rb

@@ -7,7 +7,7 @@ module Decidim
   # In order to group the relations between resources you need to specify a name.
   #
   # Some examples could be: proposals that have been created in a meeting or
-  # projects that are a result of merging diffrent proposals.
+  # projects that are a result of merging different proposals.
   class ResourceLink < ApplicationRecord
     belongs_to :from, polymorphic: true
     belongs_to :to, polymorphic: true

data/app/models/decidim/user.rb

@@ -278,18 +278,6 @@ module Decidim
       false
     end
 
-    def after_confirmation
-      return unless organization.send_welcome_notification?
-
-      Decidim::EventsManager.publish(
-        event: "decidim.events.core.welcome_notification",
-        event_class: WelcomeNotificationEvent,
-        resource: self,
-        affected_users: [self],
-        extra: { force_email: true }
-      )
-    end
-
     protected
 
     # Overrides devise email required validation.
@@ -308,11 +296,23 @@ module Decidim
       super
     end
 
+    def after_confirmation
+      return unless organization.send_welcome_notification?
+
+      Decidim::EventsManager.publish(
+        event: "decidim.events.core.welcome_notification",
+        event_class: WelcomeNotificationEvent,
+        resource: self,
+        affected_users: [self],
+        extra: { force_email: true }
+      )
+    end
+
     private
 
     # Changes default Devise behaviour to use ActiveJob to send async emails.
-    def send_devise_notification(notification, *args)
-      devise_mailer.send(notification, self, *args).deliver_later
+    def send_devise_notification(notification, *)
+      devise_mailer.send(notification, self, *).deliver_later
     end
 
     def all_roles_are_valid

data/app/models/decidim/user_group.rb

@@ -202,13 +202,13 @@ module Decidim
       document_number.present?
     end
 
-    # Overwites method in `Decidim::Validatable`, as user groups do not have a
+    # Overrides method in `Decidim::Validatable`, as user groups do not have a
     # password.
     def password_required?
       false
     end
 
-    # Overwites method in `Decidim::Validatable`, as user groups do not have a
+    # Overrides method in `Decidim::Validatable`, as user groups do not have a
     # password.
     def password
       nil

data/app/packs/src/decidim/a11y.js

@@ -190,10 +190,56 @@ const createDialog = (component) => {
   })
 }
 
+/**
+ * Announces a message to the screen reader dynamically.
+ *
+ * This should not be called consecutively multiple times because the screen
+ * reader may not read all the messages if the content is changed quickly.
+ *
+ * @param {String} message The message to be announced
+ * @param {String} mode The mode for the announcement, either "assertive"
+ *   (default) or "polite".
+ * @return {void}
+ */
+const announceForScreenReader = (message, mode = "assertive") => {
+  if (!message || typeof message !== "string" || message.length < 1) {
+    return;
+  }
+
+  let element = document.getElementById("screen-reader-announcement");
+  if (!element) {
+    element = document.createElement("div");
+    element.setAttribute("id", "screen-reader-announcement");
+    element.classList.add("sr-only");
+    element.setAttribute("aria-atomic", true);
+    document.body.append(element);
+  }
+  if (mode === "polite") {
+    element.setAttribute("aria-live", mode);
+  } else {
+    element.setAttribute("aria-live", "assertive");
+  }
+
+  element.innerHTML = "";
+
+  setTimeout(() => {
+    // Wrap the text in a span with a random attribute value that changes every
+    // time to try to indicate to the screen reader the content has changed. This
+    // helps reading the message aloud if the message is exactly the same as the
+    // last time.
+    const randomIdentifier = `announcement-${new Date().getUTCMilliseconds()}-${Math.floor(Math.random() * 10000000)}`;
+    const announce = document.createElement("span")
+    announce.setAttribute("data-random", randomIdentifier);
+    announce.textContent = message;
+    element.append(announce);
+  }, 100);
+};
+
 export {
   createAccordion,
   createDialog,
   createDropdown,
+  announceForScreenReader,
   Accordions,
   Dialogs,
   Dropdowns

data/app/packs/src/decidim/attachments/file_or_link_tabs.js

@@ -0,0 +1,55 @@
+/**
+ * This file controls the behavior of the |File| and |Link| tabs in the
+ * attachment form. It disables the |File| tab when a link is present and
+ * vice versa.
+ */
+
+const getFileButton = (container) =>
+  container.querySelector("button#trigger-file");
+const getLinkButton = (container) =>
+  container.querySelector("button#trigger-link");
+const getLinkInput = (container) =>
+  container.querySelector("input#attachment_link");
+const getUploadsContainer = (container) =>
+  container.querySelector("div[data-active-uploads]");
+
+const hasUploads = (container) => container.querySelectorAll("div").length > 0;
+
+const updateTabsState = (container) => {
+  const fileButton = getFileButton(container);
+  const linkButton = getLinkButton(container);
+  const linkInput = getLinkInput(container);
+  const uploadsContainer = getUploadsContainer(container);
+
+  const disableFileButton = Boolean(linkInput?.value);
+  const disableLinkButton = hasUploads(uploadsContainer);
+
+  fileButton.disabled = disableFileButton;
+  linkButton.disabled = disableLinkButton;
+};
+
+const initializeTabs = (container) => {
+  const linkInput = getLinkInput(container);
+  const uploadsContainer = getUploadsContainer(container);
+
+  linkInput.addEventListener("change", () => {
+    updateTabsState(container);
+  });
+
+  uploadsContainer.addEventListener("DOMSubtreeModified", () => {
+    updateTabsState(container);
+    console.log("DOMSubtreeModified");
+  });
+
+  updateTabsState(container);
+};
+
+document.addEventListener("DOMContentLoaded", () => {
+  const tabs = document.querySelectorAll(
+    "div[data-file-or-link-tabs-controller]"
+  );
+
+  tabs.forEach((container) => {
+    initializeTabs(container);
+  });
+});

data/app/packs/src/decidim/attachments/index.js

@@ -0,0 +1 @@
+import "src/decidim/attachments/file_or_link_tabs"

data/app/packs/src/decidim/back_to_list.js

@@ -2,7 +2,7 @@
  * Changes "Back to list" links to the one saved in sessionStorage API
  * To apply this to a link, at least one element must have the class "js-back-to-list".
  * For this to work it needs the filteredParams in SessionStorage, that is saved on FormFilterComponent.
- * @param {NodeList} links - Hyperlinks elements that point to the filters page that will use the fitererd params
+ * @param {NodeList} links - Hyperlinks elements that point to the filters page that will use the filtered params
  * @returns {void}
  */
 export default function backToListLink(links) {

data/app/packs/src/decidim/check_boxes_tree.js

@@ -22,7 +22,7 @@ export default class CheckBoxesTree {
   /**
    * Set checkboxes as checked if included in given values
    * @public
-   * @param {Array} checkboxes - array of checkboxs to check
+   * @param {Array} checkboxes - array of checkboxes to check
    * @param {Array} values - values of checkboxes that should be checked
    * @returns {Void} - Returns nothing.
    */

data/app/packs/src/decidim/data_consent/consent_manager.js

@@ -75,7 +75,7 @@ class ConsentManager {
 
   triggerWarnings() {
     document.querySelectorAll(".disabled-iframe").forEach((original) => {
-      if (original.querySelector(".dataconsent-warning")) {
+      if (original.querySelector("[data-dataconsent-warning]")) {
         return;
       }
 

data/app/packs/src/decidim/data_consent/consent_manager.test.js

@@ -75,7 +75,7 @@ describe("ConsentManager", () => {
             <div class="dc-switch">
               <input class="switch-input" checked="checked" id="dc-essential" type="checkbox" name="essential" disabled="">
               <label class="switch-paddle" for="dc-essential">
-                <span class="show-for-sr">Toggle Essential</span>
+                <span>Toggle Essential</span>
               </label>
             </div>
           </div>
@@ -119,7 +119,7 @@ describe("ConsentManager", () => {
             <div class="dc-switch">
               <input class="switch-input" id="dc-preferences" type="checkbox" name="preferences">
               <label class="switch-paddle" for="dc-preferences">
-                <span class="show-for-sr">Toggle Preferences</span>
+                <span>Toggle Preferences</span>
               </label>
             </div>
           </div>
@@ -139,7 +139,7 @@ describe("ConsentManager", () => {
             <div class="dc-switch">
               <input class="switch-input" id="dc-analytics" type="checkbox" name="analytics">
               <label class="switch-paddle" for="dc-analytics">
-                <span class="show-for-sr">Toggle Analytics and statistics</span>
+                <span>Toggle Analytics and statistics</span>
               </label>
             </div>
           </div>
@@ -159,7 +159,7 @@ describe("ConsentManager", () => {
             <div class="dc-switch">
               <input class="switch-input" id="dc-marketing" type="checkbox" name="marketing">
               <label class="switch-paddle" for="dc-marketing">
-                <span class="show-for-sr">Toggle Marketing</span>
+                <span>Toggle Marketing</span>
               </label>
             </div>
           </div>
@@ -182,7 +182,7 @@ describe("ConsentManager", () => {
     </div>
   `;
   const cookieWarningContent = `
-    <div class="dataconsent-warning padding-1 hide">
+    <div data-dataconsent-warning class="hide">
       <p>You need to enable all cookies in order to see this content.</p>
       <a href="#" class="button margin-vertical-2" data-open="dc-modal">
         Change cookie settings
@@ -226,7 +226,7 @@ describe("ConsentManager", () => {
       modal: modal,
       categories: categories,
       cookieName: "decidim-consent",
-      warningElement: document.querySelector(".dataconsent-warning")
+      warningElement: document.querySelector("[data-dataconsent-warning]")
     });
   });
 

data/app/packs/src/decidim/data_consent/index.js

@@ -78,7 +78,7 @@ document.addEventListener("DOMContentLoaded", () => {
     modal: modal,
     categories: categories,
     cookieName: window.Decidim.config.get("consent_cookie_name"),
-    warningElement: document.querySelector(".dataconsent-warning")
+    warningElement: document.querySelector("[data-dataconsent-warning]")
   });
 
   initDisabledIframes(manager);

data/app/packs/src/decidim/datepicker/datepicker_functions.js

@@ -0,0 +1,243 @@
+// Utility helper functions for the date and time picker functionality
+
+export const setHour = (value, format) => {
+  const hour = value.split(":")[0];
+  if (format === 12) {
+    if (Number(hour) > 12) {
+      return Number(hour) - 12;
+    };
+
+    if (Number(hour) === 0) {
+      return 12;
+    };
+
+    return Number(hour);
+  };
+
+  return Number(hour);
+};
+
+export const setMinute = (value) => {
+  const minute = value.split(":")[1];
+
+  return Number(minute);
+};
+
+export const formatInputDate = (date, formats) => {
+  const dateList = date.split("-");
+  const year = dateList[0];
+  const month = dateList[1];
+  const day = dateList[2];
+
+  if (formats.order === "d-m-y") {
+    return `${day}${formats.separator}${month}${formats.separator}${year}`;
+  } else if (formats.order === "y-m-d") {
+    return `${year}${formats.separator}${month}${formats.separator}${day}`;
+  };
+  return `${month}${formats.separator}${day}${formats.separator}${year}`;
+};
+
+export const formatInputTime = (time, format, input) => {
+  const timeList = time.split(":");
+  let hour = timeList[0];
+  const minute = timeList[1];
+
+  if (format === 12) {
+    if (Number(hour) === 12) {
+      document.getElementById(`period_pm_${input.id}`).checked = true;
+    } else if (Number(hour) > 12 && Number(hour) < 22) {
+      hour = `0${Number(hour) - 12}`;
+      document.getElementById(`period_pm_${input.id}`).checked = true;
+    } else if (Number(hour) >= 22) {
+      hour = `${Number(hour) - 12}`;
+      document.getElementById(`period_pm_${input.id}`).checked = true;
+    } else if (Number(hour) === 0) {
+      hour = "12";
+    }
+
+    return `${hour}:${minute}`;
+  };
+
+  return time;
+};
+
+export const changeHourDisplay = (change, hour, format) => {
+  let value = null;
+
+  if (change === "increase") {
+    if (format === 24) {
+      if (hour === 23) {
+        value = 0;
+      } else {
+        value = hour + 1;
+      };
+    } else if (format === 12) {
+      if (hour === 12) {
+        value = 1;
+      } else {
+        value = hour + 1;
+      }
+    }
+  } else if (change === "decrease") {
+    if (format === 24) {
+      if (hour === 0) {
+        value = 23;
+      } else {
+        value = hour - 1;
+      };
+    } else if (format === 12) {
+      if (hour === 1) {
+        value = 12;
+      } else {
+        value = hour - 1;
+      }
+    }
+  };
+
+  return value;
+};
+
+export const changeMinuteDisplay = (change, minute) => {
+  let value = null;
+
+  if (change === "increase") {
+    if (minute === 59) {
+      value = 0;
+    } else {
+      value = minute + 1;
+    };
+  } else if (change === "decrease") {
+    if (minute === 0) {
+      value = 59;
+    } else {
+      value = minute - 1;
+    };
+  };
+
+  return value;
+};
+
+let displayMinute = null;
+let displayHour = null;
+const preFix = 0;
+
+export const hourDisplay = (hour) => {
+  if (hour < 10) {
+    displayHour = `${preFix}${hour}`;
+  } else {
+    displayHour = hour;
+  };
+
+  return displayHour;
+};
+
+export const minuteDisplay = (minute) => {
+  if (minute < 10) {
+    displayMinute = `${preFix}${minute}`;
+  } else {
+    displayMinute = minute;
+  };
+
+  return displayMinute;
+};
+
+export const updateTimeValue = (time, hour, minute) => {
+  time.value = `${hourDisplay(hour)}:${minuteDisplay(minute)}`;
+};
+
+export const formatDate = (value, formats) => {
+  let newValue = value;
+  const splitValue = value.split(formats.separator);
+
+  if (formats.order === "d-m-y") {
+    newValue = `${splitValue[1]}/${splitValue[0]}/${splitValue[2]}`;
+  } else if (formats.order === "y-m-d") {
+    newValue = `${splitValue[1]}/${splitValue[2]}/${splitValue[0]}`
+  };
+
+  const date = new Date(newValue)
+
+  let day = date.getDate();
+  let month = date.getMonth() + 1;
+  let year = date.getFullYear();
+
+  if (day < 10) {
+    day = `0${day}`
+  };
+
+  if (month < 10) {
+    month = `0${month}`
+  };
+
+  return `${year}-${month}-${day}`
+};
+
+export const formatTime = (value, format, inputname) => {
+  if (format === 12) {
+    const splitValue = value.split(":");
+    let hour = splitValue[0];
+    const minute = splitValue[1];
+    if (document.getElementById(`period_am_${inputname}`).checked) {
+      switch (hour) {
+      case "12":
+        hour = "00";
+
+        return `${hour}:${minute}`;
+      default:
+        return value;
+      };
+    } else if (document.getElementById(`period_pm_${inputname}`).checked) {
+      if (Number(hour) > 0 && Number(hour) < 12) {
+        hour = `${Number(hour) + 12}`;
+      };
+
+      return `${hour}:${minute}`
+    };
+  };
+
+  return value;
+};
+
+export const updateInputValue = (input, formats, time) => {
+  input.value = `${formatDate(document.querySelector(`#${input.id}_date`).value, formats)}T${formatTime(time.value, formats.time, input.id)}`;
+};
+
+export const dateToPicker = (value, formats) => {
+  let formatArray = value.split(formats.separator);
+  let formatValue = value;
+
+  if (formats.order === "d-m-y") {
+    formatValue = `${formatArray[1]}/${formatArray[0]}/${formatArray[2]}`;
+  } else if (formats.order === "y-m-d") {
+    formatValue = `${formatArray[1]}/${formatArray[2]}/${formatArray[0]}`;
+  };
+
+
+  return formatValue;
+};
+
+export const displayDate = (value, formats) => {
+  let day = value.getDate();
+  let month = value.getMonth() + 1;
+  const year = value.getFullYear();
+
+  if (day < 10) {
+    day = `0${day}`
+  }
+  if (month < 10) {
+    month = `0${month}`
+  }
+
+  if (formats.order === "d-m-y") {
+    return `${day}${formats.separator}${month}${formats.separator}${year}`
+  } else if (formats.order === "y-m-d") {
+    return `${year}${formats.separator}${month}${formats.separator}${day}`
+  };
+
+  return `${month}${formats.separator}${day}${formats.separator}${year}`;
+};
+
+export const calculateDatepickerPos = (datePicker) => {
+  return (document.body.clientHeight - (datePicker.getBoundingClientRect().top + window.scrollY)) -
+  (document.querySelector(".item__edit-sticky").clientHeight + datePicker.clientHeight)
+};