decidim-core 0.26.7 → 0.26.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2d573b856227ec581d40d90cff9a85c1f21502a1ed6278b82aa9768a434f0e9c
-  data.tar.gz: 2ca2fff70b6e6b9314450c35ab22965b3ba60af303a2839c7716e502cb566b3a
+  metadata.gz: b439f283c43183815f29f97ff9bf9098c8378b8a5cb16191390bd874e83cbb05
+  data.tar.gz: 117c9e1d3d6f9ac3dc55c5984e717c384e5274dc198babddc8ea7a1efe29e863
 SHA512:
-  metadata.gz: 1000d09afe7e9af0586fa14c917c77a150bf9f55adee9d0a243744799a9cf9c84aa364b0f014cdec6eda3855c10b4b15772732c979d9f15e381c8a62e78aad8f
-  data.tar.gz: 1a4feafaa713a4fa540aab815e0306f125cc74d45599d5defbec2e9dd2cce8bc6223acd0148424b3973a0f1d6853b96cd3ee9a6fda7ff45cc91073fd2b84ac8a
+  metadata.gz: 299d8bcecdf6084d93845ebcb256105c79077b2b6895f177dc22ddf296cf09973f92f5c425db5a078cb5e5eb9070f4e6ea6b3ad8e4c792236a6d567de7dc155d
+  data.tar.gz: 906b497e36d89c374d4f80114b5a6c4633d3a6da254832d26c8a9d22fa41c7dfc23fb5ce93ffcd1170e3e2f4caec1859c0e44c661a7dcc3567e31b75c969832c

data/app/cells/decidim/collapsible_list/show.erb

@@ -16,7 +16,7 @@
     </span>
   </div>
 <% else %>
-  <div class="collapsible-list <%= list_class %>">
+  <div class="<%= list_class %>">
     <% list.each do |element| %>
       <% if cell_name %>
         <%= cell cell_name, element, cell_options %>

data/app/cells/decidim/version_cell.rb

@@ -77,7 +77,7 @@ module Decidim
     end
 
     def resource_path
-      resource_locator(versioned_resource).path
+      Decidim::ResourceLocatorPresenter.new(versioned_resource).path
     end
   end
 end

data/app/cells/decidim/versions_list_cell.rb

@@ -13,7 +13,7 @@ module Decidim
     end
 
     def resource_path
-      resource_locator(versioned_resource).path
+      Decidim::ResourceLocatorPresenter.new(versioned_resource).path
     end
 
     def i18n_changes_title

data/app/commands/decidim/endorse_resource.rb

@@ -31,6 +31,8 @@ module Decidim
       else
         broadcast(:invalid)
       end
+    rescue ActiveRecord::RecordNotUnique
+      broadcast(:invalid)
     end
 
     private

data/app/commands/decidim/unendorse_resource.rb

@@ -31,7 +31,7 @@ module Decidim
       query = if @current_group.present?
                 @resource.endorsements.where(decidim_user_group_id: @current_group&.id)
               else
-                @resource.endorsements.where(author: @current_user, decidim_user_group_id: nil)
+                @resource.endorsements.where(author: @current_user, decidim_user_group_id: 0)
               end
       query.destroy_all
     end

data/app/controllers/decidim/authorization_modals_controller.rb

@@ -17,7 +17,7 @@ module Decidim
     end
 
     def current_component
-      @current_component ||= Decidim::Component.find(params[:component_id])
+      @current_component ||= Decidim::Component.where(participatory_space: current_organization.participatory_spaces).find(params[:component_id])
     end
 
     def authorization_action

data/app/controllers/decidim/devise/registrations_controller.rb

@@ -37,7 +37,7 @@ module Decidim
           end
 
           on(:invalid) do
-            flash.now[:alert] = @form.errors.full_messages.join(", ") if @form.errors.full_messages.any?
+            flash.now[:alert] = t("error", scope: "decidim.devise.registrations.create")
             render :new
           end
         end

data/app/controllers/decidim/links_controller.rb

@@ -35,7 +35,7 @@ module Decidim
     end
 
     def external_url
