Pues el problema esta ahí, le asignas null
y luego quieres hacer todo lo demás. Para resolverlo tienes que crear una copia o puedes elegir optimizar el código y ahorrarte unas líneas:
- Versión como tienes el código:
crearReserva(reservaEntry: ReservaPendiente): void {
const reservaCopy = { ...reservaEntry };
this.reservaPendiente = reservaEntry;
this.reservaPendiente = null;
this.reservaService.crear(reservaCopy).subscribe(
reserva => {
console.log(reserva);
this.reservaService.eliminar(reservaCopy.id);
this.router.navigate(['/reservas'])
swal.fire('Nueva reserva', `La reserva ha sido confirmada con éxito!`, 'success')
}
);
}
Se resuelve creando una shallow copy (copia superficial) para romper la referencia al objeto original:
const reservaCopy = { ...reservaEntry };
y usando reservaCopy
para las operaciones
- Versión mejorada
crearReserva(reservaEntry: ReservaPendiente): void {
this.reservaService.crear({ ...reservaEntry }).subscribe(
reserva => {
console.log(reserva);
this.reservaService.eliminar(reservaEntry.id);
this.router.navigate(['/reservas']);
swal.fire('Nueva reserva', `La reserva ha sido confirmada con éxito!`, 'success');
}
);
}
En lugar de crear y hacer tanta asignación, enviamos directamente la copia.
Por últimoEn base a tu última pregunta, si console.log(reserva)
te regresa undefined es porque no regresas nada del método crear
, si es así comparte el código para seguirte ayudando.
Como último consejo, puedes crear un método en el servicio para manejar directamente las alertas en lugar de manejarlas a nivel de componente y ahorrarte algo más de código.