@dhruv.mittal

Updated to the latest and code and removed the merge error.
Kindly review.

🐛 | Entity Print | FileSystemInterface replace behavior constants are deprecated

Comment 17 days ago →

🐛 | File Delete (D8/D9/D10) | "Adding non-existent permissions to a role is not allowed." error

Comment 17 days ago →

🐛 | Mailchimp | Error when installing mailchimp_signup module

I followed your instruction to reproduce the error but I didn't get any error while enabling the module using drush.
Please let me know if I'm missing something.
Thank you !!

🐛 | Mailchimp | Error when installing mailchimp_signup module

Reviewing it

🐛 | Taxonomy Menu | Main menu inaccessible after disabling Taxonomy Menu

Please review

🐛 | File Delete (D8/D9/D10) | "Adding non-existent permissions to a role is not allowed." error

🐛 | Taxonomy Menu | Main menu inaccessible after disabling Taxonomy Menu

Comment 21 days ago →

🐛 | Video | Implicitly marking parameter $site_path as nullable is deprecated

Comment 21 days ago →

@divyansh.gupta please review I have removed .gitlab-ci.yml file. Now all the changes are only related to this issue.

🐛 | Video | Implicitly marking parameter $site_path as nullable is deprecated

Working on it

✨ | Editor Advanced link | Add 'lang' attribute option

Merge conflicts need to be resolved

✨ | Autofill | Support string replace option on the autofilling

Automated tests are needed to test the feature.

🐛 | Mail Login | Regular login fails if "Enable login by email address" option is not enabled

I tested The MR.
It is working fine and removing error thus moving it to RTBC.

🐛 | Market Wave | Hamburger menu not working

Mentioned issue is present in the theme. After applying the MR issue is getting resolved.
Attaching the screenshot FYR.
Before

After

So Moving it to RTBC.

📌 | Quick Edit | Deprecated trait EntityReferenceTestTrait

Comment 24 days ago →

I'm not able to see any deprecated Trait related warning.
Would you check it once again or please provide screenshots for the same.
Thanks.

🐛 | Bynder | Add hook for Bynder remote media deletion handling

📌 | Geolocation Field | PHP Deprecated: Drupal\geolocation\LocationInputManager::getForm(): Implicitly marking parameter $default_value as nullable is deprecated, the explicit nullable type must be used instead in geolocation/src/LocationInputManager.php

🐛 | Inline Entity Form | Warning : foreach() argument must be of type array|object, null given dans inline_entity_form_close_all_forms()

🐛 | Client-side Hierarchical Select | PHP 8.4 - Implicitly nullable via default value null deprecation

Thanks for the patch
Working fine for me so moving it to RTBC.

📌 | Quick Edit | Deprecated trait EntityReferenceTestTrait

Comment 26 days ago →

🐛 | Mail Login | Regular login fails if "Enable login by email address" option is not enabled

Comment 28 days ago →

🐛 | SAML Authentication | Field description for value_map does not match actual behavior

Comment 28 days ago →

🐛 | Client-side Hierarchical Select | PHP 8.4 - Implicitly nullable via default value null deprecation

As patch has been deprecated and MR is easier to review so I raised MR.
Please Review

Comment 29 days ago →

🐛 | Link checker | "Don't treat these response codes as errors" not used

🐛 | SAML Authentication | Field description for value_map does not match actual behavior

🐛 | Link checker | "Don't treat these response codes as errors" not used

MR !102 has Merge conflicts and those need to resolved. So moving it back to Needs work for the same reason.

🐛 | Link checker | "Don't treat these response codes as errors" not used

📌 | WebP | Nullable types must be explicit

I agree that Nullable types must be explicit.
And changes looks fine so marking it to RTBC

📌 | WebP | Nullable types must be explicit

🐛 | CAPTCHA | captcha_point_load($id) need to validate the $id

🌱 | Patternkit | Patternkit 9.1.2 Release Plan

slucero → credited dhruv.mittal → .

🐛 | CAPTCHA | captcha_point_load($id) need to validate the $id

📌 | Drupal core | Never bypass validation when saving a config object

