Attached patches are to add support for Australian numbers.

Use phone-1.5_au.patch for current phone-5.x-1.5 release.

CommentFileSizeAuthor
phone-1.5_au.patch12.92 KBwaddles
phone-dev_au.patch12.88 KBwaddles

Comments

thierry_gd’s picture

Assigned: Unassigned » thierry_gd
Status: Needs review » Fixed

Taken into account in 5.2.7 version

waddles’s picture

aah, whoops. I just noticed there's a bug in the match for extension. I added a space to the regex but have already stripped them in the code above it so it would never match. Please remove the space from the regex.

  /*
   * strip and save the extension (x9999) postfix if found
   */
  if (preg_match('/(x[0-9]+)$/', $phonenumber, $match)) {
    $extension = ' ('.$match[1].')';
    $phonenumber = preg_replace('/x[0-9]+$/', '', $phonenumber);
  }

We really should get some unit tests made for these.

waddles’s picture

Status: Fixed » Needs work
thierry_gd’s picture

Status: Needs work » Fixed

Taken into account in 5.2.11 version

Anonymous’s picture

Status: Fixed » Closed (fixed)

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