Project

General

Profile

Bug #3090

Bindposter giver forkert opstilling i DDB CMS - Shelfmark

Added by Kasper Hagel Madsen almost 2 years ago. Updated about 1 year ago.

Status:
Resolved (tag version)
Priority:
Urgent
Assignee:
Estimated time:
URL med eksempel:
https://fkb.dk/ting/object/870970-basis%3A22718584
Kategorier:
Søgning - Materialevisning

Description

Europa. Bind 2. 1300-1600" er en fagbog i grp. 90.

I DDB CMS ser opstillingen således ud: Biblioteket Frederiksberg. Hovedbiblioteket > Voksne > Magasin > 1300-1600

Nu er bogen pludselig blevet til en roman, for der er jo ikke klassemærker, der når op i 1300 :-)

Fejlen er sføli, at lånerbasen KUN henter sine oplysninger fra bindposten, og den indeholder kun 245 *a 1300-1600.

Hovedtitlen derimod står i den tilhørende hovedpost.

Hovedposten har også klassemærket 90.

https://fkb.dk/ting/object/870970-basis%3A22718584

 

Der må være andre lignende tilfælde hos andre biblioteker!?

Er i tvivl om denne skal til DDB eller Systematic, men da det ikke er en fejl i Cicero går vi hertil første omgang :)


Related issues

Related to DDB CMS - Bug #3825: Klassemærker mangler på flerbindsværkerClosed
Has duplicate DDB CMS - Bug #3229: 0 online + ingen forfatter i placeringClosed

History

#1 Updated by Rolf Madsen almost 2 years ago

  • Status changed from New to Need more info
  • Assignee set to Kasper Hagel Madsen
  • Target version set to Release 29-2 - Bugfixes (7.x-4.5.0)

Et par opklarende spørgsmål:

  1. Hvordan ser jeres konfiguration under /admin/config/ting/holdings ud?
  2. Hvilken brøndprofil benytter i på https://fkb.dk, konfigureret under /admin/config/ting/settings?

#2 Updated by Simon Holt almost 2 years ago

Hvis I bruger shelf_mark opstilling, skal jeg lige have kigget på det igen.

Jeg har aldrig været helt sikker på, at det jeg lavede med udvælgelsen af marc-posten var helt rigtigt, så det er muligvis det der driller.

Har også set en anden situation, hvor opstillingen heller ikke er 100% korrekt.

Tænker vi måske kunne samle det hele her?

#3 Updated by Kasper Hagel Madsen almost 2 years ago

Hej igen! Ja vi har valgt shelf_mark opstilling under  /admin/config/ting/holdings.

#4 Updated by Rolf Madsen almost 2 years ago

Og hvilken brøndprofil benytter I?

#5 Updated by Kasper Hagel Madsen almost 2 years ago

Brøndprofilen hedder:

drift - B3

 

 

#6 Updated by Simon Holt almost 2 years ago

Så er jeg klar med en løsning:

https://github.com/ding2/ting-client/pull/23 

Bemærk dette er et PR imod ting-client, hvor fejlen viste at befinde sig. Ifb. med at have kigget på koden, har jeg lavet nogle meget små rettelser i core også, som jeg håber jeg kan få sneget med her:

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

Problemet var ganske rigtig, at den fandt den forkerte post (bindposten i dette tilfælde), som ikke er komplet med alt metadata for posten. Fandt ud af, at man skal bruge den post der er markeret som 'bibliographic', da den ser ud til at indeholde det samlede metadata for hele posten.

Før kiggede vi på primaryObjectIdentifier og prøvede at finde at matchende faust. Dette virkede åbenbart i mange tilfælde. Men for årgangsposter som denne rammer den selvfølgelig bindposten, da den også har posten primære faust. Jeg fandt dette stykke dokumentation meget brugbart: http://www.metadata.dk/posttyper-danmarc2-dataleverancer 

Nu har vi jo mange eksperter herinde, så jeg vil gerne lige præsentere logikken til generering af sheflmark. Kom gerne med feedback, hvis der er noget galt med logikken:

1.
Hvis felt 652 delfelt m = sk
Vises 1. ord (efter evt. soltegn) i felt 100 delfelt a
Hvis felt 100 mangler vises 1. ord (efter evt. soltegn) i 110 delfelt a
Hvis felt 100 og felt 110 mangler vises 1. ord (efter evt. soltegn) i 245 delfelt a

2.  
Hvis felt 652 delfelt m ikke = sk og posten indeholder felt 039 
Vises indholdet i felt 039 delfelterne a + b (bogstavskoderne oversættes, se herunder) 

efterfulgt af 

Hvis felt 100 vises 1. ord (efter evt. soltegn) i 100 delfelt a
Hvis felt 100 mangler, vises 1. ord (efter evt. soltegn) i 110 delfelt a
Hvis felt 100 og felt 110 mangler, vises 1. ord (efter evt. soltegn) i 239 delfelt a
Hvis felt 100, felt 110 og felt 239 mangler, vises 1. ord (efter evt. soltegn) i 245 delfelt a

3. 
Hvis felt 652 delfelt m ikke = sk og der ikke er et felt 039

Vises indhold af felt 652 delfelt m (DK5-koden) + indhold i delfelt b

