Tickets :: [#1824] Patch for Horde_Form: allowed characters for phone numbers

Summary	Patch for Horde_Form: allowed characters for phone numbers
Queue	Horde Framework Packages
Type	Enhancement
State	Resolved
Priority	2. Medium
Owners	Horde Developers (at) , jan (at) horde (dot) org
Requester	stefan+lists (at) luethje (dot) ch
Created	04/19/2005 (7529 days ago)
Due
Updated	04/20/2005 (7528 days ago)
Assigned
Resolved	04/20/2005 (7528 days ago)
Milestone
Patch	No

04/20/2005 08:30:27 AM	Jan Schneider	Comment #3 Summary ⇒ Patch for Horde_Form: allowed characters for phone numbers State ⇒ Resolved	Reply to this comment
That's correct. We only use it in national notation to not confuse you Americans. ;-) Committed, thanks.

04/20/2005 05:43:07 AM	Chuck Hagenbuch	Comment #2 Assigned to Jan Schneider Assigned to Horde Developers State ⇒ Feedback	Reply to this comment
I've never seen a / in an international phone number, but if you say so. :) Jan?

04/19/2005 09:07:11 PM	stefan+lists (at) luethje (dot) ch	Comment #1 State ⇒ New Priority ⇒ 2. Medium Type ⇒ Enhancement Summary ⇒ Patch for LDAP: allowed characters for phone numbers Queue ⇒ Horde Framework Packages	Reply to this comment
In Germany we use often the slash as delimiter for phone numbers, so the following patch should fix this: --- ./lib/Horde/Form.php.save 2005-04-13 20:48:18.000000000 +0200 +++ ./lib/Horde/Form.php 2005-04-13 20:48:47.000000000 +0200 @@ -1246,11 +1246,11 @@ $valid = strlen(trim($value)) > 0; if (!$valid) { $message = _("This field is required."); } } else { - $valid = preg_match('/^\+?[\d()\- ]$/', $value); + $valid = preg_match('/^\+?[\d()\- \/]$/', $value); if (!$valid) { $message = _("You have to enter a valid cellphone number, digits only with an optional '+' for the international dialing prefix."); } }