I just discovered that if you pass NULL as the value to a placeholder in a log entry, this will blow up when using dblog and visiting the “Recent log messages” page with the following error:
TypeError: Drupal\Component\Utility\Html::escape(): Argument #1 ($text) must be of type string, null given, called in core/lib/Drupal/Component/Render/FormattableMarkup.php on line 238 in Drupal\Component\Utility\Html::escape() (line 431 of core/lib/Drupal/Component/Utility/Html.php).
Apparently those are handled fine by t(), but not by the dblog module.
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #3
drunken monkeyFixed in this MR, please test/review.
While most of the places in the code concerned entity labels (search indexes in most cases, to be precise) which can be
NULLtheoretically, the most important changes were probably the ones concerning fields, in theAddHierarchyandRenderedItemprocessors, as field labels are probably more oftenNULLthan entity labels.Comment #5
drunken monkey