decidim-participatory_processes 0.31.1 → 0.31.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9f77c394790a10232a3e5f8bc7f033a711933f6e316a35313f190d9de3435d28
-  data.tar.gz: 69891e0aca851db40a8bf558a2327d5567a89ca984594d237698c9e700ed1956
+  metadata.gz: 3a7aad4fa28b53aa35cfe06f58dfd59052a1e6a9474d7c39c3a1a658fe7c1483
+  data.tar.gz: 2180d0923275741be1c04255971c82d33f0e5e387dd1bfe75b9f0fd82a9d899f
 SHA512:
-  metadata.gz: 49f5edb0088e0b504ee21a8ff96801340b3d98c8d8bfb7ea695547848dcebbef408a7c5dea688b2a3f82407459bf2d4363318a541aac160f33453d88a37d1425
-  data.tar.gz: f7d23748d5e79ddd81368fd2f54b2b26a14f249fa02e687c474b04c3700df45c483282c890fab1d180cfda15d58d91f160f7ff1262771a014af3dc4473150b4b
+  metadata.gz: 7cb3f501e500106bdbf4d7ae41f95829a312c763cae7f16930f9acb1a6142ac4128b4ce9bbcb41016e94d9d9a64428dda74448a2766cda1c0c9a0b65cac4c3bc
+  data.tar.gz: 7a7b206aaf34f6083530414c20eb3fc8cbe43bf06b6c18b8cb40ccc0ca6f7e49b78cf20fae20be08425d4f0c7cafec73b89bdaf73f52970533e1d5954ba5cc05

data/app/commands/decidim/participatory_processes/admin/import_participatory_process.rb

@@ -29,7 +29,7 @@ module Decidim
             add_admins_as_followers(@imported_process)
           end
 
-          broadcast(:ok, @imported_process)
+          broadcast(:ok, @imported_process, @warnings)
         end
 
         private
@@ -37,13 +37,15 @@ module Decidim
         attr_reader :form
 
         def import_participatory_process
-          importer = Decidim::ParticipatoryProcesses::ParticipatoryProcessImporter.new(form.current_organization, current_user)
+          @warnings = []
           participatory_processes.each do |original_process|
+            importer = Decidim::ParticipatoryProcesses::ParticipatoryProcessImporter.new(form.current_organization, current_user)
             Decidim.traceability.perform_action!("import", Decidim::ParticipatoryProcess, current_user) do
               @imported_process = importer.import(original_process, current_user, title: form.title, slug: form.slug)
               importer.import_participatory_process_steps(original_process["participatory_process_steps"]) if form.import_steps?
               importer.import_folders_and_attachments(original_process["attachments"]) if form.import_attachments?
               importer.import_components(original_process["components"]) if form.import_components?
+              @warnings.concat(importer.warnings)
               @imported_process
             end
           end

data/app/controllers/decidim/participatory_processes/admin/participatory_process_imports_controller.rb

@@ -18,8 +18,9 @@ module Decidim
           @form = form(ParticipatoryProcessImportForm).from_params(params)
 
           ImportParticipatoryProcess.call(@form) do
-            on(:ok) do
+            on(:ok) do |_process, warnings|
               flash[:notice] = I18n.t("participatory_process_imports.create.success", scope: "decidim.admin")
+              flash[:warning] = warnings.join("<br>") if warnings.any?
               redirect_to participatory_processes_path
             end
 

data/app/forms/decidim/participatory_processes/admin/participatory_process_import_form.rb

@@ -30,12 +30,14 @@ module Decidim
         attribute :import_components, Boolean, default: true
         attribute :document, Decidim::Attributes::Blob
 
+        validates :document, presence: true
         validates :document, file_content_type: { allow: ACCEPTED_TYPES.values }
         validates :slug, presence: true, format: { with: Decidim::ParticipatoryProcess.slug_format }
         validates :title, translatable_presence: true
         validate :slug_uniqueness
 
         validate :document_type_must_be_valid, if: :document
+        validate :document_must_have_content, if: -> { document.present? && errors[:document].none? }
 
         def document_text
           @document_text ||= document&.download
