Merge pull request #17337 from grokability/fixes-#17112-ldap-location-set-to-0

snipe · web-flow · commit 526a7ddea693 · 2025-07-02T17:20:34.000+01:00
Fixed #17112 - Set location ID to null instead of 0
diff --git a/app/Console/Commands/LdapSync.php b/app/Console/Commands/LdapSync.php
@@ -182,7 +182,7 @@ public function handle()
             // Inject location information fields
             for ($i = 0; $i < $results['count']; $i++) {
                 $results[$i]['ldap_location_override'] = false;
-                $results[$i]['location_id'] = 0;
+                $results[$i]['location_id'] = null;
             }
 
             // Grab subsets based on location-specific DNs, and overwrite location for these users.

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ public function handle()`
`182`	`182`	`// Inject location information fields`
`183`	`183`	`for ($i = 0; $i < $results['count']; $i++) {`
`184`	`184`	`$results[$i]['ldap_location_override'] = false;`
`185`		`- $results[$i]['location_id'] = 0;`
	`185`	`+ $results[$i]['location_id'] = null;`
`186`	`186`	`}`
`187`	`187`
`188`	`188`	`// Grab subsets based on location-specific DNs, and overwrite location for these users.`