Fix legal terms regression

3scale · Sep 18, 2024 · 4924c8d · 4924c8d
1 parent 33e9a2b
commit 4924c8d
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/app/models/cms/builtin/legal_term.rb b/app/models/cms/builtin/legal_term.rb
@@ -38,6 +38,6 @@ def content_type
   private
 
   def publish_draft
-    self.published = draft unless published
+    self.published = draft
   end
 end
diff --git a/test/unit/cms/builtin/legal_term_test.rb b/test/unit/cms/builtin/legal_term_test.rb
@@ -23,4 +23,14 @@ def setup
     assert_not_empty lt.errors[:system_name]
   end
 
+  test 'published is filled with draft value' do
+    value = 'some text'
+    lt = @provider.builtin_legal_terms.create(system_name: 'signup_licence', draft: value)
+    assert_equal value, lt.published
+
+    new_value = 'updated text'
+    lt.update(draft: new_value)
+    assert_equal new_value, lt.published
+  end
+
 end