Project

General

Profile

Bug #1460

Reserveringsknap på årbøger giver problemer

Added by Tina Skjærbæk Søeborg over 3 years ago. Updated 15 days ago.

Status:
Need more info
Priority:
Urgent
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Forsiden, Søgning - Materialevisning, Driftsvedligehold - Manual (udarbejdes før udrulning)

Description

Der er et problem med reserveringsknapper på årbøger, som er en speciel variant af fænomenet med reserveringsknap på tidsskrifter og periodika

Sagen er, at nogle årbøger har årgangen og numre, andre har det ikke.

Hvis man aktiverer reserveringsknappen for årbøger, vises knappen på hovedposten uanset om materialet har årgange og numre. Hvis en låner klikker på knappen, får de en fejlmeddelelse "Only library user can make reservations."

Se fx https://hvidovre-stg.ddbcms.dk/ting/object/870970-basis%3A51174275

Hvis man derimod fjerner hak i reserveringsknap for årbøger, fjernes knappen på hovedposten på ALLE årbøger, også dem der ikke har årgange og numre. For deres vedkommende betyder det så, at de slet ikke kan reserveres.

Sådan har vi sat det op på vores produktionsmiljø for at undgå ovenstående fejl.
Se fx https://www.hvidovrebib.dk/ting/object/870970-basis%3A51837231

Der er altså behov for at finde en løsning, så reserveringsknappen vises på hovedposten, hvis der ikke er årgange eller numre, og vises på de enkelte årgange/numre, hvis der er det.

Skærmbillede 2019-06-11 kl. 10.59.36.png (70.1 KB) Skærmbillede 2019-06-11 kl. 10.59.36.png Der er res.knap, men materialet kan ikke reserveres alligevel. Gitte Barlach, 06/11/2019 11:22 AM
Skærmbillede 2019-06-11 kl. 09.26.22.png (63.9 KB) Skærmbillede 2019-06-11 kl. 09.26.22.png Der er res.knap, men materialet kan ikke reserveres alligevel. Gitte Barlach, 06/11/2019 11:22 AM
Skærmbillede 2019-06-11 kl. 09.32.01.png (95.4 KB) Skærmbillede 2019-06-11 kl. 09.32.01.png Værkvisning. Nogle har res.knap andre har ikke Gitte Barlach, 06/11/2019 11:23 AM
Skærmbillede 2019-06-11 kl. 09.32.27.png (89.7 KB) Skærmbillede 2019-06-11 kl. 09.32.27.png Fordi man skal helt ind på materialevisningen og folde ud for at ses res.knappen Gitte Barlach, 06/11/2019 11:24 AM
1460-årbog-værk-med-pif-og-alm-poster.PNG (57.2 KB) 1460-årbog-værk-med-pif-og-alm-poster.PNG Simon Holt, 06/12/2019 12:36 PM
1460-værkvisning-på-greve-med-resknap.PNG (351 KB) 1460-værkvisning-på-greve-med-resknap.PNG Simon Holt, 06/12/2019 12:48 PM
1460-periodical-undefinde-reservability-check.PNG (71.9 KB) 1460-periodical-undefinde-reservability-check.PNG Simon Holt, 06/12/2019 01:52 PM

Related issues

Related to DDB CMS - Bug #2530: Problemer med reservering af tidsskrifterResolved (tag version)
Related to DDB CMS - Enhancement #3325: Abstraktion af materialetyper for tidsskrifterResolved (tag version)
Is duplicate of DDB CMS - Bug #1545: Visse periodikum kan ikke reserveresClosed

History

#1 Updated by Rolf Madsen over 3 years ago

  • Status changed from New to Open (waiting)
  • Priority changed from Normal to Urgent
  • Target version set to DDB CMS 2016 1. opgradering

#2 Updated by Tina Skjærbæk Søeborg over 3 years ago

Min beskrivelse ovenfor af scenariet med reserveringsknap på hovdposten er ikke helt korrekt.

Fejlmeddelelse "Only library user can make reservations." skyldtes, at jeg var logget ind som webmaster, da jeg forsøgte at lave reservationen.

Problemet eksisterer som beskrevet, men hvis man klikker på reserver-knappen på en årbogs hovedpost får man teksten "[xxx] kan ikke reserveres."

Så fejlmeddelelsen er altså mere meningsfuld, en jeg først skrev, men den betyder stadig, at folk ikke opdager, at de ville kunne reservere indiduelle årgange, hvis de klappede niveauerne ud.

Det angivne eksempel var heller ikke en god illustration, da den kun har en årgang, der dog er oprettet som årgang (3. variant), så den kunne man resevere.

Et bedre eksempel er denne, hvor der er flere årgange og man får fejlmeddelelsen:
https://hvidovre-stg.ddbcms.dk/ting/object/716700-katalog%3A90196049

#4 Updated by Rolf Madsen over 3 years ago

  • Target version changed from DDB CMS 2016 1. opgradering to DDB CMS - Analyse og prioritering udestår

#5 Updated by Rolf Madsen over 2 years ago

  • Is duplicate of Bug #1545: Visse periodikum kan ikke reserveres added

#6 Updated by Rolf Madsen over 2 years ago

  • Priority changed from Urgent to None
  • Kategorier Søgning - Materialevisning added

Hej Tina.

Jeg har vurderet at Steens beskrivelse på #1545 er mere udfoldet og jeg vil derfor lukke dette issue.

Hvis der er pointer i dette issue som du mener Steen har overset vil du så ikke overføre dem?

På forhånd tak!!

#7 Updated by Tina Skjærbæk Søeborg over 2 years ago

Hej Rolf

Jeg mener ikke, det er samme problem. Steen beskriver, at der er noget uoverensstemmelse mellem typen i dkabm-posten og marc-posten, som giver problemer for periodika.

Problemet for årbøger er ikke, at typen nogle gange er forkert. Problemet er, at der er årbøger både med og uden årgange og numre. For at kunne reservere de enkelte årgange for de materialers vedkommen, der har årgange, er man nødt til at fjerne hakket i "provider availability holdings", så der ikke kommer en reserver knap på hovedposten.
Men det betyder så for de årbøger, der IKKE har årgange og numre, slet ikke har nogen reserveringsknap, selvom de burde kunne reserveres, og det er jo ærgerligt.

Periodika vil altid have numre, så for deres vedkommende er det ikke noget problem.

#8 Updated by Rolf Madsen over 2 years ago

  • Status changed from Open (waiting) to Needs analysis
  • Priority changed from None to High
  • Target version changed from DDB CMS - Analyse og prioritering udestår to Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1)

Jeg kan godt se hvad du siger.

