🇪🇺🇫🇷 Paris, France
Account created on 24 June 2010, almost 15 years ago
#

Merge Requests

More

Recent comments

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

phily created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

You should use {{ 'test@example.com'|obfuscateMail }}

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

add the "Language: set code"

Perfect tip, that works!
Thanks @zengenuity

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Are you sending these emails to a user account on the site and want to use the language from their account settings?
Yes

Does it work if you set the recipient_uid field on the easy email entity?
The email is sent but in the website default language.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #97 works for me using Drupal 11.1.2 😋

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Sorry for being late 😁.
The updated bpmn_io.css in MR92 seems good for me, thank you both.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

OK, so let's add more lines to your Workarounds section in bpmn_io.css:

.bio-properties-panel-add-entry {
  fill: var(--add-entry-hover-fill-color);
}

It's the rule the (+) get when it is hovered (so it will have the same state hovered or not).
So your Workarounds section looks like :

/* Workarounds for broken template select button */
#bpmn-io .bio-properties-panel-group-header-buttons .bio-properties-panel-group-header-button:not(.bio-properties-panel-arrow), /* your proposition */
#bpmn-io .bio-properties-panel-dropdown-button .bio-properties-panel-group-header-button {
  background-color: var(--select-template-background-color);
}

.bio-properties-panel-add-entry {
  fill: var(--add-entry-hover-fill-color); /* fix for the black (+) */
}
🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Jurgen, your proposition adds a weird blue styling to the 'Extension properties ' (+) button
PS: I'm using Gin theme.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

phily created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

phily created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Meanwhile, I found the Content Entity Source sub module.
Then on the TMGMT Sources tab, you can select taxonomy terms and check all terms...

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

phily created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Both MRs work for me using Drupal 11.1.1

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

MR 20 works for me using Drupal 11.1.1

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #92 works for me with Drupal 10.4.0 as well ;-)

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch works for offsetting time only, could it be extended to handle date offset so one can get a date with a '-3 months' offset?

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #51 works for me using Drupal 10.3.9

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

#7 works for me using Drupal 10.3.6, patch is attached.
It should work with Drupal 11 as the views.views.inc file is the same for these 2 releases.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #2 works for me using Drupal 10.3.6 and PHP 8.3
Same result for altcom_neil in the above comment, so moving this issue to RTBC.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

phily created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

I looking for this too.
I thought they would be on the admin/config/regional/config-translation page (Drupal 10.3.6).

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #14 works for me with:

  • Drupal 10.3.6
  • facets 3.0.0-beta1 + patch "3360426 - Updated php 8.2 compatibility"
  • views_infinite_scroll 2.0.3
  • AJAX enable on the view (block display)

Thanks

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Thanks @boinkster for sharing kentrichards.net blog post, 4 easy steps and it now works!

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

phily created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #5 works for me.
I didn't alter my composer.json file for other reasons so I manually downloaded to lootie player to the libraries/lottiefiles--lottie-player/dist folder.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Similar module with such functionnality : https://www.drupal.org/project/back_to_top_with_pi

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

phily created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France
🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Renamed patch file for naming consistency.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch at #2 has 2 actions: fixing the missing schema.yml file and modifing the NbspFilter.php file.
The weird extra folders might come from that second part of the patch where deleted commented lines are replaced by non commented lines.
Anyway, it might better to split these fixes in two issues and keep the missing schema fix in this one.
So, here is an updated patch fixing the missing file only, credits go to Jejddx.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Using Drupal 10.3.1, I confirm @hudri at comment 28 🐛 CKEditor 5 balloons invisible when CKEditor 5 is used inside a modal Needs work :

#27 does not work, the property --ck-z-modal is no longer used in v10.3.0. The relevant property was renamed to --ck-z-panel

So patch#27 updated to use --ck-z-panel (instead of --ck-z-modal or using both for retrocompatibility) will be OK.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #92 works for me with Drupal 10.3

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

@grimal: see #35 at issues 3419024 🐛 Incorrect content-length header breaks HTTP/2 with net::err_http2_protocol_error Active where the suggested addition to .htaccess file might help you.
Header always unset Content-Length

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #8 works for me using Drupal 10.2.6 with Bootstrap 5 classes (button colors and sizes).

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

An update hook will be needed for existing tac_lite setups as all users accesses need to be set again.
Syncing "tab" and "field" could be the best of both worlds… something that has been imagined in issue https://dgo.to/2057915

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

I do think this is a major feature for tons of reasons:

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #5 works for using Drupal 10.2.5

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #2 works for using using Drupal 10.2.5
Maybe another reviewer could give it a test to help switching this issue to RTBC status.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Allowing to bypass all tac-lite schemes may be a bit too wide.

Without a 'bypass tac_lite permission for XXX scheme', related issue Add the ability to include all taxonomy terms without having to select them may give a finer control.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #2 works for using using Drupal 10.2.5

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #9 works for using using Drupal 10.2.5

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #5 works for using using Drupal 10.2.5

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #2 works for using using Drupal 10.2.5
Same issue as in #7 about MR !7

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

