🇮🇳India @Sivaji_Ganesh_Jojodae

Updated to reflect comment #55.

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

sivaji_ganesh_jojodae → changed the visibility of the branch 3514936-modulesuninstallform.php-assigns-string to hidden.

Closing due to branch issue mentioned in #16.

Fork #3514936 allows to create a features branch from 11.x.

sivaji_ganesh_jojodae → created an issue.

+1 for @prudloff.

If you grep there are several routes with _access: 'TRUE' as appropriate.

grep -rni \_access: core/modules/*/*.routing.yml | grep -i TRUE

There are several routes

core/modules/big_pipe/big_pipe.routing.yml:9:    _access: 'TRUE'
core/modules/filter/filter.routing.yml:7:    _access: 'TRUE'
core/modules/image/image.routing.yml:55:    _access: 'TRUE'
core/modules/system/system.routing.yml:7:    _access: 'TRUE'
core/modules/system/system.routing.yml:15:    _access: 'TRUE'
core/modules/system/system.routing.yml:23:    _access: 'TRUE'
core/modules/system/system.routing.yml:31:    _access: 'TRUE'
core/modules/system/system.routing.yml:342:    _access: 'TRUE'
core/modules/system/system.routing.yml:352:    _access: 'TRUE'
core/modules/system/system.routing.yml:360:    _access: 'TRUE'
core/modules/system/system.routing.yml:400:    _access: 'TRUE'
core/modules/system/system.routing.yml:407:    _access: 'TRUE'
core/modules/system/system.routing.yml:414:    _access: 'TRUE'
core/modules/system/system.routing.yml:421:    _access: 'TRUE'
core/modules/system/system.routing.yml:459:    _access: 'TRUE'
core/modules/system/system.routing.yml:479:    _access: 'TRUE'
core/modules/system/system.routing.yml:489:    _access: 'TRUE'
core/modules/system/system.routing.yml:519:    _access: 'TRUE'
core/modules/system/system.routing.yml:526:    _access: 'TRUE'
core/modules/user/user.routing.yml:138:    _access: 'TRUE'
core/modules/user/user.routing.yml:148:    _access: 'TRUE'
core/modules/user/user.routing.yml:192:    _access: 'TRUE'
core/modules/user/user.routing.yml:233:    _access: 'TRUE'
core/modules/views/views.routing.yml:6:    _access: 'TRUE'

I don't find reference to original interface name change. However I can see usage change in issue #2002134 → commit id f6161fad

The existing branches are outdated. I'm not able to create a new branch from 11.x.

Hitting create branch button throws error

Failed to create branch '3311358-module-uinstall': invalid reference name '11.x'.

sivaji_ganesh_jojodae → changed the visibility of the branch 3311358-module-uninstall-form to hidden.

sivaji_ganesh_jojodae → changed the visibility of the branch 3311358-module-uninstall-form to active.

sivaji_ganesh_jojodae → changed the visibility of the branch 3311358-module-uninstall-form to hidden.

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

I think the feedback is to change as follow,

- @trigger_error('typedConfigManager() is deprecated in drupal:11.2.0 and is removed from drupal:12.0.0. This method is no longer in use instead $this->typedConfigManager to be used. See https://www.drupal.org/project/drupal/issues/3477616', E_USER_DEPRECATED);
+ @trigger_error('typedConfigManager() is deprecated in drupal:11.2.0 and is removed from drupal:12.0.0. The replacement is to use the $this->typedConfigManager as is done in ::validateForm(). See https://www.drupal.org/project/drupal/issues/3477616', E_USER_DEPRECATED);

I encountered the same issue. I had to clear cache for the new hook_mail() implementation to get recognized by Drupal.

Good catch @nod_

Interesting issue. The bug indeed is still around. I can reproduce the issue with the attached test module.

Attached are the screenshots of before and after the MR.

The patch looks straightforward and pipeline looks good as well.

Inside core/modules/basic_auth/tests I can still find the usage of \Drupal::. Should we change the following occurances as well?

./modules/basic_auth_test/src/BasicAuthTestController.php:16:    \Drupal::state()->set('basic_auth_test.state.controller_executed', TRUE);
./modules/basic_auth_test/src/BasicAuthTestController.php:25:    \Drupal::service('page_cache_kill_switch')->trigger();
./modules/basic_auth_test/src/BasicAuthTestController.php:28:      '#markup' => \Drupal::state()->get('basic_auth_test.state.controller_executed') ? 'yep' : 'nope',

Drupal 7 is EOL and I don't think this affects D10+.

sivaji_ganesh_jojodae → changed the visibility of the branch 3504629-process-all-view-arguments to hidden.

The MR looks fine to me.

> changed target branch from 11.x to 11.1.x

Should be 11.x

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

The issue seems to exist in both Drupal 10.x and 11.x.

Could someone share any inputs or direction on how to resolve?

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

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

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

Updated the MR to reflect comment #34 and earlier.

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

The updated code visually looking good to me.

+1 for RTBC.

Created MR to reflect the above comments.

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

When $element['#title'] is set, $element['#title'] should be used not NULL.

@joachim, noted. Thank you!

It's a quick-fix and pipeline has passed already.

+1 to mark this issue as duplicate in favor of #2972846.

Patch to be updated for core policy change to meet the deprecated code removal → .

Pipeline has failed with following error,

Behat\Mink\Exception\ResponseTextException: The text "Has taxonomy term (= term1)" was not found anywhere in the text of the current page.

.

Thanks. I will fix the comment line. But there is an error from unit test. See below,

I've addressed the comment posted on MR.

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

Patch #3 re-rolled for branch 11.x.

sivaji_ganesh_jojodae → changed the visibility of the branch 3504567-breadcrumb-ada-fix to hidden.

sivaji_ganesh_jojodae → changed the visibility of the branch 3504567-breadcrumb-ada-fix-2 to hidden.

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

On a related note, shouldn't we use wildcard /* instead of empty array [] to indicate the global visibility to be inline with the page visibility settings?

sivaji_ganesh_jojodae → changed the visibility of the branch 2345807-default-time-zone to hidden.

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

@joachim, not sure what I'm missing. Any inputs on the failing test results?

Working on merge request.

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

Drupal 7 security support has ended as of 5 January 2025. This is the final extension of Drupal 7 end of life – Drupal 7 will no longer receive security updates or compatibility updates

See Drupal 7 End of Life → .

One of the views has a URL ending with a space. I used ctrl+F on pattern "[space])" to find the occurrence.

See the screenshot below,

Seems like it was configured incorrectly in the first place.

Facing the same issue with the views admin page. Downgrading symfony/http-foundation to 6.4.12 makes the error go away. Not sure what will be the reliable fix for the issue.

greggles → credited sivaji_ganesh_jojodae → .

I am attaching a patch for some interim fixes.

Sivaji_Ganesh_Jojodae → created an issue.

On a related note, we are getting the following error with the flatten method.

TypeError: Drupal\rest_export_nested\Plugin\views\display\RestExportNested::flatten(): Argument #1 ($data) must be of type array, null given, called in /usr/share/nginx/html/project-consumer-drupal/modules/contrib/rest_export_nested/src/Plugin/views/display/RestExportNested.php on line 45 in Drupal\rest_export_nested\Plugin\views\display\RestExportNested->flatten() (line 92 of /usr/share/nginx/html/project-consumer-drupal/modules/contrib/rest_export_nested/src/Plugin/views/display/RestExportNested.php).

I confirm that #7 fixes the issue.

Unable to apply the patch from merge request.

I've attached something that works for me.

Steps in comment #17 helped to work around the issue. Thanks @rduplock!

We are requiring this feature in our project. Would like to see this added shortly.

Thanks a lot. I will check commerce_sync → and let you know.

Let me know if anyone has any inputs. Any help or direction will be much appreciated.

I think we should remove the core and change the core_version_requirement to support only 9 & 10 (core_version_requirement: ^9 || ^10) since D8 is no longer supported anyway.

Note that the module also supports Drupal 8 which requires core to be present.

core_version_requirement: ^8.8 || ^9 || ^10

Sivaji_Ganesh_Jojodae → created an issue.

I have similar issues. Getting 504 timeout in the interim of the batch process after 3 minutes of starting. Not sure if it is the timeout on the client (guzzle, php-fpm, nginx, or database) or the server's.

Someone from our team indicated checking HTTP 499 which is the client's initiation to discard the request from processing further and the server could respect the same.

The issue seems to be with the entity reference field,

If you have access to your server and PHP configuration, you can extend the max_execution_time in the php.ini.

If there is the usage of PHP-FPM to serve PHP scripts, you have also to adjust the request_terminate_timeout in your FPM config to the same value.

If PHP-FPM has a reverse proxy like Nginx in the front to the user, the fastcgi_read_timeout also needs to be set to the same value, and to all other proxies in the front, you may adjust the proxy_read_timeout value.

Anyway I'm going to try https://www.drupal.org/docs/contributed-modules/entity-share/troubleshoo... →

@Igor Mashevskyi, let me know if there is anything I can help with. Also, could you provide some direction on resolving the issue?

Thanks for working on drupal 10.

Uncaught PHP Exception Error: "Class "Drupal\\commerce_cart_api\\Normalizer\\FieldNormalizer" not found" at core/lib/Drupal/Component/DependencyInjection/Container.php line 259

We are getting the above issue after the upgrade. The site fails to load with a couple of other errors.

Thanks a lot. I can help with testing.

We have a complex structure of products and variations with reference to other variations and paragraphs.

Sivaji_Ganesh_Jojodae → created an issue.

Sivaji_Ganesh_Jojodae → created an issue. See original summary → .

#36 works. Downgrading drupal/coder (8.3.21 => 8.3.13).

Sivaji_Ganesh_Jojodae → created an issue.

To fix this issue, uninstall login_emailusername and use mail login → instead.

pameeela → credited Sivaji_Ganesh_Jojodae → .