πŸ‡§πŸ‡ͺBelgium @michaelsoetaert

Account created on 17 February 2016, over 8 years ago
#

Merge Requests

Recent comments

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

Attached patch does as described.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

When using the patch, I got the following error:
TypeError: array_filter(): Argument #1 ($array) must be of type array, null given in array_filter() (line 174 of modules/contrib/layout_builder_lock/layout_builder_lock.module).

I needed to add the #parents-configuration to the layout_builder_lock_regions form element to fix the issue.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

michaelsoetaert β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

Sadly, the patch in comment #13 πŸ› Timeout on large sites nodes during install Needs review still resulted in timeouts on our higher environments (due to different PHP values). The issue seemed to be the large amount of data being loaded in views_url_alias_rebuild_path, since it's retrieving the complete Entity-object of each path alias.

I decided to try the approach @dstorozhuk suggested (using the Batch API). Only loading the path alias IDs in views_url_alias_rebuild_path, splitting the list in chunks and only loading the Entity-objects of the given path alias IDs in each batch operation. That seems to have fixed the timeouts.

Attached patch with the described functionality.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

I've rerolled the changes from MR#17 on branch 8.x-2.x-with-issue-3396154 on the latest version of the 3.x branch.

We needed multilingual support (different URL aliases for different translations), which branch 3.x provides, but we were also getting timeouts because of the size of the website (where the changes in this issue come into play).

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

michaelsoetaert β†’ changed the visibility of the branch 3438163-videoplayerformatter-wrongfully-not to hidden.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

When used in conjunction with #3364019 ✨ Add feature to clone entities by reference fields like Paragraphs RTBC to clone referenced entities, the entities would still be loaded in the current language and not the requested translation language. Properly setting the translation language in the $form_state fixes this issue.

Attached is the updated patch.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

The previous patch contained changes from another patch. I've removed that (wrongfully added) code.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

The underlying issue has finally been fixed for the ff_mpeg module and a new release ( 8.x-1.2 β†’ ) is available. Updating to this new release will solve this issue.

Closing this issue.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

The patch in comment #19 ✨ Output multiple values of a query param with repeating param name Needs work didn't apply on Drupal Core 10.0.9, so I've rerolled it.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

Issue has been fixed and added to latest release 1.0.0-alpha2.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

Issue has been fixed and added to latest release 1.0.0-alpha2.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

michaelsoetaert β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

michaelsoetaert β†’ made their first commit to this issue’s fork.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

Make sure that your hosting supports this. I was having the same issue on Acquia and after some research, I found it only supports the -t flag. If your hosting doesn't support the -bs flag, you can configure a new sendmail command in your settings.php file.

This is the documentation for Acquia I followed.

πŸ‡§πŸ‡ͺBelgium michaelsoetaert

I've tested the proposed solution and can confirm that it works with drupal/token 1.11.0.

Production build 0.71.5 2024