Backport 'Fix Empty participatory process group is created when importing a PP …' to v0.26

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

@@ -53,6 +53,11 @@ def import(attributes, _user, opts)
       end
 
       def import_process_group(attributes)
+        title = compact_translation(attributes["title"] || attributes["name"])
+        description = compact_translation(attributes["description"])
+
+        return if title.blank? && description.blank?
+
         Decidim.traceability.perform_action!("create", ParticipatoryProcessGroup, @user) do
           group = ParticipatoryProcessGroup.find_or_initialize_by(
             title: attributes["title"] || attributes["name"],
@@ -152,6 +157,11 @@ def import_components(components)
 
       private
 
+      def compact_translation(translation)
+        translation["machine_translations"] = translation["machine_translations"].reject { |_k, v| v.blank? } if translation["machine_translations"].present?
+        translation.reject { |_k, v| v.blank? }
+      end
+
       def create_attachment_collection(attributes)
         return unless attributes.compact.any?
 

decidim-participatory_processes/spec/serializers/decidim/participatory_processes/participatory_process_importer_spec.rb

@@ -94,6 +94,19 @@ module Decidim::ParticipatoryProcesses
           expect(group.title).to eq(group_data["name"])
         end
       end
+
+      context "when the process group is empty" do
+        let(:group_data) do
+          {
+            "title" => Decidim::Faker::Localized.localized { "" },
+            "description" => Decidim::Faker::Localized.localized { "" }
+          }
+        end
+
+        it "does not create a process group" do
+          expect { subject }.not_to change(Decidim::ParticipatoryProcessGroup, :count)
+        end
+      end
     end
   end
 end