Bug #4041
Sletning af reservering udløser fejl
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"
Related issues
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?
#20
Updated by Simon Holt 11 months ago
Bare i orden, Nhi :)
#22
Updated by Gitte Barlach 11 months ago
- Status changed from Resolved to Resolved (tag version)