Style fixes (code quality)

phalcon/Acl/Adapter/Memory.zep

@@ -19,7 +19,6 @@ use Phalcon\Acl\Exception;
 use Phalcon\Events\Manager as EventsManager;
 use Phalcon\Acl\RoleAware;
 use Phalcon\Acl\ComponentAware;
-use Phalcon\Acl\RoleInterface;
 use Phalcon\Acl\ComponentInterface;
 use ReflectionFunction;
 

phalcon/Annotations/Annotation.zep

@@ -10,7 +10,6 @@
 
 namespace Phalcon\Annotations;
 
-use Phalcon\Annotations\Annotation;
 use Phalcon\Annotations\Exception;
 
 /**

phalcon/Application/AbstractApplication.zep

@@ -5,7 +5,7 @@
  * (c) Phalcon Team <team@phalcon.io>
  *
  * For the full copyright and license information, please view the LICENSE.txt
- * file that was distributed with this source code.
+ * file that was dis§tributed with this source code.
  */
 
 namespace Phalcon\Application;
@@ -94,7 +94,7 @@ abstract class AbstractApplication extends Injectable implements EventsAwareInte
     /**
      * Handles a request
      */
-    abstract public function handle();
+    abstract public function handle(var! uri);
 
     /**
      * Register an array of modules present in the application

phalcon/Cli/Console.zep

@@ -34,13 +34,17 @@ class Console extends AbstractApplication
     /**
      * Handle the whole command-line tasks
      */
