Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Follow-up to #2508735: Code injection via preg_replace()
Problem/Motivation
#2508735: Code injection via preg_replace() did go in without test coverage.
Comments
Comment #13
quietone CreditAttribution: quietone at PreviousNext commentedThree files were changed in the originating issue.
core/modules/system/src/MachineNameController.php
The change in \Drupal\system\MachineNameController::transliterate is tested in \Drupal\Tests\system\Unit\Transliteration\MachineNameControllerTest
core/modules/image/src/PathProcessor/PathProcessorImageStyles.php
the change is in \Drupal\image\PathProcessor\PathProcessorImageStyles::processInbound and the code is tested in \Drupal\Tests\image\Functional\ImageDimensionsTest
core/lib/Drupal/Core/Block/BlockBase.php
That code is now in \Drupal\Core\Block\BlockPluginTrait::getMachineNameSuggestion and it is tested in \Drupal\Tests\block\Kernel\BlockInterfaceTest
Therefor, closing as outdated.