🇫🇷France @mitsuko

#8 works always with 8.x-3.24 (and Drupal 11)

Hi @randalv,

Thanks for clarifying and adding the functionality in the last release 1.0.18.
It works like a charm and there is no reason to do more !

Kind regards

Hi @randalv,

First, thanks for your response and your commits !

Overall, your way of solving the problem suits me very well. Even if I find the functionality is more implicit like that, and there is potentially the risk that the user validates the form without choosing any role...

Just for my information, I don't understand your comment : "The authenticated role *should* always be selected, so removing it from the automatically selected roles might produce strange results.".

Other roles necessarily inherit the permissions given to the role "authenticated role", so if this role is selected or not makes no difference. Besides, when I tried with my patch, everything worked perfectly when I only selected the "content editor" role... but maybe I'm missing something in the code...

Once again, thank you !

Digging a little deeper to see that this was the intended behavior. I opened a merge request to add this functionality.

As much as the private browser window do the trick for the anonymous role, it seems essential to me to be able to test as an "Authenticated user" only.

mitsuko → created an issue.

Hi,

I just try this module with a drupal 11 installation and this patch.

There is a bug when I use views formatter for my entity reference field which accept multiple values, only the first value is displayed.
(Views settings are OK, pagination's parameter is "display all values")

I don't know if it's specific to Drupal 11 yet...

Hi Samit

Thanks for pointing out this info.
You're totally right, the problem is fixed with MR 20 !

Sorry for missing out

The module is functional with the patch above but the config tabs "anonymous" and "Authenticated Behavior" give the following errr :

ArgumentCountError: Too few arguments to function Drupal\Core\Form\ConfigFormBase::__construct(), 1 passed in /var/www/html/web/modules/contrib/r4032login/src/Form/AnonymousSettingsForm.php on line 35 and exactly 2 expected in Drupal\Core\Form\ConfigFormBase->__construct() (line 43 of core/lib/Drupal/Core/Form/ConfigFormBase.php).

Mitsuko → created an issue.

Hi,

I post here for others.
I had the same error with the 4.0.5 release of select2 library.
Update to latest stable release of select2 (4.0.13) did the trick !

Reply to myself, brooke_heaton and other folks who could land here :

be careful to delimiter :

If you have multivalues field like that : "Apple, Banana, Cherry"
Delimiter must be ", " not ","

The extra space avoid bad duplicate behavior... which is ultimately a understandable behavior.

Maybe this issue could be closed with the "Work as designed" statut ?

Hi Everybody,

I'm a little surprised

5 years since this issue was open and I have the same result/bug.
When I use the entity_generate plugin, I get one duplicate by term...

As the nodes (parent's entity) are distributed over the 2 terms each time, the plugin is completely unusable as is.

So I wonder if this is a true bug or if I'm missing something?

Modules involved in my migration process :

• Migrate
• Migrate Plus
• Migrate CSV Source
• Migrate Tools

Thanks a lot for any clue...

Mitsuko → created an issue.

Version 8.x is no longer even displayed on the module presentation page because it is only compatible with Drupal 9, which is no longer maintained... :-(

I can confirm the issue on a fresh drupal 10 installation.
So , maybe the issue's priority is not high enough ?
Patch works for me too !

Hello there,

Can someone commit the patch ?
It works like a charm just with this modification.

Thank you

Hi maxwellkeeble,

First, thanks to share this module. It's very useful !

I wanted to know if you were planning an update to the development branch for Drupal 10 in the more or less near future ?

Thanks a lot for your response

Hi artmania

Thank you for your explanations.
I must confess that I haven't made extensive use of this module so far.

My feeling is that there are already a lot of options and that the interface is not super light with the side panel of Layout Builder.
So I'm going to take my time before adding new elements...

if I understand correctly "margin options" and "preserve colors" are mainly for section/layout so the "Layout builder Styles" module seems the way to follow for me (not teste so far).

For grid reverse, maybe a free extra classes field could do the trick...

I fixed supported versions and background-image error (not related to jquery) with the new release, so I will close this issue but we can continue this discussion...

Update the patch for drupal 10 compatibility

Hi artmania,

Once again, thank you for your feedbacks !
I'm glad you use this module too.

I hadn't planned to spend a lot of time on this one, that's why there is only a dev version so far...
... But I will take a look the coming days to have a stable version, compatible with Drupal 10.

I like your idea to improve this module and would try to implement them.

Sponsoring is welcome ! You can contact me in MP if you want talk about that.

Thanks

Hi artmania,

Thanks for your constant review and sorry for that.
I've been too confident on the patch provided...

Can you tell me, if the new release 8.x-1.9 works as expected

Thank you both for the cleanup...

Mitsuko → created an issue.

Mitsuko → made their first commit to this issue’s fork.

Hi artmania,

Thank your for your review. It was in my ToDo' list.

I just publish a new release officially compatible with D10.
I have updated the readme to include your emark about use in views.
(This behaviour is logic because the module override field.html.twig)

Adapt the variables to fix to xampp 8.1.x

Great news !

Thank you for your quick answer.
Yes, it works with the 2.0.5 release !

Yes.

Structure Sync 2.04
Drupal 10.0.5
PHP 8.1.11

Mitsuko → created an issue.

Mitsuko → created an issue.

Sorry the problem was theme related, the ommerce module is not in question, it came from an evolution of the theme.
All my apologies

Mitsuko → created an issue.

Thank you !