🇲🇦Morocco @h_kac

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

@Darvanen thank you

Hello,
I fixed some errors shown at level 2 to level 6 can u check that ?

For the error below, I have checked the interface containing the signature of the function "get" we have only one param "CountryCode", I think we should remove the second param in the class

------ ------------------------------------------------------------------------------- 
  Line   src/Plugin/GraphQL/DataProducer/AddressFormatRepository.php                    
 ------ ------------------------------------------------------------------------------- 
  98     Method                                                                         
         CommerceGuys\Addressing\AddressFormat\AddressFormatRepositoryInterface::get()  
         invoked with 2 parameters, 1 required.                                         
 ------ ------------------------------------------------------------------------------- 

What do you think ?

For me, when I started working with Drupal at the beginning, I always confuse between plugins and modules.
I used to work with Wordpress CMS and what we call "Module" in Drupal is called "Plugin" in WP.
Drupal use both terms "Module" and "Plugin", but what plugin means in Drupal is different : it allows to extend fonctionnalities of a module...

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

we had the same error when we upgraded to php8.1, after we investigated we found that we implemented hook_views_view_field where we changed $variables['output'], instead of having a string we had an array, and that explains why we had the error.
After fix it, everything works as expected.

I faced this issue with drupal 9.3.22 when we upgraded from PHP 7.3 to php 8.1. The comment #21 worked for me. I just saved the page without making any changes and the error was gone. When I do "drush locale-update" no more error are shown.

I tried the path #14 with Drupal 9.3.22 and module redirect 1.7 I am getting an error : You have requested a non-existent service "entity.manager". in Drupal\Component\DependencyInjection\Container->get() when I changed it to "entity_type.manager " in createInstance method it works

h_kac → created an issue.

@mike yes I run the tests using phpunit and all is Green.

I updated services, form Class and the controller to use Dependency Injection.

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

suggestion : for the commit with id=0cdd9537 we can remove the "else block" in this file FormStateInterface; and keep just the "return" statement.