🇧🇪Belgium @Jonasanne

This fix has been included in 10.3.x so I guess this issue can me closed and assumed fixed?

Uploaded the diff of the MR for ease of use as a patch

I made a small mistake with the prev patch.
Goodmorning :D

Added a diff from merge request for ease of use as patch.

Thankyou @scott_euser for updating this.

pjonckiere → credited jonasanne → .

borisson_ → credited jonasanne → .

borisson_ → credited jonasanne → .

borisson_ → credited jonasanne → .

Rerolled #93

Reroll patch from #2 for 2.0.2

I accidentally uploaded the wrong patch.

This one works correctly for me at 10.3.2

Rerolled patch for 10.3.2

I Replace deprecated watchdog_exception() with \Drupal\Core\Utility\Error::logException()

- Inject logger service into the File class.
- Update logResults method to use \Drupal\Core\Utility\Error::logException() for logging exceptions.
- Ensure proper dependency injection and autoloading configuration.

PHPUNIT testing still fails and needs to be checked
We also need to wait on a new D11 ready release of drupal/monitoring

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

+1
Can we please have a new release D11 compatible?

Kind regards,

Hi Dieter,

I tested everything this morning, and it all seems to be working well in one of our projects. I've also created a merge request.

If there's anything else I need to do, please let me know.

Best regards,

I have made the summarized changes. However, it still needs to be tested with a valid Flexmail account since I don't have one at the moment.

Jonasanne → created an issue.

I did the changes and tested it with upgrade status.

Jonasanne → created an issue.

Uploaded patch here for ease of use

I have tested the MR and I can confirm this resolves the problem and the error is gone.

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

I tested this on 10.2.6 with php 8.1 (branch 2.0.x and branch 3.0.x) and on 10.3.0 with 8.3 while following the steps above and I do not have the error.

Looks great to me!

Jonasanne → created an issue.

I tested this PR inside configuration inspector and it looks good.

Jonasanne → changed the visibility of the branch CozyDrupalNerd-2.0.x-patch-a239 to active.

Jonasanne → changed the visibility of the branch CozyDrupalNerd-2.0.x-patch-a239 to hidden.

Jonasanne → created an issue.

Jonasanne → created an issue.

I updated the file with the correct naming conventions and tested it in config inspector.

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

I added the config schema for view area plugin and tested it via the configurator inspector.
No warnings or errors.

Jonasanne → changed the visibility of the branch 3376126-config-schema-missing to active.

Jonasanne → changed the visibility of the branch 3376126-config-schema-missing to hidden.

I like this proposition. I also updated the issue information.

I updated the check with the correct modules and tested this again.

I added the defensive check to see if menu module is enabled.

I removed the version from the library file

Jonasanne → changed the visibility of the branch 3457317-fatal-error-when to hidden.

Jonasanne → created an issue.

I added the icon to the field_ui form and encoded it to a base_64.

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

Rerolled the patch for 10.3.x

As proposed from @vlad.dancer I created a patch to check the block type.

Just checking in for an update.
Also, I'm happy to help out with co-maintaining if needed.

Jelle_S and I did some research together on how to reproduce this on a clean Drupal install:

1. Install and enable the menu_multilingual and block_field modules
2. Create a field on a node or a paragraph with type 'Block (plugin)' and set display mode to 'block field'
3. Edit the field or paragraph and set the field value to a menu reference
4. View the node

I updated the issue summary and this issue also accurs on 2.0.0

Rerolled patch

Patch reroll

Rerolled patch

Jonasanne → created an issue.

In this code modification, I enhanced the UserMailUniqueValidator class in the UserMailUniqueValidator.php file. The changes involve refactoring the constructor to leverage property promotion, which is a feature introduced in PHP 8.0. This helps simplify the code by declaring and initializing class properties directly within the constructor parameters.

Here's a breakdown of the changes:
Property Promotion in Constructor:

I replaced the individual property declarations and assignments in the constructor with property promotion.
Introduced property promotion for the AuthmapInterface, OpenidConnectRealmManagerInterface, EntityFieldManagerInterface, and EntityTypeManagerInterface properties.

Constructor Parameter Removal:
Removed the unnecessary parameters from the constructor, as property promotion takes care of both declaration and initialization.

Dependency Injection Enhancement:
Updated the create method to include the new dependencies (EntityFieldManagerInterface and EntityTypeManagerInterface) when instantiating the class.

Jonasanne → created an issue.

I created a small patch for this issue.

Reroll patch #46 for 10.2.x

Rerolled patch to 10.2.x

Patch rerolled for D10.2

Has there been an update on this issue?
I still have this problem in D10.1.4.