The MR !34 works for me without any issues - RTBC.
The patch #8 is updated to apply to the latest 4.x
Also, I don't think #26 changes the solution, just extending it.
The MR!8743 is updated to pass the code styles check. Please review.
The PR is updated.
The updated patch can be found in the attachment.
Attached is an updated patch with flashing as it's done on the Flush Single Image Style module.
This should be reopened as it causes incompatibility with Drupal 10.2.7.
Please see
the comment
📌
Change the 1.1.x branch to support Drupal <= 10.2 for a smoother update process and revert new changes
Fixed
on bootstrap_styles module.
The operator ( `<=10.2` ) limits anything higher than 10.2.0 on the 10.2.x range.
This should be reopened as I also faced incompatibility with Drupal 10.2.7.
Agree with @vangelisp
Combined the changes from MR!6125 and #26 to MR!8743.
The MR!6125 is hidden as redundant.
dburiak → changed the visibility of the branch 3041174-adjust-layout-builder to hidden.
It looks like it's related to the Drupal core issue https://www.drupal.org/project/drupal/issues/2504115 → .
The workaround is to render the form without Ajax and send a taxonomy term ID through the URL query parameter.
The patch is attached.
The issue is still relevant for 2.0.x-dev.
I can confirm that the attached patch fixes the issue.
The patch works for me on Drupal 11.
Patch #15 has been updated to work on the latest codebase 3.1.x-dev.
dburiak → made their first commit to this issue’s fork.
@orakili I added 2 comments to your PR about DeprecationHelper class and code duplication.
Correct me if I'm wrong @sthomen but your changes make 3.x not compatible with < Drupal 10.2.
Isn't it better to use DeprecationHelper to support older versions as well?
if (!class_exists(DeprecationHelper::class)) {
$element['#upload_validators']['file_validate_extensions'][0] = implode(' ', $extensions);
}
else {
DeprecationHelper::backwardsCompatibleCall(
\Drupal::VERSION,
'10.2',
static function () use ($extensions, &$element) {
$element['#upload_validators']['FileExtension'][0] = implode(' ', $extensions);
},
static function () use ($extensions, &$element) {
$element['#upload_validators']['file_validate_extensions'][0] = implode(' ', $extensions);
}
);
}@smustgrave the summary is updated. Please review.
dburiak → changed the visibility of the branch 3419621-typeerror-inarray-argument to hidden.
@smustgrave The string in the original issue was calling from Geofield Map module (already fixed). The summary is updated with a proposed solution section.
This issue is not related to the ckeditor_html_embed and duplicates
https://www.drupal.org/project/drupal/issues/3364884
🐛
JavaScript operators in
Needs work
On the referenced issue there is a patch to fix this.
I think this one can to be closed.
Added MR to 11.x: https://git.drupalcode.org/issue/drupal-3419621/-/tree/3419621-typeerror...
Please review.
dburiak → changed the visibility of the branch 3419621-typeerror-inarray-argument-d11 to hidden.
dburiak → changed the visibility of the branch 3419621-typeerror-inarray-argument-d11 to active.
Updated test-only patch with fixed coding standards.
Here is the test-only patch showing the issue.
Without patching, the test will fail.
Added a patch based on the proposed solution.
Sure @smustgrave
Thank you for review.
The doTestHomePageLinks test method is updated to fix the subdirectory multisite case for the PHPUnit testing run.
Added a patch with the changes above to simplify using with composer.
+
The summary is updated. Also, the full patch is attached.
@larowlan the changes from the patch above cause another regression with front page links. I created a new issue: 🐛 [regression] Language switcher block returns links to node on the frontpage Fixed