Project

General

Profile

Bug #5225

Se Online-knap benytter forkert url

Added by Gitte Barlach 4 months ago. Updated about 2 months ago.

Status:
Resolved (tag version)
Priority:
High
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Min konto - Login/Adgang og proxy, Søgning - Søgeresultat efter søg - Brønd

Description

Fejlen ser ud til at være introduceret via #4877. 

Heri beskrives rækkefølgen som flg.:

1. Check om postens type er en af online typerne under  /admin/config/ting/online_types
2. Check dcterms:URI
3. Check om postens relationer har en hasOnlineAccess

Men det er ikke korrekt. Hvis posten har en hasOnlineAccess, er det den, der skal benyttes. 
Det ses f.eks. i kilden Books24x7 - posten 150083-ebog:bks000113290



 

#1

Updated by Gitte Barlach 4 months ago

  • Priority changed from High to Normal
#2

Updated by Steen Larsen 4 months ago

Jeg (et script) har fundet følgende kilder hvor urlen findes både i dkabm-posten og som relation til posten og hvor de to urler er forskellige.
Scriptet har kigget på de første 10 poster fundet ved facet.acsource="...".

Der er vist et forslag til evt. workaround der kan benyttes i proxy-settings, admin/config/ting/proxy

Der er dog ingen garanti at det virker for alle 73.000 poster fra de viste kilder eller om der er poster fra andre kilder (udover de 10) der fejler

 

books24x7
Værtsnavn: www.books24x7.com
Udtryk: /www.books24x7.com\/marc.asp\?bookid=/
Erstatning: 2masteritezproxy.skillport.com/sso/skillport?context=

ebook central
Værtnavn: ebookcentral.proquest.com (findes i forvejen hvis man har kilden aktiveret)
Udtryk: /(ddbdk-public|\[PROVIDERSLIBRARYID\])/
Erstatning (ændres ikke)

Hvis udtrykket kun indeholder /\[PROVIDERSLIBRARYID\]/ så opdateres udtryk som vist med parentes.
Hvis der allerede findes en parentes i søge-udtrykket med flere elementer så tilføjes 'ddbdk-public' på passende vis med lodret streg.

filmstriben
Værtsnavn: www.filmstriben.dk
Udtryk: /\/bibliotek\//
Erstatning: /fjernleje/
Kryds i "Do not use proxy prefix for this hostname"

Kan ikke bruges hvis man  har kilden "Filmstriben på biblioteket" aktiveret i vip-søgeprofilen
Er heller ikke 100% sikker da det kan afhænge af rækkefølgen af linkene i posten, hvor nogle af filmcentralens (gamle) urler vises.
 

gyldendals røde online ordbøger
turteori

- kan ikke løses da det er generelle link der findes i posten. Der er dog kun få poster i de to kilder.

videnskab.dk
- ser ok ud

bibliotekernes podcasts
- linker til bibliotek.kk.dk istedet for ereolen men ser ud til at virke.

 

#3

Updated by Gitte Barlach 4 months ago

  • Assignee set to Thomas Hansen

Hej Fini
Har du mulighed for at tage et kig på denne?

#4

Updated by Thomas Hansen 4 months ago

Den beskrevne rækkefølge afhænger også af hvor det er henne...

Men hvis det bare er rækkefølgen: https://github.com/ding2/ding2/pull/1815

Men hvorfor benyttede man ikke lige chancen for at få ryddet op i is('online') og isOnline()?

#5

Updated by Gitte Barlach 4 months ago

  • Status changed from Ready for development to Funktionstest (pull-requests)
  • Assignee changed from Thomas Hansen to Gitte Barlach
#6

Updated by Ninna Rasmussen 3 months ago

  • Assignee changed from Gitte Barlach to Thomas Hansen
#7

Updated by Gitte Barlach 3 months ago

okay, forstår ikke altid vores platform.sh miljø. Har gennem flere testforsøg fået flg. fejl, der har forhindret mig i at komme videre:

 

Warning: Invalid argument supplied for foreach() i ding_facetbrowser_content_type_render() (linje 33 af /app/public/profiles/ding2/modules/ding_facetbrowser/plugins/content_types/ding_facetbrowser/ding_facetbrowser.inc).


Gennemgik konfigs. flere gange uden at se lyset. Men nu pludselig i dag udebliver fejlen. Så vi gør et nyt forsøg på at teste. 

#8

Updated by Lotte Tøstesen 3 months ago

Håber I kan patche en rettelse på hurtigt - det er ret kritisk for os.

Vh Lotte/Roskilde

#9

Updated by Tue Gaston 3 months ago

Lotte Tøstesen wrote:

Håber I kan patche en rettelse på hurtigt - det er ret kritisk for os.

Vh Lotte/Roskilde

Kan du ikke få Steens workaround som beskrevet i https://platform.dandigbib.org/issues/5225#note-2 til at virke?

#10

Updated by Tue Gaston 3 months ago

  • Assignee changed from Thomas Hansen to Lotte Tøstesen
#11

Updated by Gitte Barlach 3 months ago

Testet på platform.sh 
"Se online"-urlerne fungerer nu korrekt. Derfor vil jeg gerne sende denne videre til code review!

#12

Updated by Gitte Barlach 3 months ago

  • Status changed from Funktionstest (pull-requests) to Needs code review
  • Assignee changed from Lotte Tøstesen to Jørgen Nielsen
#13

Updated by Lotte Tøstesen 3 months ago

Tue Gaston skrev:

Lotte Tøstesen wrote:

Håber I kan patche en rettelse på hurtigt - det er ret kritisk for os.

Vh Lotte/Roskilde

Kan du ikke få Steens workaround som beskrevet i https://platform.dandigbib.org/issues/5225#note-2 til at virke?

Jo, nu har jeg fået Ebook Central og Books 24x7 til at virke - men det er jo ikke alle der lige sidder og roder med den slags, så jeg håber nu I vil sætte den til høj prioritet. Jeg har ikke haft tid til at tjekke om alle de andre e-ressourcer virker.

#14

Updated by Ninna Rasmussen 3 months ago

  • Priority changed from Normal to High
#15

Updated by Jørgen Nielsen 3 months ago

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

reviewet og godkendt

#16

Updated by Gitte Barlach 3 months ago

Klar til merge, og derefter integrationstest. 

#17

Updated by Kasper Garnæs 3 months ago

  • Status changed from Reviewed to Integrationstest (Versioner)
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

#18

Updated by Kasper Garnæs 3 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach
#19

Updated by Gitte Barlach about 2 months ago

  • Assignee changed from Gitte Barlach to Steen Larsen

@Steen: kan du hjælpe med at teste denne?

#20

Updated by Steen Larsen about 2 months ago

Har testet kilderne nævnt i note 2 og et par øvrige kilder - alle virker som forventet med det korrekte link i "Se online"-knappen på detalje-siden

Jeg opdagede at der også i hasonlineaccess-relationen kan tilbydes flere urler og her vælges så den sidste, hvilket må være korrekt indtil videre. Kilden "bibliotekernes podcasts" er et eksempel på dette.

Opgaven må betragtes som løst.

#21

Updated by Gitte Barlach about 2 months ago

  • Status changed from Integrationstest (Versioner) to Resolved (tag version)

Tusind tak for test, Steen!!

Jeg sætter hermed sagen til resolved. 

Also available in: Atom PDF