Merge pull request #109 from vrdist-john/hotfix/update-booleans

PrintNodeApiRequestor incorrectly casts booleans to strings

Author: rawilk

src/Api/PrintNode/PrintNodeApiRequestor.php

@@ -72,12 +72,12 @@ private static function encodeObjects(mixed $objects): mixed
             return Util::utf8($objects->id);
         }
 
-        if ($objects === true) {
-            return 'true';
+        if ($objects === 'true') {
+            return true;
         }
 
-        if ($objects === false) {
-            return 'false';
+        if ($objects === 'false') {
+            return false;
         }
 
         if (is_array($objects)) {

tests/Feature/Api/PrintNode/PrintNodeApiRequestorTest.php

@@ -59,11 +59,19 @@
     ],
     'boolean true' => fn () => [
         'value' => true,
-        'expected' => 'true',
+        'expected' => true,
+    ],
+    'string boolean true' => fn () => [
+        'value' => 'true',
+        'expected' => true,
     ],
     'boolean false' => fn () => [
         'value' => false,
-        'expected' => 'false',
+        'expected' => false,
+    ],
+    'string boolean false' => fn () => [
+        'value' => 'false',
+        'expected' => false,
     ],
 ]);