Project

General

Profile

Bug #4764

Galleri på Materialevisningssiden

Added by Rolf Madsen 5 months ago. Updated 9 days ago.

Status:
Resolved (tag version)
Priority:
High
Assignee:
Estimated time:
URL med eksempel:
https://github.com/danskernesdigitalebibliotek/ddb-react/pull/90
Kategorier:
Søgning - Materialevisning, Driftsvedligehold - Oversættelser (udarbejdes før udrulning)

Description

Problemstilling

Med overgangen til de to nye Material List- og Follow Searches services fjernes det gamle P2 kodekompleks, og dermed fjernes også Serendipitetsmotoren der hidtil har leveret materialeforslag i DDB CMS.

Den materialekarrusel der hidtil har været vist på Materialevisningssiden fik materialer netop fra Serendipitetsmotoren, hvilket betyder at denne materialekarrusel fjernes med Release 31.1.

Formål

Tilføj en ny funktion der kan levere materialeforslag til lånerne på Materialevisningssiden.

Løsningsforslag

Gallerivisningen har til formål at vise materialer relateret til den pågældende materialevisning.

 

Desktopvisning

NB. Bookmark-ikonet skal ikke vises.

Mobilvisning

NB. Bookmark-ikonet skal ikke vises.

Det gør vi ved at søge efter andre materialer med samme emner (specifikt subject DBCS), målgruppe og kilde.
Derudover udelukker vi den titel der vises på den pågældende materialevisningsside og sorterer galleriet efter nyeste først.

Eksempelpost: Harry Potter og Fønixordenen

https://bibliotek.kk.dk/ting/object/870970-basis%3A54871996

  • Emneord - <dc:subject xsi:type="dkdcplus:DBCS">
    • <dc:subject xsi:type="dkdcplus:DBCS">magi</dc:subject>
    • <dc:subject xsi:type="dkdcplus:DBCS">troldmænd</dc:subject>
  • Målgruppe - <dcterms:audience>
    • <dcterms:audience>børnematerialer</dcterms:audience>
  • Kilde - <ac:source>
    • <ac:source>Bibliotekskatalog</ac:source>
    • <ac:source>ereolen</ac:source>
  • <ac:source>ereolen global</ac:source>
    • <ac:source>comics plus</ac:source>
    • <ac:source>ebook central</ac:source>
    • <ac:source>rbdigital magazine</ac:source>
  • NOT pågældende titel - <dc:title>
    • <dc:title>Harry Potter og Fønixordenen</dc:title>
  • Sortering - ?sort=date_descending&
    • ?sort=date_descending
    • <ns1:sort>date_descending</ns1:sort>

Eksempelsøgning Harry Potter og Fønixordenen

https://bibliotek.kk.dk/ting/object/870970-basis%3A54871996

 

Skabelon for søgestreng:

term.subject any "<> <>" and

term.category any "<> <>" and

term.acSource any "<> <>" not

phrase.title ="<>"

?sort=date_descending

 

Eksempel på søgestreng:

term.subject any "magi troldmænd" and

term.category any "børnematerialer" and

term.acSource any "bibliotekskatalog 'ereolen' ‘ereolen global’ 'comics plus' 'ebook central' 'rbdigital magazines'" not

phrase.title="harry potter og fønixordenen"

 

https://bibliotek.kk.dk/search/ting/term.subject%20any%20%22magi%20troldm%C3%A6nd%22%20and%20term.category%20any%20%22b%C3%B8rnematerialer%22%20and%20term.acSource%20any%20%22bibliotekskatalog%20%27ereolen%27%20%27ereolen%20global%27%20%27comics%20plus%27%20%27ebook%20central%27%20%27rbdigital%20magazines%27%22%20not%20phrase.title%3D%22harry%20potter%20og%20f%C3%B8nixordenen%22?sort=date_descending&

 

Eksemple katalogpost

NB. Nedenstående er responses fra henholdsvis Open Platform og Opensearch, men på den helt samme post, da jeg ikke har adgang til København fra Postman klienten.

