- Issue created by @Grevil
- Assigned to Grevil
- 🇩🇪Germany Anybody Porta Westfalica
Nice finding, simple but important fix!
- last update
almost 2 years ago 27 pass - @grevil opened merge request.
- Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 9:27am 20 June 2023 - 🇩🇪Germany Grevil
The "description" change shouldn't break anything, as it is properly set in the fences.yml. But we should make sure, that the fallback or rather "default" is properly set.
- 🇩🇪Germany Anybody Porta Westfalica
Thanks @Grevil! That's fine, nice one.
Could you explain the test removal? Seems unrelated and even dangerous?
- 🇩🇪Germany Grevil
Array has 2 duplicate keys with value 'Classes and tags, with label' ('Classes and tags, with label', 'Classes and tags, with label')." which can be simply removed.
The same tests already exists under a different name.
- Status changed to RTBC
almost 2 years ago 10:32am 20 June 2023 - 🇩🇪Germany Anybody Porta Westfalica
As I see, it already exists as "Default field, default items wrapper, no label" AND the "Classes and tags, with label" was double-used. You removed the right one.
All clear now, RTBC!
- last update
almost 2 years ago 27 pass - Status changed to Fixed
almost 2 years ago 10:38am 20 June 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Fixed
over 1 year ago 7:59am 30 September 2023 Still in current stable, time for a new release?
Deprecated function: Creation of dynamic property Drupal\fences\TagManager::$themeHandler is deprecated in Drupal\fences\TagManager->__construct() (line 43 of modules/contrib/fences/src/TagManager.php).
- 🇩🇪Germany Anybody Porta Westfalica
@thomas.frobieter indeed! Tagged 3.0.1:
https://www.drupal.org/project/fences/releases/3.0.1 →