@dhruv.mittal

dhruv.mittal → created an issue.

Comment 14 days ago →

Comment 14 days ago →

I am able to reproduce the issue using config inspector
also attaching the screenshot of config inspector
after applying the MR error was not there so moving it to RTBC

Comment 14 days ago →

Reviewing it

Got it, providing MR soon for the same soon !

I've checked .module file where simplify_form_alter hook present but it's empty.
Thank you

Working on it

Comment 21 days ago →

Comment 21 days ago →

Comment 22 days ago →

I've tried to reproduce the issue and successfully reproduced it.
After reproducing the error MR is able to resolve the error.
Attaching the Screenshots for the reference and moving it to RTBC.

Comment 23 days ago →

Moving it to RTBC as I was able to reproduce the error and patch was resolving the error but forgot to move it RTBC earlier.

Comment 23 days ago →

Reviewing it

Comment 23 days ago →

Comment 24 days ago →

I've applied the MR and warnings are removed.
So moving it to RTBC

Comment 25 days ago →

Reviewing it

I can confirm that this warning persist and after applying the MR the warning is resolved.

Thank you @lawxen for quick merge, but I think you forgot to assign credits.
Thank you :)

Reviewing it

Resolved conflicts please review

Hey @ccjjmartin
Thank you for you contribution
But In Raised MR pipeline is failing, but in targeted branch it's all green.
So for the same reason putting it back to needs work

Working on it

Working on it

Working on it

✨ | Status pages | Return proper 404 and 403 status codes

Reviewing it

🐛 | Key | Support multivalue key type (arrays) in key overrides

✨ | Domain | Add "Is all of" exposed filter (extend ManyToOne filter plugin)

Able to see the "Is all of" button after applying the MR
So moving it to RTBC
Thanks

✨ | Domain | Add "Is all of" exposed filter (extend ManyToOne filter plugin)

Reviewing it

📌 | Node Access Rebuild Progressive | Missing drush/drush dependency

📌 | Features | Fix PHP 8.3/8.4 issues

But in the targeted branch PHPunit tests are passing but in your Merge Request Phpunit tests are failing.

🐛 | Google Tag | Environment ID max length should be extended

✨ | Webform | Make form_file_limit error message configurable

Please review

✨ | Webform | Make form_file_limit error message configurable

📌 | Features | Fix PHP 8.3/8.4 issues

unit tests are failing
So moving it back to the Needs work\
Thanks

📌 | Node Access Rebuild Progressive | Missing drush/drush dependency

🐛 | Config Pages | entity.config_pages.edit_form is never used

Thank you for approving the merge request but I think you forgot to assign credit :)

🐛 | Upgrade Status | Doesn't like !project_page nor !project_issue_queue

Please review

📌 | Features | Fix PHP 8.3/8.4 issues

Reviewing it

🐛 | Upgrade Status | Doesn't like !project_page nor !project_issue_queue

🐛 | Paragraphs Sets | PHP 8.3 Deprecation Notice: Increment operator on non-numeric types is deprecated in paragraphs_sets module

🐛 | Paragraphs Sets | PHP 8.3 Deprecation Notice: Increment operator on non-numeric types is deprecated in paragraphs_sets module

✨ | Configuration Read-only mode | Save settings button disabled (as it should be), but Delete button still works

✨ | Configuration Read-only mode | Save settings button disabled (as it should be), but Delete button still works

Hey @alberto56
I've tried to reproduce the issue and I got to know that delete button is already disabled if you click on delete button then it will redirect to confirmation page where you can easily see, delete button is disabled.
Thanks

🐛 | Orejime | Schema errors for orejime.settings

🐛 | Drupal core | Warning: Undefined variable $source_name in Drupal\content_translation\Controller\ContentTranslationController->overview()

I can confirm this issue persists with provided steps to reproduce
and patch is able to resolve the issue.
Thanks for your contribution @johnatas
Moving it to RTBC

🐛 | Drupal core | Warning: Undefined variable $source_name in Drupal\content_translation\Controller\ContentTranslationController->overview()