@@ -75,6 +77,15 @@ module Decidim
 
           errors.add(:slug, :taken)
         end
+
+        def document_must_have_content
+          return if document_text.blank?
+
+          parsed = JSON.parse(document_text)
+          errors.add(:document, :empty) if parsed.is_a?(Array) && parsed.empty?
+        rescue JSON::ParserError
+          errors.add(:document, :invalid_json)
+        end
       end
     end
   end

data/app/models/decidim/participatory_process_step.rb

@@ -9,9 +9,11 @@ module Decidim
     include Traceable
     include Loggable
 
+    delegate :can_participate?, to: :participatory_process
+
     translatable_fields :title, :description, :cta_text
 
-    belongs_to :participatory_process, foreign_key: "decidim_participatory_process_id", class_name: "Decidim::ParticipatoryProcess"
+    belongs_to :participatory_process, foreign_key: "decidim_participatory_process_id", class_name: "Decidim::ParticipatoryProcess", touch: true
     has_one :organization, through: :participatory_process
 
     alias participatory_space participatory_process

data/app/serializers/decidim/participatory_processes/participatory_process_importer.rb

@@ -4,9 +4,12 @@ module Decidim
   module ParticipatoryProcesses
     # A factory class to ensure we always create ParticipatoryProcesses the same way since it involves some logic.
     class ParticipatoryProcessImporter < Decidim::Importers::Importer
+      attr_reader :warnings
+
       def initialize(organization, user)
         @organization = organization
         @user = user
+        @warnings = []
       end
 
       # Public: Creates a new ParticipatoryProcess.
@@ -43,7 +46,7 @@ module Decidim
             private_space: attributes["private_space"],
             participatory_process_group: process_group
           )
-          @imported_process.attached_uploader(:hero_image).remote_url = attributes["remote_hero_image_url"] if attributes["remote_hero_image_url"].present?
+          import_hero_image(attributes["remote_hero_image_url"])
 
           @imported_process.save!
           @imported_process
@@ -63,7 +66,7 @@ module Decidim
             organization: @organization
           )
 
-          group.remote_hero_image_url = attributes["remote_hero_image_url"] if remote_file_exists?(attributes["remote_hero_image_url"])
+          import_group_hero_image(group, attributes["remote_hero_image_url"])
           group.save!
           group
         end
@@ -91,20 +94,37 @@ module Decidim
         return if attachments["files"].nil?
 
         attachments["files"].map do |file|
-          next unless remote_file_exists?(file["remote_file_url"])
-
-          file_tmp = URI.parse(file["remote_file_url"]).open
+          url = file["remote_file_url"]
+          next if url.blank?
+
+          error = remote_file_error(url)
+          if error.present?
+            @warnings << I18n.t(
+              "decidim.participatory_processes.admin.imports.attachment_error",
+              title: attachment_title(file),
+              error:
+            )
+            next
+          end
 
           Decidim.traceability.perform_action!("create", Attachment, @user) do
             attachment = Attachment.new(
               title: file["title"],
               description: file["description"],
-              content_type: file_tmp.content_type,
               attached_to: @imported_process,
-              weight: file["weight"],
-              file: file_tmp, # Define attached_to before this
-              file_size: file_tmp.size
+              weight: file["weight"]
             )
+            begin
+              attachment.attached_uploader(:file).remote_url = url
+              attachment.set_content_type_and_size
+            rescue OpenURI::HTTPError, Errno::ENOENT, Errno::ECONNREFUSED, SocketError, Net::OpenTimeout, Net::ReadTimeout => e
+              @warnings << I18n.t(
+                "decidim.participatory_processes.admin.imports.attachment_error",
+                title: attachment_title(file),
+                error: format_error(e)
+              )
+              next
+            end
             attachment.create_attachment_collection(file["attachment_collection"])
             attachment.save!
             attachment
@@ -148,7 +168,7 @@ module Decidim
         attachment_collection
       end
 
-      def remote_file_exists?(url)
+      def remote_file_error(url)
         return if url.nil?
 
         accepted = ["image", "application/pdf"]
@@ -156,10 +176,53 @@ module Decidim
         http_connection = Net::HTTP.new(url.host, url.port)
         http_connection.use_ssl = true if url.scheme == "https"
         http_connection.start do |http|
