When create a new node for example article and after adding another language. But after that, you try to edit another language you always follow to English version.
So that's problem, you can't edit another language.

I did the small modification for generating the link for edit another language.

for example:
https://drupal.dev/node/NODE_ID/edit - now, but must be https://drupal.dev/de/node/NODE_ID/edit

Comments

IvanBerezhnov created an issue. See original summary.

ivan berezhnov’s picture

Title: Edit link in node/%node/translate wrong for language » Edit link in node/%node/translate wrong for another language
ivan berezhnov’s picture

Issue summary: View changes
ivan berezhnov’s picture

Title: Edit link in node/%node/translate wrong for another language » Node edit link in node/%node/translate wrong for another language
StatusFileSize
new621 bytes
ivan berezhnov’s picture

Issue summary: View changes
ivan berezhnov’s picture

Status: Active » Needs review

Status: Needs review » Needs work
joseph.olstad’s picture

Ivan, I rerolled your patch. Haven't actually analysed it though.

To avoid such problems when creating patches please refer to "Making a Drupal patch with git" documentation.

Resubmitting.

joseph.olstad’s picture

Version: 7.x-1.2 » 7.x-1.x-dev
Status: Needs work » Needs review

was this actually intended for 7.x-1.2 ? (very old release)
or 7.x-1.x dev?

joseph.olstad’s picture

a human review (or two) would be appreciated.

ivan berezhnov’s picture

I checked your patch all ok. Thank you!

joseph.olstad’s picture

Hi @IvanBerezhnov, by the sounds of it, I like this change, I just haven't had time to test it yet. I'll hold off committing it to dev until at least after 7.x-1.15 is released.

joseph.olstad’s picture

seems like a good change to make. Lets target this for 7.x-1.16, however put it in dev after 7.x-1.15 and let it sit in dev for a while.

joseph.olstad’s picture

Status: Needs review » Fixed

7.x-1.20 just released, (not with this though)

put this in dev branch for possibly 7.x-1.21 (next release?) when? not sure.

joseph.olstad’s picture

Status: Fixed » Needs review

going to revert this for now

  • joseph.olstad committed e0a46aa on 7.x-1.x
    revert Issue #2831861 by Ivan Berezhnov, joseph.olstad: Node edit link...
joseph.olstad’s picture

Status: Needs review » Fixed

this was released in 7.x-1.22

Status: Fixed » Closed (fixed)

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

joseph.olstad’s picture

Status: Closed (fixed) » Needs review

Actually it was reverted

joseph.olstad’s picture

It'd be nice to have an extra review or two for this patch. appreciate it.

joseph.olstad’s picture

I had a another look at this patch, it seems like this patch should be the right thing to do.

It was reverted as a precaution, there was an unrelated issue so I had backed off some recent changes just to be safe.

Now reviewing this, it looks like it makes sense but more reviews would be nice.