Update api/app/Http/Requests/UserFormRequest.php

JhumanJ · coderabbitai[bot] · web-flow · commit 29108db28445 · 2025-02-14T15:14:43.000Z
Co-authored-by: coderabbitai[bot] &lt;136622811+coderabbitai[bot]@users.noreply.github.com&gt;
diff --git a/api/app/Http/Requests/UserFormRequest.php b/api/app/Http/Requests/UserFormRequest.php
@@ -19,11 +19,12 @@ protected function prepareForValidation()
         $data = $this->all();
 
         if (isset($data['properties']) && is_array($data['properties'])) {
-            foreach ($data['properties'] as &$property) {
-                if (isset($property['help']) && strip_tags($property['help']) === '') { // Remove help if it's empty
+            $data['properties'] = array_map(function ($property) {
+                if (isset($property['help']) && is_string($property['help']) && strip_tags($property['help']) === '') {
                     $property['help'] = null;
                 }
-            }
+                return $property;
+            }, $data['properties']);
         }
 
         $this->merge($data);

Original file line number	Diff line number	Diff line change
`@@ -19,11 +19,12 @@ protected function prepareForValidation()`
`19`	`19`	`$data = $this->all();`
`20`	`20`
`21`	`21`	`if (isset($data['properties']) && is_array($data['properties'])) {`
`22`		`- foreach ($data['properties'] as &$property) {`
`23`		`- if (isset($property['help']) && strip_tags($property['help']) === '') { // Remove help if it's empty`
	`22`	`+ $data['properties'] = array_map(function ($property) {`
	`23`	`+ if (isset($property['help']) && is_string($property['help']) && strip_tags($property['help']) === '') {`
`24`	`24`	`$property['help'] = null;`
`25`	`25`	`}`
`26`		`- }`
	`26`	`+ return $property;`
	`27`	`+ }, $data['properties']);`
`27`	`28`	`}`
`28`	`29`
`29`	`30`	`$this->merge($data);`