-          return http.head(url.request_uri)["Content-Type"].start_with?(*accepted)
+          response = http.head(url.request_uri)
+          content_type = response["Content-Type"]
+          next if response.is_a?(Net::HTTPSuccess) && content_type&.start_with?(*accepted)
+
+          message = response.message.presence || Rack::Utils::HTTP_STATUS_CODES[response.code.to_i]
+          message = message.presence || "Error"
+          next "#{response.code} #{message}"
         end
-      rescue StandardError
-        nil
+      rescue StandardError => e
+        format_error(e)
+      end
+
+      def attachment_title(file)
+        title = file["title"]
+        return "" if title.blank?
+
+        return title unless title.is_a?(Hash)
+
+        title.values.find(&:present?) || ""
+      end
+
+      def import_hero_image(url)
+        return if url.blank?
+
+        @imported_process.attached_uploader(:hero_image).remote_url = url
+      rescue OpenURI::HTTPError, Errno::ENOENT, Errno::ECONNREFUSED, SocketError, Net::OpenTimeout, Net::ReadTimeout => e
+        @warnings << I18n.t("decidim.participatory_processes.admin.imports.hero_image_error", error: format_error(e))
+      end
+
+      def import_group_hero_image(group, url)
+        return if url.blank?
+
+        group.attached_uploader(:hero_image).remote_url = url
+      rescue OpenURI::HTTPError, Errno::ENOENT, Errno::ECONNREFUSED, SocketError, Net::OpenTimeout, Net::ReadTimeout => e
+        @warnings << I18n.t("decidim.participatory_processes.admin.imports.hero_image_error", error: format_error(e))
+      end
+
+      def format_error(error)
+        return error.message unless error.respond_to?(:io) && error.io.respond_to?(:status)
+
+        status = error.io.status
+        return error.message if status.blank? || status.first.blank?
+
+        code = status[0]
+        message = status[1].presence || Rack::Utils::HTTP_STATUS_CODES[code.to_i]
+        message = message.presence || error.message
+        "#{code} #{message}"
       end
     end
   end

data/app/views/decidim/participatory_processes/admin/participatory_process_imports/_form.html.erb

@@ -1,8 +1,12 @@
 <div class="form__wrapper">
   <div class="card pt-4" id="processes">
     <div class="card-section">
+      <div class="row column mb-4">
+        <%= cell("decidim/announcement", { body: t("decidim.participatory_processes.admin.new_import.help_html") }, callout_class: "info") %>
+      </div>
+
       <div class="row column">
-        <%= form.translated :text_field, :title, autofocus: true %>
+        <%= form.translated :text_field, :title, autofocus: true, aria: { label: :title } %>
       </div>
 
       <div class="row column">
@@ -10,7 +14,7 @@
       </div>
 
       <div class="row column">
-        <%= form.upload :document, required: true, label: t(".document_legend"), button_class: "button button__sm button__transparent-secondary" %>
+        <%= form.upload :document, label: t(".document_legend"), button_class: "button button__sm button__transparent-secondary", help_i18n_scope: "decidim.forms.file_help.import_file", help_i18n_messages: ["message_1"] %>
       </div>
 
       <div class="card-divider">
@@ -32,7 +36,6 @@
         </div>
       </div>
 
-      </div>
     </div>
   </div>
 </div>

data/app/views/decidim/participatory_processes/admin/participatory_process_steps/index.html.erb

@@ -24,7 +24,7 @@
             <tr data-id="<%= step.id %>">
               <td data-label="<%= t("models.participatory_process_step.fields.title", scope: "decidim.admin") %>">
                 <div class="flex items-center gap-x-2">
-                  <%= icon "drag-move-2-line" %>
+                  <%= icon("draggable", class: "dragger hover:cursor-grab") %>
                   <% if step.active? %>
                     <span class="icon-active"></span>
                   <% end %>

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

@@ -38,6 +38,6 @@
   </div>
 
   <section class="participatory-space__block-reference">
