Project

General

Profile

Bug #4041

Sletning af reservering udløser fejl

Added by Tue Gaston 12 months ago. Updated 11 months ago.

Status:
Resolved (tag version)
Priority:
Urgent
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Min konto - Reserveringer

Description

Jeg er logget ind på upgrade med testbruger Caroline Conrad DDB.

Jeg går ind på "Reserveringer i kø" under lånerstatus https://upgrade-fbs.ddbcms.dk/user/me/status-reservations

Jeg vælger et materiale.

Jeg klikker på "Slet valgte reserveringer"

Jeg klikker på "Ja" under "Er du sikker på at du vil slette valgte reserveringer"

Så sker der ikke noget i meget lang tid.

Efter 5-10 minutter kommer denne fejlmeddelelse:

... og reserveringen er IKKE slettet.

Samme fejl er registreret på vanilla, og også ved klik på "Slet alle reserveringer" i stedet for blot "Slet valgte reserveringer"

Bug4014.png (65 KB) Bug4014.png Bug#4041 Nhi A Sy, 01/24/2019 02:35 PM

Related issues

Related to DDB CMS - Bug #3936: Visning af fjernlån i lånerstatus (reserveringer og lån) fejler og giver "Error: missing/unknown/inaccessible record..." LØSES MED #4512Resolved (tag version)

History

#1 Updated by Christel Krabbenhøft 12 months ago

  • Assignee changed from Christel Krabbenhøft to Simon Holt

Simon, kan dette på nogen måde hænge sammen med MWA-liste implementeringen? Issue #3901?

#2 Updated by Simon Holt 12 months ago

Nej, det er ikke det kan jeg se på fejlen. Men har en ide om, hvad det er. Jeg kigger lige på det.

#3 Updated by Tue Gaston 12 months ago

Jeg skal lige tilføje, at da jeg efterfølgende forsøgte at genskabe problemet, modtog jeg ikke den fejlmeddelelse, jeg har sat et screenshot ind af.
Men der skete stadig ingenting efter klik på "Ja" under "Er du sikker på at du vil slette valgte reserveringer" - og jeg får stadig ikke slettet reserveringerne.

#4 Updated by Tue Gaston 12 months ago

Tue Gaston wrote:

Jeg skal lige tilføje, at da jeg efterfølgende forsøgte at genskabe problemet, modtog jeg ikke den fejlmeddelelse, jeg har sat et screenshot ind af.
Men der skete stadig ingenting efter klik på "Ja" under "Er du sikker på at du vil slette valgte reserveringer" - og jeg får stadig ikke slettet reserveringerne.

Og kan det være samme problem som https://platform.dandigbib.org/issues/4039 i ny forklædning?

#5 Updated by Simon Holt 12 months ago

Ja, tror det er samme problem.

#6 Updated by Simon Holt 12 months ago

PR der fikser fejlen ved sletning af reservationer: https://github.com/ding2/ding2/pull/1311

Fejlen der er postet i beskrivelsen er denne sag, har ikke noget at gøre med fejlen ved sletning af reservation. Den fejl der er synderen her er:

Recoverable fatal error: Argument 2 passed to fbs_reservation_delete() must be of the type array, string given i fbs_reservation_delete()

Det skyldes en mindre fejl/misforståelse ifb med arbejdet i https://platform.dandigbib.org/issues/1261.

Efter denne rettelse havde jeg heller ikke problemer med at reservere. Men søge-spinneren kom stadig frem og har altså ikke noget at gøre med at reseveringen fejler. Søge spinneren kom frem, men efter jeg lukkede pop-up vinduet, der kom frem efter reservation, forsvandt spinneren også og resevationen var foretaget. Jeg foreslår vi kigger på hvorfor spinneren pludselig kommer frem ved reservation i #4039 (og ændrer titlen, for der sker ingen fejl, men det er bare spinneren der fejlagtigt dukker op)

UPDATE: Se bort fra ovenstående. Tue viser netop også i sagen #4039, at reseveringen går godt og at problemet kun er at søge-spinneren pludselig dukker op.

