Account created on 22 April 2014, almost 11 years ago
#

Merge Requests

More

Recent comments

🇧🇪Belgium joevagyok

Thanks for the review comments. The MR is in a work in progress.

🇧🇪Belgium joevagyok

joevagyok made their first commit to this issue’s fork.

🇧🇪Belgium joevagyok

I think this issue should be closed in favour of Drush 13 update where Drush 12 should be the minimum version required. https://www.drupal.org/project/extra_field/issues/3511072 📌 Drush 13 compatibility Active

🇧🇪Belgium joevagyok

The MR had missing pieces from the patch, I pushed it.
I attached a patch file for composer patching against 1.19 version.

🇧🇪Belgium joevagyok

joevagyok made their first commit to this issue’s fork.

🇧🇪Belgium joevagyok

joevagyok made their first commit to this issue’s fork.

🇧🇪Belgium joevagyok

Thanks for the fix, can we have some assertions for this please?

🇧🇪Belgium joevagyok

Thank you for the proposal! Test coverage for this would be much appreciated!

🇧🇪Belgium joevagyok

@neclimdul can we have a merge request with this please?

🇧🇪Belgium joevagyok

Also some basic test coverage of the code would be much apprechiated!

🇧🇪Belgium joevagyok

I like the proposal and the patch! Can we get a merge request with it please? :)

🇧🇪Belgium joevagyok

The solution is looking good for me! Thanks @scott_euser

🇧🇪Belgium joevagyok

Opened another MR against the 3.x branch as well.
Uploading the patch file against the 3.0.0-rc20 release for composer patching.

🇧🇪Belgium joevagyok

Thank you all for the contribution!

🇧🇪Belgium joevagyok

Removed the big border to have only the image.

🇧🇪Belgium joevagyok

Thank you for the simplified version, I find it way too simple. :)

I fixed the numbering on the previous.

🇧🇪Belgium joevagyok

I recently merged a fix related to this. I believe it is fixed by 3478937 🐛 CkEditor5 Counter gets attached in the wrong place after ajax call Active .

Can you please confirm?

🇧🇪Belgium joevagyok

I understand that you would expect from the config to act as a definitive limit when turned on and truncate the string once it is switched on.
However, it implies that the field can hold more characters just well, before the option was not turned on, so having the string truncated on form load, would render part of your pages unreadable due to the loss of strings after the save operation. I find that a bit harsh and disturbing from the editor point of view.

🇧🇪Belgium joevagyok

joevagyok made their first commit to this issue’s fork.

🇧🇪Belgium joevagyok

This issue is not close to be merged and here are some reasons why:

  • The MR still introduces a bug which creates more cloned entities in certain conditions that in should. This can be only detected with automated tests asserting the amount of entities in the database after each clone.
  • The base class meant for content entities should not contain layout builder specific logic, not every site has layout builder.
  • There are certain entity types and entity reference combinations that are not entirely clear how we should proceed.
  • The MR needs automated tests.

These have to be addressed before we make a review.

🇧🇪Belgium joevagyok

Thanks for reporting the problem. It seems that the new version of AI module introduces change that the plugin needs to accommodate.

Currently, I am away for couple of weeks, but after I will fix the issue, unless someone contributes the fix before.

🇧🇪Belgium joevagyok

Thank you for the contribution.
Please add a test case functional test case to validate we actually have the issue and preferably the fix in a separate commit to show it fixes the problem.

Production build 0.71.5 2024