Open Platform

    {
      "dcTitle": [
        "Harry Potter og Fønixordenen"
      ],
      "subjectDBCS": [
        "magi",
        "troldmænd"
      ],
      "audience": [
        "børnematerialer"
      ],
      "acSource": [
        "Bibliotekskatalog"
      ]
   },

Opensearch

<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/" xmlns:ac="http://biblstandard.dk/ac/namespace/" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:dcterms="http://purl.org/dc/terms/" xmlns:dkabm="http://biblstandard.dk/abm/namespace/dkabm/" xmlns:dkdcplus="http://biblstandard.dk/abm/namespace/dkdcplus/" xmlns:oss="http://oss.dbc.dk/ns/osstypes" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://oss.dbc.dk/ns/opensearch">
    <SOAP-ENV:Body>
        <searchResponse>
            <result>
                <hitCount>1</hitCount>
                <collectionCount>1</collectionCount>
                <more>false</more>
                <searchResult>
                    <collection>
                        <resultPosition>1</resultPosition>
                        <numberOfObjects>1</numberOfObjects>
                        <object>
                            <dkabm:record>
                                <ac:identifier>54871996|870970</ac:identifier>
                                <dc:identifier xsi:type="dkdcplus:ISBN">9788702272482</dc:identifier>
                                <ac:source>Bibliotekskatalog</ac:source>
                                <dc:source>Harry Potter and the Order of the Phoenix</dc:source>
                                <dc:title>Harry Potter og Fønixordenen</dc:title>
                                <dc:title xsi:type="dkdcplus:full">Harry Potter og Fønixordenen</dc:title>
                                <dcterms:alternative>Harry Potter og Fønixordenen</dcterms:alternative>
                                <dc:creator xsi:type="dkdcplus:aut">Joanne K. Rowling</dc:creator>
                                <dc:creator xsi:type="oss:sort">Rowling, Joanne K.</dc:creator>
                                <dc:subject xsi:type="dkdcplus:DK5-Text">Skønlitteratur</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DBCS">fantasy</dc:subject>
                                <dc:subject xsi:type="dkdcplus:genre">fantasy</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DBCN">for 12 år</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DBCN">for 13 år</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DBCN">for 14 år</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DBCN">for 15 år</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DBCN">for 16 år</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DBCS">magi</dc:subject>
                                <dc:subject xsi:type="dkdcplus:DK5">sk</dc:subject>
                               <dc:subject xsi:type="dkdcplus:DBCS">troldmænd</dc:subject>
                                <dcterms:abstract>
Fantasy. Da Harry Potter vender tilbage til Hogwarts er meget ændret. Man tror, at han lyver angående Voldemort, og ministeriet sender en repræsentant til skolen, der snart er delt i to fjendtlige lejre
</dcterms:abstract>
                                <dc:description>På ryggen: 5</dc:description>
                                <dc:description>
Joanne K. Rowling bruger også pseudonymet Robert Galbraith
</dc:description>
                                <dc:description xsi:type="dkdcplus:series">5. del af: Harry Potter og De Vises Sten</dc:description>
                                <dcterms:audience xsi:type="dkdcplus:age">fra 12 år</dcterms:audience>
                                <dcterms:audience>børnematerialer</dcterms:audience>
                                <dkdcplus:version>6. udgave, 1. oplag (2018)</dkdcplus:version>
                                <dc:publisher>Gyldendal</dc:publisher>
                                <dc:contributor xsi:type="dkdcplus:trl">Hanna Lützen</dc:contributor>
                                <dc:date>2018</dc:date>
                                <dc:type xsi:type="dkdcplus:BibDK-Type">Bog</dc:type>
                                <dcterms:extent>841 sider</dcterms:extent>
                                <dc:language xsi:type="dcterms:ISO639-2">dan</dc:language>
                                <dc:language>Dansk</dc:language>
                            </dkabm:record>
                            <identifier>870970-basis:54871996</identifier>
                            <primaryObjectIdentifier>870970-basis:54871996</primaryObjectIdentifier>
                            <recordStatus>active</recordStatus>
                            <creationDate>2018-09-19</creationDate>
                            <formatsAvailable>
                                <format>dkabm</format>
                                <format>marcxchange</format>
                            </formatsAvailable>
                            <objectsAvailable>
                                <identifier>870970-basis:54871996</identifier>
                            </objectsAvailable>
                        </object>
                    </collection>
                </searchResult>
                <facetResult />
                <statInfo>
                    <fedoraRecordsCached>1</fedoraRecordsCached>
                    <fedoraRecordsRead>0</fedoraRecordsRead>
                    <time>0.068504</time>
                    <trackingId>