Fejlen med "Unsupported operand types" der nævnes i beskrivelsen af denne sag er alvorlig, men foreslå vi tager den i en anden sag, da det er et symptom på et mere generel problem, der opstået efter vi er gået over til getObject, når der skal hentes flere objekter ned (#2223). getObject opfører sig meget underligt i nogle tilfælde (speciel når den ikke kan finde objekterne) og vil foreslå vi tager den i https://platform.dandigbib.org/issues/3936, da det er her problemet først blev opdaget. Jeg har allerede lavet et PR i denne sag, men det var en lappeløsning lige til dette problem og som vi ser, er der flere problemer med at getObject returnerer noget underligt i visse tilfælde.

Det er vigtigt at ovenstående læses omhyggeligt igennem. Ellers ender det med at blive meget kompliceret og forvirrende det her, hvis vi får blandet for mange ting sammen.

#7 Updated by Simon Holt 12 months ago

Jeg har givet en mere detaljeret forklaring på problemerne med objekt request og det der returneres i: https://platform.dandigbib.org/issues/3936#note-31

#8 Updated by Simon Holt 12 months ago

  • Related to Bug #3936: Visning af fjernlån i lånerstatus (reserveringer og lån) fejler og giver "Error: missing/unknown/inaccessible record..." LØSES MED #4512 added

#9 Updated by Simon Holt 12 months ago

Vil lige tilføje, at den fejl Tue oplever i #4039 med at reservationen slet ikke bliver foretaget, meget vel kan have noget at gøre med problemerne med Object request som jeg beskriver i ovenstående. Men er rimelig sikker på, det ikke har noget at gøre med at søge-spinneren pludselig kommer og at det er en separat problematik.

UPDATE: Arrg.. beklager, Tue: du skriver jo netop at reseveringen faktisk bliver foretaget i #4039. Ved ikke hvordan jeg fik det en i hovedet. Se bort fra ovenstående.

#10 Updated by Christel Krabbenhøft 12 months ago

  • Status changed from New to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

Tak for super hurtig action, Simon! Du er en knag:-) Vi får den sagen code reviewet hurtigst muligt i det nye år. //Christel

#11 Updated by Gitte Barlach 11 months ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs
  • Target version set to Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0)

#12 Updated by Gitte Barlach 11 months ago

  • Priority changed from Normal to Urgent

#13 Updated by Kasper Garnæs 11 months ago

  • Status changed from Needs code review to Reviewed

Reviewed og godkendt.

#14 Updated by Kasper Garnæs 11 months ago

  • Status changed from Reviewed to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Merged.

#15 Updated by Nhi A Sy 11 months ago

  • File Bug4014.png Bug4014.png added
  • Status changed from Technical test to Needs code review
  • Assignee changed from Gitte Barlach to Simon Holt

Lige prøvet og slette en reservation med samme bruger, den responder med en error 500, se venligst vedhæftet fil

#16 Updated by Simon Holt 11 months ago

Den siger "with message". Kan du prøve at finde message fra response og poste den her?

#17 Updated by Nhi A Sy 11 months ago

  • Status changed from Needs code review to Technical test
  • Assignee changed from Simon Holt to Gitte Barlach

#18 Updated by Nhi A Sy 11 months ago

Glemt det lige Simon, der er mig der har misforstået noget :-)

#19 Updated by Nhi A Sy 11 months ago

  • Assignee changed from Gitte Barlach to Nhi A Sy

#20 Updated by Simon Holt 11 months ago

Bare i orden, Nhi :)

#21 Updated by Nhi A Sy 11 months ago

  • Status changed from Technical test to Resolved

Testet på release 7.x-4.6.0-rc6 på upgrade-fbs.ddbcms.dk

Sletning af 1 eller flere reservering er ok, med brugeren "
Caroline Conrad DDB"

#22 Updated by Gitte Barlach 11 months ago

  • Status changed from Resolved to Resolved (tag version)

Also available in: Atom PDF