Scott.Idem/OSIT-AE-App-Svelte
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: person lookup failing due to legacy person_id_random field name

Browse Source 
V3 CRUD returns 'id' as the random identifier, not 'person_id_random'.
The person check and assignment were using the old field name, causing
the 'no person record' alert even when the lookup returned valid data.

Now checks person_rec.id ?? person_rec.person_id_random as a fallback
for backwards compatibility.
This commit is contained in:
Scott Idem
2026-03-06 22:51:29 -05:00
parent 0dab64a8d6
commit e7ac5168f6
1 changed files with 14 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
src/lib/app_components/e_app_sign_in_out.svelte
View File

@@ -524,17 +524,15 @@
}) })
.then((person_response) => { .then((person_response) => {
// Safety Check: Ensure the response is valid and contains at least one record before accessing index 0. // Safety Check: Ensure the response is valid and contains at least one record before accessing index 0.
if ( // V3 CRUD returns 'id' as the random identifier; 'person_id_random' is a legacy field name.
person_response && const person_rec = person_response?.[0];
person_response.length > 0 && if (person_rec && (person_rec.id || person_rec.person_id_random)) {
person_response[0].person_id_random
) {
console.log( console.log(
`Successfully loaded person for user_id_random (${user_id}):`, `Successfully loaded person (${user_id}):`,
person_response[0] person_rec
); );
person_obj = person_response[0]; person_obj = person_rec;
person_id = person_obj.person_id_random; person_id = person_rec.id ?? person_rec.person_id_random;
trigger = true; // Set trigger to true to indicate we can now sign in trigger = true; // Set trigger to true to indicate we can now sign in
} else { } else {
@@ -544,7 +542,6 @@
} }
}) })
.then(() => { .then(() => {
// Once all promises are resolved, we can check if we have both user_id and person_id
if (user_id && person_id) { if (user_id && person_id) {
console.log(`Successfully authenticated and loaded user and person records: user_id: ${user_id}, person_id: ${person_id}`); console.log(`Successfully authenticated and loaded user and person records: user_id: ${user_id}, person_id: ${person_id}`);
} else { } else {
@@ -623,17 +620,15 @@
}) })
.then((person_response) => { .then((person_response) => {
// Safety Check: Ensure the response is valid and contains at least one record before accessing index 0. // Safety Check: Ensure the response is valid and contains at least one record before accessing index 0.
if ( // V3 CRUD returns 'id' as the random identifier; 'person_id_random' is a legacy field name.
person_response && const person_rec = person_response?.[0];
person_response.length > 0 && if (person_rec && (person_rec.id || person_rec.person_id_random)) {
person_response[0].person_id_random
) {
console.log( console.log(
`Successfully loaded person for user_id_random (${user_id}):`, `Successfully loaded person (${user_id}):`,
person_response[0] person_rec
); );
person_obj = person_response[0]; person_obj = person_rec;
person_id = person_obj.person_id_random; person_id = person_rec.id ?? person_rec.person_id_random;
trigger = true; // Set trigger to true to indicate we can now sign in trigger = true; // Set trigger to true to indicate we can now sign in
} else { } else {
@@ -643,7 +638,6 @@
} }
}) })
.then(() => { .then(() => {
// Once all promises are resolved, we can check if we have both user_id and person_id
if (user_id && person_id) { if (user_id && person_id) {
console.log(`Successfully authenticated and loaded user and person records: user_id: ${user_id}, person_id: ${person_id}`); console.log(`Successfully authenticated and loaded user and person records: user_id: ${user_id}, person_id: ${person_id}`);
} else { } else {