Thanks for the patch. I also still had the problem. After the 3rd iteration of the update hook and the query for $token_users, $sandbox[‘current’] was set to the last item in the pl table. Therefore the $token_users array was empty and the following query for $valid_users failed.
I have tried the patch from #39, but it does not solve our problem with the css files not being found. Deactivating the option 'Enforce clean and canonical URLs.' on the other hand solves the problem.
* Server: Apache
* Drupal 10.2.2
* redirect 1.9 (patched)
* multilingual
We are still trying to find the reason for this
For all of you who, like me, first landed here in search of a solution and somehow ended up there:
https://www.drupal.org/project/redirect/issues/3373123
🐛
Setting 'Enforce clean and canonical URLs.' breaks CSS aggregation on multilingual Drupal 10.1.x with browser caching enabled
RTBC
. If I deactivate the 'Enforce clean and canonical URLs' option of the redirect module on one of our multilingual pages, the aggregated CSS files are loaded correctly. All suggested solutions in this issue did not work. The patch in the linked redirect-issue applies, but does not work in our case either. However, the cause is the same. Perhaps this is a hint for all those for whom the CSS aggregation also no longer works. Checking the server configuration with an updated patch from
https://www.drupal.org/project/drupal/issues/3384272
✨
Allow to validate Apache/Nginx configuration requirement for aggregation folder before enabling aggregation.
Needs review
shows that everything is ok/correctly configured.
* Server: Apache
* Drupal 10.2.2
* redirect 1.9 (patched)
* multilingual
Rebased and updated the patch.
Still a problem with Drupal 10.1.2 and Seven 1.0.0 (contrib)
re-rolled patch
We have this problem because of:
https://www.drupal.org/project/paragraphs_features/issues/3353704
🐛
Add-in only works if active theme is claro or gin
RTBC
https://git.drupalcode.org/project/paragraphs_features/-/merge_requests/... solves the issue but the styling for the add in between buttons ist gone with the latest versions of paragraphs_ee and paragraphs_features
Re-rolled patch. Use correct base branch.
Update D10 compatibility from #13
- remove obsolete "core: 8.x" from .info.yml
Re-roll patch
- Fix error for taxonomies
- update for D10 compatibility
A possible solution is to add needed config described here: https://ckeditor.com/docs/ckeditor4/latest/api/CKEDITOR_config.html#cfg-... via hook_wysiwyg_editor_settings_alter()
/**
* Implements hook_wysiwyg_editor_settings_alter().
*/
function hook_wysiwyg_editor_settings_alter(&$settings, $context): void {
if ($context['profile']->editor == 'ckeditor') {
$settings['iframe_attributes'] = [
'sandbox' => 'allow-scripts allow-same-origin',
'allow' => 'autoplay',
];
}
}