Created an Italian Address Field handler - with all the valid provinces.

CommentFileSizeAuthor
#12 formatter-italian.patch1 KBaturetta
#10 addressfield-formatter--IT.tpl_.txt849 bytesaturetta
#8 addressfield-it-1162636.patch1.29 KBaturetta
#7 addressfield-it-1162636.patch1.29 KBaturetta
#6 addressfield-it#1162636.patch1.29 KBaturetta
#5 addressfield-it.patch410 bytesaturetta
italian_address_handler.patch4.73 KBronald_istos

Comments

rszrama’s picture

rszrama commented
Status: Active » Fixed

Committed, thanks! And just so I know, is there an expectation with the Italian postal service that a province be selected?

ronald_istos’s picture

ronald_istos commented

Thanks! And yes they expect the province to be there although the postcode is technically unique and enough to identify province/city pair

rszrama’s picture

rszrama commented

Great, thanks for getting this in to us. : )

I see there are a bunch more waiting to be added, too. I guess they aren't tweeting about their patches. ^_^

Status: Fixed » Closed (fixed)

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

aturetta’s picture

aturetta commented
Priority: Normal » Critical
Status: Closed (fixed) » Needs review
StatusFileSize
new addressfield-it.patch410 bytes

The patch committed did not add the italian include file to the info's files section. As a result the module gives now a WSOD if the default language is italian.

Also, please close #1062166: Italian provinces which is almost a duplicate of this one, but this one has slightly better names for some italian provinces.

aturetta’s picture

aturetta commented
StatusFileSize
new addressfield-it#1162636.patch1.29 KB

While I'm at it, I corrected some misspelled province names. Please use this patch and not #5.

aturetta’s picture

aturetta commented
StatusFileSize
new addressfield-it-1162636.patch1.29 KB

It seems # is not a valid character for filenames in issues..... sorry

aturetta’s picture

aturetta commented
StatusFileSize
new addressfield-it-1162636.patch1.29 KB

OMG, I hope this time all is good. The previous one was not UTF-8.....

rszrama’s picture

rszrama commented
Status: Needs review » Fixed

Fixed, thanks for sticking with this.

aturetta’s picture

aturetta commented
Priority: Critical » Normal
Status: Fixed » Needs review
StatusFileSize
new addressfield-formatter--IT.tpl_.txt849 bytes

Followup.

Add a customized formatter for Italian.

The attached file should be renamed as .tpl.php and committed to the theme directory.

aturetta’s picture

aturetta commented
Assigned: ronald_istos » Unassigned

Ping

aturetta’s picture

aturetta commented
StatusFileSize
new formatter-italian.patch1 KB

Re-submitting as patch, maybe it is easier.

damien tournoud’s picture

damien tournoud commented
Status: Needs review » Closed (won't fix)

The -reloaded branch has been merged. Tested by aturetta yesterday, this doesn't appear to be needed anymore.