Så grundlæggende kan vi ikke nøjes med at tjekke availability før vi indsætter Reservér knappen.

Vi er nødt til at tjekke beholdningen, og hvis årgang/nummer:

  1. findes på posten skal Reservér knappen IKKE vises på posten, men ud for hver årgang/nummer visning som vi er vant på tidsskrifter
  2. IKKE findes på posten skal Reservér knappen vises på posten som vi er vant til fra bøger og andre materialer end tidsskrifterne.

#9 Updated by Rolf Madsen over 2 years ago

  • Assignee set to Tina Skjærbæk Søeborg

#10 Updated by Tina Skjærbæk Søeborg over 2 years ago

Lige præcis - og så i øvrigt kommunikere ud til alle os, der har slået knappen fra lige nu pga. fejlen, at vi skal slå den til igen :)

#11 Updated by Steen Larsen over 2 years ago

Jeg har ikke checket hvordan sagen er med FBS og brønd 3.5 - mine gamle kommentarer i #1545 går på Alma og brønd 3.0
Meget af min kommentar dér går på at dkabm-posten ikke altid giver et godt bud på om knappen skal vises på den ene eller anden måde.

Måske skal vi simpelthen droppe dkabm-posten og udelukkende se på hvad provideren giver - som jeg skriver:
"Der bør kigges på om visning af beholdningsoplysningerne udelukkende baseres på provider-oplysningerne istedet for også at afhænge af hvad dkabm-posten leverer"

Dvs uanset hvad dkabm-posten fortæller (og som vi viser i titler, beskrivelser m.m.) så er oplysningerne derfra ikke ansvarlige om der skal være reserver-knapper på den ene eller anden måde - de vises afhængig af provideren svarer.

Går vi den vej kan de to sager samles i en sag.

#12 Updated by Rolf Madsen over 2 years ago

  • Assignee changed from Tina Skjærbæk Søeborg to Thomas Hansen
  • Target version changed from Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1) to Release 27 - Bugfixes (Reload)

#13 Updated by Thomas Hansen about 2 years ago

  • Status changed from Needs analysis to Needs code review

Fixed: https://github.com/ding2/ding2/pull/588

Løsningen er en smule hackery med at skifte over til at bruge holdings istedet for items hvis der er en reserveringsknap på siden, men det var det der lige var til uden at skulle refaktorere ding_reservations, ding_availability og ding_periodical.

#14 Updated by Gitte Barlach about 2 years ago

  • Assignee changed from Thomas Hansen to Jørgen Nielsen
  • Target version changed from Release 27 - Bugfixes (Reload) to DDB CMS 2017 1. opgradering (Reload sprintbacklog)

#15 Updated by Jørgen Nielsen about 2 years ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

reviewet og godkendt

#16 Updated by Kasper Garnæs about 2 years ago

  • Status changed from Reviewed to Technical test

Merged.

#17 Updated by Rolf Madsen about 2 years ago

  • Status changed from Technical test to Closed

#18 Updated by Rolf Madsen about 2 years ago

  • Status changed from Closed to Technical test

#19 Updated by Simon Holt about 2 years ago

Jeg har testet dette her ifb. med #1455 og det ser desværre ikke ud til at virke.

Jeg har verificeret at den rent faktisk finder reserveringsknappen her: https://github.com/ding2/ding2/blob/master/modules/ding_availability/js/ding_availability.js#L39, men det får den ikke fjernet alligevel og både "hovedreserveringsknappen" og reserveringsknapperne under "Årgang og numre" vises dermed.

Jeg kan dog godt lide valget af løsning:
1. Vi aktiverer reserveringsknap for alle årbøger under admin/config/ting/reservable
2. Hvis opstillingen viser sig at være en PIF-post og have issues, fjerner/dsiable vi bare "hovedreserveringsknappen"

Vil dog foreslå vi laver noget i den her retning i ding_periodical:

https://github.com/ding2/ding2/pull/642

Har testet ovenstående og får nu ikke længere en ikke-virkende reserveringsknap på årbøger lavet som en PIF-post

#20 Updated by Lotte Tøstesen almost 2 years ago

  • Status changed from Technical test to Need more info
  • Assignee changed from Gitte Barlach to Simon Holt

Hej Simon
Skal lige forstå det rigtigt, har du lavet en rettelse her, som vi skal have codereviewet og merged, eller er det et løsningsforslag som ikke er færdiglavet endnu?? Jeg synes da, det ser ud til at virke - men kan du give mig nogle eksempelmaterialer som fejler?

#21 Updated by Simon Holt almost 2 years ago

Hej Lotte

Der er ikke noget der er klar til at bliver merget her.

Jeg kunne ikke få Thomas' fiks til at virke, så prøvede en anden tilgang og fik det til at virke med at den fjernede reservationsknappen, hvis ding_periodical fandt periodiske oplysninger i beholdningsdata.

Det er efterfølgende gået op for mig, at jeg ikke havde taget højde for, at der også skal vises de normale holdings i disse tilfælde.

Men spørgsmålet er, om vi ikke skal droppe det. For har fundet ud af i #1455 at reservation alligevel vil fejle for årbøger, når der ikke kommer en "Årgang og numre", da det er et udtryk for manglende periodica-oplysninger i beholdingen. Der er simpelthen en regel i FBS, der forhindrer reservation, som jeg har forklaret nærmere i #1455. Så hvis "den enkelte" reserveringsknap skal virke, skal vi gentænke koden i disse tilfælde og få periodica-oplysninger til at sende med til FBS API et andet sted.

Har spurgt Systematic om de vil kigge på at ændre den regel der forhindrer reservation, men det skal formuleres som et ændringsønske til Kombit.

Mit forslag er, at vi opfordrer biblioteker til at finde disse problematiske årbøger i Cicero og manuelt at sørge for at periodica-oplysninger er korrekt angivet i behold.

#22 Updated by Steen Larsen almost 2 years ago

Måske skulle vi dykke ned i hvorfor pull-requestet ovenfor ikke virker?

Det er vigtigt at vi ikke tester på materialer som helt mangler de periodiske oplysninger for det virker ikke.
Men der er flere materialer som ikke er de kendte periodika-typer men som faktisk alligevel har reelle år/volumen/nummer-informationer.

Som jeg har gjort tidligere i #1545 kan disse materialer søges frem ved at søge på ma=pe og evt. inkludere de underlige typer eller udelade de kendte.

Via facet-søgning på ma=pe kan man finde disse term.typer (her for aakb) som har materialer der er periodika:

  1. årbog, periodikum, tidsskrift
  2. avis, serie
  3. bog, cd (musik), cd-rom, dvd-rom, dvd, graphic novel, mikroform, pc-spil, playstation 2, playstation, sammensat materiale, tegneserie
  4. netdokument, periodikum (net), avis (net), tidsskrift (net)