os:2020-02-28T13:17:39:432768:7934:cisterne:opensearch-m-5-2-m-deploy-7b46447bcd-rxbnt
</trackingId>
                </statInfo>
            </result>
        </searchResponse>
    </SOAP-ENV:Body>
</SOAP-ENV:Envelope>
Skærmbillede 2020-06-23 kl. 10.22.04.png (198 KB) Skærmbillede 2020-06-23 kl. 10.22.04.png Gitte Barlach, 06/23/2020 10:22 AM
FireShot Capture 009 - Materiale - Elefanter i køkkenhaven - - upgrade-fbs.ddbcms.dk.png (1.26 MB) FireShot Capture 009 - Materiale - Elefanter i køkkenhaven - - upgrade-fbs.ddbcms.dk.png en af de relaterede materialer vises uden sin forside Gitte Barlach, 07/03/2020 02:11 PM
FireShot Capture 008 - Materiale - Spis køkkenhaven - - upgrade-fbs.ddbcms.dk.png (1.39 MB) FireShot Capture 008 - Materiale - Spis køkkenhaven - - upgrade-fbs.ddbcms.dk.png nogle af de relaterede materialer vises uden deres forsider Gitte Barlach, 07/03/2020 02:13 PM
FireShot Capture 010 - Materiale - Hej have! _ familiens bog om naturen lige uden for vindue_ - upgrade-fbs.ddbcms.dk.png (1.33 MB) FireShot Capture 010 - Materiale - Hej have! _ familiens bog om naturen lige uden for vindue_ - upgrade-fbs.ddbcms.dk.png flere relaterede materialer vises uden deres forsider Gitte Barlach, 07/03/2020 02:14 PM
FireShot Capture 001 - Materiale - Politikens bog om clematis og andre klatreplanter -_ - upgrade-fbs.ddbcms.dk.png (404 KB) FireShot Capture 001 - Materiale - Politikens bog om clematis og andre klatreplanter -_ - upgrade-fbs.ddbcms.dk.png Post med tom grå bjælke Gitte Barlach, 07/09/2020 01:25 PM
Skærmbillede 2020-07-09 kl. 14.30.52.png (61.3 KB) Skærmbillede 2020-07-09 kl. 14.30.52.png Gitte Barlach, 07/09/2020 02:32 PM
Skærmbillede 2020-07-09 kl. 14.30.37.png (61.8 KB) Skærmbillede 2020-07-09 kl. 14.30.37.png Gitte Barlach, 07/09/2020 02:32 PM

Related issues

Related to DDB CMS - Bug #4886: Galleri - viser materialer som ikke er i beholdningReady for development

History

#1 Updated by Rolf Madsen 5 months ago

@Christel, så vidt jeg kan se er Lasse ikke oprettet i Redmine, så jeg kan ikke assigne sagen til ham.

Er det korrekt at den skal lægges under Release 31.1 eller blev vi enige om at den først skulle med senere?

#2 Updated by Rolf Madsen 5 months ago

  • Target version changed from Release 31-1 - Lists and Opensearch (7.x-6.0.0) to Release 31-2 - Bug fixes (7.x-5.?.?)

#3 Updated by Rolf Madsen 4 months ago

  • Description updated (diff)

#4 Updated by Rolf Madsen 4 months ago

  • Description updated (diff)

