Fix NULL error with weight field in assemblies & processes (#7491)

Co-authored-by: LauraJaimeC <75725233+LauraJaimeC@users.noreply.github.com>
decidim · Mar 2, 2021 · 7aae7de · 7aae7de
1 parent e4c21f7
commit 7aae7de
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/decidim-assemblies/app/forms/decidim/assemblies/admin/assembly_form.rb b/decidim-assemblies/app/forms/decidim/assemblies/admin/assembly_form.rb
@@ -45,7 +45,7 @@ class AssemblyForm < Form
         attribute :parent_id, Integer
         attribute :participatory_processes_ids, Array[Integer]
         attribute :scope_id, Integer
-        attribute :weight, Integer
+        attribute :weight, Integer, default: 0
 
         attribute :is_transparent, Boolean
         attribute :promoted, Boolean
@@ -80,6 +80,8 @@ class AssemblyForm < Form
         validates :banner_image, passthru: { to: Decidim::Assembly }
         validates :hero_image, passthru: { to: Decidim::Assembly }
 
+        validates :weight, presence: true
+
         alias organization current_organization
 
         def ensure_parent_cannot_be_child

diff --git a/...y_processes/app/forms/decidim/participatory_processes/admin/participatory_process_form.rb b/...y_processes/app/forms/decidim/participatory_processes/admin/participatory_process_form.rb
@@ -32,7 +32,7 @@ class ParticipatoryProcessForm < Form
         attribute :scope_id, Integer
         attribute :related_process_ids, Array[Integer]
         attribute :scope_type_max_depth_id, Integer
-        attribute :weight, Integer
+        attribute :weight, Integer, default: 0
 
         attribute :private_space, Boolean
         attribute :promoted, Boolean
@@ -59,6 +59,8 @@ class ParticipatoryProcessForm < Form
         validates :banner_image, passthru: { to: Decidim::ParticipatoryProcess }
         validates :hero_image, passthru: { to: Decidim::ParticipatoryProcess }
 
+        validates :weight, presence: true
+
         alias organization current_organization
 
         def map_model(model)