1) kender vi
2) er nok nogle vi har glemt
3) er de underlige typer
4) skal ikke vise beholdning af andre årsager

De periodika der mangler oplysninger er desværre også inkluderet i ovenstående 1-3

#23 Updated by Simon Holt almost 2 years ago

> Men der er flere materialer som ikke er de kendte periodika-typer men som faktisk alligevel har reelle år/volumen/nummer-informationer.

Hvis et materiale har år/volumen/nummer-informationer i bibliotekssystemet kommer "Årgang og numre" automatisk frem uanset materialetype. Der er ikke noget check i DDB CMS på term.type eller lignende her, den kalder bare availability holdings og leder efter issues.

Dette hænger ikke så godt sammen med den valgte fremgangsmåde med at vælge hvilke term.typer der skal have en enkelt reserveringsknap øverst under admin/config/ting/reservable. Så hvis vi skal arbejde videre med det her, synes jeg vi skal kigge på at gentænke dette. Umiddelbart ser jeg to løsninger:

1. Vi beholder modellen med at vinke af under admin/config/ting/reservable, men refaktorerer ding_periodical (der viser "Årgang og numre") og ding_availability (der viser holdings), så den spiller sammen med det. Ergo, der bliver aldrig vist både "Årgang og numre" plus en reserveringsknap øverst og almindelige holdings. Og hvis der eksempelvis kun er en/et år/volumen/nummer præsenterer vi bare en enkelt reserveringsknap. Man vinker bare af under admin/config/ting/reservable og så finder systemet selv ud af at præsentere reservingsmuligheder på den mest hensigstmæssige måde.
2. Vi dropper admin/config/ting/reservable og baserer det hele på beholdingsoplysninger fra bibliotekssystemet, som det er nævnt i http://platform.dandigbib.org/issues/1545#note-7.

2. lyder fristende, men kan der være nogle situationer hvor biblioteker rent faktisk ønsker den kontrol, som 1. giver?

Selv om vi fikser dette, vil det alligevel ikke være muligt at reservere ma=pe materialer uden periodica-oplysninger. Vi kan ikke på nogen måde reservere disse materialer via DDB CMS. Har prøvet at sende tom-streng/null med i alle periodica-felterne, men intet virker i disse tilfælde.

Så problematikken, som den her sag startede med, med at der ikke kommer "Årgang og numre", da der mangler periodica-oplysninger i bibliotekssystemet og disse materailer dermed ikke kan reserveres, kan vi simpelthen ikke løse med rettelser til DDB CMS.

#24 Updated by Rolf Madsen almost 2 years ago

  • Status changed from Need more info to Needs analysis
  • Target version changed from DDB CMS 2017 1. opgradering (Reload sprintbacklog) to Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1)
  • Kategorier Inspiration - Forsiden added

#25 Updated by Rolf Madsen over 1 year ago

  • Related to Bug #2530: Problemer med reservering af tidsskrifter added

#26 Updated by Rolf Madsen over 1 year ago

  • Description updated (diff)

Er fejlen omkring "at der ikke kommer "Årgang og numre", da der mangler periodica-oplysninger i bibliotekssystemet og disse materailer dermed ikke kan reserveres" meldt ind til Systematic i Sagsnr: 55687, som fremgår af https://platform.dandigbib.org/issues/2530?next_issue_id=2511&prev_issue_id=2532#note-30?

Løser forslag 2 så grundlæggende at Reservér knappen så ikke vises på posten som et "normalt" materiale, hvis et materiale har årgang/nummeroplysninger?

#27 Updated by Rolf Madsen over 1 year ago

  • Target version changed from Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1) to Release 29-2 - Bugfixes (7.x-4.5.0)

#28 Updated by Steen Larsen over 1 year ago

Sagen der blev sendt til systematic i #2530#note-30 skulle være løst, men jeg har mine forbehold jvf #2530#note-49

For at kunne håndtere tom streng/null-værdier i periodical skulle jsonmapper opdateres hvorefter fejl i swagger pludselig blev synlige.
Det vil vise sig om der stadig er fejl.

I #2530#note-42 har jeg nævnt de 3 mulige fejlkilder der er kendt - pkt 1 håndteres af #2530, pkt 2 skal håndteres af personale (tilføj noget i et af de 3 felter) og pkt 3. også af personale (kasser og opret igen) som jeg har forsøgt at beskrive i #3247.

