henrystivens / upload-image-and-data

core/libs/upload/upload.php

Doc Comments +1 added lines, -1 removed lines

@@ -135,7 +135,7 @@
     /**
      * Asigna las extensiones de archivos permitidas
      *
-     * @param array|string $value lista de extensiones para archivos, si es string separado por |
+     * @param string[] $value lista de extensiones para archivos, si es string separado por |
      */
     public function setExtensions($value) {
         if (!is_array($value)) {

core/libs/i18n/i18n.php

Spacing +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
     
     private static function sprintf($sentence, $args, $offset)
     {
-        return vsprintf( $sentence, array_slice( $args, $offset));
+        return vsprintf($sentence, array_slice($args, $offset));
         
     }
 }

core/libs/logger/logger.php

Spacing +13 added lines, -13 removed lines

@@ -90,7 +90,7 @@ 
      * 
      * @param string $name
      */
-    public static function initialize($name='')
+    public static function initialize($name = '')
     {
         self::$log_path = APP_PATH . 'temp/logs/'; //TODO poder cambiar el path
         if ($name === '') {
@@ -129,14 +129,14 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function log($type='DEBUG', $msg, $name_log)
+    public static function log($type = 'DEBUG', $msg, $name_log)
     {  
         if (is_array($msg)) {
             $msg = print_r($msg, true);
         }
         //TODO poder añadir otros formatos de log
         $date = date(DATE_RFC1036);
-        $msg = "[$date][$type] " . $msg ;
+        $msg = "[$date][$type] " . $msg;
         if (self::$transaction) {
             self::$queue[] = $msg;
             return;
@@ -151,7 +151,7 @@ 
      */
     protected static function write($msg, $name_log)
     {
-        self::initialize($name_log);  //TODO dejarlo abierto cuando es un commit
+        self::initialize($name_log); //TODO dejarlo abierto cuando es un commit
         fputs(self::$fileLogger, $msg . PHP_EOL);
         self::close();
     }
@@ -209,7 +209,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function warning($msg, $name_log='')
+    public static function warning($msg, $name_log = '')
     {
         self::log('WARNING', $msg, $name_log);
     }
@@ -221,7 +221,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function error($msg, $name_log='')
+    public static function error($msg, $name_log = '')
     {
         self::log('ERROR', $msg, $name_log);
     }
@@ -233,7 +233,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function debug($msg, $name_log='')
+    public static function debug($msg, $name_log = '')
     {
         self::log('DEBUG', $msg, $name_log);
     }
@@ -245,7 +245,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function alert($msg, $name_log='')
+    public static function alert($msg, $name_log = '')
     {
         self::log('ALERT', $msg, $name_log);
     }
@@ -257,7 +257,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function critical($msg, $name_log='')
+    public static function critical($msg, $name_log = '')
     {
         self::log('CRITICAL', $msg, $name_log);
     }
@@ -269,7 +269,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function notice($msg, $name_log='')
+    public static function notice($msg, $name_log = '')
     {
         self::log('NOTICE', $msg, $name_log);
     }
@@ -281,7 +281,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function info($msg, $name_log='')
+    public static function info($msg, $name_log = '')
     {
         self::log('INFO', $msg, $name_log);
     }
@@ -293,7 +293,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function emergence($msg, $name_log='')
+    public static function emergence($msg, $name_log = '')
     {
         self::log('EMERGENCE', $msg, $name_log);
     }
@@ -305,7 +305,7 @@ 
      * @param string $msg
      * @param string $name_log
      */
-    public static function custom($type='CUSTOM', $msg, $name_log='')
+    public static function custom($type = 'CUSTOM', $msg, $name_log = '')
     {
         self::log($type, $msg, $name_log);
     }

core/libs/upload/adapters/image_upload.php

Spacing +1 added lines, -1 removed lines

@@ -138,7 +138,7 @@
 
         $image = $this->_imgInfo;
         // Verifica que sea un archivo de imagen
-        if (!$image){
+        if (!$image) {
             Flash::error('Error: el archivo debe ser una imagen');
             return FALSE;
         }

Braces +6 added lines, -2 removed lines

@@ -187,10 +187,14 @@
     protected function _validatesTypes()
     {
         // Verifica que sea un archivo de imagen
-        if (!$this->_imgInfo) return FALSE;
+        if (!$this->_imgInfo) {
+            return FALSE;
+        }
 
         foreach ($this->_types as $type) {
-            if ($this->_imgInfo['mime'] == "image/$type") return TRUE;
+            if ($this->_imgInfo['mime'] == "image/$type") {
+                return TRUE;
+            }
         }
 
         return FALSE;

core/libs/kumbia_auth/kumbia_auth.php

Spacing +2 added lines, -2 removed lines

@@ -18,8 +18,8 @@
  * @license    http://wiki.kumbiaphp.com/Licencia     New BSD License
  */
 
-class KumbiaAuth extends KumbiaFacade{
-    protected static function getAlias(){
+class KumbiaAuth extends KumbiaFacade {
+    protected static function getAlias() {
         return 'auth';
     }
 }

core/libs/redirect/redirect.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         if ($seconds) {
             header("Refresh: $seconds; url=$route");
         } else {
-            header('Location: '.$route, TRUE, $statusCode);
+            header('Location: ' . $route, TRUE, $statusCode);
             $_SESSION['KUMBIA.CONTENT'] = ob_get_clean();
             View::select(null, null);
         }

Braces +3 added lines, -2 removed lines

@@ -86,8 +86,9 @@
             $vars = $url;
         }
         
-        if (++$cyclic > 1000)
-            throw new KumbiaException('Se ha detectado un enrutamiento cíclico. Esto puede causar problemas de estabilidad');
+        if (++$cyclic > 1000) {
+                    throw new KumbiaException('Se ha detectado un enrutamiento cíclico. Esto puede causar problemas de estabilidad');
+        }
 
         Router::to($vars, TRUE);
     }

core/libs/security/security.php

Spacing +2 added lines, -2 removed lines

@@ -42,12 +42,12 @@
         }
     }
 
-    public static function createSecureRSAKey($kumbia=true)
+    public static function createSecureRSAKey($kumbia = true)
     {
         $config = Config::read('config');
         if ($config->kumbia->secure_ajax) {
             if ($_SESSION['rsa_key']) {
-                if ((time() % 8) == 0) {
+                if ((time()%8) == 0) {
                     return self::generateRSAKey($kumbia);
                 } else {
                     echo "<input type='hidden' id='rsa32_key' value=\"{$_SESSION['rsa_key']}\"/>";

core/libs/validate/validations.php

Spacing +11 added lines, -11 removed lines

@@ -33,7 +33,7 @@ 
     /**
      * Solo letras
      */
-    const IS_ALPHA    = '/^(?:[^\W\d_]|([ ]))*$/mu';
+    const IS_ALPHA = '/^(?:[^\W\d_]|([ ]))*$/mu';
 
     /**
      * Almacena la Expresion Regular
@@ -48,7 +48,7 @@ 
      * @param  mixed $check Valor a ser chequeado
      * @return bool
      */
-    public static function numeric($check){
+    public static function numeric($check) {
         return is_numeric($check);
     }
 
@@ -74,14 +74,14 @@ 
      */
     public static function maxlength($value, $param)
     {
-        $max= isset($param['max'])?$param['max']:0;
+        $max = isset($param['max']) ? $param['max'] : 0;
         return !isset($value[$max]);
     }
 
     /**
      * Valida longitud de la cadena
      */
-    public static function length($value, $param){
+    public static function length($value, $param) {
         $param = array_merge(array(
             'min' => 0,
             'max' => 9e100,
@@ -137,7 +137,7 @@ 
      */
     public static function url($url, $param)
     {
-        $flag = isset($param['flag'])? $param['flag'] : 0;
+        $flag = isset($param['flag']) ? $param['flag'] : 0;
         return filter_var($url, FILTER_VALIDATE_URL, FILTER_FLAG_HOST_REQUIRED | $flag);
     }
 
@@ -197,7 +197,7 @@ 
      */
     public static function date($value, $param)
     {
-        $format = isset($param['format'])? $param['format'] : 'Y-m-d';
+        $format = isset($param['format']) ? $param['format'] : 'Y-m-d';
         $date = DateTime::createFromFormat($format, $value);
         return $date && $date->format($format) == $value;
     }
@@ -211,7 +211,7 @@ 
      */
     public static function pattern($check, $param)
     {
-        $regex = isset($param['regexp'])? $param['regexp'] : '/.*/';
+        $regex = isset($param['regexp']) ? $param['regexp'] : '/.*/';
         return empty($check) || FALSE !== filter_var($check, FILTER_VALIDATE_REGEXP, array('options' => array('regexp' => $regex)));
     }
 
@@ -224,7 +224,7 @@ 
      */
     public static function decimal($value, $param)
     {
-        $decimal = isset($param['decimal'])? $param['decimal'] : ',';
+        $decimal = isset($param['decimal']) ? $param['decimal'] : ',';
         return filter_var($value, FILTER_VALIDATE_FLOAT, array('options' => array('decimal' => $decimal)));
     }
 
@@ -238,7 +238,7 @@ 
      */
     public static function equal($value, $param, $obj)
     {
-        $equal = isset($param['to'])? $param['to'] : '';
+        $equal = isset($param['to']) ? $param['to'] : '';
         return ($obj->$equal == $value);
     }
 
@@ -247,8 +247,8 @@ 
      * @param string $key
      * @return string
      */
-    public static function getMessage($key){
-        $arr  = array(
+    public static function getMessage($key) {
+        $arr = array(
             'required' => 'Este campo es requerido',
             'alphanum' => 'Debe ser un valor alfanumérico',
             'alpha'    => 'Solo caracteres alfabeticos',

core/libs/validate/validate.php

Indentation +3 added lines, -3 removed lines

@@ -114,7 +114,7 @@ 
             return false;
         }
         if(!$obj->$ruleName($field, $param)){
-           $this->addError($param, $field, $ruleName);
+            $this->addError($param, $field, $ruleName);
         }
         return true;
     }
@@ -126,7 +126,7 @@ 
      * @param string $rule Nombre de la regla
      */
     protected function addError(Array $param, $field, $rule){
-         $this->messages[$field][] = isset($param['error']) ?
+            $this->messages[$field][] = isset($param['error']) ?
                 $param['error']: Validations::getMessage($rule);
     }
 
@@ -137,7 +137,7 @@ 
      * @return string
      */
     protected static function getRuleName($ruleName, $param){
-         /*Evita tener que colocar un null cuando no se pasan parametros*/
+            /*Evita tener que colocar un null cuando no se pasan parametros*/
         return is_integer($ruleName) && is_string($param)?$param:$ruleName;
     }
 

Spacing +26 added lines, -26 removed lines

@@ -19,7 +19,7 @@ 
  * @copyright  Copyright (c) 2005 - 2017 Kumbia Team (http://www.kumbiaphp.com)
  * @license    http://wiki.kumbiaphp.com/Licencia     New BSD License
  */
-require __DIR__.'/validations.php';
+require __DIR__ . '/validations.php';
 class Validate
 {
     /**
@@ -63,9 +63,9 @@ 
      * @param mixed $obj Objecto o Array a validar
      * @param array $rules Aray de reglas a validar
      */
-    public function __construct($obj, Array $rules){
+    public function __construct($obj, Array $rules) {
         $this->is_obj = is_object($obj);
-        $this->obj = (object)$obj;
+        $this->obj = (object) $obj;
         $this->rules = $rules;
     }
 
@@ -73,20 +73,20 @@ 
      * Ejecuta las validaciones
      * @return bool Devuelve true si todo es válido
      */
-    public function exec(){
+    public function exec() {
         /*Recorrido por todos los campos*/
-        foreach ($this->rules as $field => $fRule){
+        foreach ($this->rules as $field => $fRule) {
             $value = self::getValue($this->obj, $field);
             /*Regla individual para cada campo*/
             foreach ($fRule as $ruleName => $param) {
                 $ruleName = self::getRuleName($ruleName, $param);
-                $param =  self::getParams($param);
+                $param = self::getParams($param);
                 /*Ignore the rule is starts with "#"*/
-                if($ruleName[0] == '#') continue;
+                if ($ruleName[0] == '#') continue;
                 /*Es una validación de modelo*/
-                if($ruleName[0] == '@'){
+                if ($ruleName[0] == '@') {
                     $this->modelRule($ruleName, $param, $field);
-                }elseif(!Validations::$ruleName($value, $param, $this->obj)){
+                }elseif (!Validations::$ruleName($value, $param, $this->obj)) {
                     $this->addError($param, $field, $ruleName);
                 }
             }
@@ -102,18 +102,18 @@ 
      * @param string $field Nombre del campo
      * @return bool
      */
-    protected function modelRule($rule, $param, $field){
-        if(!$this->is_obj){
+    protected function modelRule($rule, $param, $field) {
+        if (!$this->is_obj) {
             trigger_error('No se puede ejecutar una validacion de modelo en un array', E_USER_WARNING);
             return false;
         }
         $ruleName = ltrim($rule, '@');
         $obj = $this->obj;
-        if(!method_exists($obj, $ruleName)){
+        if (!method_exists($obj, $ruleName)) {
             trigger_error('El metodo para la validacion no existe', E_USER_WARNING);
             return false;
         }
-        if(!$obj->$ruleName($field, $param)){
+        if (!$obj->$ruleName($field, $param)) {
            $this->addError($param, $field, $ruleName);
         }
         return true;
@@ -125,9 +125,9 @@ 
      * @param string $field Nombre del campo
      * @param string $rule Nombre de la regla
      */
-    protected function addError(Array $param, $field, $rule){
+    protected function addError(Array $param, $field, $rule) {
          $this->messages[$field][] = isset($param['error']) ?
-                $param['error']: Validations::getMessage($rule);
+                $param['error'] : Validations::getMessage($rule);
     }
 
     /**
@@ -136,9 +136,9 @@ 
      * @param mixed $param
      * @return string
      */
-    protected static function getRuleName($ruleName, $param){
+    protected static function getRuleName($ruleName, $param) {
          /*Evita tener que colocar un null cuando no se pasan parametros*/
-        return is_integer($ruleName) && is_string($param)?$param:$ruleName;
+        return is_integer($ruleName) && is_string($param) ? $param : $ruleName;
     }
 
     /**
@@ -146,8 +146,8 @@ 
      * @param mixed $param
      * @return array
      */
-    protected static function getParams($param){
-        return is_array($param)?$param:array();
+    protected static function getParams($param) {
+        return is_array($param) ? $param : array();
     }
 
     /**
@@ -156,35 +156,35 @@ 
      * @param string $field
      * @return mixed
      */
-    protected static function getValue($obj, $field){
-        return !empty($obj->$field)?$obj->$field:null;//obtengo el valor del campo
+    protected static function getValue($obj, $field) {
+        return !empty($obj->$field) ? $obj->$field : null; //obtengo el valor del campo
     }
 
     /**
      * Devuelve los mensajes de error
      *
      */
-    public function getMessages(){
+    public function getMessages() {
         return $this->messages;
     }
 
     /**
      * Version de instancias para flush error
      */
-    public function flash(){
+    public function flash() {
         self::errorToFlash($this->getMessages());
     }
 
-    public static function fail($obj, Array $rules){
+    public static function fail($obj, Array $rules) {
         $val = new self($obj, $rules);
-        return $val->exec() ? false:$val->getMessages();
+        return $val->exec() ? false : $val->getMessages();
     }
 
     /**
      * Envia los mensajes de error via flash
      * @param Array $error
      */
-    public static function errorToFlash(Array $error){
+    public static function errorToFlash(Array $error) {
         foreach ($error as $value)
             Flash::error($value);
     }

Braces +7 added lines, -4 removed lines

@@ -82,11 +82,13 @@ 
                 $ruleName = self::getRuleName($ruleName, $param);
                 $param =  self::getParams($param);
                 /*Ignore the rule is starts with "#"*/
-                if($ruleName[0] == '#') continue;
+                if($ruleName[0] == '#') {
+                    continue;
+                }
                 /*Es una validación de modelo*/
                 if($ruleName[0] == '@'){
                     $this->modelRule($ruleName, $param, $field);
-                }elseif(!Validations::$ruleName($value, $param, $this->obj)){
+                } elseif(!Validations::$ruleName($value, $param, $this->obj)){
                     $this->addError($param, $field, $ruleName);
                 }
             }
@@ -185,7 +187,8 @@ 
      * @param Array $error
      */
     public static function errorToFlash(Array $error){
-        foreach ($error as $value)
-            Flash::error($value);
+        foreach ($error as $value) {
+                    Flash::error($value);
+        }
     }
 }