Redirect correct language page after node save

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

Still happening, updated IS with some clearer steps to reproduce.

Re-rolled #4

I have tested the issue in drupal version 10.1.x (https://simplytest.me/). Applied above patch and i got the solution. Adding Screenshots for the reference . Need RTBC+1

The latest patch does not seem to apply anymore against core 10.2.

Supports 10.2.x

@leo liao thanks for the 10.2 patch. Any code fixes are applied to the latest dev branch - 11.x.

The https://www.drupal.org/files/issues/2024-03-01/3096570-22.patch → patch applies cleanly to 11.x. I have created the MR 7350 to move this ticket along.
All patches on this issue have been hidden. The MR should be used for any updates.

PENDING WORK - see https://www.drupal.org/project/drupal/issues/3096570#comment-14954234 🐛 Redirect correct language page after node save Needs work

1. Need a change record to be created and trigger_error updated
2. Needs test coverage of trigger_error.

PENDING WORK

1. DONE - Need a change record to be created and trigger_error updated. See Draft CR at https://www.drupal.org/node/3438992 →
2. PENDING - Needs test coverage of trigger_error.

Attached is static patch of the MR 7350 for 11.x and a patch adapted for 10.3.x

Cleaner patch for 10.3.x with no tests

Could we use the node language instead? In that case, we don't need to introduce a new language service too.
['language' => $node->language()]

@peterwcm thank you. Using the node language worked for me and is a lot cleaner. I will update the PR.

MR passed. Attached is an equivalent 10.3.x for any builds still using that.

let's try adding that 10.3.x patch again

10.3.x patch with no tests

Thanks @recrit
The changes look good. The issue applies to new node creation and editing the source language of an existing node, which I think should be the default behaviour. Tested and it worked as suggested.

Committed and pushed 6c01bb31917 to 11.x and f86f182a989 to 11.1.x. Thanks!

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