mirror of
https://github.com/mauriceboe/TREK.git
synced 2026-06-19 13:21:46 +00:00
fix(reservations): always update place_id when saving hotel accommodation
When clearing the accommodation place from a hotel reservation, the update branch that runs without a place_id omitted the column from its UPDATE statement, leaving the old place linked in day_accommodations. Collapse the two branches into one that always writes place_id (null or value).
This commit is contained in:
@@ -278,13 +278,8 @@ export function updateReservation(id: string | number, tripId: string | number,
|
|||||||
const { place_id: accPlaceId, start_day_id, end_day_id, check_in, check_out, confirmation: accConf } = create_accommodation;
|
const { place_id: accPlaceId, start_day_id, end_day_id, check_in, check_out, confirmation: accConf } = create_accommodation;
|
||||||
if (start_day_id && end_day_id) {
|
if (start_day_id && end_day_id) {
|
||||||
if (resolvedAccId) {
|
if (resolvedAccId) {
|
||||||
if (accPlaceId) {
|
|
||||||
db.prepare('UPDATE day_accommodations SET place_id = ?, start_day_id = ?, end_day_id = ?, check_in = ?, check_out = ?, confirmation = ? WHERE id = ?')
|
db.prepare('UPDATE day_accommodations SET place_id = ?, start_day_id = ?, end_day_id = ?, check_in = ?, check_out = ?, confirmation = ? WHERE id = ?')
|
||||||
.run(accPlaceId, start_day_id, end_day_id, check_in || null, check_out || null, accConf || confirmation_number || null, resolvedAccId);
|
.run(accPlaceId || null, start_day_id, end_day_id, check_in || null, check_out || null, accConf || confirmation_number || null, resolvedAccId);
|
||||||
} else {
|
|
||||||
db.prepare('UPDATE day_accommodations SET start_day_id = ?, end_day_id = ?, check_in = ?, check_out = ?, confirmation = ? WHERE id = ?')
|
|
||||||
.run(start_day_id, end_day_id, check_in || null, check_out || null, accConf || confirmation_number || null, resolvedAccId);
|
|
||||||
}
|
|
||||||
} else if (accPlaceId) {
|
} else if (accPlaceId) {
|
||||||
const accResult = db.prepare(
|
const accResult = db.prepare(
|
||||||
'INSERT INTO day_accommodations (trip_id, place_id, start_day_id, end_day_id, check_in, check_out, confirmation) VALUES (?, ?, ?, ?, ?, ?, ?)'
|
'INSERT INTO day_accommodations (trip_id, place_id, start_day_id, end_day_id, check_in, check_out, confirmation) VALUES (?, ?, ?, ?, ?, ?, ?)'
|
||||||
|
|||||||
Reference in New Issue
Block a user