Cleaned small mistakes.

msyk · Nov 2, 2024 · 5d5733a · 5d5733a
1 parent 90dacb8
commit 5d5733a
Show file tree

Hide file tree

Showing 5 changed files with 13 additions and 8 deletions.
diff --git a/src/php/DB/Settings.php b/src/php/DB/Settings.php
@@ -915,7 +915,7 @@ public function getAuthentication(): ?array
      * @param string|null $key
      * @return float|int|mixed|string|null
      */
-    public function getAuthenticationItem(?string $key) :mixed
+    public function getAuthenticationItem(?string $key): mixed
     {
         if (isset($this->authentication[$key])) {
             return $this->authentication[$key];

diff --git a/src/php/DB/Support/ProxyVisitors/DescribeVisitor.php b/src/php/DB/Support/ProxyVisitors/DescribeVisitor.php
@@ -22,7 +22,7 @@ public function visitIsAuthAccessing(OperationElement $e): bool
 
     /**
      * @param OperationElement $e
-     * @return void
+     * @return bool
      */
     public function visitCheckAuthentication(OperationElement $e): bool
     {

diff --git a/src/php/DB/Support/ProxyVisitors/OperationVisitor.php b/src/php/DB/Support/ProxyVisitors/OperationVisitor.php
@@ -5,7 +5,6 @@
 use Exception;
 use INTERMediator\DB\Logger;
 use INTERMediator\DB\Proxy;
-use INTERMediator\DB\Support\ProxyElements\CheckAuthenticationElement;
 use INTERMediator\DB\Support\ProxyElements\OperationElement;
 use INTERMediator\FileUploader;
 use INTERMediator\IMUtil;
@@ -346,8 +345,7 @@ function CreateReplaceImpl(string $access): void
     /**
      * @return void
      */
-    protected
-    function defaultHandleChallenge()
+    protected function defaultHandleChallenge(): void
     {
         $proxy = $this->proxy;
         Logger::getInstance()->setDebugMessage("[handleChallenge] access={$proxy->access}, succeed={$proxy->authSucceed}", 2);

diff --git a/src/php/IMUtil.php b/src/php/IMUtil.php
@@ -243,7 +243,7 @@ public static function isPHPExecutingUNIX(): bool
      * @param $str
      * @return array|string
      */
-    public static function removeNull($str):array|string
+    public static function removeNull($str): array|string
     {
         return str_replace("\x00", '', $str ?? "");
     }
@@ -846,8 +846,15 @@ public static function isRunAsWebApp(): bool
         return true;
     }
 
-    public static function getFromProfileIfAvailable(string $str): string
+    /**
+     * @param string|null $str
+     * @return string|null
+     */
+    public static function getFromProfileIfAvailable(?string $str): string|null
     {
+        if (is_null($str)) {
+            return null;
+        }
         $comp = array_map(function ($elm) {
             return strtolower(trim($elm));
         }, explode('|', $str));

diff --git a/src/php/Params.php b/src/php/Params.php
@@ -57,7 +57,7 @@ private static function readParamsPHPFile(): void
     }
 
     /**
-     * @param mixed $vName
+     * @param string|array $vName
      * @param mixed $defValue
      * @return mixed
      */