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

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

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: http://api.drupal.org/api/drupal/includes--locale.inc/function/country_g...

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
FileSize
2.48 KB

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/standard.inc
@@ -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/standard.inc
@@ -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

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
FileSize
394 bytes

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.