Update for PHP 8.3 and Phalcon 5.8, fix tests.

src/AdminCabinet/Controllers/FirewallController.php

@@ -432,22 +432,26 @@ public function disableAction(): void
     }
 
     /**
-     * Sort array. The localnet and 0.0.0.0 should be at the first position on the list
+     * Compare two network entries for sorting
      *
-     * @param $a
-     * @param $b
-     *
-     * @return bool
+     * @param array $a First network entry
+     * @param array $b Second network entry
+     * @return int Returns -1 if $a should be placed before $b,
+     *             1 if $a should be placed after $b,
+     *             0 if they are considered equal
      */
-    private function sortArrayByNetwork($a, $b): bool
+    private function sortArrayByNetwork(array $a, array $b): int
     {
+        // If second entry is permanent and first is not 0.0.0.0/0
         if ($b['permanent'] && $a['network'] !== '0.0.0.0/0') {
-            return true;
+            return 1; // Move $a after $b
         }
+
+        // If second entry is 0.0.0.0/0
         if ($b['network'] === '0.0.0.0/0') {
-            return true;
+            return 1; // Move $a after $b
         }
 
-        return false;
+        return -1; // In all other cases, move $a before $b
     }
 }

src/AdminCabinet/Views/IncomingRoutes/modify.volt

@@ -26,7 +26,7 @@
 {{ partial("partials/playAddNewSound", ['label': t._('iv_PlaySound'), 'id':'audio_message_id', 'fieldClass':'eleven wide field', 'fieldId':'']) }}
 <h3 class="ui dividing header ">{{ t._("ir_CallTransferTo") }}</h3>
 
-<div class="field max-width-500">
+<div class="field max-width-800">
     <label for="extension"></label>
     {{ form.render('extension') }}
 </div>

src/PBXCoreREST/Middleware/AuthenticationMiddleware.php

@@ -1,4 +1,5 @@
 <?php
+
 /*
  * MikoPBX - free phone system for small business
  * Copyright © 2017-2023 Alexey Portnov and Nikolay Beketov
@@ -30,7 +31,6 @@
 use Phalcon\Mvc\Micro;
 use Phalcon\Mvc\Micro\MiddlewareInterface;
 
-
 /**
  * Class AuthenticationMiddleware
  *
@@ -70,18 +70,19 @@ public function call(Micro $application): bool
             return false;
         }
 
-        if (true !== $isNoAuthApi
-         && true !== $request->isLocalHostRequest()
-         && true !== $request->isAllowedAction($application)) {
+        if (
+            true !== $isNoAuthApi
+            && true !== $request->isLocalHostRequest()
+            && true !== $request->isAllowedAction($application)
+        ) {
              $this->halt(
                  $application,
-                $response::FORBIDDEN,
-                'The route is not allowed'
-            );
+                 $response::FORBIDDEN,
+                 'The route is not allowed'
+             );
             return false;
         }
 
         return true;
     }
-
-}
+}