-    <%= resource_reference(current_participatory_space) %>
+    <p><%= resource_reference(current_participatory_space) %></p>
   </section>
 </main>

data/app/views/layouts/decidim/admin/participatory_process_group.html.erb

@@ -1,3 +1,5 @@
+<% add_secondary_root_menu(:admin_participatory_process_group_menu) %>
+
 <% content_for :breadcrumb_context_menu do %>
   <div class="process-title-content-breadcrumb-container-right">
     <% if allowed_to? :create, :process_group %>
@@ -8,17 +10,6 @@
         </span>
       <% end %>
     <% end %>
-    <div class="inline-block relative">
-      <%= button_tag id: "processes-menu-trigger", data: { controller: "dropdown", target: "processes-dropdown-menu-settings" }, class: "dropdown__trigger button button__transparent" do %>
-        <span>
-          <%= t("menu.manage", scope: "decidim.admin") %>
-        </span>
-        <%= icon "arrow-down-s-line" %>
-      <% end %>
-      <div id="processes-dropdown-menu-settings" class="process-title-content-dropdown">
-        <%= dropdown_menu(:admin_participatory_process_group_menu).render %>
-      </div>
-    </div>
   </div>
 <% end %>
 

data/config/locales/ar.yml

@@ -130,7 +130,6 @@ ar:
         destroy:
           success: تم حذف مجموعة العملية التشاركية بنجاح.
         edit:
-          title: تحرير مجموعة العملية
           update: تحديث
         new:
           create: إنشاء
@@ -141,7 +140,6 @@ ar:
       participatory_process_imports:
         new:
           import: استيراد
-          title: استيراد مسار تشاركي
       participatory_process_publications:
         create:
           error: كانت هناك مشكلة في نشر هذه العملية التشاركية.
@@ -161,13 +159,11 @@ ar:
         destroy:
           success: تم حذف مرحلة العملية التشاركية بنجاح.
         edit:
-          title: تحرير مرحلة العملية التشاركية
           update: تحديث
         index:
           steps_title: المراحل
         new:
           create: إنشاء
-          title: مرحلة عملية تشاركية جديدة
         ordering:
           error: كانت هناك مشكلة في إعادة ترتيب هذه المراحل من العملية التشاركية.
         update:
@@ -181,8 +177,6 @@ ar:
           success: تمت إزالة المشرف بنجاح من هذه العملية التشاركية.
         edit:
           update: تحديث
-        index:
-          process_admins_title: مدراء العملية التشاركية
         new:
           create: إنشاء
         update:
@@ -200,7 +194,6 @@ ar:
           published: نشرت
         new:
           create: إنشاء
-          title: عملية تشاركية جديدة
         update:
           error: حدثت مشكلة أثناء تحديث هذه العملية التشاركية.
           success: عملية المشاركة تم تحديثها بنجاح.

data/config/locales/bg.yml

@@ -99,8 +99,6 @@ bg:
           label: По група на процеса
       menu:
         participatory_process_groups: Групи процеси
-        participatory_process_groups_submenu:
-          info: Информация
         participatory_processes: Процеси
         participatory_processes_submenu:
           attachment_collections: Папки
@@ -148,7 +146,6 @@ bg:
         destroy:
           success: Групата на процеса на участие беше изтрита успешно.
         edit:
-          title: Редактиране на групата на процеса
           update: Актуализация
         new:
           create: Създаване
@@ -163,7 +160,6 @@ bg:
         new:
           import: Импортиране
           select: Изберете кои данни искате да импортирате
-          title: Импортиране на процес на участие
       participatory_process_publications:
         create:
           error: Възникна проблем при публикуването на този процес на участие.
@@ -193,13 +189,11 @@ bg:
               Оставихме го тук, за да можете да мигрирате CTA, от които се нуждаете. Полетата <i>Call to action path</i> и
               <i>Текстът на подканващата фраза</i> ще бъде премахнат в следващата версия.
             title: CTA е остарял
-          title: Редактиране на етап на процес на участие
           update: Актуализация
         index:
-          steps_title: Етапи
+          steps_title: Фази
         new:
           create: Създаване
-          title: Нов етап на процес на участие
         ordering:
           error: Възникна проблем при промяната на реда на тези етапи на процеса на участие.
         update:
