Argument #1 ($type) must be of type string, null given

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

According to autologout_post_update_8014 this should be set as 'status'. I wonder if it's possible for this config to be updated to still be empty, without digging through the form/config we could set a fallback in case the $status is empty?

MR raised.

I can see the MR got failed. is it possible to share any patch for the issue?

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

Hi @jrglasgow,
I've made the necessary updates to merge request !56, resolving all the CS errors. As you can see, PHPCS now has a green tick across the board.
Thanks

hi amandeep,

will the PHPCS issue resolve the issue?

@soutams, the error for MR! is still there. But there were some cs warning which was a quickfix, which I have made. I am yet not aware about the unit test cases. So I worked only on the some warnings. Please & Thank You!!

The failure is because, I believe, the D9 PHP unit tests are using PHP7.4.

thanks for your comment.

We are using drupal 10 and php version is 8.1.24.

Can you please provide a patch for the fix?

Sure, as attached (this will include the phpcs changes from @Amandeep123)

"Hello @Rajeshreeputra, your code successfully achieved the green checkmark on D9. Thank you.

Simplified the code and committed.

Automatically closed - issue fixed for 2 weeks with no activity.