Forslag 2 burde også kunne håndtere afvigende periodikatyper som f.eks. term.type=serie - som på trods af krydser/ikke-krydser i /admin/config/ting/reservable og /admin/config/ting/holdings stadig ikke kan reserveres (jvf den lukkede sag #1545).

Nærværende sag har vel også allerede fået implementeret noget kode jvf note 16 ovenfor og min undren over manglende reserverknap i #3247

 

 

#29 Updated by Rolf Madsen over 1 year ago

Vi har lavet et udviklingsønske til Kombit om at availability response udvides med oplysing om hvorvidt et tidsskrift har årgang- og nummeroplysninger:

/external/{agencyid}/catalog/availability/v3 Get availability of bibliographical records

Response Class AvailabilityV3 {

recordId (string): The FAUST number of the Bibliographic record

reservations (integer): Total number of current active reservations of the Bibliographic record

reservable (boolean): True if materials can be reserved

available (boolean): True if materials is available on-shelf at some placement, false if all materials are lent out

}

Udvides med 

volNumberAvailable (boolean): True if Periodical or Yearbook has a Volume/Number structure

Kan det bidrage til løsningen af ovenstående?

#30 Updated by Steen Larsen over 1 year ago

Nu kan det være lidt svært at gætte hvad funktionaliteten egentlig betyder - jeg vil antage at "or Yearbook" er overflødig og at Year mangler - så der istedet burde stå "has a Volumen/Number/Year structure."

Og så gætter jeg at det er en smart genvej til at checke forekomsten af "periodical"-feltet for alle materialerne (for et givet faust) og så er det IKKE en løsning.

Vi kan løse reserveringerne for de fleste værker/materialer som beskrevet tidligere, men der er undtagelser som ikke kan løses i ddbcms pga fejl i materialernes registreringer (jvf note28). Fejl som FBS-API gladeligt sender videre til ddbcms.

 

#31 Updated by Simon Holt over 1 year ago

Det er noget kompliceret noget det her og der har været lidt forskellige problemer i spil.

Sagen her startede med følgende problem (som også er beskrevet i #1545, der er blevet lukket til fordel for denne sag): Indstillingen for om en materialetype skal have en reseveringsknap, giver ikke mening ift. årbøger.

Som det er nævnt flere gange i denne sag, er det fordi at nogle årbøger er laves som enkeltposter og skal have en reserveringsknap mens andre har den interne struktur med Volumen/Number/Year og skal ikke have en reserveringsknap, men i stedet have en for hver udgivelse under "Årgang og numre". Dette spiller altså ikke sammen med modellen, hvor man en gang for alle tager stilling til, om en materiale-type skal have en reseveringsknap

Der er blevet præsenteret to løsningforslag til dette i denne sag og de er begge lidt hackede løsninger ovenpå den eksisterende model.

https://github.com/ding2/ding2/pull/588 (merged)

https://github.com/ding2/ding2/pull/642 (ikke-merged)

Men vi kom så frem til, at det ikke rigtig gav mening at løse det på den måde alligevel pga den måde FBS håndterer periodica: hvis en post har ma=pe bliver materialerne på posten automatisk inddelt i "eksemplarer" og ved reservation skal man angive nok information i Volumen/Number/Year felterne til at den kan finde det rigtige eksemplar til reservation. 

I de tilfælde hvor der ikke kommer en "Årgang og numre" er det fordi der mangler periodica-oplysninger på eksemplarerne i bibliotekssystemet, så de kan ikke reserveres.

Så jeg er lidt i tvivl om, hvad vi skal lave i denne sag. Biblioteker der ikke får "Årgang og numre" på nogle af deres årbøger, bør rette disse poster i bibliotekssystemet.

Det er ikke sådan lige til at finde alle årbøger, der mangler periodica-oplysninger. Jeg tror vi (Vejle) har fundet og rettet de fleste efterhånden. Vi har bl.a. fået en stor hjælp af Steen, Aarhus, der har en metode til at finde dem.

Så for at opsummere

Det man som bibliotek skal gøre er at deaktivere reserveringsknappen for årbøger under admin/config/ting/reservable og sørge for at periodica-oplysninger er indtastet korrekt, så man får reserveringsknapper under "Årgang og numre".

#32 Updated by Steen Larsen over 1 year ago

Der er to typer reserver-knapper (en "monografi"-knap og "periodika"-knap).

Jeg synes at vi burde arbejde henimod at visningen af knapperne afhang af FBS-API-svaret istedet for dkabm-typerne (som de to pullrequest måske delvist håndterer?).

Udfordringen er dog at der er fejl i FBSAPI-svaret i nogle tilfælde så vi skal have en passende algoritme.
Men måske kan man bruge statistik - er de fleste materialer periodical ifølge FBS er det jo nok et sådant.

#33 Updated by Simon Holt over 1 year ago

Der er to typer reserver-knapper (en "monografi"-knap og "periodika"-knap).

Fint.. lad os bruge den terminologi :) Det er det samme jeg beskriver, bare med andre ord.

Jeg synes at vi burde arbejde henimod at visningen af knapperne afhang af FBS-API-svaret istedet for dkabm-typerne (som de to pullrequest måske delvist håndterer?).

Enig, synes bare det er out of scope i denne sag.

Der er jo også projektet med guldknappen, som muligvis laver om på alt det her.

 

#34 Updated by Simon Holt over 1 year ago

@Steen ved nærmere eftertanke er jeg ikke helt sikker på, jeg har forstået dig rigtig.

Jeg synes at vi burde arbejde henimod at visningen af knapperne afhang af FBS-API-svaret istedet for dkabm-typerne (som de to pullrequest måske delvist håndterer?).

Mener du bare i det her tilfælde med periodika eller mere generelt, så vi helt fjerner det med at man skal indstille hvor der skal være reserveringsknap under /admin/config/ting/reservable?

Det ene PR kigger på om der er mere end en reseveringsknap i frontend og skifter til holdings, hvis det er. Det andet kigger på serveren, om der blev returneret issues i holdings og hvis ja indstiller en javascript setting, der deaktiverer "monografi-knappen" i frontend. Så i den forstand, kan man vel godt sige den kigger på svaret fra FBS API.

Og lige en rettelse:

Har kigget lidt nærmere på den sag med guldknappen: https://platform.dandigbib.org/issues/1460 

Det ser ud til det kun er på avisartikler, tidsskriftsartikler og så vidt jeg kan forstå, kommer det ikke til at ændre på reserveringsknapperne ved periodika.

#35 Updated by Steen Larsen over 1 year ago

Jeg mener der udelukkende skal kigges på svaret fra FBS så der skal netop ikke være mulighed for indstillinger.
På den måde kan du også håndtere de tilfælde hvor dkabm-typen er afvigende (jvf #1545 og #1460#note-22).

Selvfølgelig er der nogle hjørner der så stadig ikke kan håndteres men det kan FBS-API jo heller.

#36 Updated by Simon Holt over 1 year ago

Jeg kan godt se din pointe med at vi har et problem her.

Felter der viser "Årgang og numre" bliver kun vist, hvis invokering af hook_ding_entity_is returnerer at det er en periodical. Som du selv har nævnt, er det lige nu kun hvis dkabm-typen er 'tidsskrift', 'periodikum' eller 'årbog'. Se: https://github.com/ding2/ding2/blob/master/modules/ting/ting.module#L522 

Når jeg laver en ma=pe søgning på vejlebib.dk får jeg 6.045 resultater. Af dem er 4.571 enten 'tidsskrift', 'periodikum' eller 'årbog', hvilket vil sige vi har 6.045 - 4.571 = 1477 materialer, der ikke kan reserveres.

**Rettelse til ovenstående udregning: Jvf. note-22 her. Skal selvfølgelig lige fratrække dem som ikke skal vise beholdning: netdokument, periodikum (net), avis (net), tidsskrift (net). Trækkes disse fra bliver det 1.123 materialer, der ikke kan reseveres.

Måske kunne vi lave det sådan her:

1. ding_reservation modulet tjekker altid om der er holdings. Hvis der er holdings og der ikke returneres periodical informatiom fra holdings kaldet til FBS API, laver den en reseveringsknap.

2. ding_periodical tjekker ligeledes efter holdings, men laver selvfølglelig kun reserveringsknapper hvis der returneres periodical information

Dette vil nok ikke være så svært at lave.

Alternativt, hvis det er for omfattende at ændre i modellen med indstilling af reserveringsknapper for dkabm-typer, kan vi prøve helt at fjerne det tjek på om et materialet er en periodical. Eller måske tilføje de ekstra typer som tilsyneladende også opfattes som periodical i visse situationer.

#37 Updated by Simon Holt over 1 year ago

  • Status changed from Needs analysis to Needs design decision
  • Assignee changed from Simon Holt to Rolf Madsen

#38 Updated by Steen Larsen over 1 year ago

Mht at tilføje ekstra typer til checket så synes jeg ikke det er en god ide men det vil selvfølgelig løse problemet med "avis" og "serie" som (vistnok?) ikke findes i en ikke-periodisk type.

Øvrige typer kan ses/findes ved denne søgning:
ma=pe not term.type any "tidsskrift serie årbog periodikum netdokument"
og det er ganske vist kun få materialer (hos os).

Men det burde kunne laves simpelt:

Spørg FBS hvis kilden er bibliotekskatalog og hvis der er reserver-bare materialer så sæt den rette knap afhængig af feltet periodical.

#39 Updated by Simon Holt over 1 year ago

Men det burde kunne laves simpelt:

Spørg FBS hvis kilden er bibliotekskatalog og hvis der er reserver-bare materialer så sæt den rette knap afhængig af feltet periodical.

Det handler jo også om, ikke at ændre for meget i hvordan tingene er implementeret nu. Lige nu håndteres de to forskellige reseveringsknapper to forskellige steder i to forksellige moduler og det er nok bedst at beholde det sådan.

God pointe med at vi lige skal sikre det er bibliotekskatalog, da vi laver opslag i FBS-API uden namespace. Så:

1. ding_reservation modulet tjekker om der er reserverbare materialer, hvis kilden er bibliotekskatalog. Hvis der er og der ikke returneres periodical informatiom fra holdings kaldet til FBS API, laver den en reseveringsknap.

2. ding_periodical tjekker ligeledes efter reserverbare materialer hvis kilden er bibliotekskatalog, men laver selvfølglelig kun reserveringsknapper, hvis der returneres periodical information.

 

#40 Updated by Rolf Madsen over 1 year ago

  • Status changed from Needs design decision to Ready for development
  • Assignee changed from Rolf Madsen to Simon Holt

#41 Updated by Rolf Madsen over 1 year ago

#42 Updated by Rolf Madsen over 1 year ago

  • Target version changed from Release 29-2 - Bugfixes (7.x-4.5.0) to Release 29-2 - Bugfixes (Vejle)

#43 Updated by Kasper Garnæs over 1 year ago

1. ding_reservation modulet tjekker om der er reserverbare materialer, hvis kilden er bibliotekskatalog. Hvis der er og der ikke returneres periodical informatiom fra holdings kaldet til FBS API, laver den en reseveringsknap.

Reservationsknapper afhænger af hvorvidt materialet bliver identificeret som reservable. Det vil vi i så fald gerne undgå her. Dette kan implementeres igennem noget á la:

function fbs_ding_entity_is($entity, $class) {
  if ($class == 'reservable') {
    // Return FALSE if holdings contain periodical information to make ding_entity_is return FALSE
    // no matter what other implementations might return.
  }
}

Check for kilde er vel allerede muligt på /admin/config/ting/reservable.

 

#44 Updated by Rolf Madsen over 1 year ago

OBS. Vær opmærksom på Kaspers kommentar i https://platform.dandigbib.org/issues/3325#note-4 som behandler "Abstraktion af materialetyper for tidsskrifter".

#45 Updated by Rolf Madsen about 1 year ago

  • Target version changed from Release 29-2 - Bugfixes (Vejle) to Release 33 - Bugfixes

#47 Updated by Simon Holt about 1 year ago

Er i gang med at kigge på den her. Ifb. med dette har jeg fundet et yderligere problem. For nogle poster påstår FBS fejlagtigt, at det er et periodikum. Pga. det hack der blev lavet i https://platform.dandigbib.org/issues/1460#note-13, får vi nu slet ingen reserveringsknap på disse poster. Et par eksempler:

https://vejlebib.dk/ting/object/870970-basis%3A42793825 

https://vejlebib.dk/ting/object/870970-basis%3A27719228 

FBS returnerer ellers at de er reserverable: 

https://vejlebib.dk/ding_availability/items/42793825

https://vejlebib.dk/ding_availability/items/27719228

Men når holdings hentes, er de markeret som periodical, så reserveringsknappen fjernes pga https://platform.dandigbib.org/issues/1460#note-13:

https://vejlebib.dk/ding_availability/holdings/42793825

https://vejlebib.dk/ding_availability/holdings/27719228

Jeg har tjekket et par andre biblioteker og de har ikke problemet. I hvert fald ikke på disse poster, men måske på andre?

Min pointe er, at vi tilsyneladende ikke engang kan regne med at FBS har fuldstændig styr på om en post er periodikum.

UPDATE 23-07-2018:

Efter jeg gravede videre i dette, fandt jeg ud af at FBS API returnerer periodical information på et af materialerne og derfor bliver det af DDB CMS markeret som periodical. Hvorfor den returnerer periodical information på disse materialer har jeg ikke regnet ud i endnu. Posten er ikke ma=pe og derfor har vi heller ikke mulighed for at fjerne periodical-informationerne i Cicero. Vi har lavet en sag ved Systematic. Jeg tænker det er noget der er gået galt ifb. med vores data-migrering til FBS. 

Men problemet viser, at det er muligt for FBS API at smide periodical-informationer afsted, selv om en post ikke er ma=pe og man derfor ikke har mulighed for at redigere/indtaste periodical-information i Cicero. Måske kunne vi lave periodical-tjekket mere robust i DDB CMS, sådan at den tjekker alle materialerne for periodical-information. Det bedste ville selvfølgelig være, hvis det blev fikset i FBS sådan at den ikke returnerer periodical for materialer der ikke er ma=pe også selv om der ligger et eller andet databasen.

#48 Updated by Simon Holt 12 months ago

Så er der PR: https://github.com/ding2/ding2/pull/1139

Den primære ændring i dette PR er, at vi går fra at det er opensearch, der skal bestemme hvilke materialer der kan reserveres, til at det er library provideren som f.eks. FBS. Det er også den, der burde vide bedst.

Man skal således ikke længere indstille hvilke kilder/typer, der skal have reserveringsknap under /admin/config/ting/reservable. Det finder bibliotekssystemet selv ud af. Det skulle dermed bl.a. fikse det problem vi har med reserveringsknap på årbøger (4.) plus alle de andre problemer nævnt i nedenstående.

Man skal dog stadig (af performance årsager) tage stilling til hvilke kilder, systemet skal spørge om availability-information om. Som standard er det sat til "bibliotekskatalog" i FBS/Opensearch, men man har altså mulighed for at vælge flere typer, hvis der skulle blive brug for det. Lige for at understrege: Denne indstilling medfører ikke, at der kommer reserveringsknap. Den medfører bare at systemet vil spørge bib-systemet om availability og dermed kommer der måske en reserveringsknap for disse materialer.

Problematikkerne med at det ikke er bibliotekssystemet, der afgør om poster kan reserveres, er nævnt flere steder. Jeg prøver lige at opsummere her:

1. Hvilke materialer der skal kunne reserveres, administreres allerede i bibliotekssystemerne. Og for FBS' vedkommende kan man ovenikøbet styre det helt ned på materialeniveau. Så det hænger ganske enkelt ikke sammen med at vi i DDB CMS udelukkende prøver at afgøre udfra materialetyper fra brønden. Det er også lidt dobbelt-konfetti, at det administreres to steder.

2. Når vi får nye materialetyper, skal man altid huske at indstille dette i DDB CMS også. Dette glemmer man at gøre, som vi for eksempel så med Switch-spil, der i en lang periode slet ikke kunne reserveres, fordi bibliotekerne havde glemt at indstille det.

3. For FBS vedkommende: Brøndens typer stemmer ikke overens med det metadata FBS for eksempel anvender til at afgøre om et materiale er periodikum. Det kan giver problemer med poster, der bliver fuldstændig låst udefra at kunne reserveres. Dette er uddybet i https://platform.dandigbib.org/issues/1460#note-36.

4. For nogle materialetyper er det ikke muligt at sige, at den altid er det eller det andet. For eksempel med årbøger, skal nogle af dem have en reserveringsknap fordi de er lavet som enkeltposter, mens andre ikke skal have det, fordi den har periodika-oplysning i beholdning.

Jeg tror grunden til at man har lavet det i første omgang på denne måde har været performance-relateret (for at undgå det ekstra kald til bib-system API). Men med så mange problemer med det, mener jeg det er bedre at løse det på den rigtige måde og lade det system der ved bedst afgøre om et materialer skal kunne reserveres.

#49 Updated by Simon Holt 12 months ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

#50 Updated by Steen Larsen 11 months ago

Lige en kommentar til din update i note47 er netop det tilfælde jeg beskriver i #3247

dvs du ser et materiale som er en almindelig bog og i FBS materialevisning vises den også som en sådan (dvs ingen år/volumen/nummer felter) men felterne ER skam udfyldt - og man kan se det i materiale-søgelisten som i dette eksempel med 3 materialer - læg mærke til () som er den sædvanlige tilføjelse af år/nummer/volumen til titlen

Fejlen er opstået enten ved overgangen fra Ddelibra eller (teoretisk) ved en postændring af DBC

Det er desværre vældigt omstændig at finde dem i FBS og rette dem som beskrevet i #3247. Der er heller ingen hjælp fra Systematic.

 

#51 Updated by Gitte Barlach 9 months ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#52 Updated by Gitte Barlach 7 months ago

  • Target version changed from Release 33 - Bugfixes to Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0)

Relateret til #3739; sagen fremrykkes derfor.

#53 Updated by Simon Holt 7 months ago

Lyder godt :)