-      @external_url ||= URI.parse(params[:external_url])
+      @external_url ||= URI.parse(URI::Parser.new.escape(params[:external_url]))
     end
   end
 end

data/app/controllers/decidim/user_timeline_controller.rb

@@ -12,7 +12,7 @@ module Decidim
     helper_method :activities, :resource_types, :user
 
     def index
-      raise ActionController::RoutingError, "Not Found" if current_user != user
+      raise ActionController::RoutingError, "Not Found" unless user && current_user == user
     end
 
     private

data/app/forms/decidim/account_form.rb

@@ -24,7 +24,7 @@ module Decidim
     validates :nickname, presence: true, format: { with: Decidim::User::REGEXP_NICKNAME }
 
     validates :nickname, length: { maximum: Decidim::User.nickname_max_length, allow_blank: true }
-    validates :password, confirmation: true
+    validates :password, confirmation: { message: I18n.t("errors.messages.password_confirmation_message") }
     validates :password, password: { name: :name, email: :email, username: :nickname }, if: -> { password.present? }
     validates :password_confirmation, presence: true, if: :password_present
     validates :avatar, passthru: { to: Decidim::User }

data/app/forms/decidim/notifications_settings_form.rb

@@ -41,13 +41,5 @@ module Decidim
     def direct_message_types
       allow_public_contact ? "all" : "followed-only"
     end
-
-    def user_is_moderator?(user)
-      Decidim.participatory_space_manifests.map do |manifest|
-        participatory_space_type = manifest.model_class_name.constantize
-        return true if participatory_space_type.moderators(user.organization).exists?(id: user.id)
-      end
-      false
-    end
   end
 end

data/app/forms/decidim/registration_form.rb

@@ -17,7 +17,7 @@ module Decidim
     validates :name, presence: true, format: { with: Decidim::User::REGEXP_NAME }
     validates :nickname, presence: true, format: { with: Decidim::User::REGEXP_NICKNAME }, length: { maximum: Decidim::User.nickname_max_length }
     validates :email, presence: true, "valid_email_2/email": { disposable: true }
-    validates :password, confirmation: true
+    validates :password, confirmation: { message: I18n.t("errors.messages.password_confirmation_message") }
     validates :password, password: { name: :name, email: :email, username: :nickname }
     validates :password_confirmation, presence: true
     validates :tos_agreement, allow_nil: false, acceptance: true

data/app/forms/url_validator.rb

@@ -6,7 +6,7 @@
 #
 class UrlValidator < ActiveModel::EachValidator
   def validate_each(record, attribute, value)
-    record.errors[attribute] << (options[:message] || "must be a valid URL") unless url_valid?(value)
+    record.errors.add attribute, :url_format, **options unless url_valid?(value)
   end
 
   # a URL may be technically well-formed but may

data/app/helpers/decidim/layout_helper.rb

@@ -74,8 +74,11 @@ module Decidim
       classes = _icon_classes(options) + ["external-icon"]
 
       if path.split(".").last == "svg"
+        icon_path = application_path(path)
+        return unless icon_path
+
         attributes = { class: classes.join(" ") }.merge(options)
-        asset = File.read(application_path(path))
+        asset = File.read(icon_path)
         asset.gsub("<svg ", "<svg#{tag_builder.tag_options(attributes)} ").html_safe
       else
         image_pack_tag(path, class: classes.join(" "), style: "display: none")
@@ -83,9 +86,17 @@ module Decidim
     end
 
     def application_path(path)
-      img_path = asset_pack_path(path)
-      img_path = URI(img_path).path if Decidim.cors_enabled
-      Rails.root.join("public/#{img_path}")
+      # Force the path to be returned without the protocol and host even when a
+      # custom asset host has been defined. The host parameter needs to be a
+      # non-nil because otherwise it will be set to the asset host at
+      # ActionView::Helpers::AssetUrlHelper#compute_asset_host.
+      img_path = asset_pack_path(path, host: "", protocol: :relative)
+      path = Rails.public_path.join(img_path.sub(%r{^/}, ""))
+      return unless File.exist?(path)
+
+      path
+    rescue ::Webpacker::Manifest::MissingEntryError
+      nil
     end
 
     # Allows to create role attribute according to accessibility rules