#5 Updated by Christel Krabbenhøft 4 months ago

  • Assignee changed from Christel Krabbenhøft to Lasse Borly
  • Priority changed from Normal to High

#6 Updated by Lasse Borly 4 months ago

  • URL med eksempel set to https://github.com/danskernesdigitalebibliotek/ddb-react/pull/90

#7 Updated by Christel Krabbenhøft 4 months ago

  • Assignee changed from Lasse Borly to Kasper Garnæs

#9 Updated by Rolf Madsen 4 months ago

@Lasse mangler der mere udvikling, hvor Kasper er involveret eller kan status ændres til Needs code review?

#10 Updated by Christel Krabbenhøft 4 months ago

Yes, Kasper skal integrere modulet.

#11 Updated by Lasse Borly 4 months ago

Rolf Madsen skrev:

@Lasse mangler der mere udvikling, hvor Kasper er involveret eller kan status ændres til Needs code review?

Jeg er usikker på hvad proceduren er herfra må jeg indrømme da jeg ikke har været inde over den her del af processen før.
Det må lige være op til Kasper hvad der sker herfra inden vi ændrer status på den tænker jeg.

#12 Updated by Rolf Madsen 4 months ago

  • Status changed from Ready for development to Development

Det er meget fint Lasse, og tak for det!

Jeg kan forstå at Kasper mangler at foretage integrationen af React komponenten i DDB CMS, så jeg har ændret status til Development da sagen stadig er under aktiv udvikling.

Det er først når det endelige pull request er leveret at vi ændrer status til Needs code review.

#13 Updated by Kasper Garnæs 3 months ago

  • Status changed from Development to Needs code review
  • Assignee changed from Kasper Garnæs to Gitte Barlach

PR på integtation med DDB CMS: https://github.com/ding2/ding2/pull/1630

#14 Updated by Rolf Madsen 3 months ago

@Kasper, kan det passe at der ikke er sat kontekst på oversættelser ved t() funktionerne, og bør vi ikke have ændret det så det bliver et krav fremover?

#15 Updated by Gitte Barlach 3 months ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#16 Updated by Kasper Garnæs 2 months ago

@Rolf: Det er korrekt at der ikke er sat kontekst på oversættelserne.

Jeg synes det er fint at have dette på som krav fremadrettet. I så fald skal vi sørge for at få det med ind i vores reviews og/eller som et automatiseret check.

#17 Updated by Rolf Madsen 2 months ago

Helt fint.

Jeg har talt med Gitte om at vi tager det med på et Core team møde inden sommerferien!

#18 Updated by Jesper Kristensen 2 months ago

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

Et par sprøgsmål til udbydende kommentar i koden.

Alt det med tokens skal tages ud af dette PR, da det kommer i #1617 (og er lavet om der). Det vil gøre det nemmer når de skal merges ind i samme release.

Derudover er det hele lidt "magick", da selve det som laver noget ligger i ding react, som ikke er en del af core (af mig ukendt årsager) og derfor lidt svært at sige noget om om det virker eller er en god måde det er lavet på. Så det vil mest bare være kode stil kommentar.

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

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

Jeg har besvaret kommentarerne.

#20 Updated by Jesper Kristensen about 2 months ago

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

Godkendt

#21 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.

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

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#23 Updated by Gitte Barlach about 2 months ago

Testet på upgrade-fbs med 7.x-6.1.0-beta4
Mac - browser Chrome

På materialevisningssiderne ses kun en grå skygge; det virker som om der er et element, men det bliver aldrig vist.
Konfigurationer burde være ok, herunder client id og secrets, hvilket jeg har verificeret på den måde at jeg kan konstatere at min bruger kan se Min liste samt Mine gemte søgninger.
Se vedhæftede screendump

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

Ændringen er afhængig af opgraderingen til cover service v2 i #4793. Denne er ikke blevet reviewed og derfor heller ikke merged endnu.