Kommer til at se, jeg lige skal have lavet en rebase før den kan merges.

#54 Updated by Gitte Barlach 7 months ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jesper Kristensen to Simon Holt

ok, tak, Simon:-)
Så ændrer jeg lige status og assigner sagen til dig. 

#55 Updated by Simon Holt 7 months ago

Det er super :) jeg får kigget på det!

#56 Updated by Christel Krabbenhøft 6 months ago

Hej Simon. Vil du kigge på denne efter din eksamen? :-)

#57 Updated by Simon Holt 6 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

PR er rebased og opdateret med et par ekstra rettelser!

https://github.com/ding2/ding2/pull/1139

Det primære i denne rettelse er, at vi spørger FBS hver gang om et materiale er reserverbar. Det holder ikke at basere det på materialetype om et materiale skal have en reserverknap, da der kan være forskel på om materialer er reserverbare indenfor en materialetype. Desuden har det også givet problemer HVER gang der er kommet en ny materialetype og DDB CMS ikke er opdateret til at vise reserveringsknap for den. Hvad der kan reserveres administreres allerede i FBS/Cicero og der er ingen grund til det også skal administreres i DDB CMS.

Jeg er klar over, at der har været lidt problemer med forsinket reserverknap. Men har lige testet PR og selvom den skal lave et kald til FBS for at tjekke reserverbar, kommer alle knapper lynhurtigt. Og det er uden rettelserne fra #3905 og #3739, som får det til at køre endnu bedre.

