Merge pull request #16816 from tashik/fix-managerinterface

Add priority parameter to attach method in ManagerInterface

Author: niden

phalcon/Events/Manager.zep

@@ -21,8 +21,6 @@ use SplPriorityQueue;
  */
 class Manager implements ManagerInterface
 {
-    const DEFAULT_PRIORITY = 100;
-
     /**
      * @var bool
      */

phalcon/Events/ManagerInterface.zep

@@ -15,12 +15,13 @@ namespace Phalcon\Events;
  */
 interface ManagerInterface
 {
+    const DEFAULT_PRIORITY = 100;
     /**
      * Attach a listener to the events manager
      *
      * @param object|callable handler
      */
-    public function attach(string! eventType, handler) -> void;
+    public function attach(string! eventType, handler, int! priority = self::DEFAULT_PRIORITY) -> void;
 
     /**
      * Detach the listener from the events manager

tests/integration/Mvc/Dispatcher/Helper/DispatcherTestDefaultNoNamespaceController.php

@@ -1,10 +1,7 @@
-<?php
+<?php // phpcs:disable PSR1.Classes.ClassDeclaration.MissingNamespace
 
-// @codingStandardsIgnoreStart
 use Phalcon\Mvc\Controller;
 
-// @codingStandardsIgnoreSEnd
-
 /**
  * \DispatcherTestDefaultNoNamespaceController
  * Dispatcher Controller for testing different dispatch scenarios
@@ -23,8 +20,8 @@
  */
 class DispatcherTestDefaultNoNamespaceController extends Controller
 {
-    const RETURN_VALUE_STRING = 'string';
-    const RETURN_VALUE_INT    = 5;
+    public const RETURN_VALUE_STRING = 'string';
+    public const RETURN_VALUE_INT    = 5;
 
     public function beforeExecuteRoute()
     {