Remove unnecessary sanitising

Existing descriptions have been sanitised in a migration. New descriptions are sanitised when assigned. That should cover everything.
openfoodfoundation · Sep 5, 2024 · d061fe8 · d061fe8
1 parent 53286c2
commit d061fe8
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 10 deletions.
diff --git a/app/models/enterprise.rb b/app/models/enterprise.rb
@@ -247,11 +247,6 @@ def self.distinct_count
     count(distinct: true)
   end
 
-  # Remove any unsupported HTML.
-  def long_description
-    HtmlSanitizer.sanitize(super)
-  end
-
   # Remove any unsupported HTML.
   def long_description=(html)
     super(HtmlSanitizer.sanitize(html))

diff --git a/spec/models/enterprise_spec.rb b/spec/models/enterprise_spec.rb
@@ -404,11 +404,6 @@
       subject.long_description = "Hello <script>alert</script> dearest <b>monster</b>."
       expect(subject.long_description).to eq "Hello alert dearest <b>monster</b>."
     end
-
-    it "sanitises existing HTML in long_description" do
-      subject[:long_description] = "Hello <script>alert</script> dearest <b>monster</b>."
-      expect(subject.long_description).to eq "Hello alert dearest <b>monster</b>."
-    end
   end
 
   describe "callbacks" do