#58 Updated by Gitte Barlach 6 months ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#59 Updated by Christel Krabbenhøft 6 months ago

  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0) to Release 31 - bugfixes

Pga. tidspres bliver vi nødt til at skubbe sagen til rel. 31. //Christel

#60 Updated by Jørgen Nielsen 5 months ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jesper Kristensen to Gitte Barlach

reviewet og godkendt

#61 Updated by Simon Holt 4 months ago

Til orientering. Ifb med inkludering af denne rettelse i vores egen kodebase opdagede jeg en mindre fejl som gav en PHP warning. Jeg har opdateret PR med et commit der fikser denne warning:

https://github.com/ding2/ding2/pull/1139/commits/6eb604bc68ae5aa747cc127b1c224d17b957b376 

Det er en meget lille ændring, der ikke har betydning for funktionaliteten.

#62 Updated by Kasper Garnæs about 2 months ago

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

Merged.

#63 Updated by Kasper Garnæs about 2 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#64 Updated by Gitte Barlach about 1 month ago

jeg tester denne.

#65 Updated by Gitte Barlach about 1 month ago

testet på upgrade-fbs med 7.x-4.8.0-rc1

Mit udgangspunkt for testen er denne opsummering:

"nogle årbøger er lavet som enkeltposter og skal have en reserveringsknap ('monografi-knap') mens andre har den interne struktur med Volumen/Number/Year og skal ikke have en reserveringsknap, men i stedet have en for hver udgivelse under "Årgang og numre". ('periodica.knap')

a) den lange liste med materialetyper i backend under /admin/config/ting/reservable  er nu fjernet, da vi ikke længere baserer os på Brønden for at afgøre, om der skal vises res.knap eller ej. Testet og OK. Dog burde hjælpeteksten måske rettes så der ikke længere står "types" men kun "sources": "Update the lists of known types and sources by asking the search provider for all types and sorces." Men det er en detalje. OBS: Manualen skal opdateres!
b) res-flow alm. materialer (monografier og periodica (tss.)). Testet og ok. 
c) roller: kun provider / biblioteksbrugere skal kunne reservere. Testet og ok.
d) årbøger der tidligere fejlagtigt ikke fik en res.knap. Jeg mangler nogle konkrete eksmepler for at kunne teste.
Jeg lægger dog mærke til at der nu re res.knapper i værkvisningen på nogle poster, der ikke tidligere har haft det. Inde på seslve materialet er der dog res.knapper - både før og nu:
FØR:https://www.grevebibliotek.dk/ting/collection/870970-basis%3A06604544
NU: https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A06604544

