From bd09c30efc89c263d0f4f7d8807b613d3b224070 Mon Sep 17 00:00:00 2001 From: ecoreng Date: Thu, 31 Jul 2014 22:55:20 -0700 Subject: [PATCH 1/3] refactor to avoid foreach when filtering --- src/Valitron/Validator.php | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/Valitron/Validator.php b/src/Valitron/Validator.php index b9b4e41..746a179 100644 --- a/src/Valitron/Validator.php +++ b/src/Valitron/Validator.php @@ -78,11 +78,7 @@ class Validator { // Allows filtering of used input fields against optional second array of field names allowed // This is useful for limiting raw $_POST or $_GET data to only known fields - foreach ($data as $field => $value) { - if (empty($fields) || (!empty($fields) && in_array($field, $fields))) { - $this->_fields[$field] = $value; - } - } + $this->_fields = !empty($fields) ? array_intersect_key($data, array_flip($fields)) : $data; // set lang in the follow order: constructor param, static::$_lang, default to en $lang = $lang ?: static::lang(); From 85e353604a56afc100ef218be32bb0057829e3a5 Mon Sep 17 00:00:00 2001 From: ecoreng Date: Thu, 31 Jul 2014 23:24:48 -0700 Subject: [PATCH 2/3] added instanceOf to spanish language --- lang/es.php | 1 + 1 file changed, 1 insertion(+) diff --git a/lang/es.php b/lang/es.php index 41d1b21..04ea7b2 100644 --- a/lang/es.php +++ b/lang/es.php @@ -30,4 +30,5 @@ return array( 'creditCard' => "debe ser un numero de tarjeta de credito valido", "lengthMin" => "debe contener mas de %d caracteres", "lengthMax" => "debe contener menos de %d caracteres" + "instanceOf" => "dene ser una instancia de '%s'" ); From 9d266482337eba829127b173942d9723cc347af5 Mon Sep 17 00:00:00 2001 From: Elias Coronado Date: Thu, 31 Jul 2014 23:27:11 -0700 Subject: [PATCH 3/3] typo fix --- lang/es.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lang/es.php b/lang/es.php index 04ea7b2..75602d8 100644 --- a/lang/es.php +++ b/lang/es.php @@ -30,5 +30,5 @@ return array( 'creditCard' => "debe ser un numero de tarjeta de credito valido", "lengthMin" => "debe contener mas de %d caracteres", "lengthMax" => "debe contener menos de %d caracteres" - "instanceOf" => "dene ser una instancia de '%s'" + "instanceOf" => "debe ser una instancia de '%s'" );