๐Ÿ‡ช๐Ÿ‡ธSpain @tonibarbera

Account created on 5 September 2016, over 8 years ago
#

Merge Requests

More

Recent comments

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

This issue is still present.

I've created a new approach to solve this error: https://git.drupalcode.org/project/group_content_menu/-/merge_requests/34

Here the patch file.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I've tested installed on Drupal 10.3.1 and the patch #3 layout_builder_st-plugin-not-found-3463435-3.patch fixes the error.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Rerolled for the version 8.x-1.8

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

The patch doesnยดt apply to Drupal 10.3.1. Rerolled the patch for that version

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I've rerolled the #50 patch for the Drupal 10.3.1 version

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Rerolled the patch for the 10.3.1 branch

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I've updated the constructor. It should work now with Drupal 10.3

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I am working on a feature that requires this patch.

The provided solution here does not allow you to select the view from the field configuration menu.

Therefore, I'm pushing here a simple patch that provides a contextual menu in the field configuration that allows selecting the default view and applying it in the media modal window.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I've created a patch to have context in the NavigationBlock.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Added new commit to fix the error:

Drupal\Core\Entity\Query\QueryException: Entity queries must explicitly set whether the query should be access checked or not. See Drupal\Core\Entity\Query\QueryInterface::accessCheck(). in Drupal\Core\Entity\Query\Sql\Query->prepare() (line 141 of core/lib/Drupal/Core/Entity/Query/Sql/Query.php).

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I've merged the changes in src/Controller/GroupWebformController.php from the issue 3214003.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

For more info: the error was related to the contrib module bootstrap4_modal. It adds the a JS library that is in conflict in our project with the Drupal 10.1.x Modal libraries.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Closing this issue. Checked in a Drupal without any contrib module and is not reproducible and works fine.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

The MR was updated to solve a Javascript encoding typo.

Here the patch and the interdiff

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I can't find a way to reproduce the error only with core modules, but the JS in that file has an error, present in Drupal 10.2.x and Drupal 11:

262 $(window).on('dialog:aftercreate', (e, dialog, $element, settings) => {
263 $element.on('click.dialog', '.dialog-cancel', (e) => {
264 dialog.close('cancel');
265 e.preventDefault();
266 e.stopPropagation();
267 });
268 });

dialog is not passed to the function definition

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Here the patch to solve this error.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

This is an error dependent on issue: 3166181. I'm closing this one.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Sorry for my mistake: The previous patch is not correct, here the complete patch and the interdiff for 4.1 brach running with Drupal 10.1.x

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I've updated this patch for Drupal 10. Here the patch and the interdiff.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

This is an error related to issue: 3166181, please close this one.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Here the patch for D10 with CKEditor 4

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

I've rerolled a new patch that works on Drupal 10, based on the issue: 3287825.
Here is the new patch and the interdiff.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Rerolled patch for Drupal 10.1.5 and the interdiff

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Rerolled the MR for Drupal 10.1.5 and attached interdiff.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Sorry, this is the correct patch:

Tested with PHP8.2, group_media 8.x-2.x, and drupal/core 10.1.5

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Provided a patch tested by group_media 8.x-2.x and drupal/core 10.1.5

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Here a patch for the url_embed version 2.0.0-alpha1

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

This patch includes the current MR changes. Tested in Drupal 10.1.5 and bootstrap_barrio 5.5.14

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

Ok, then I've updated the MR with this ticket fix and I'm providing a new patch version.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

The solution provided by the user sarwan_verma is not correct. Is not fixing the reported problem in this ticket and it modified lines that are not related to this ticket at all.

I'm providing a correct fix.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ช๐Ÿ‡ธSpain tonibarbera

tonibarbera โ†’ made their first commit to this issueโ€™s fork.

Production build 0.71.5 2024