This month there's a new digit in some Brazilian states phone numbers, so this patch is to solve it to Drupal 7 version of this module (I've sent a similar patch to D6 version #1710980: Brazil new phone number ).

CommentFileSizeAuthor
#1 brazilianphoned7-1728142-1.patch1.26 KBrfsbsb
brazilianphoned7.patch1.26 KBrfsbsb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rfsbsb’s picture

rfsbsb
Primary language Portuguese, Brazil
Location Kitchener
CreditAttribution: rfsbsb commented
FileSize
brazilianphoned7-1728142-1.patch1.26 KB

Rerolling the patch with improvements on regex as suggested on issue #1710980: Brazil new phone number

astutonet’s picture

astutonet
Location Sao Paulo, SP
CreditAttribution: astutonet commented

This patch works as desired, why wasn't commited yet?

Someone can help us, please?

Tks.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans commented
Status: Needs review » Reviewed & tested by the community

This looks good. Leaving for rfsbsb to commit - he's the 1.x maintainer.

astutonet’s picture

astutonet
Location Sao Paulo, SP
CreditAttribution: astutonet commented

The changes made in D7 version are the same of the D6 version available here.

There is only one problem in D7 version: missing a number in the last example of the error message, which it should be '99999-9999 ', as in the D6 version.

rfsbsb’s picture

rfsbsb
Primary language Portuguese, Brazil
Location Kitchener
CreditAttribution: rfsbsb commented
Status: Reviewed & tested by the community » Fixed

Hi @astutonet, thank you for point out the missing error message. I fixed the patch and applied it.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Some typo fixing