-    public function handle(array arguments = null)
+    public function handle(var! arguments)
     {
         var className, container, dispatcher, eventsManager, module, moduleName,
             moduleObject, modules, path, router, task;
 
         let container = this->container;
 
+        if unlikely typeof arguments != "array" {
+            throw new Exception("Argument should be an array");
+        }
+
         if unlikely typeof container != "object" {
             throw new Exception(
                 Exception::containerServiceNotFound("internal services")

phalcon/Factory/AbstractFactory.zep

@@ -13,7 +13,7 @@ namespace Phalcon\Factory;
 use Phalcon\Config;
 use Phalcon\Factory\Exception;
 
-class AbstractFactory
+abstract class AbstractFactory
 {
     /**
      * @var array

phalcon/Forms/Element/Select.zep

@@ -11,7 +11,7 @@
 namespace Phalcon\Forms\Element;
 
 use Phalcon\Forms\Element\AbstractElement;
-use Phalcon\Tag\Select;
+use Phalcon\Tag\Select as SelectTag;
 
 /**
  * Phalcon\Forms\Element\Select
@@ -73,7 +73,7 @@ class Select extends AbstractElement
         /**
          * Merged passed attributes with previously defined ones
          */
-        return Select::selectField(
+        return SelectTag::selectField(
             this->prepareAttributes(attributes),
             this->optionsValues
         );

phalcon/Forms/Form.zep

@@ -14,7 +14,6 @@ use Countable;
 use Iterator;
 use Phalcon\Di\Injectable;
 use Phalcon\Di\DiInterface;
-use Phalcon\FilterInterface;
 use Phalcon\Filter\FilterInterface;
 use Phalcon\Forms\Exception;
 use Phalcon\Forms\Element\ElementInterface;

phalcon/Helper/Fs.zep

@@ -13,7 +13,8 @@ namespace Phalcon\Helper;
 /**
  * This class offers file operation helper
  */
-class Fs {
+class Fs
+{
 
     /**
      * Gets the filename from a given path, Same as PHP's basename() but has non-ASCII support.
@@ -30,8 +31,8 @@ class Fs {
     	var filename, matches;
         let uri = rtrim(uri, DIRECTORY_SEPARATOR);
         let filename = preg_match(
-        	"@[^" . preg_quote(DIRECTORY_SEPARATOR, "@") . "]+$@", 
-        	uri, 
+        	"@[^" . preg_quote(DIRECTORY_SEPARATOR, "@") . "]+$@",
+        	uri,
         	matches
         ) ? matches[0] : "";
         if suffix {

phalcon/Html/Attributes/RenderInterface.zep

@@ -14,7 +14,8 @@ namespace Phalcon\Html\Attributes;
 *
 * Interface Phalcon\Html\Attributes\RenderInterface
 */
-interface RenderInterface {
+interface RenderInterface
+{
     /**
      * Generate a string represetation
      */

phalcon/Http/RequestInterface.zep

@@ -191,7 +191,7 @@ interface RequestInterface
      * @param bool onlyPath If true, query part will be omitted
      * @return string
      */
-    final public function getURI(bool onlyPath = false) -> string;
+    public function getURI(bool onlyPath = false) -> string;
 
     /**
      * Gets a variable from the $_POST superglobal applying filters if needed

phalcon/Image/Adapter/Imagick.zep

@@ -10,7 +10,7 @@
 
 namespace Phalcon\Image\Adapter;
 
-use Imagick;
+use Imagick as ImagicNative;
 use ImagickDraw;
 use ImagickPixel;
 use Phalcon\Image\Enum;
@@ -50,7 +50,7 @@ class Imagick extends AbstractAdapter
 
         let this->file = file;
 
-        let this->image = new Imagick();
+        let this->image = new ImagicNative();
 
         if file_exists(this->file) {
             let this->realpath = realpath(this->file);
@@ -105,7 +105,7 @@ class Imagick extends AbstractAdapter
      */
     public function __destruct()
     {
-        if this->image instanceof \Imagick {
+        if this->image instanceof ImagicNative {
             this->image->clear();
             this->image->destroy();
         }
@@ -138,7 +138,7 @@ class Imagick extends AbstractAdapter
     /**
      * Get instance
      */
-    public function getInternalImInstance() -> <\Imagick>
+    public function getInternalImInstance() -> <ImagicNative>
     {
         return this->image;
     }
@@ -166,7 +166,7 @@ class Imagick extends AbstractAdapter
 
         let pixel2 = new ImagickPixel("transparent");
 
-        let background = new Imagick();
+        let background = new ImagicNative();
 
         this->image->setIteratorIndex(0);
 
@@ -324,7 +324,7 @@ class Imagick extends AbstractAdapter
     {
         var mask, ret;
 
-        let mask = new Imagick();
+        let mask = new ImagicNative();
 
         mask->readImageBlob(
             image->render()
@@ -417,7 +417,7 @@ class Imagick extends AbstractAdapter
         }
 
         let pseudo = fadeIn ? "gradient:black-transparent" : "gradient:transparent-black",
-            fade = new Imagick();
+            fade = new ImagicNative();
 
         fade->newPseudoImage(
             reflection->getImageWidth(),
@@ -454,7 +454,7 @@ class Imagick extends AbstractAdapter
 
         fade->destroy();
 
-        let image = new Imagick(),
+        let image = new ImagicNative(),
             pixel = new ImagickPixel(),
             height = this->image->getImageHeight() + height;
 
@@ -808,7 +808,7 @@ class Imagick extends AbstractAdapter
         var watermark, ret;
 
         let opacity = opacity / 100,
-            watermark = new Imagick();
+            watermark = new ImagicNative();
 
         watermark->readImageBlob(
             image->render()

phalcon/Mvc/Application.zep

@@ -79,13 +79,17 @@ class Application extends AbstractApplication
     /**
      * Handles a MVC request
      */
-    public function handle(string! uri) -> <ResponseInterface> | bool
+    public function handle(var! uri)
     {
         var container, eventsManager, router, dispatcher, response, view,
             module, moduleObject, moduleName, className, path, implicitView,
             returnedResponse, controller, possibleResponse, renderStatus,
             matchedRoute, match;
 
+        if unlikely typeof uri != "string" {
+            throw new Exception("Argument should be a string");
+        }
+
         let container = this->container;
 
         if unlikely typeof container != "object" {

phalcon/Mvc/Model.zep

@@ -41,7 +41,6 @@ use Phalcon\Mvc\Model\TransactionInterface;
 use Phalcon\Mvc\Model\ValidationFailed;
 use Phalcon\Mvc\ModelInterface;
 use Phalcon\Validation\ValidationInterface;
-use Phalcon\Events\ManagerInterface as EventsManagerInterface;
 use Serializable;
 
 /**

phalcon/Mvc/Model/ManagerInterface.zep

@@ -262,7 +262,7 @@ interface ManagerInterface
      * );
      * ```
      */
-    final public function isVisibleModelProperty(<ModelInterface> model, string property) -> bool;
+    public function isVisibleModelProperty(<ModelInterface> model, string property) -> bool;
 
     /**
      * Sets if a model must keep snapshots

phalcon/Mvc/Model/Query.zep

@@ -30,7 +30,6 @@ use Phalcon\Di\InjectionAwareInterface;
 use Phalcon\Mvc\Model\RelationInterface;
 use Phalcon\Mvc\Model\TransactionInterface;
 use Phalcon\Db\DialectInterface;
-use Phalcon\Cache\Adapter\AdapterInterface;
 
 /**
  * Phalcon\Mvc\Model\Query

phalcon/Mvc/ModelInterface.zep

@@ -13,11 +13,9 @@ use Phalcon\Db\Adapter\AdapterInterface;
 use Phalcon\Di\DiInterface;
 use Phalcon\Messages\MessageInterface;
 use Phalcon\Mvc\Model\CriteriaInterface;
-use Phalcon\Mvc\Model\ModelInterface;
 use Phalcon\Mvc\Model\MetaDataInterface;
 use Phalcon\Mvc\Model\ResultsetInterface;
 use Phalcon\Mvc\Model\TransactionInterface;
-use Phalcon\Mvc\ModelInterface;
 
 /**
  * Phalcon\Mvc\ModelInterface

phalcon/Validation/Validator/File/Size/Equal.zep

@@ -12,7 +12,6 @@ namespace Phalcon\Validation\Validator\File\Size;
 
 use Phalcon\Messages\Message;
 use Phalcon\Validation;
-use Phalcon\Validation\Validator\File\Size\Equal;
 use Phalcon\Validation\Validator\File\AbstractFile;
 
 /**

phalcon/Validation/Validator/File/Size/Max.zep

@@ -12,7 +12,6 @@ namespace Phalcon\Validation\Validator\File\Size;
 
 use Phalcon\Messages\Message;
 use Phalcon\Validation;
-use Phalcon\Validation\Validator\File\Size\Max;
 use Phalcon\Validation\Validator\File\AbstractFile;
 
 /**

phalcon/Validation/Validator/File/Size/Min.zep

@@ -12,7 +12,6 @@ namespace Phalcon\Validation\Validator\File\Size;
 
 use Phalcon\Messages\Message;
 use Phalcon\Validation;
-use Phalcon\Validation\Validator\File\Size\Min;
 use Phalcon\Validation\Validator\File\AbstractFile;
 
 /**