Skip to main content
se añadieron 22 caracteres en el cuerpo
Origen Enlace
René
  • 1.2k
  • 3
  • 14

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:

  1. 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

  1. 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.

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:

  1. 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

  1. 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 último, 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.

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:

  1. 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

  1. 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.

En 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.

Origen Enlace
René
  • 1.2k
  • 3
  • 14

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:

  1. 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

  1. 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 último, 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.