@@ -212,13 +206,9 @@ bg:
         destroy:
           success: Администраторът успешно премахнат от този процес на участие.
         edit:
-          title: Актуализиране на администратора на процеса на участие
           update: Актуализация
-        index:
-          process_admins_title: Администратори на процес на участие
         new:
           create: Създаване
-          title: Нов администратор на процес на участие
         update:
           error: Възникна проблем при актуализирането на администратор за този процес на участие.
           success: Администраторът беше актуализиран успешно за този процес на участие.
@@ -235,7 +225,6 @@ bg:
           unpublished: Непубликувано
         new:
           create: Създаване
-          title: Нов процес на участие
         update:
           error: Възникна проблем при актуализирането на този процес на участие.
           success: Процесът на участие беше актуализиран успешно.

data/config/locales/bs-BA.yml

@@ -1,12 +1,6 @@
 ---
 bs:
   decidim:
-    admin:
-      participatory_process_user_roles:
-        edit:
-          title: Ažuriraj administratora participatornog procesa
-        new:
-          title: Novi administrator participatornog prostora
     events:
       participatory_process:
         role_assigned:

data/config/locales/ca-IT.yml

@@ -66,6 +66,8 @@ ca-IT:
           attributes:
             document:
               allowed_file_content_types: 'Tipus de document no vàlid. Només s''accepten arxius amb les següents extensions: %{types}'
+              empty: El document està buit
+              invalid_json: El document no és un JSON vàlid
     models:
       decidim/participatory_process_step_activated_event: Fase activada
       decidim/participatory_process_step_changed_event: Fase modificada
@@ -106,7 +108,7 @@ ca-IT:
       menu:
         participatory_process_groups: Grups de processos
         participatory_process_groups_submenu:
-          info: Informació
+          info: Quant a aquest grup de processos
           landing_page: Disposició de la pàgina de destinació
         participatory_processes: Processos
         participatory_processes_submenu:
@@ -168,7 +170,7 @@ ca-IT:
           error: Hi ha hagut un error en eliminar el grup de processos participatius.
           success: El grup de processos participatius s'ha eliminat correctament.
         edit:
-          title: Edita grup de procés
+          title: Editar el grup de procés
           update: Actualitza
         new:
           create: Crea
@@ -183,7 +185,7 @@ ca-IT:
         new:
           import: Importar
           select: Selecciona quines dades vols importar.
-          title: Importar el procés participatiu
+          title: Importar un procés participatiu
       participatory_process_publications:
         create:
           error: S'ha produït un error en publicar aquest procés participatiu.
@@ -212,13 +214,13 @@ ca-IT:
               Ara això es fa a la <b>Página d'inici</b>, fent servir el bloc de contingut <b>Imatges principal i botó d'acció</b>.
               L'hem deixar aquí per a que migris els CTAs que necessitis. Els camps <i>Ruta del botó d'acció</i> i <i>Text del botó d'acció</i> s'eliminaran a la propera versió.
             title: CTA obsolet
-          title: Edita fase de procés participatiu
+          title: Editar la fase de procés participatiu
           update: Actualitza
         index:
           steps_title: Fases del procés
         new:
           create: Crea
-          title: Nova fase de procés participatiu
+          title: Nova fase del procés participatiu
         ordering:
           error: S'ha produït un error en la reordenació de les fases d'aquest procés participatiu.
         update:
@@ -231,7 +233,7 @@ ca-IT:
         destroy:
           success: L'administradora s'ha eliminat correctament d'aquest procés participatiu.
         edit:
-          title: Actualitzar administradora de procés participatiu
+          title: Actualitzar l'administradora del procés participatiu
           update: Actualitza
         index:
           process_admins_title: Administradores del procés participatiu
@@ -494,9 +496,13 @@ ca-IT:
             all: Tots
             max_results: Quantitat màxima d'elements que es mostraran
             selection_criteria: Criteris de selecció
+        imports:
+          attachment_error: No s'ha pogut importar l'arxiu adjunt "%{title}" (%{error}).
+          hero_image_error: No s'ha pogut importar la imatge principal (%{error}).
         new_import:
           accepted_types:
             json: JSON
