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

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
StatusFileSize
new2.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

StatusFileSize
new427 bytes

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
StatusFileSize
new394 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.