[tipos commit] r1613 - trunk/Tipos/Filter

0 views
Skip to first unread message

codesite...@google.com

unread,
Oct 11, 2008, 10:39:25 AM10/11/08
to tipo...@googlegroups.com
Author: rodrigo.moraes
Date: Sat Oct 11 07:38:26 2008
New Revision: 1613

Modified:
trunk/Tipos/Filter/SanitizeEmailList.php
trunk/Tipos/Filter/ValidateEmailList.php

Log:
Sanitizing to an array was not a good idea. Now it returns a string.

Modified: trunk/Tipos/Filter/SanitizeEmailList.php
==============================================================================
--- trunk/Tipos/Filter/SanitizeEmailList.php (original)
+++ trunk/Tipos/Filter/SanitizeEmailList.php Sat Oct 11 07:38:26 2008
@@ -25,6 +25,8 @@
* one by line in textareas).
*
* @param mixed $value The value to validate.
+ *
+ * @param string $sep List separator.
*
* @return bool True if valid, false if not.
*
@@ -54,6 +56,6 @@
}

// done
- return $emails;
+ return implode($sep, $emails);
}
}

Modified: trunk/Tipos/Filter/ValidateEmailList.php
==============================================================================
--- trunk/Tipos/Filter/ValidateEmailList.php (original)
+++ trunk/Tipos/Filter/ValidateEmailList.php Sat Oct 11 07:38:26 2008
@@ -21,15 +21,20 @@
* Validates a list of emails.
*
* @param mixed $value The value to validate.
+ *
+ * @param string $sep List separator.
*
* @return bool True if valid, false if not.
*
*/
- public function validateEmailList($value)
+ public function validateEmailList($value, $sep = ',')
{
if (! $this->_filter->getRequire() && $value === null) {
return true;
}
+
+ // transform into an array
+ $value = explode($sep, $value);

// drop duplicated values
$value = array_unique($value);

Reply all
Reply to author
Forward
0 new messages