Please Review.

📌 | Drupal core | Never bypass validation when saving a config object

📌 | Geolocation Field | PHP Deprecated: Drupal\geolocation\LocationInputManager::getForm(): Implicitly marking parameter $default_value as nullable is deprecated, the explicit nullable type must be used instead in geolocation/src/LocationInputManager.php

Working on this

🐛 | Inline Entity Form | Warning : foreach() argument must be of type array|object, null given dans inline_entity_form_close_all_forms()

Thank you for your contribution @undersound3.
But as Patch is deprecated and MR is easy to review and apply, So I have converted your patch into MR.
Thank you

✨ | Text field formatter | Provide the option to specify the element class

📌 | Geolocation Field | PHP Deprecated: Drupal\geolocation\LocationInputManager::getForm(): Implicitly marking parameter $default_value as nullable is deprecated, the explicit nullable type must be used instead in geolocation/src/LocationInputManager.php

🐛 | Pathauto | Fatal error: Declaration of Drupal\pathauto\PathautoFieldItemList::computeValue() must be compatible with Drupal\path\Plugin\Field\FieldType\PathFieldItemList::computeValue(): void in contrib/pathauto/src/PathautoFieldItemList.php

Hey @mkalkbrenner
Thank you for the patch but would you please share the steps to reproduce.

🐛 | Book PDFs | Exception when used with contrib book

🐛 | Universal media formatter | Cannot load the "%s" entity with NULL ID

🐛 | Universal media formatter | Cannot load the "%s" entity with NULL ID

📌 | Language Cookie | Fix php 8.4 deprecations

Comment 2 months ago →

📌 | Bakery Single Sign-On System | Remove the "restrict access" label from the Bypass Bakery permission

Comment 2 months ago →

📌 | Language Cookie | Fix php 8.4 deprecations

📌 | Coffee | core_version_requirement: ^10.2 || ^11.0

Please review

📌 | Currency | PHP 8.4 implicit nullable is deprecated

I have successfully reproduced the issue.
And after applying the MR the issue is gone.
so Moving it to RTBC.

🐛 | Conditional Fields | When adding required to textarea with summary, both became required.

Hey impol!
This issue is fixed in the newer version of the conditional_fields and it no more persists.
Thank you !

📌 | Coffee | core_version_requirement: ^10.2 || ^11.0

🐛 | Password Separate Form | Deprecated function: trim(): Passing null to parameter #1 ($string) of type string is deprecated in Drupal\change_pwd_page\Form\ChangePasswordForm->validateForm() (line 148 of modules/contrib/change_pwd_page/src/Form/ChangePasswordForm.php)

🐛 | Password Separate Form | Deprecated function: trim(): Passing null to parameter #1 ($string) of type string is deprecated in Drupal\change_pwd_page\Form\ChangePasswordForm->validateForm() (line 148 of modules/contrib/change_pwd_page/src/Form/ChangePasswordForm.php)

Created MR against 2.x branch kindly review

🐛 | Password Separate Form | Deprecated function: trim(): Passing null to parameter #1 ($string) of type string is deprecated in Drupal\change_pwd_page\Form\ChangePasswordForm->validateForm() (line 148 of modules/contrib/change_pwd_page/src/Form/ChangePasswordForm.php)

🐛 | Password Separate Form | Deprecated function: trim(): Passing null to parameter #1 ($string) of type string is deprecated in Drupal\change_pwd_page\Form\ChangePasswordForm->validateForm() (line 148 of modules/contrib/change_pwd_page/src/Form/ChangePasswordForm.php)

Rebased the MR as it was some commit behind. and will create a new MR that will target 2.x

🐛 | Password Separate Form | Deprecated function: trim(): Passing null to parameter #1 ($string) of type string is deprecated in Drupal\change_pwd_page\Form\ChangePasswordForm->validateForm() (line 148 of modules/contrib/change_pwd_page/src/Form/ChangePasswordForm.php)

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

📌 | Currency | PHP 8.4 implicit nullable is deprecated

🐛 | Conditional Fields | When adding required to textarea with summary, both became required.