data/app/helpers/decidim/resource_helper.rb

@@ -67,6 +67,8 @@ module Decidim
 
     # Returns an instance of ResourceLocatorPresenter with the given resource
     def resource_locator(resource)
+      return resource.resource_locator if resource.respond_to?(:resource_locator)
+
       ::Decidim::ResourceLocatorPresenter.new(resource)
     end
 

data/app/jobs/decidim/data_portability_export_job.rb

@@ -13,6 +13,9 @@ module Decidim
       save_or_upload_file(user, path)
 
       ExportMailer.data_portability_export(user, filename, password).deliver_later
+
+      # Deletes temporary file
+      File.delete(path)
     end
 
     private

data/app/jobs/decidim/open_data_job.rb

@@ -11,6 +11,8 @@ module Decidim
       raise "Couldn't generate Open Data export" unless exporter.export.positive?
 
       organization.open_data_file.attach(io: File.open(path, "rb"), filename: organization.open_data_file_path)
+      # Deletes the temporary file file
+      File.delete(path)
     end
   end
 end

data/app/models/decidim/organization.rb

@@ -87,6 +87,12 @@ module Decidim
       @top_scopes ||= scopes.top_level
     end
 
+    def participatory_spaces
+      @participatory_spaces ||= Decidim.participatory_space_manifests.flat_map do |manifest|
+        manifest.participatory_spaces.call(self)
+      end
+    end
+
     def public_participatory_spaces
       @public_participatory_spaces ||= Decidim.participatory_space_manifests.flat_map do |manifest|
         manifest.participatory_spaces.call(self).public_spaces

data/app/models/decidim/user.rb

@@ -202,7 +202,7 @@ module Decidim
     end
 
     def admin_terms_accepted?
-      return true if admin_terms_accepted_at
+      admin_terms_accepted_at.present?
     end
 
     # Whether this user can be verified against some authorization or not.
@@ -253,6 +253,14 @@ module Decidim
       Arel.sql(%{("decidim_users"."last_sign_in_at")::text})
     end
 
+    def moderator?
+      Decidim.participatory_space_manifests.map do |manifest|
+        participatory_space_type = manifest.model_class_name.constantize
+        return true if participatory_space_type.moderators(organization).exists?(id: id)
+      end
+      false
+    end
+
     protected
 
     # Overrides devise email required validation.

data/app/packs/src/decidim/geocoding/attach_input.js

