Add Deprecations (#27)

Co-authored-by: onairmarc <[email protected]>

.git-blame-ignore-revs

@@ -13,3 +13,4 @@ c10c3ac2613e54de290a9135afe024036174ef20
 aef6f8b59ab2ff054b1a823efe242e0cb0cd1164
 d521d82e9b14a5f411f3492529800c6b39f17740
 11b88b5f8641574fc93d2a4f443aeaf819381c91
+f74b50a0c5a93e2042dd0899f67dc0d77d9f37b2

src/Exceptions/FilesystemExceptions/DirectoryNotFoundException.php

@@ -10,9 +10,9 @@ class DirectoryNotFoundException extends BaseException
     public function __construct(?string $path = null)
     {
         if (is_null($path)) {
-            $msg = str_concat_space('Unable to locate directory on line', $this->line, 'in', $this->file);
+            $msg = "Unable to locate directory on line {$this->line} in {$this->file}";
         } else {
-            $msg = str_concat_space('Unable to locate directory', $path, 'on line', $this->line, 'in', $this->file);
+            $msg = "Unable to locate directory {$path} on line {$this->line} in {$this->file}";
         }
 
         parent::__construct($msg, ExitCode::RESOURCE_UNAVAILABLE, $this->getPrevious());

src/Exceptions/FilesystemExceptions/FileNotFoundException.php

@@ -10,9 +10,9 @@ class FileNotFoundException extends BaseException
     public function __construct(?string $path = null)
     {
         if (is_null($path)) {
-            $msg = str_concat_space('Unable to locate file on line', $this->line, 'in', $this->file);
+            $msg = "Unable to locate file on line {$this->line} in {$this->file}";
         } else {
-            $msg = str_concat_space('Unable to locate file', $path, 'on line', $this->line, 'in', $this->file);
+            $msg = "Unable to locate file {$path} on line {$this->line} in {$this->file}";
         }
 
         parent::__construct($msg, ExitCode::RESOURCE_UNAVAILABLE, $this->getPrevious());

src/Exceptions/ImproperBooleanReturnedException.php

@@ -16,7 +16,7 @@ public function __construct(bool $expected = true)
             $actual = 'true';
         }
 
-        $msg = str_concat_space('Improper boolean returned. Expected', $expect, 'but got', $actual);
+        $msg = "Improper boolean returned. Expected {$expect} but got {$actual}";
 
         parent::__construct($msg, ExitCode::DATA_ERROR, $this->getPrevious());
     }

src/Exceptions/NotImplementedException.php

@@ -8,7 +8,7 @@ class NotImplementedException extends BaseException
 {
     public function __construct()
     {
-        $msg = str_concat_space('Method is not implemented on line', $this->line, 'in', $this->file);
+        $msg = "Method is not implemented on line {$this->line} in {$this->file}";
 
         parent::__construct($msg, ExitCode::RESOURCE_UNAVAILABLE, $this->getPrevious());
     }

src/ObjectHelpers/arr_helpers.php

@@ -6,6 +6,8 @@
 if (!function_exists('arr_to_short_syntax')) {
     /**
      * @throws ArgumentNullException
+     *
+     * @deprecated No Replacement
      */
     function arr_to_short_syntax(array|string $value): array|string
     {

src/ObjectHelpers/dir_helpers.php

@@ -3,30 +3,43 @@
 use EncoreDigitalGroup\StdLib\Objects\Directory;
 
 if (!function_exists('dir_current')) {
+    /** @deprecated */
     function dir_current(): string
     {
         return Directory::current();
     }
 }
 
 if (!function_exists('dir_hash')) {
-    /** @experimental */
+    /**
+     * @experimental
+     *
+     * @deprecated
+     * */
     function dir_hash(string $dir): string
     {
         return Directory::hash($dir);
     }
 }
 
 if (!function_exists('dir_scan')) {
-    /** @experimental */
+    /**
+     * @experimental
+     *
+     * @deprecated
+     */
     function dir_scan(string $dir): array
     {
         return Directory::scan($dir);
     }
 }
 
 if (!function_exists('dir_scan_recursive')) {
-    /** @experimental */
+    /**
+     * @experimental
+     *
+     * @deprecated
+     */
     function dir_scan_recursive(string $path): array
     {
         return Directory::scanRecursive($path);

src/ObjectHelpers/file_helpers.php

@@ -12,15 +12,23 @@ function file_save_contents(string $path, mixed $data): void
 }
 
 if (!function_exists('file_copy')) {
-    /** @throws FileNotFoundException */
+    /**
+     * @throws FileNotFoundException
+     *
+     * @deprecated
+     */
     function file_copy(string $source, string $destination): void
     {
         File::copy($source, $destination);
     }
 }
 
 if (!function_exists('file_delete')) {
-    /** @throws FileNotFoundException */
+    /**
+     * @throws FileNotFoundException
+     *
+     * @deprecated
+     */
     function file_delete(string $path): void
     {
         File::delete($path);

src/ObjectHelpers/num_helpers.php

@@ -3,6 +3,7 @@
 use EncoreDigitalGroup\StdLib\Objects\Number;
 
 if (!function_exists('num_to_int')) {
+    /** @deprecated */
     function num_to_int(mixed $value): int
     {
         return Number::toInt($value);

src/ObjectHelpers/php_function_helpers.php

@@ -2,8 +2,6 @@
 
 if (!function_exists('get_type')) {
     /**
-     * @noinspection PhpMissingReturnTypeInspection
-     *
      * @phpstan-ignore-next-line
      */
     function get_type(mixed $value): string

src/ObjectHelpers/str_helpers.php

@@ -17,15 +17,23 @@ function str_camel(string $string): string
 }
 
 if (!function_exists('str_concat')) {
-    /**@param array<string, string> ...$str */
+    /**
+     * @param  array<string, string>  ...$str
+     *
+     * @deprecated
+     */
     function str_concat(mixed ...$str): string
     {
         return Str::concat($str);
     }
 }
 
 if (!function_exists('str_concat_space')) {
-    /**@param array<string, string> ...$str */
+    /**
+     * @param  array<string, string>  ...$str
+     *
+     * @deprecated
+     */
     function str_concat_space(mixed ...$str): string
     {
         return Str::concatSpace($str); //@phpstan-ignore-line
@@ -70,6 +78,7 @@ function str_lower(?string $str = null): ?string
 }
 
 if (!function_exists('str_max_length')) {
+    /** @deprecated use str_limit() instead. */
     function str_max_length(string $str, ?int $length = null): string
     {
         return Str::maxLength($str, $length);

src/ObjectHelpers/val_helpers.php

@@ -1,14 +1,15 @@
 <?php
 
 if (!function_exists('not_null')) {
+    /** @deprecated */
     function not_null(mixed $value): bool
     {
         return !is_null($value);
     }
 }
 
 if (!function_exists('val_not_null')) {
-    /** @deprecated Use not_null() instead. */
+    /** @deprecated */
     function val_not_null(mixed $value): bool
     {
         return not_null($value);

src/Objects/Arr.php

@@ -2,18 +2,20 @@
 
 namespace EncoreDigitalGroup\StdLib\Objects;
 
-use EncoreDigitalGroup\StdLib\Exceptions\ArgumentNullException;
+use EncoreDigitalGroup\StdLib\Exceptions\NullExceptions\ArgumentNullException;
 use Illuminate\Support\Arr as ArraySupport;
 use TypeError;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class Arr extends ArraySupport
 {
+    /** @deprecated use Arr::toShort() instead. */
     public static function convertToShortSyntax(array|string $code): array|string
+    {
+        return static::toShort($code);
+    }
+
+    public static function toShort(array|string $code): array|string
     {
         // Regular expression to match long array syntax
         $pattern = '/array\s*\(([^()]*(?:(?R)[^()]*)*)\)/m';
@@ -22,13 +24,14 @@ public static function convertToShortSyntax(array|string $code): array|string
         $callback = function ($matches): string {
             // Recursively convert nested arrays
             $innerArray = $matches[1];
-            $convertedInnerArray = self::convertToShortSyntax($innerArray);
+            $convertedInnerArray = self::toShort($innerArray);
 
             // Handle proper formatting of nested arrays
             $convertedInnerArray = preg_replace('/\n\s*/', ' ', $convertedInnerArray);
 
             if (!is_string($convertedInnerArray)) {
-                throw new TypeError(str_concat_space('Expected string, got ', get_type($convertedInnerArray)));
+                $type = get_type($convertedInnerArray);
+                throw new TypeError("Expected string, got {$type}");
             }
 
             return "[\n    " . $convertedInnerArray . "\n]";

src/Objects/Directory.php

@@ -7,11 +7,7 @@
 use RecursiveDirectoryIterator;
 use RecursiveIteratorIterator;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class Directory
 {
     /**

src/Objects/File.php

@@ -4,11 +4,7 @@
 
 use EncoreDigitalGroup\StdLib\Exceptions\FileNotFoundException;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class File
 {
     /**

src/Objects/Json.php

@@ -2,11 +2,7 @@
 
 namespace EncoreDigitalGroup\StdLib\Objects;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class Json
 {
     /**

src/Objects/Number.php

@@ -4,11 +4,7 @@
 
 use Illuminate\Support\Number as NumberSupport;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class Number extends NumberSupport
 {
     /**

src/Objects/Php.php

@@ -2,11 +2,7 @@
 
 namespace EncoreDigitalGroup\StdLib\Objects;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class Php
 {
     /**

src/Objects/Str.php

@@ -10,11 +10,7 @@
 
 use Illuminate\Support\Str as StringSupport;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class Str extends StringSupport
 {
     /**@param array<string, string> ...$str */

src/Objects/Url.php

@@ -2,11 +2,7 @@
 
 namespace EncoreDigitalGroup\StdLib\Objects;
 
-/**
- * @api
- *
- * @internal
- */
+/** @api */
 class Url
 {
     public static function encode(mixed $data): string

src/Objects/Value.php

@@ -8,8 +8,6 @@
  * @api
  *
  * @deprecated
- *
- * @internal
  */
 #[Deprecated]
 class Value