Needed to not fail with newer `plone.outputfilters` from this PR: https://github.com/plone/plone.outputfilters/pull/49
Comments go through the outputfilters, and the new branch calls `soup.prettify()` from `Beautifulsoup`, leading to more white space.
Sample test failures on Jenkins, see https://jenkins.plone.org/job/plip-plip-image-srcsets-3.8/5/#showFailuresLink
```
'<p>\n Go to http://www.plone.org\n</p>' != '<p>Go to http://www.plone.org</p>'
- <p>
- Go to http://www.plone.org
? ^ ^
+ <p>Go to http://www.plone.org</p>? ^^^ ^^^^
- </p>
File "/srv/python3.8/lib/python3.8/unittest/case.py", line 60, in testPartExecutor
yield
File "/srv/python3.8/lib/python3.8/unittest/case.py", line 676, in run
self._callTestMethod(testMethod)
File "/srv/python3.8/lib/python3.8/unittest/case.py", line 633, in _callTestMethod
method()
File "/home/jenkins/.buildout/eggs/cp38/plone.app.discussion-4.0.0a7-py3.8.egg/plone/app/discussion/tests/test_comment.py", line 180, in test_getText_doesnt_link
self.assertEqual(
File "/srv/python3.8/lib/python3.8/unittest/case.py", line 912, in assertEqual
assertion_func(first, second, msg=msg)
File "/srv/python3.8/lib/python3.8/unittest/case.py", line 1292, in assertMultiLineEqual
self.fail(self._formatMessage(msg, standardMsg))
File "/srv/python3.8/lib/python3.8/unittest/case.py", line 753, in fail
raise self.failureException(msg)
```
Moderator is not forced to delete a comment or to let it pending:
Workflow has two more states "rejected" and "spam" to existing review workflow.
Moderation view extended showing all states. Filter by state.
This is part of PloneHotfix20161129.
Updated metadata.xml version to 1000. This leaves more room for
profile version increases in earlier releases. We apply the rolemap
step again to avoid accidentally missing it.
Otherwise, the email field might get set to required when an anonymous
user visits, and then remain required when an authenticated user
visits, making it impossible for an authenticated user to fill in the
form without validation error. Or when in the control panel the field
is set as not required anymore, that change would have no effect until
the instance was restarted.