@@ -125,7 +125,10 @@ export default function attachGeocoding($input, options, callback) {
   $input.on("geocoder-suggest-coordinates.decidim", (_ev, coordinates) => {
     setCoordinates(coordinates);
     geocoded = true;
-    callback(coordinates)
+    if (typeof callback === "function") {
+      callback(coordinates);
+      return;
+    }
   });
 
   // Set the initial values if the field defines the coordinates

data/app/packs/src/decidim/geocoding/provider/here.js

@@ -34,26 +34,28 @@ $(() => {
       if (`${query}`.trim().length < queryMinLength) {
         return;
       }
-
+      // Changes to the autocomplete api call based on:
+      // https://developer.here.com/documentation/geocoding-search-api/migration_guide/migration-geocoder/topics-api/autocomplete.html
       currentSuggestionQuery = setTimeout(() => {
         $.ajax({
           method: "GET",
-          url: "https://autocomplete.geocoder.ls.hereapi.com/6.2/suggest.json",
+          url: "https://autocomplete.search.hereapi.com/v1/autocomplete",
           data: {
             apiKey: config.apiKey,
-            query: query,
-            language: language
+            // eslint-disable-next-line
+            q: query,
+            lang: language
           },
           dataType: "json"
         }).done((resp) => {
-          if (resp.suggestions) {
-            return callback(resp.suggestions.map((item) => {
+          if (resp.items) {
+            return callback(resp.items.map((item) => {
               const label = generateAddressLabel(item.address, addressFormat);
 
               return {
                 key: label,
                 value: label,
-                locationId: item.locationId
+                locationId: item.id
               }
             }));
           }
@@ -65,30 +67,24 @@ $(() => {
     $input.on("geocoder-suggest-select.decidim", (_ev, selectedItem) => {
       $.ajax({
         method: "GET",
-        url: "https://geocoder.ls.hereapi.com/6.2/geocode.json",
+        url: "https://lookup.search.hereapi.com/v1/lookup",
         data: {
           apiKey: config.apiKey,
-          gen: 9,
-          jsonattributes: 1,
-          locationid: selectedItem.locationId
+          id: selectedItem.locationId
         },
         dataType: "json"
       }).done((resp) => {
-        if (!resp.response || !Array.isArray(resp.response.view) ||
-          resp.response.view.length < 1
+        if (!resp || Object.keys(resp).length < 1
         ) {
           return;
         }
-
-        const view = resp.response.view[0];
-        if (!Array.isArray(view.result) || view.result.length < 1) {
-          return;
+        const position = resp.position;
+        if (!position || !position.lat || !position.lng) {
+          return
         }
-
-        const result = view.result[0];
         const coordinates = [
-          result.location.displayPosition.latitude,
-          result.location.displayPosition.longitude
+          position.lat,
+          position.lng
         ];
 
         $input.trigger(

data/app/packs/src/decidim/geocoding/provider/photon.js

@@ -59,7 +59,7 @@ $(() => {
               return {
                 key: label,
                 value: label,
-                coordinates: item.geometry.coordinates
+                coordinates: item.geometry.coordinates.reverse()
               }
             }));
           }

data/app/packs/src/decidim/map/controller/drag_marker.js

@@ -1,6 +1,4 @@
-import * as L from "leaflet";
 import MapController from "src/decidim/map/controller"
-import "src/decidim/vendor/leaflet-tilelayer-here"
 
 export default class MapDragMarkerController extends MapController {
   start() {

data/app/packs/src/decidim/map/controller/markers.js

@@ -1,5 +1,4 @@
 import "src/decidim/vendor/jquery-tmpl"
-import * as L from "leaflet";
 import MapController from "src/decidim/map/controller"
 import "leaflet.markercluster";
 

data/app/packs/src/decidim/map/controller/static.js

@@ -1,4 +1,3 @@
-import * as L from "leaflet";
 import MapController from "src/decidim/map/controller"
 
 const openLink = window.open;

data/app/packs/src/decidim/map/controller.js

@@ -1,5 +1,3 @@
-import * as L from "leaflet";
-import "src/decidim/map/icon"
 import MapControllerRegistry from "src/decidim/map/controller_registry"
 
 export default class MapController {

data/app/packs/src/decidim/map/factory.js

@@ -1,3 +1,5 @@
+import "src/decidim/map/icon"
+
 import MapMarkersController from "src/decidim/map/controller/markers"
 import MapStaticController from "src/decidim/map/controller/static"
 import MapDragMarkerController from "src/decidim/map/controller/drag_marker"
@@ -22,7 +24,8 @@ import MapDragMarkerController from "src/decidim/map/controller/drag_marker"
  *   window.Decidim.createMapController = (mapId, config) => {
  *     if (config.type === "custom") {
  *       // Obviously you need to implement CustomMapController for this to
- *       // work.
+ *       // work. You can find an example at:
+ *       // decidim-dev/app/packs/src/decidim/dev/test/custom_map_factory.js
  *       return new window.Decidim.CustomMapController(mapId, config);
  *     }
  *

data/app/packs/src/decidim/map/icon.js

@@ -1,4 +1,3 @@
-import * as L from "leaflet";
 import { SVGIcon } from "leaflet-svgicon"
 
 L.DivIcon.SVGIcon = SVGIcon;

data/app/packs/src/decidim/map/legacy.js

@@ -1,6 +1,5 @@
 /* eslint-disable require-jsdoc */
 
-import * as L from "leaflet";
 import "src/decidim/map/factory"
 
 /**

data/app/packs/src/decidim/map/provider/default.js

@@ -1,3 +1,5 @@
+import "leaflet"
+
 /**
  * NOTE:
  * This has to load before decidim/map in order for it to apply correctly when

data/app/packs/src/decidim/map/provider/here.js

@@ -1,4 +1,5 @@
-import * as L from "leaflet"
+import "leaflet"
+import "src/decidim/vendor/leaflet-tilelayer-here"
 
 /**
  * NOTE:

data/app/queries/decidim/metrics/users_metric_manage.rb

@@ -11,15 +11,15 @@ module Decidim
       private
 
       def query
-        return @query if @query
-
-        @query = Decidim::User.where(organization: @organization)
-        @query = @query.where("created_at <= ?", end_time)
-        @query
+        @query ||= Decidim::User.where(organization: @organization)
+                                .where("deleted_at IS NULL OR deleted_at > ?", end_time)
+                                .where("blocked_at IS NULL OR blocked_at > ?", end_time)
+                                .confirmed
+                                .where("created_at <= ?", end_time)
       end
 
       def quantity
-        @quantity ||= @query.where("created_at >= ?", start_time).count
+        @quantity ||= query.where("created_at >= ?", start_time).count
       end
     end
   end

data/app/views/decidim/notifications_settings/show.html.erb

@@ -45,7 +45,7 @@
     </label>
   </div>
 
-  <% if @notifications_settings.user_is_moderator?(current_user) %>
+  <% if current_user.moderator? %>
     <p><strong><%= t(".administrators") %></strong></p>
     <div class="switch tiny switch-with-label email_on_moderations">
       <label>

data/config/environment.rb

@@ -0,0 +1,3 @@
+# frozen_string_literal: true
+
+# Empty line for playing nice with tpope/vim-rails

data/config/locales/ar.yml

@@ -391,7 +391,7 @@ ar:
           request_confirmation_instructions: طلب تعليمات التأكيد
           title: تأكيد البريد الإلكتروني
       show:
-        close_modal: إغلاق مشروط
+        close_modal: اغلاق النموذج
     block_user_mailer:
       notify:
         body_1: تم حظر الحساب الخاص بك.
@@ -514,9 +514,8 @@ ar:
           already_have_an_account?: هل لديك حساب؟
           newsletter: تلقي رسالة إخبارية من حين لآخر مع المعلومات ذات الصلة
           newsletter_title: الاتصال إذن
-          nickname_help: الاسم المستعار الخاص بك في %{organization}. يمكن أن يحتوي فقط على أحرف وأرقام، و '-' و '_'.
+          nickname_help: الاسم المستعار الخاص بك في %{organization}. يمكن أن يحتوي فقط على أحرف وأرقام '-' و '_'.
           password_help: "%{minimun_characters} أحرف كحد أدنى، يجب أن لا تكون شائعة جدا (مثل 123456) ويجب أن تكون مختلفة عن اسمك المستعار وعن بريدك الإلكتروني."
-          sign_in: تسجيل الدخول
           sign_up: انشئ حسابًا
           sign_up_as:
             legend: الاشتراك باسم
@@ -537,7 +536,7 @@ ar:
         newsletter_modal:
           buttons:
             check: تحقق واستمر
-            close_modal: إغلاق مشروط
+            close_modal: اغلاق النموذج
             uncheck: إبقاءه غير مختار
           notice: |-
             <p>مهلا ، هل أنت متأكد أنك لا تريد تلقي رسالة إخبارية؟<br>
@@ -574,7 +573,7 @@ ar:
         success: تم تحميل الصورة بنجاح
       drag_and_drop_help: يمكن إضافة صور عن طريق سحبها وإسقاطها أو لصقها.
     endorsable:
-      endorsements: موافقات
+      endorsements: الدعم والتأييد
       endorsements_count: عد أصوات التأييدات
     endorsements:
       identities:
@@ -921,7 +920,7 @@ ar:
         body_1: أنت على وشك زيارة رابط خارجي ونرجو منك أن تتحلى بالحذر فيما يتعلق بالمحتوى الموجود في الموقع الخارجي.
         body_2: الرجاء التحقق من الرابط الذي توشك على زيارته والتأكد من التعرف عليه كموقع آمن قبل المتابعة.
         cancel: إلغاء
-        close_modal: إغلاق مشروط
+        close_modal: اغلاق النموذج
         proceed: المواصلة
         title: فتح رابط خارجي
     log:
@@ -1011,7 +1010,7 @@ ar:
           ok: حسناً
         index:
           ago: منذ
-          close: إغلاق مشروط
+          close: اغلاق النموذج
           from: من
           groups: فِرَقي
           last_message: آخر رسالة
@@ -1143,7 +1142,6 @@ ar:
           proposals: اقتراحات
           proposals_explanation: تقديم مقترحات ، ودعم المقترحات الحالية وتعزيز التغييرات التي تريد رؤيتها.
         footer_sub_hero:
-          footer_sub_hero_body: دعونا نبني مجتمعًا أكثر انفتاحًا وشفافية وتعاونًا.<br /> انضموا وشاركوا وقرّروا.
           footer_sub_hero_headline: مرحبًا بكم على المنصة التشاركية لـ %{organization}.
           register: تسجيل
         hero:
@@ -1174,7 +1172,7 @@ ar:
         refuse:
           modal_body: إذا رفضت ، فلن تتمكن من استخدام النظام الأساسي ، يمكنك <a href="%{data_portability_path}">تنزيل بياناتك</a> و / أو <a href="%{delete_path}">حذف حسابك</a>.
           modal_btn_exit: سأراجعها لاحقًا
-          modal_close: إغلاق مشروط
+          modal_close: اغلاق النموذج
           modal_title: هل ترفض حقًا الأحكام والشروط المحدثة؟
         required_review:
           alert: لقد قمنا بتحديث شروط الخدمة الخاصة بنا ، يرجى مراجعتها.
@@ -1268,7 +1266,7 @@ ar:
           future: مستقبل
           past: الماضي
       filters_small_view:
-        close_modal: إغلاق مشروط
+        close_modal: اغلاق النموذج
         filter: منقي
         filter_by: مصنف بواسطة
         unfold: كشف
@@ -1376,7 +1374,7 @@ ar:
       index:
         add_max_users: 'يمكن إضافة مستخدمين إلى المحادثة: 9 مستخدمين كحد أقصى'
         add_users_placeholder: البحث…
-        close: غلق
+        close: قريب
         conversations_info: المحادثات خاصة. لا يمكن لأحد آخر أن يراها. يمكن بدء محادثة مع أي مشارك أو مجموعة مسجلة على المنصة.
         from: من
         last_message: آخر رسالة
@@ -1397,7 +1395,7 @@ ar:
         back: عرض كافة المحادثات
         deleted_accounts: لا يمكن أن يكون لديك محادثة مع حسابات محذوفة.
         not_allowed: هذا المستخدم لم يعد يستقبل المزيد من الرسائل المباشرة.
-        title: محادثة مع %{usernames}
+        title: المحادثة مع %{usernames}
       update:
         error: لم يتم إرسال الرسالة. حاول مرة أخرى في وقت لاحق
     user_interests:
@@ -1564,8 +1562,6 @@ ar:
       updated: تم تحديث حسابك بنجاح.
     sessions:
       already_signed_out: تم الخروج بنجاح
-      new:
-        sign_in: تسجيل الدخول
       signed_in: تم تسجيل الدخول بنجاح
       signed_out: تم الخروج بنجاح
     shared:
@@ -1574,7 +1570,6 @@ ar:
         didn_t_receive_confirmation_instructions: لم تتلق تعليمات التأكيد؟
         didn_t_receive_unlock_instructions: لم تتلق تعليمات فتح؟
         forgot_your_password: نسيت رقمك السري؟
-        sign_in: تسجيل الدخول
         sign_in_with_provider: تسجيل الدخول مع %{provider}
         sign_up: انشئ حسابًا
       minimum_password_length: