Node already has translation" = "Drupal\node\NodeTranslationController" in annotation

CommentFileSizeAuthor
#3 1977790-3.patch1.35 KBandypost
#1 1977790-1.patch591 bytesandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Status: Active » Needs review
Issue tags: +D8MI
FileSize
1977790-1.patch591 bytes

Let's see

Status: Needs review » Needs work

The last submitted patch, 1977790-1.patch, failed testing.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Status: Needs work » Needs review
FileSize
1977790-3.patch1.35 KB

Seems the test is broken too.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
CreditAttribution: plach commented
Status: Needs review » Reviewed & tested by the community

Looks good, thanks.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented

BTW the change that left this dead code around was in http://drupal.org/node/1945906.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed http://drupalcode.org/project/drupal.git/commit/5578271 and pushed to 8.x. Thanks!

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Updated issue summary.