Account created on 8 May 2008, about 16 years ago
#

Merge Requests

More

Recent comments

πŸ‡§πŸ‡ͺBelgium kensae

Which line is failing?
When I'm using the issue fork directly in my composer file to load, it seems to work.

πŸ‡§πŸ‡ͺBelgium kensae

A d10 release would be great indeed. It's only a matter of changing the core_requirements and we're good to go.

πŸ‡§πŸ‡ͺBelgium kensae

kensae β†’ created an issue.

πŸ‡§πŸ‡ͺBelgium kensae

This issue is introduced again with facets dev-3.0.x, using facets exposed filters, search_api 1.34.0 and views_bulk_operations 4.2.6 in Drupal 10.2.5

I've updated the patch for the combination of this versions.

πŸ‡§πŸ‡ͺBelgium kensae

I've fixed the failing test. It didn't include the hostnames.

πŸ‡§πŸ‡ͺBelgium kensae

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

πŸ‡§πŸ‡ͺBelgium kensae

kensae β†’ changed the visibility of the branch 3241153-move-helper-functions to hidden.

πŸ‡§πŸ‡ͺBelgium kensae

This would be very useful to extend ckeditor: for example to implement the 'Balloon editor' in a custom module.

πŸ‡§πŸ‡ͺBelgium kensae

Problem #2 is caused by the fact I'm rendering the view with twig_tweak's function drupal_view, which calls the views_embed_view function. Because currently the SearchApiDisplay is searching for arguments on the url or the from the arguments in the ajax request, the argument set directly in views_embed_view are being ignored.
I've managed to get the original contextual arguments from the 'old_view' array from the ViewExecutable object. It seems a little dirty to me, but for now, it does the trick.

πŸ‡§πŸ‡ͺBelgium kensae

Unfortunately my changes do not work, because set arguments are being overwritten again in ViewsBlock plugin

πŸ‡§πŸ‡ͺBelgium kensae

I've tested this patch with Facets 3.0.x-dev and better exposed filters 6.0.3 in an ajax enabled view, and it works for me.

πŸ‡§πŸ‡ͺBelgium kensae

I can confirm the patch in #9 πŸ› CKEditor 5 balloons invisible when CKEditor 5 is used inside a modal Needs work works for me and typing is possible. This patch also solves the following issue for me: https://www.drupal.org/project/drupal/issues/3351603 πŸ› CKEditor 5 link broken in Dialog if another CKEditor exists Needs work

πŸ‡§πŸ‡ͺBelgium kensae

The patch in #134 failed for Drupal 10.2 on the media.routing.yml file, so I've adapted the patch slightly.

πŸ‡§πŸ‡ͺBelgium kensae

I've tested the merge request in #2 and it works for our use case.

πŸ‡§πŸ‡ͺBelgium kensae

Instead of using the patching the 2.x-dev version, I've added the issue fork repository in my composer.json. Patching doesn't work because composer is checking the version requirements before the patch is applied.

πŸ‡§πŸ‡ͺBelgium kensae

I've added the check suggested in #66

πŸ‡§πŸ‡ͺBelgium kensae

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

πŸ‡§πŸ‡ͺBelgium kensae

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

πŸ‡§πŸ‡ͺBelgium kensae

I guess this issue can be closed because the changes are targeted in the 3.0.x-dev branch to avoid confusion?
I've accidentally opened an issue fork for this issue because I didn't care to look at the 3.0.x-dev branch myself.

πŸ‡§πŸ‡ͺBelgium kensae

I guess it would be better to keep both issues separated and remove the changes for the Mollie for Drupal Webform submodule in this patch?
I wonder If we should even include patches for Mollie for Ubercart and Mollie for Drupal - Customers API or move them to separate issues?

πŸ‡§πŸ‡ͺBelgium kensae

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

πŸ‡§πŸ‡ͺBelgium kensae

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

πŸ‡§πŸ‡ͺBelgium kensae

Would it be possible to move this changes into a new release?

πŸ‡§πŸ‡ͺBelgium kensae

I think we should reopen this issue, because it generates the following error: Error: Class "Drupal\migrate_tools\Commands\MigrateToolsCommands" not found in include() (line 27 of /app/www/modules/contrib/path_redirect_import/src/Commands/PathRedirectImportCommands.php) #0 /app/vendor/composer/ClassLoader.php(571): include()

The solution to this is to change
use Drupal\migrate_tools\Commands\MigrateToolsCommands;
to
use Drupal\migrate_tools\Drush\MigrateToolsCommands;
in PathRedirectImportCommands

πŸ‡§πŸ‡ͺBelgium kensae

This configuration works for me:

$settings['cache']['default'] = 'cache.backend.file_system';
$settings['cache']['bins']['config'] = 'cache.backend.database';
$settings['cache']['bins']['discovery'] = 'cache.backend.database';
$settings['cache']['bins']['bootstrap'] = 'cache.backend.database';
$settings['cache']['bins']['data'] = 'cache.backend.database';
$settings['filecache']['directory']['default'] = '/app/private/filecache';
πŸ‡§πŸ‡ͺBelgium kensae

I've created a patch from the issue fork. I had to remove the changes from the .info.yml file, because they caused the patch to fail because of the following issue: https://www.drupal.org/project/drupalorg/issues/3066468 πŸ“Œ Packaging info from .info.yml often creates conflicts when patching (ddo) Active

πŸ‡§πŸ‡ͺBelgium kensae

This issue was cause by a custom form_alter we used in a specific project. So we can close this issue.

Production build 0.69.0 2024