Curacao is a country since 10 October 2010, and needs to appear in the countries list of this module. Thanks.

#13 curacao_added_to_country_list-1313342-13.patch394 bytesRaulMuroc
PASSED: [[SimpleTest]]: [MySQL] 37,029 pass(es). View
#7 curacao_added_to_country_list-1313342-6.patch427 bytesCrookedNumber
PASSED: [[SimpleTest]]: [MySQL] 33,972 pass(es). View
#4 curacao_added_to_country_list-1313342.patch2.48 KBCrookedNumber
PASSED: [[SimpleTest]]: [MySQL] 33,977 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


iMiksu’s picture

Project: Address Field » Drupal core
Version: 7.x-1.0-beta2 » 7.x-dev
Component: Code » locale.module

I think this should be fixed by core. However, you can alter countries according to API documentation:

CrookedNumber’s picture

Title: Curacao » Curacao should appear standard_country_list()

Better title.

CrookedNumber’s picture

Version: 7.x-dev » 8.x-dev
CrookedNumber’s picture

Assigned: Unassigned » CrookedNumber
Status: Active » Needs review
2.48 KB
PASSED: [[SimpleTest]]: [MySQL] 33,977 pass(es). View

Attaching patch.

CrookedNumber’s picture

Title: Curacao should appear standard_country_list() » Curacao should appear in standard_country_list()
kathyh’s picture

Status: Needs review » Needs work

Remove extra spaces at the end of the Curacao line

+++ b/core/includes/
@@ -76,6 +76,7 @@ function standard_country_list() {
+    'CW' => $t('Curaçao'),    ¶

Only Curacao should be added in the patch. Persian, Hebrew and Urdu should not be affected.

+++ b/core/includes/
@@ -302,7 +303,7 @@ function standard_language_list() {
+    'ar' => array('Arabic', /* Left-to-right marker "" */ 'العربية', LANGUAGE_RTL),
@@ -323,7 +324,7 @@ function standard_language_list() {
+    'fa' => array('Persian, Farsi', /* Left-to-right marker "" */ 'فارسی', LANGUAGE_RTL),
@@ -333,7 +334,7 @@ function standard_language_list() {
+    'he' => array('Hebrew', /* Left-to-right marker "" */ 'עברית', LANGUAGE_RTL),
@@ -392,7 +393,7 @@ function standard_language_list() {
+    'ur' => array('Urdu', /* Left-to-right marker "" */ 'اردو', LANGUAGE_RTL),
CrookedNumber’s picture

427 bytes
PASSED: [[SimpleTest]]: [MySQL] 33,972 pass(es). View

Thanks. Not sure why that happened. Re-rolling.

CrookedNumber’s picture

Status: Needs work » Needs review
kathyh’s picture

Status: Needs review » Reviewed & tested by the community

Verified that CW Curacao is in ISO 3166-1 alpha-2 and patch load OK.

claudiu.cristea’s picture

Issue tags: +needs backport to D7

D7 candidate.

webchick’s picture

Version: 8.x-dev » 7.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed and pushed to 8.x. Needs a re-roll for 7.x.

webchick’s picture

Issue tags: +Novice

And this would be a nice easy patch for new contributors, so tagging Novice.

RaulMuroc’s picture

Assigned: CrookedNumber » RaulMuroc
Status: Patch (to be ported) » Needs review
394 bytes
PASSED: [[SimpleTest]]: [MySQL] 37,029 pass(es). View

Here is a patch re-rolled for 7.x.

webchick’s picture

Status: Needs review » Fixed

That looks right! Thanks a lot.

Committed and pushed to 7.x. w00t!

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