#25 Updated by Gitte Barlach about 2 months ago

  • Subject changed from Galleri på Materialevisningssiden to Galleri på Materialevisningssiden (TEST AFVENTER #4793)

#26 Updated by Gitte Barlach about 1 month ago

Testet på upgrade-fbs med 

a) der vises nu relaterede materialer på materialevisningssiden
a1) i nogle tilfælde (uden jeg dog kan gennemskue hvorfor) er der "huller" i visning, dvs på nogle af de relaterede materialer vises der  ingen forside; men man kan stadig klikke i det tomme areal og derved komme til selve materialevisningen. Og her vises faktisk en forside. 
eksempler:
https://upgrade-fbs.ddbcms.dk/ting/object/870971-avis%3A89659345 
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A28702167
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis:46508122

Noget kunne tyde på at det hænger sammen med størrelsen på forsidebilledet. 
Disse materialers forsider vises f.eks. ikke når de optræder som releaterede materialer:
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis:27157084
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis:46811755

b) til højre ovenover de relaterede materialer vises flg. tekst, der skal oversættes i fald vi skal beholde den:
​​​​​​"Show search result for related materials"
 

c) klikker man på "Show search result for related materials" udføres søgningen godt nok, men man får flg. fejl øverst på søgeresultat siden:
Eksempel:

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '%' AS w10, t.word LIKE '%niveau'%' AS w11, t.word LIKE '%børnematerialer%' AS w' at line 1

Eksempel:
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '%' AS w8, t.word LIKE '%niveau'%' AS w9, t.word LIKE '%voksenmaterialer%' AS w10' at line 1

d) det ser ikke ud til at der vises /skal vises relateret materiale på tidsskrifter; men der vises alligevel en grå boks som om der mangler et element på siden.
Der er også andre poster hvor relaterede materialer ikke vises - f.eks. https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A29783594

e) nogle gange er det meget ikke-relevante materialer der vises:

Eksempler:
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A07253044
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A23192721
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A27601073



 

 

#27 Updated by Gitte Barlach about 1 month ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Kasper Garnæs
  • Kategorier Driftsvedligehold - Oversættelser (udarbejdes før udrulning) added

Hej Kasper
kan du se på denne igen?
tænker først og fremmest på a) og c)

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

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

a1) i nogle tilfælde (uden jeg dog kan gennemskue hvorfor) er der "huller" i visning, dvs på nogle af de relaterede materialer vises der  ingen forside; men man kan stadig klikke i det tomme areal og derved komme til selve materialevisningen. Og her vises faktisk en forside. 

Jeg har tilrettet React komponenterne så materialer uden en forside i tilstrækkelig størrelse ekskluderes fra visningen af de relaterede materialer: https://github.com/danskernesdigitalebibliotek/ddb-react/pull/100

 

c) klikker man på "Show search result for related materials" udføres søgningen godt nok, men man får flg. fejl øverst på søgeresultat siden

Dette ser ud til at skyldes en eksisterende problem med søgning i redaktionelt indhold, der kommer til udtryk, når søgestrengen indeholder ' (gåseøje). Det virker som CQL imod OpenSearch men ikke imod Drupals database. I de fleste situationer er ' ligegyldigt. Jeg foreslår at vi fjerner det: https://github.com/ding2/ding2/pull/1660

 

d) det ser ikke ud til at der vises /skal vises relateret materiale på tidsskrifter; men der vises alligevel en grå boks som om der mangler et element på siden.
Der er også andre poster hvor relaterede materialer ikke vises - f.eks. https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A29783594

Hvis materialet ikke indeholder emner så genereres der en tom streng i CQL søgningen som OpenSearch fejler på. Jeg har opdateret React komponenten så dele af søgestrengen der måtte indeholde tomme søgestrenge ekskluderes.

Det betyder at hvis et materiale fx. ikke har nogle emner så udelades "term.subject any "${subjects}"" fra søgestregnen i stedet for at blive til "term.subject any """: https://github.com/danskernesdigitalebibliotek/ddb-react/pull/101

#29 Updated by Jesper Kristensen about 1 month ago

  • Status changed from Needs code review to Reviewed

