raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-27 01:43:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary sanitising

Browse Source 
Existing descriptions have been sanitised in a migration. New
descriptions are sanitised when assigned. That should cover everything.
This commit is contained in:
Maikel Linke
2024-09-05 12:38:33 +10:00
parent 53286c22ba
commit d061fe8ad9
2 changed files with 0 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/models/enterprise.rb
View File

@@ -247,11 +247,6 @@ class Enterprise < ApplicationRecord
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))

5
spec/models/enterprise_spec.rb
View File

@@ -404,11 +404,6 @@ RSpec.describe Enterprise 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
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