Hvis felt 652 delfelt b ikke findes - vises indhold i delfelt a + delfelt h (kommasepareret)
(Forklarende tekst til 652 m vises ikke)

efterfulgt af (ligesom ovenfor)

Hvis felt 100 vises 1. ord (efter evt. soltegn) i 100 delfelt a
Hvis felt 100 mangler, vises 1. ord (efter evt. soltegn) i 110 delfelt a
Hvis felt 100 og felt 110 mangler, vises 1. ord (efter evt. soltegn) i 245 delfelt a

#7 Updated by Simon Holt almost 2 years ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Kasper Hagel Madsen to Rolf Madsen

#8 Updated by Rolf Madsen almost 2 years ago

  • Assignee changed from Rolf Madsen to Gitte Barlach

#9 Updated by Rolf Madsen almost 2 years ago

  • Subject changed from bindposter giver forkert opstilling i DDB CMS to Bindposter giver forkert opstilling i DDB CMS - Shelfmark

#10 Updated by Rolf Madsen almost 2 years ago

  • Has duplicate Bug #3229: 0 online + ingen forfatter i placering added

#11 Updated by Steen Larsen almost 2 years ago

Gad vide om det kunne være muligt at få den lavet som et felt i dkabm? Men så skal alle jo være enige og det jo nok tage umanerlig lang tid at få implementeret.

Det forhindrer dog ikke at fejlen bliver rettet i den eksisterende kode.

#12 Updated by Steen Larsen almost 2 years ago

Som en kommentar til algoritmen tror jeg der mangler noget med felt 239t - fra kat-format.dk vedr 239:
Delfelt *t indeholder en uniform titel, som følger praksis for folkebiblioteker for katalogisering af musikværker, en såkaldt "standardtitel" eller en dansk "standardtitel" på en film.

Eksempler:

52720567 - her ser det ud til at det er den engelske titel der benyttes (fra 245) istedet for 239t
52484480 - hvor opstillingsfeltet er tomt.

#13 Updated by Gitte Barlach over 1 year ago

  • Status changed from Needs code review to Need more info
  • Assignee changed from Gitte Barlach to Simon Holt

Simon, hvad tænker du om Steen kommentar? skal der rettes yderligere eller er vi klar til code review?

#14 Updated by Simon Holt over 1 year ago

Hej Gitte

Vil meget gerne rette aloritmen til, men er ikke helt sikker på hvor jeg skal bruge 239t.

Lige nu bruger vi eksempelvis 239a for musik: "Hvis felt 100 og felt 110 mangler, vises 1. ord (efter evt. soltegn) i 239 delfelt a"

Skal vi så bare bruge 239t her i stedet eller skal det have en højere prioritering? Måske kan Steen kaste lys over det?

Jeg sender i hvert fald lige det Steen har skrevet til de eksperter, der har været med til udtænke algoritmen.

#15 Updated by Steen Larsen over 1 year ago

Mht titlen er målet vel at "aflure" hvordan dkabm-feltet <dc:title xsi:type="dkdcplus:full"> dannes, så vi selv kan håndtere tekst før/efter tegnet ¤ (som allerede er fjernet i dkabm-feltet).

En sådan vejledning/algoritme kan man vel få fra DBC?

#16 Updated by Simon Holt over 1 year ago

Har lige fået at vide, at jeg bare skal bruge 239t i stedet for 239a. Så er det ikke bare det vi gør?

#17 Updated by Steen Larsen over 1 year ago

Udover det med titel fra hovedpost/bindpost er der også det med klassemærket som beskrevet i sagen ovenfor.

Her endnu et eksempel med faust 20142995 hvor både titel og klassemærke fejler (forkert titel, klassemærke mangler)

#18 Updated by Simon Holt over 1 year ago

@Steen har lige tjekket og opstilling for posten med faust 20142995 ser ud til at blive fikset med rettelsen fra #note-6.

#19 Updated by Rolf Madsen over 1 year ago

  • Status changed from Need more info to Development

#20 Updated by Rolf Madsen over 1 year ago

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

#21 Updated by Simon Holt over 1 year ago

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

@Steen kan du ikke oprette en ny sag angående det med felt 239t?

Vi skal altså have denne vigtige sag i mål snart. Lige nu vises opstilling jo forkert for en masse bindposter, så synes ikke den lille rettelse med 239t skal stå i vejen og har derfor sat dent til code review!

Og lige for god ordens skyld: det der skal code-reviewes er:

https://github.com/ding2/ting-client/pull/23

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

#22 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

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

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

Reviewed. Jeg har et par rettelser jeg gerne ser lavet.

#24 Updated by Simon Holt over 1 year ago

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

Begge PR er opdateret.

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

  • Status changed from Needs code review to Reviewed

Godkendt. Afventer release.

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

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

Merged.

#27 Updated by Gitte Barlach about 1 year ago

  • Status changed from Technical test to Resolved (tag version)

Testet på upgrade-29 med 4.5.0-rc2

Rettelsen ser ud til at fungere efter hensigten. Godkender derfor denne. 

 

#28 Updated by Simon Holt about 1 year ago

  • Related to Bug #3825: Klassemærker mangler på flerbindsværker added

Also available in: Atom PDF