FØR: https://www.grevebibliotek.dk//ting/collection/870970-basis%3A50928977
NU: https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A50928977

e) Guldknappen skal med stor sansynlighed refaktureres som følge af ændringen
f) lidt tanker omkring UX som jeg tillader mig at bringe på banen, selvom det nok ligger lidt udenfor dette issue:
f1) Der er materialer der har en reserver-knap, men som alligevel ikke kan reserveres.
Eksempel:
https://upgrade-fbs.ddbcms.dk/ting/object/725300-katalog%3A126221843Meddelelse til brugeren: "Du har ikke tillladelse..." 

Eksempel:
https://upgrade-fbs.ddbcms.dk/ting/object/725300-katalog%3A29646031
Meddelelse til brugeren: "Reservering ikke mulig.."
 

f2) I samme værkklynge er der en res.knap på nogle titler, men ikke på andre, hvor man i stedet skal helt ind på selve posten og folde årgange og numre ud for at se knappe
Eksempel:
https://upgrade-fbs.ddbcms.dk/ting/collection/725300-katalog%3A51794664
Statistisk årbog. 2015 (119. årgang) har en res.knap allerede i værkvisningen. (Men jeg kan alligevel ikke reservere den.)
Statistisk årbog (2017) har ingen res.knap i værkoversigten - man skal helt ind på materialet og folde årgange og numre ud for at se knappen. 

Samme problematik gælder f.eks. i https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A06604544

Ovenstående (f1 og f2) er nok faktisk ikke en fejl ift. tidligere. Det er lidt svært når jeg ikke kan se hvilke evt. begrænsninger Greve Bib har på deres materialer eller hvordan de har indtastet dem i FBS. MEN: hvis jeg skal se bort fra div. tekniske indsigter jeg måske ikke helt har samt det forhold at at alt måske faktisk er "korrekt" set ift hvordan systemerne er, så vil jeg egentlig bare sige at jeg ikke tror brugerne forstår det. 

g)Anonym bruger - inaktive res.knapper: 
i denne værkklynge:https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A29058865 sker der ingenting hvis jeg klikker på reserver-knappen ud for materialet "Og det var Danmark : de bedste striber. Årgang 4 (2009). Jeg prøvede derefter  at klikke på "reserver" for nogle af de andre poster i værkklyngen. De virkede i første omgang, dvs jeg blev prompted for at logge ind. Men efter lidt flere forsøg, kan jeg pludselig ikke få nogen af res.knapperne til at virke. Der sker ingenting når jeg klikker på dem. 
Jeg venter derfor lidt, og prøver igen. Pludselig kan jeg få én af de andre res-knapper til at virke i samme værkklynge, men den næste jeg klikker på virker så ikke. Dette sker flere gange, men tilfældigt hvilke der virker og hvilke der ikke virker. 
Som indlogget bruger får jeg flg. medd.hver gang jeg klikker på res.knappen: "Reservering ikke mulig. Kontakt biblioteket."

H) ingen res. knap:
der er ingen reserverings-knap på denne: https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A23209373
 

#66 Updated by Gitte Barlach about 1 month ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Simon Holt

#67 Updated by Simon Holt about 1 month ago

@Gitte tak for test og feedback! kigger på det i dag.

#68 Updated by Simon Holt about 1 month ago

d) årbøger der tidligere fejlagtigt ikke fik en res.knap. Jeg mangler nogle konkrete eksmepler for at kunne teste

Det er ikke kun vigtigt at teste at visse årbøger får en reserveringsknap, men også at årbøger der er PIF-poster ikke får en primær reserveringsknap, da det i disse tilfælde er knapperne under "Årgang og numre" der skal bruges. De biblioteker, der har slået reservering til for materialetype årbog, har haft problemet med at der i visse tilfælde fejlagtigt blev vist en ikke-fungerende primær reserveringsknap. Biblioteker der har slået det fra, har haft problemet med manglende reserveringsknap, således det slet ikke var muligt at reservere.

Et godt udgangspunkt for at teste dette er dette værk, da den ser ud til at indeholde både almindelige og PIF-poster (se vedhæftning for at se hvordan værket ser ud for mig lige nu: 1460-årbog-værk-med-pif-og-alm-poster.PNG).

https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A07145756 

Alle dem der ikke har reserveringsknap i ovenstående værk er også PIF-poster med "Årgang og numre", og dem der har er almindelige poster, der skal have en primær reservationsknap.

Vil mene at dette eksempel viser, at det primære problem i denne sag er løst!

Det vil kræve meget størrer ændringer, hvis vi også skal vise reserveringsknap på PIF-poster i værkvisning. Hele ding_periodical skal nok skrives om og man bør nok også lave et nyt ding_holdings modul og så flytte ansvaret for at vise reserveringsknapper på periodical fra ding_periodical til ding_reservation. Og hvilken årgang/nummer skulle denne reserveringsknap så resevere? Jeg tror vi bevæger os lidt ud i noget, som det med guldknappen egentlig prøvede at løse? Under alle omstændigheder synes jeg ikke det er noget vi skal prøve at rette i denne sag, da det er for omfattende, men i stedet lave en ny sag.

Jeg har også en kommentar angående dette i PR under "Further improvements out of scope of this PR":

  • Make new ding_holdings module which merges the holding functionality from ding_availability and ding_periodical into one. This module could define one field, which just shows the holdings, if any, returned from the library provider. The field could handle both periodical and non-periodical holdings, ensuring that we never get the confusing display of two types of holdings.
  • Remove all holdings functionality from ding_availability, such that the module is only concerned with availability and relevant fields.
  • Remove the ding_periodical module entirely. All holdings display is now handled by the dedicated ding_holdings module. It also contains some handling of periodical reservation, which could be moved to ding_reservation.

Jeg lægger dog mærke til at der nu re res.knapper i værkvisningen på nogle poster, der ikke tidligere har haft det. Inde på seslve materialet er der dog res.knapper - både før og nu:
FØR:https://www.grevebibliotek.dk/ting/collection/870970-basis%3A06604544
NU: https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A06604544

FØR: https://www.grevebibliotek.dk//ting/collection/870970-basis%3A50928977
NU: https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A50928977

Der ser ud til være gået noget "Værkmatch har hukommelse" i dine links, for bliver bare omdirrigeret indtil postvisning nu, når jeg trykker på tre af dem.