Reviewed and approved

#30 Updated by Gitte Barlach about 1 month ago

  • Subject changed from Galleri på Materialevisningssiden (TEST AFVENTER #4793) to Galleri på Materialevisningssiden

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

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

Alle tre ovenstående PRs er merged.

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

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#33 Updated by Gitte Barlach about 1 month ago

Testet på upgrade-fbs med 7.x-6.1.0-beta7

a1)
Der vises nu kun materialer, der har en forside der "passer" i størrelsen. Godkendt med den bemærkning at det skal overvejes om det kan blive muligt at vise de materialer, der er relevante, uanset størrelsen på forsiden. 
b) teksten "Show search result for related materials" er desværre stadig på engelsk
c) jeg får desværre stadig flg. fejl når jeg klikker på "Show search result for related materials":
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '%' AS w11, t.word LIKE '%global'%' AS w12, t.word LIKE '%'comics%' AS w13, t.wor' at line 1
d) ændringen betyder at der nu vises relaterede materialer på eksempelvis tidsskrifter; spørgsmålet er dog om de materialer, der vises, er relevante.
se f.eks.
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A20401206
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A43843141

 

#34 Updated by Gitte Barlach about 1 month ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Ift. til d)
Visning af relaterede materialer skal helt udelades på poster, der ikke har emneord tilknyttede.

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

Ændringerne var ikke kommet med i 7.x-6.1.0-beta7 pga. en fejl i releaseprocessen. Dette medfører problemerne b) og c).

PR der fjerner visningen af relaterede materialer for materialer uden emneord: https://github.com/ding2/ding2/pull/1663

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

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

#37 Updated by Gitte Barlach about 1 month ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#38 Updated by Jesper Kristensen about 1 month ago

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

Approved

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

  • Status changed from Reviewed to Needs code review

Gitte har bemærket at materialerne der vises i karousellen vises i en anden rækkefølge end i søgeresultatet. Det skyldes at visningen af de relaterede materialer pt. forudsætter at cover service returnerer materialer i samme rækkefølge som de bliver efterspurgt. Det er ikke nødvendigvis givet.

Følgende PR baserer visningen af relaterede materialer på rækkefølgen fra søgeresultatet: https://github.com/danskernesdigitalebibliotek/ddb-react/pull/102

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

Gitte har også fundet et issue hvor kald til Cover service fejler hvis den foregående søgning efter relaterede materialer ikke giver nogen resultater.

Det har medført en par ændringer til hvordan situationen med manglende relaterede materialer med covers håndteres generelt: https://github.com/danskernesdigitalebibliotek/ddb-react/pull/103

#41 Updated by Gitte Barlach about 1 month ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#42 Updated by Jesper Kristensen about 1 month ago

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

LGTM. Og observationerne omkring cover serive er helt rigtige.

#43 Updated by Gitte Barlach about 1 month ago

Testet på upgrade-fbs med  7.x-6.1.0-beta8

Testcases: 

1) poster uden emneord og visning af relaterede materialer
På poster hvor der ikke indgår emneord som en del af den bibliografiske beskrivelse (eksempelvis PIF poster) vises relaterede materialer ikke. Godkendt.
 
2) Show search result for related materials
2.1. Oversættelse
Teksten er oversat til "Vis søgeresultat for relaterede materialer". Godkendt. 

2.2. rækkefølge af poster i søgeresultat
Når man klikker på linket i "Vis søgeresultat for relaterede materialer" vises posterne i søgeresultatet nu i samme rækkefølge som de vises i på galleriet. Godkendt. 
 

3) Håndtering af manglende relaterede materialer med covers 
3.1. Såfremt der ikke er tilstrækkeligt mange materialer til at fylde hele Galleriet ud, tager visningen nu pænt højde for dette, dvs. der er ingen "tommepladser". Godkendt.

