Closed (outdated)
Project:
Drupal core
Version:
6.x-dev
Component:
language system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
11 Mar 2012 at 13:52 UTC
Updated:
2 Mar 2016 at 22:18 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
JThan commentedThis is my first patch uploaded, so please help me with learning if I did anything wrong. Thank you.
This is the patch for Drupal 8.
Comment #2
JThan commentedAnd here is one for Drupal 7.
Comment #3
JThan commentedSorry, forgot to set the status.
Comment #4
JThan commentedOkay, I did not know that the TestBot is testing against the version the issue is set to. So I did change the Version too early (should have waited until the test bot is finished with the D8 code).
As the D7 code has sent a request for testing already, I hope it will stay at D7 Branch, and switch the issue back to D8 for retesting. Sorry people :(.
Comment #5
JThan commented#1: drupal8.rename-language-Uyghur.1477516.0.patch queued for re-testing.
Comment #6
Emin Sulaiman commentedThank you very much.
Comment #7
Emin Sulaiman commented#2: drupal7.rename-language-Uyghur.1477516.0.patch queued for re-testing.
Comment #9
JThan commentedYou should not retest that patch. It was green, now it is red :).
Please, everyone, the patch is good.
@Emin Sulaiman: Please test the patch and have some others to test it. After the test, someone can set the status to "Reviewed and tested"
Comment #10
Emin Sulaiman commented#2: drupal7.rename-language-Uyghur.1477516.0.patch queued for re-testing.
Comment #12
JThan commentedIt failed because of wrong settings in the issue header. It was green before retesting - so it is working. Please test the patch and set RTBC.
Comment #13
JThan commentedOh and I will unassign this. It should have never been assigned to Emin Sulaiman because he was not the making the patch. Now it needs to be tested, RTBC and commited. Thats all stuff I cannot / should not do.
Comment #14
Emin Sulaiman commented#1: drupal8.rename-language-Uyghur.1477516.0.patch queued for re-testing.
Comment #15
lazysoundsystem commentedLooks good to me.
Comment #16
webchickCommitted and pushed to 8.x and 7.x. Thanks!
Comment #18
David_Rothstein commentedThis one might be worth backporting to Drupal 6, since it's zero risk. (Assuming there will be another Drupal 6 bugfix release at some point, that is.)
Comment #19
Emin Sulaiman commented#2: drupal7.rename-language-Uyghur.1477516.0.patch queued for re-testing.
Comment #20
Emin Sulaiman commented#1: drupal8.rename-language-Uyghur.1477516.0.patch queued for re-testing.
Comment #22
lazysoundsystem commentedHere's a backport for 6.x.
Comment #22.0
lazysoundsystem commenteduyghur
Comment #23
areke commentedThe patch didn't apply, so I re-rolled it.
Comment #24
areke commentedComment #25
areke commented