`Markdown` view helper does not allow `null` for `text` argument
tomudding opened this issue · comments
Tom Udding commented
Current behaviour
A lot of historical data of activities does not have correct localisation, as such null
is always passed to
gewisweb/module/Application/src/View/Helper/Markdown.php
Lines 24 to 27 in 24d805e
This is not allowed and a 500 is returned.
Desired behaviour
Allow null
and automatically change it to ''
(empty string).
Steps to reproduce
https://gewis.nl/en/activity/view/2063
Website version
latest
What operating are you seeing the problem on?
No response
What browsers are you seeing the problem on?
No response
Other information
No response
Tom Udding commented
Perhaps it is actually better to change the LocaliseText
view helper instead. Never have it return null
but ''
(empty string) instead.