Apologies assigned mistakenly
so unassigned myself

🐛 | Orejime | Schema errors for orejime.settings

Reviewing it

🐛 | Views PHP | Deprecate curly brace syntax for accessing array elements

Indeed the use of curly braces {} for array or object access, which has been deprecated in PHP 7.4 and removed entirely in PHP 8.0.
so for the same moving it to RTBC

🐛 | Views PHP | Deprecate curly brace syntax for accessing array elements

Reviewing it

🐛 | Page Manager | Route "entity.page_variant.edit_form" does not exist.

Now I can confirm this issue persists and able to resolve the issue using the patch
Thank you

✨ | Domain | Add views filter "Available on current domain" for users

Please review

✨ | Domain | Add views filter "Available on current domain" for users

Working on it

🐛 | Domain | Domain access: TypeError in hook_node_grants can completely crash site

✨ | Domain | Permission allows access to view domain entity

✨ | Domain | Permission allows access to view domain entity

🐛 | Domain | Domain access: TypeError in hook_node_grants can completely crash site

✨ | Domain | Permission allows access to view domain entity

🐛 | Page Manager | [PHP8.4] Misc null deprecation warning on a few files

Changes looks good to me so moving it to RTBC

🐛 | Page Manager | Route "entity.page_variant.edit_form" does not exist.

Hey @karavkars
I tried to reproduce the error following the steps to reproduce
But could not able to do so I'm attaching the Video for your reference please have a look and Let me know If I'm missing something
Thank You

🐛 | Page Manager | [PHP8.4] Misc null deprecation warning on a few files

reviewing it

🐛 | Page Manager | Route "entity.page_variant.edit_form" does not exist.

🐛 | Menu Manipulator | Error when navigation to settings form

I'm able to reproduce the error and after applying the MR error is resolved. screenshots are attached FYR
So moving to the RTBC.

🐛 | Menu Manipulator | Error when navigation to settings form

Reviewing it

🐛 | Link checker | Rename config value blacklist to blocklist

Instead of changing the blacklist to blocklist, We can just make a cspell file in which we can ignore blacklist, blacklisted, blacklists, whitelist word
Let me know if this solution sounds good then I'll proceed.

🐛 | Babel | Service Container Anti-Pattern

Changes look good to me as use of \Drupal::service() is not ideal way.
So Moving it to RTBC+.
Thank you

🐛 | Babel | Service Container Anti-Pattern

🐛 | Media file delete | Call to a member function getDisplayName() on null in Drupal\media_file_delete\Form\MediaDeleteForm->buildForm() (line 58 MediaDeleteForm.php)

Reviewing it

🐛 | Link checker | Rename config value blacklist to blocklist

🐛 | Social Auth Roles | user_roles() deprecated in D11.

please review

🐛 | Social Auth Roles | user_roles() deprecated in D11.

✨ | Paragraphs Modal Edit | Make modal dialog width configurable

🐛 | Media file delete | Call to a member function getDisplayName() on null in Drupal\media_file_delete\Form\MediaDeleteForm->buildForm() (line 58 MediaDeleteForm.php)

Added a configuration Form and Modal width configurable.

🐛 | JSON:API Resources | Use own _auth if configured

✨ | Paragraphs Modal Edit | Make modal dialog width configurable

🐛 | Parameters | PHP 8.4 compatibility

Please review

🐛 | Parameters | PHP 8.4 compatibility

🐛 | Display Suite | PHP 8.4 - Implicitly nullable via default value null deprecation

🐛 | JSON:API Resources | Code results in "array_merge() does not accept unknown named parameters"

Changes looks good to me, so moving it to RTBC

🐛 | JSON:API Resources | Use own _auth if configured

🐛 | Anti-Duplicates | ArgumentCountError: Too few arguments to function Drupal\Core\Form\ConfigFormBase::__construct()

Created MR as Remaining task suggested and moving it to needs review

💬 | Paragraphs Previewer | PHP 8.4 compatibility issue

Changes looks good to me, So moving it to RTBC

📌 | Varnish purger | composer require 'drupal/varnish_purge' does not install dependencies