3.2. fejl ift. visning af tom grå bjælke og intet Galleri
Jeg har fundet denne fejl:
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis:25696921 - posten har et emneord tilknyttet
https://upgrade-fbs.ddbcms.dk/ting/object/870971-avis%3A82434267 - posten har et emneord tilknyttet
Af en eller anden grund vises der intet Galleri på materialet; den grå boks fylder mere i højden end jeg har set ved tidligere test..
Fejlen optræder også hyppigt på avisartikler
Jeg lægger mærke til at der optræder en hel del fejl i db loggen af flg. type:
"Loaded empty collection with ID 870970-basis:47935369, this could be an error."

3.3. Rolf har oplevet at de grå billedbaggrunde stod og blinkede i meget lang tid før den gav op. TIl gengæld fjernede den så overskrift og link til søgeresultat.

4) relevans
Som jeg ser det går der noget galt med denne:
https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A27601073
De materialer der vises som relevante er det ikke. 

#45 Updated by Steen Larsen about 1 month ago

I note 43, pkt 4 henvises til en søgning som ikke giver det rigtige resultat.

Det ser ud til at man tror enkeltapostrof har en betydning i søgningerne men dette tegn har ingen betydning i CQL.

Og det ser heller ikke ud til at det har en betydning for ddbcms-til-cql oversætteren.

Lav f.eks. søgningen term.acSource any " 'ereolen global' "

og kig på hvilke kilder der returneres poster fra:

ereolen (39221)
global issues in context (17970)
ereolen global (12445)

Det er en term-søgning og der returneres poster fra kilder som indeholder én eller flere af de angivne ord.

Et andet eksempel:

term.subject any " 'Tage Andersen (f. 1947)' blomster "

som svarer til

term.subject any " Tage Andersen f 1947 blomster "

Altså også inkluderet alle forfattere hvor årstal er angivet med "f"

Term-søgningen kombineret med "any" dur egentlig kun hvor de enkelte emneord ikke indeholder mellemrum

 

Der er en anden søgemåde (som muligvis ikke er CQL) som kunne bruges - her med nogle emneord med mellemrum som vi ønsker at finde som subject:

emne1 ord1
emne2 ord2
emne3 ord3

som giver denne søgning:

term.subject = ("emne1 ord1" or "emne2 ord2" or "emne3 ord3")

som igen svarer til

term.subject = "emne1 ord1" or term.subject = "emne2 ord2" or term.subject = "emne3 ord3"

Jeg har ikke overvejet om man istedet skal bruge phrase-søgninger men en sådan søgning er også mulig: phrase.subject = ( ... or ... or ... )

#46 Updated by Rolf Madsen about 1 month ago

Jeg har lige testet den sidste mulighed.

#47 Updated by Rolf Madsen about 1 month ago

Skabelon for søgestreng:

phrase.subject=("<>" or "<>") and

term.category=("<>" or "<>") and

term.acSource=("<>" or "<>") not

phrase.title ="<>"

?sort=date_descending

Eksempel på søgestreng:

phrase.subject=("magi" or "troldmænd") and 

term.category=("børnematerialer") and 

term.acSource=("bibliotekskatalog" or "ereolen" or "ereolen global" or "comics plus" or "ebook central" or "rbdigital magazines") not 

phrase.title=("harry potter og fønixordenen")

?sort=date_descending

Eksempelsøgning

https://upgrade-fbs.ddbcms.dk/search/ting/phrase.subject%3D%28%22magi%22%20or%20%22troldm%C3%A6nd%22%29%20and%20term.category%3D%28%22b%C3%B8rnematerialer%22%29%20and%20term.acSource%3D%28%22bibliotekskatalog%22%20or%20%22ereolen%22%20or%20%22ereolen%20global%22%20or%20%22comics%20plus%22%20or%20%22ebook%20central%22%20or%20%22rbdigital%20magazines%22%29%20not%20phrase.title%3D%28%22harry%20potter%20og%20f%C3%B8nixordenen%22%29?sort=date_descending

#48 Updated by Gitte Barlach about 1 month ago

  • Status changed from Reviewed to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Kasper Garnæs

#49 Updated by Steen Larsen about 1 month ago