Kan komme ind til følgende værkvisning: https://www.grevebibliotek.dk//ting/collection/870970-basis%3A50928977 og her får jeg et værk med reserveringsknap (se vedhæftning: 1460-værkvisning-på-greve-med-resknap.PNG). Så vidt jeg kan se kører grevebib endnu ikke med rettelsen her, så det med reserveringknapper på værkvisninger har vist ikke noget med sagen her at gøre? Men kan godt følge dine bekymringer om det.

f2) I samme værkklynge er der en res.knap på nogle titler, men ikke på andre, hvor man i stedet skal helt ind på selve posten og folde årgange og numre ud for at se knappe
Eksempel:
https://upgrade-fbs.ddbcms.dk/ting/collection/725300-katalog%3A51794664
Statistisk årbog. 2015 (119. årgang) har en res.knap allerede i værkvisningen. (Men jeg kan alligevel ikke reservere den.)
Statistisk årbog (2017) har ingen res.knap i værkoversigten - man skal helt ind på materialet og folde årgange og numre ud for at se knappen. 

Angående at man skal ind og folde "Årgang og numre" ud på nogle af dem: se ovenstående.

g)Anonym bruger - inaktive res.knapper: 
i denne værkklynge:https://upgrade-fbs.ddbcms.dk/ting/collection/870970-basis%3A29058865 sker der ingenting hvis jeg klikker på reserver-knappen ud for materialet "Og det var Danmark : de bedste striber. Årgang 4 (2009). Jeg prøvede derefter  at klikke på "reserver" for nogle af de andre poster i værkklyngen. De virkede i første omgang, dvs jeg blev prompted for at logge ind. Men efter lidt flere forsøg, kan jeg pludselig ikke få nogen af res.knapperne til at virke. Der sker ingenting når jeg klikker på dem. 
Jeg venter derfor lidt, og prøver igen. Pludselig kan jeg få én af de andre res-knapper til at virke i samme værkklynge, men den næste jeg klikker på virker så ikke. Dette sker flere gange, men tilfældigt hvilke der virker og hvilke der ikke virker. 
Som indlogget bruger får jeg flg. medd.hver gang jeg klikker på res.knappen: "Reservering ikke mulig. Kontakt biblioteket."

Kan godt genskabe denne. Kan se det er en javascript fejl, der forhindrer det. Se vedhæftning: 1460-reserveringsfejl-værkvisning.PNG og https://www.drupal.org/project/bootstrap/issues/3013236.

Det lader til at opstå fordi jQuery UI mangler på værkvisningen. Så snart jeg inkluderer jQuery UI i backend virker det. Vil lige dykke ned i hvorfor jQuery UI mangler, hvorfor det skal bruges og hvorfor det er inkluderet på postvisning men ikke på værkvisning.

H) ingen res. knap:
der er ingen reserverings-knap på denne: https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A23209373

FBS API siger at den ikke er reservable: https://upgrade-fbs.ddbcms.dk/ding_reservation/870970-basis:23209373/is_reservable. Hvis den bør være reservable, er det FBS API der udleverer en forkert status og det bør vi kigge nærmere på.

Udover dette har jeg også fundet en anden fejl selv: Den prøver unødvendigt at lave reservability checks for alle reserveringsknapper under "Årgang og numre". ding_periodical gør på nuværende tidspunkt dette selv, så der er ingen grund til det. Det giver også en masse fejlopslag på formen  https://upgrade-fbs.ddbcms.dk/ding_reservation/undefined/is_reservable. Se vedhæftning: 1460-periodical-undefinde-reservability-check.PNG.

Kasper er også kommet med et forslag til optimering af performance på Github, som jeg synes er vigtig at få med. Det er desuden også et par smårettelser.

Så jeg får lige fremsat en nyt PR :)

#69 Updated by Simon Holt about 1 month ago

Angående den ikke-fungerende reserveringsknap på værker: har fundet ud af det skyldes en rettelse i #4088 og postet en kommentar om det i sagen.

Jeg er lidt i tvivl om hvordan fejlen bedst løses, så derfor tror jeg det er bedre at få Benjamin til at kigge på det igen i #4088.

#70 Updated by Simon Holt about 1 month ago

Så er der nyt PR: https://github.com/ding2/ding2/pull/1460 (hehe.. 1460)

Indeholder:

1. Optimering af reserverbar-tjek.

2. Tjek ikke reserbar for "Årgang og numre"-reserverknap

3. Håndter fejl hvis der er problemer med FBS-API eller ikke opsat korrekt (hvis den her merges før mit PR fra #4379, er det PR unødvendigt).

4. Mindre småting fra sidste code review.

#71 Updated by Simon Holt about 1 month ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

#72 Updated by Gitte Barlach about 1 month ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#73 Updated by Gitte Barlach about 1 month ago

  • Priority changed from High to Urgent

#74 Updated by Kasper Garnæs about 1 month ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Simon Holt

Jeg har reviewet https://github.com/ding2/ding2/pull/1460 og kommet med kommentarer.

#75 Updated by Tue Gaston 23 days ago

Jeg er med på, at seneste PR ikke er merget.
Jeg er også med på, at den bug, jeg her oplever muligvis slet ikke er relateret til dette issue - men nu skriver jeg det her alligevel;

"Reservér-knappen" er efter opgradering til rel. 7.x-4.8.0-rc2 forsvundet fra alle materialer i redaktionelt indhold:

https://stg.aakb.dk/nyheder/inspiration/noget-der-ligner-adler-olsen vs https://www.aakb.dk/nyheder/inspiration/noget-der-ligner-adler-olsen 

Skal jeg oprette et separat issue?

#76 Updated by Gitte Barlach 22 days ago

Hej Simon 
Hat du mulighed for at se på denne i starten af næste uge ?

#77 Updated by Simon Holt 18 days ago

Jeps, går i gang med at færdiggøre denne nu!

@Tue jeg tror rettelsen her vil fikse de manglende knapper, men kigger lige på det.

#78 Updated by Simon Holt 16 days ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

PR er opdateret.

#79 Updated by Gitte Barlach 16 days ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#80 Updated by Simon Holt 16 days ago

Seneste PR vil give bedre performance på forsiden og andre sider med karuseller/lister. Det viser sig, at der var nogle steder den stadig tjekkede reservabiliy synkront. Jeg har skrevet mere om det her: https://platform.dandigbib.org/issues/4379#note-21

#81 Updated by Jesper Kristensen 15 days ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jesper Kristensen to Simon Holt

#82 Updated by Simon Holt 15 days ago

Interessant spørgsmål :) Har svaret på Github.

#83 Updated by Simon Holt 15 days ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Simon Holt to Jesper Kristensen

Also available in: Atom PDF