+          help_html: Aquesta funció d'importació et permet crear un nou procés participatiu a partir d'un fitxer JSON exportat. Pots exportar un procés participatiu d'una altra organització o de la mateixa organització. El procés participatiu importat inclourà la seva configuració, els components i els fitxers adjunts (si s'han seleccionat).
         participatory_process_duplicates:
           form:
             slug_help_html: 'Els segments d''URL s''utilitzen per generar les URL que apunten a aquest procés. Només accepta lletres, números i guions, i ha de començar amb una lletra. Exemple: %{url}'

data/config/locales/ca.yml

@@ -66,6 +66,8 @@ ca:
           attributes:
             document:
               allowed_file_content_types: 'Tipus de document no vàlid. Només s''accepten arxius amb les següents extensions: %{types}'
+              empty: El document està buit
+              invalid_json: El document no és un JSON vàlid
     models:
       decidim/participatory_process_step_activated_event: Fase activada
       decidim/participatory_process_step_changed_event: Fase modificada
@@ -106,7 +108,7 @@ ca:
       menu:
         participatory_process_groups: Grups de processos
         participatory_process_groups_submenu:
-          info: Informació
+          info: Quant a aquest grup de processos
           landing_page: Disposició de la pàgina de destinació
         participatory_processes: Processos
         participatory_processes_submenu:
@@ -168,7 +170,7 @@ ca:
           error: Hi ha hagut un error en eliminar el grup de processos participatius.
           success: El grup de processos participatius s'ha eliminat correctament.
         edit:
-          title: Edita grup de procés
+          title: Editar el grup de procés
           update: Actualitza
         new:
           create: Crea
@@ -183,7 +185,7 @@ ca:
         new:
           import: Importar
           select: Selecciona quines dades vols importar.
-          title: Importar el procés participatiu
+          title: Importar un procés participatiu
       participatory_process_publications:
         create:
           error: S'ha produït un error en publicar aquest procés participatiu.
@@ -212,13 +214,13 @@ ca:
               Ara això es fa a la <b>Página d'inici</b>, fent servir el bloc de contingut <b>Imatges principal i botó d'acció</b>.
               L'hem deixar aquí per a que migris els CTAs que necessitis. Els camps <i>Ruta del botó d'acció</i> i <i>Text del botó d'acció</i> s'eliminaran a la propera versió.
             title: CTA obsolet
-          title: Edita fase de procés participatiu
+          title: Editar la fase de procés participatiu
           update: Actualitza
         index:
           steps_title: Fases del procés
         new:
           create: Crea
-          title: Nova fase de procés participatiu
+          title: Nova fase del procés participatiu
         ordering:
           error: S'ha produït un error en la reordenació de les fases d'aquest procés participatiu.
         update:
@@ -231,7 +233,7 @@ ca:
         destroy:
           success: L'administradora s'ha eliminat correctament d'aquest procés participatiu.
         edit:
-          title: Actualitzar administradora de procés participatiu
+          title: Actualitzar l'administradora del procés participatiu
           update: Actualitza
         index:
           process_admins_title: Administradores del procés participatiu
@@ -494,9 +496,13 @@ ca:
             all: Tots
             max_results: Quantitat màxima d'elements que es mostraran
             selection_criteria: Criteris de selecció
+        imports:
+          attachment_error: No s'ha pogut importar l'arxiu adjunt "%{title}" (%{error}).
+          hero_image_error: No s'ha pogut importar la imatge principal (%{error}).
         new_import:
           accepted_types:
             json: JSON
+          help_html: Aquesta funció d'importació et permet crear un nou procés participatiu a partir d'un fitxer JSON exportat. Pots exportar un procés participatiu d'una altra organització o de la mateixa organització. El procés participatiu importat inclourà la seva configuració, els components i els fitxers adjunts (si s'han seleccionat).
         participatory_process_duplicates:
           form:
             slug_help_html: 'Els segments d''URL s''utilitzen per generar les URL que apunten a aquest procés. Només accepta lletres, números i guions, i ha de començar amb una lletra. Exemple: %{url}'