Merge pull request #14 from magento-engcom/MAGETWO-89875

MAGETWO-89875: null passed to count() call in Zend/Validate/File/Uplo…

Author: Joan He

library/Zend/Validate/File/FilesSize.php

@@ -155,10 +155,6 @@ public function isValid($value, $file = null)
             }
         }
 
-        if (count($this->_messages) > 0) {
-            return false;
-        }
-
-        return true;
+        return !empty($this->_messages);
     }
 }

library/Zend/Validate/File/ImageSize.php

@@ -338,11 +338,7 @@ public function isValid($value, $file = null)
             $this->_throw($file, self::HEIGHT_TOO_BIG);
         }
 
-        if (count($this->_messages) > 0) {
-            return false;
-        }
-
-        return true;
+        return !empty($this->_messages);
     }
 
     /**

library/Zend/Validate/File/Size.php

@@ -312,11 +312,7 @@ public function isValid($value, $file = null)
             }
         }
 
-        if (count($this->_messages) > 0) {
-            return false;
-        }
-
-        return true;
+        return !empty($this->_messages);
     }
 
     /**

library/Zend/Validate/File/Upload.php

@@ -161,7 +161,7 @@ public function setFiles($files = array())
      */
     public function isValid($value, $file = null)
     {
-        $this->_messages = null;
+        $this->_messages = [];
         if (array_key_exists($value, $this->_files)) {
             $files[$value] = $this->_files[$value];
         } else {
@@ -223,11 +223,7 @@ public function isValid($value, $file = null)
             }
         }
 
-        if (count($this->_messages) > 0) {
-            return false;
-        } else {
-            return true;
-        }
+        return !empty($this->_messages);
     }
 
     /**