[D7] AddressField CO

Hi pixelula,

To get the regular stuff out of the way:
- you should check out the review by pareview and fix those small issues.
- check out these tips for a template on what a good project page looks like.

For my manual review:
- To avoid having that long array of states and cities in a code file that you may have to work on periodically I would suggest you put that array into a variable at install and delete the variable at uninstall. That way when you need the list you can just do variable_get('addressfield_co_states_list', array()). But that's just my opinion on keeping clutter out of your working file. SOmeone else may say different.

- I installed the module and created a test address with the state of Guainía and city of Mapiripana. After saving the node and viewing I just see a value of 94663 which is just a combination of the numeric values you have for the state (94) and city (663) in the select forms.

I tested this module, and it works, with one small bug.

The key instead of the value is shown in the when viewing the node.

There is an option render_option_value that solves this problem:

$format['locality_block']['administrative_area']['#render_option_value'] = true;

We are currently quite busy with all the project applications and we prefer projects with a review bonus. Please help reviewing and put yourself on the high priority list, then we will take a look at your project right away :-)

Also, you should get your friends, colleagues or other community members involved to review this application. Let them go through the review checklist and post a comment that sets this issue to "needs work" (they found some problems with the project) or "reviewed & tested by the community" (they found no major flaws).

I'm a robot and this is an automated message from Project Applications Scraper.

After a recent discussion with one of the addressfield maintainers we would like to ask you to submit this as patch against the core addressfield module instead. Module duplication and fragmentation is a huge problem on drupal.org and we prefer collaboration over competition.

If your patch is accepted please close this project application and feel free to come back to us whenever you have a new project to promote.

If your patch is rejected for whatever reason please revert the status of this project application and add a section to the issue summary where you explain why this could not go into addressfield core directly. And a link to the issue in the addressfield issue queue where that was discussed would be nice in the issue summary.

----
Top Shelf Modules - Crafted, Curated, Contributed.

Closing due to lack of activity. Feel free to reopen if you are still working on this application (see also the project application workflow).

I'm a robot and this is an automated message from Project Applications Scraper.