I gave a try to the tac_lite module which also determines node access with the "matching taxonomy terms" setup: facets counters are then good and related to how many nodes the user has access to.

So, maybe to the way access_policy declares access to node is different from how Drupal 10 handles it (which seems to be used by facets)...

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Sorry to reopen the issue, but I'm facing the exact same behavior and same fix with a brand new subtheme generated using today's downloaded dev release of Radix (composer require 'drupal/radix:6.0.x-dev@dev'), either setting the subtheme or radix as the default theme: webform field wrapper class is gone...

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

As a first hint, I'd say that main.script.js used in the subtheme is querying for data-toggleattribute while Bootstrap 5 uses data-bs-toggle.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Yes, the fix from the latest dev release fixes it ;-), thanks.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Of course, after copying the component to the child theme, all references to {% include 'radix:block' %} in the 'templates' folder should be updated to {% include 'SUBTHEME:block' %}

That's why to radix component should be fixed.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #75 works for me using Drupal 10.1.7 + CKEditor 5 on a full display mode.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Code from weseze at #7 updated by fonant at #9 works for me using Drupal 9.5.11 and Search API with a DB backend.
This is a lifesaver for the project I need random sorting on a search page.

I was wondering if it could be upgraded to get the actual reset_seed_custom value from the view itself rather than hard coding it in the custom code.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #90 works for me using Drupal 10.1.5

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

I forgot to say I'm using Drupal 10.1.5

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

PhilY created an issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Works for me using Drupal 10.1.5

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #122 Also use text editor (CKEditor) for "summary" of a text field Needs work works for me using Drupal 10.1.4 (don't forget to force reload your browser page to allow new js to refresh).

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #3 works for me using Drupal 10.1.3

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

https://www.drupal.org/project/structure_sync/issues/3073810#comment-13215771 works me using Drupal 10.1.3, structure_sync 2.0.5 and menu_link_attributes 1.3.0
Thanks

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Couldn't find a way to avoid [0] to be replaced as a token at one step or another so I ended encoding the url so /search?f[0]=tags:[term:id] is written in ECA as /search?f%5B0%5D=tags%3A[term:id] and it worked.
Thanks @jurgenhaas anyway for your kind support for an non-related ECA issue.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Thanks, I've tried but the search and replace result shoud be stored in a temporary custom token to be used as url for the redirect action.
In that last action, I shoud use 'replace tokens' to be able to use my custom token and then [0] (token in token) is still removed from the string...

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

I face same kind of error using Drupal 10.1.2 with stock node full display mode (no Layout Builder), either with Olivero or a Radix child theme.
Disabling or removing the comment field from my content type display mode removes the error.

A temporary fix was to duplicate the field.html.twig template to field--node--uid.html.twig in my custom theme (or to write your own twig as long as the file exists).

Then another similar error yells about missing field--node--created.html.twig file.
Again, duplicating the field.html.twig template to field--node--created.html.twig fixes it.

I've also tried uninstalling manage_display module by curiosity: it removed the 'Written by' (uid) and 'Written on' (created) fields from content type form displays and view modes!!!

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Works for using Drupal 10.1.2

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Patch #22/#24 works for me using Scenario 1, views_data_export 8.x-1.3 (with xls_serialization addon) and Drupal 9.5.9
Thanks

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Here is a patch for suggested fix by marieAscomedia in #15
It works on Drupal 10 as well (still not fixed).
I don't know which of both patches is the cleanest way to fixed the issue: #2 or #24

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France
🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

@mansspams: issue #3130512 doesn't fix hotlinking Lotties
Browser still reports 404 for json while it's 302 with proper rewriting to production server for png/jpg/webp....

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

By experiencing the same issue with external_link_popup module which has been fixed the same as per comment #8 by joelpittet, attached patch adds the missing #[ReturnTypeWillChange] statement.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

@jurgenhaas:
The process I'm trying to set up should react on both "Insert content entity" and "Update content entity" events.
Basically, it's an email notification which recipent depends on the updated state.

For "update", the core patch you provided at https://www.drupal.org/project/drupal/issues/3346430 🐛 Wrong revision can be loaded into original in \Drupal\Core\Entity\EntityStorageBase::doPreSave Needs work works using Drupal 9.5.5, ECA 1.1.2 & PHP 8.1.6

For "insert", both before and after moderation states have the same value (the new one) but that may be designed by core as there is no previous state.
Anyway, I just need to add a test in the loop for new entities.

So, I can now check previous moderation state against new one on node update event, thanks a lot.

🇫🇷France PhilY 🇪🇺🇫🇷 Paris, France

Hotlinking fails when using Lottie animations which are saved as .json files (website using 'Media entity Lottie' contrib module to handle looties).

<lottie-player autoplay="autoplay" loop="" background_transparent="1" src="/sites/default/files/2023-01/lottie-01.json" background="transparent"></lottie-player>)

Production build 0.71.5 2024