Merge pull request #9 from NuclearDog/master

travisghansen · web-flow · commit 4a320b14035a · 2023-10-17T09:09:43.000-05:00
Fix notice returning reference in DotAccess::get
diff --git a/src/KubernetesClient/Dotty/DotAccess.php b/src/KubernetesClient/Dotty/DotAccess.php
@@ -88,7 +88,10 @@ public static function &get(&$currentValue, $key, $default = null) {
             $currentValue = &self::propGet($currentValue, $currentKey);
         }
 
-        return $currentValue === null ? $default : $currentValue;
+        if ($currentValue === null)
+            return $default;
+        else
+            return $currentValue;
     }
 
     public static function exists(&$currentValue, $key) {

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,10 @@ public static function &get(&$currentValue, $key, $default = null) {`
`88`	`88`	`$currentValue = &self::propGet($currentValue, $currentKey);`
`89`	`89`	`}`
`90`	`90`
`91`		`- return $currentValue === null ? $default : $currentValue;`
	`91`	`+ if ($currentValue === null)`
	`92`	`+ return $default;`
	`93`	`+ else`
	`94`	`+ return $currentValue;`
`92`	`95`	`}`
`93`	`96`
`94`	`97`	`public static function exists(&$currentValue, $key) {`