- Merge request !34Issue #3299045: Duplicate templates in same namespace is valid use case β (Open) created by parisek
- last update
almost 2 years ago 35 pass, 1 fail - π¨πΏCzech Republic parisek
Added MR and disabled logging warning "Found multiple files for the", seems it's not wide issue, patch is enough :)
- First commit to issue fork.
- last update
over 1 year ago 35 pass, 1 fail - Status changed to Needs review
over 1 year ago 12:21pm 18 July 2023 - last update
over 1 year ago CI aborted - last update
over 1 year ago CI aborted Attached a patch which allows sites to opt out of the warning using their settings.php file:
// Don't attempt to track duplicate templates. $settings['components_ignore_duplicate_templates'] = TRUE;
Or on the theme/module level:
components: # Ignore duplicate templates. ignore_duplicate_templates: true
This still requires tests to be implemented, but works for my use case. So feedback is more than welcome on the current implementation.
- last update
over 1 year ago 61 pass - Status changed to RTBC
about 1 month ago 3:30pm 27 February 2025 - π¬π§United Kingdom John Cook
I had problems applying the diff / patch using
cweagans/composer-patches
, with a segment being rejected incomponents.info.yml
. This was due to versioning information added by the packaging script on Drupal.org. I manually added the two line change before testing. I don't see this being a problem once the issue has been merged.I tested with just the patch to make sure that the duplicates are reported by default. As expected, the errors appeared in the watchdog log.
Then I tried again after adding
$settings['components_ignore_duplicate_templates'] = TRUE;
to mysettings.php
. After I cleared the cache and refreshed the same page, the errors are no longer logged.Moving to RTBC.
- π¬π§United Kingdom John Cook
As a side note, I noticed this bug because I have
/web
as a symlink to/docroot
after changing host providers.