Ja, det er bedre.

En anden ting er jo så det der "fyldes i"...

https://upgrade-fbs.ddbcms.dk/ting/object/870970-basis%3A27601073 er der et link til relaterede materialer og linket indeholder (her med den gamle skabelon): term.category any "'alment niveau' voksenmaterialer"

Det giver ikke mening at søge på "alment niveau" - term.category kan kun have to værdier voksenmaterialer og børnematerialer.

Det giver dog ikke fejl så længe ét af de to gyldige værdier er med i søgningen.

Værdierne har jo ikke mellemrum så her kan any-versionen i øvrigt godt bruges.

I nogle tilfælde kan man komme ud for at få søgningen: term.category any "voksenmaterialer børnematerialer" - den bidrager nok ikke med så meget, men fejl er det jo heller ikke.

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

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


PRs på følgende ændringer:

Omskrivning af queries efter relaterede materialer baseret på kommentar 47:

- https://github.com/danskernesdigitalebibliotek/ddb-react/pull/104

- https://github.com/ding2/ding2/pull/1664

 

Fjernelse af relatede materialer, hvis der ikke kan findes nogen ud fra søgning som også har covers:

- https://github.com/danskernesdigitalebibliotek/ddb-react/pull/105

#51 Updated by Gitte Barlach about 1 month ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

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

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

Merged efter aftale med Gitte Barlach.

Søgningen på "alment niveau" kommer ved at tage værdier fra dcterms:audience feltet for materialet som angivet øverst i dette issue.

#53 Updated by Gitte Barlach about 1 month ago

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

Testet på upgrade-fbs med Release - 7.x-6.1.0-beta9

Har testet den omskrevne søgning efter relaterede materialer baseret på kommentar 47
Eksempel: phrase.subject=("Tage Andersen (f. 1947)" or "blomster" or "blomsterdekorationer" or "havekunst" or "interiører" or "Andersen Tage (f. 1947)") and term.category=("alment niveau" or "voksenmaterialer") and term.acSource=("bibliotekskatalog" or "ereolen" or "ereolen global" or "comics plus" or "ebook central" or "rbdigital magazines") not phrase.title=("Enthusiasm")
Det ser ok ud. 

Har testet fjernelse af relatede materialer, hvis der ikke kan findes nogen ud fra søgning som også har covers:
f.eks. https://upgrade-fbs.ddbcms.dk/ting/object/870971-avis%3A82434267

Godkender hermed sagen. 

#54 Updated by Christian Hundebøl Høj 21 days ago

Jeg har lige åbnet en post på kolding-stg.ddbcms.dk -> https://kolding-stg.ddbcms.dk/ting/object/870970-basis:48016022

Her kan man se, i gallerivisningen, at man bliver anbefalet romanen "Fredsmaskinen", FAUST 47706165.

Men vi har ikke den bog? https://kolding-stg.ddbcms.dk/ting/object/870970-basis:47706165

#55 Updated by Christian Hundebøl Høj 20 days ago

Tillæg: Jeg kan se, at den givne bog, "Fredsmaskinen", er på eReolen. Men det er jo ikke eReolen-posten, der linkes til.

Samtidig er en af de andre anbefalinger denne: https://kolding-stg.ddbcms.dk/ting/object/870970-basis:48163572 -- det er et materiale som kun tre biblioteker har, men den er ikke på eReolen?

#56 Updated by Rolf Madsen 9 days ago

@Christian, det er godt spottet og du har så ganske ret. Tak fordi du gør opmærksom på problemet!

Det lader til at vi har glemt at tilføje "and holdingsitem.agencyid="762100"" til søgestrengen der ligger under karusellen.

Jeg opretter et issue på det!

#57 Updated by Rolf Madsen 9 days ago

  • Related to Bug #4886: Galleri - viser materialer som ikke er i beholdning added

#58 Updated by Rolf Madsen 9 days ago

Galleri - viser materialer som ikke er i beholdning - #4886

Also available in: Atom PDF