Skip to content

No bell vars striptags

Matthias Larisch requested to merge no-bell-vars-striptags into master

What does this MR do?

Remove strip_tags in addBell because it might have broken serialized values and did not do good for the others. I checked all usages of the bell data: It is only in the navbar where it is all printed from vue.js in a safe way.

How confident are you it won't break things if deployed?

quite

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))

Merge request reports