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.
By tstoeckler on
Change record status:
Published (View all published change records)
Project:
Introduced in branch:
8.0.x
Introduced in version:
8.0.0-rc1
Issue links:
Description:
ConfigNamesMapper::hasTranslatable()
previously had two logic flaws:
- If no config names were registered in the mapper it would return
TRUE
. This does not make sense, because if nothing is contained in the mapper certainly nothing has translatable data. - If multiple config names were registered in the mapper it would only return
TRUE
if all of the config names have translatable data. This does not make sense, because surely if one of the things in the list contains translatable data then the list as such does too.
The second bullet point was even documented on ConfigMapperInterface::hasTranslatable()
and tested but it is still wrong.
Impacts:
Module developers