DDB CMS

Carsten Vilhelmsen, kan du hjælpe min hukommelse på gled omkring det her spørgsmål?

Er det en ændring du er opmærksom på?

Der er et generelt problem med syndikering af relaterede materialer, så jeg er pt. ikke i stand til at teste det specifikke spørgmål.

Test af workflow:

• Jeg finder Pingprisen 2020 – Danmarks største tegneseriepris i oversigten (dump 1)
  
• Klikker "gennemse" og ser listen over tilknyttede materialer (dump 2)
  
• Artiklen er klar til download, men indholdsfeltet er tomt (dump 3)
• Artiklen er klar til download, men materialefeltet er tomt (dump 3)
  
• Artiklen publiceres uden relaterede materialer https://taarnbybib.dk/nyheder/litteratur/pingprisen-2020-danmarks-stoerste-tegneseriepris
  

Tak for opklaringen Carsten, det ser lidt giftigt ud.

@Ninna jeg tror vi skal have den omkring ITK, hvis det har noget med migreringen af indhold til Paragraphs at gøre.

Jeg har gentaget workflowet og støder stadig ind i de samme problemer

• Artiklen findet i BPI (dump 1)
• Previewet viser artiklen indholder relaterede materialer (dump 2)
• Syndikeringen af artikel viser at der mangler en manchet tekst (dump 3)
• Og at der ikke er kommet relaterede materialer med (dump 4)

Som tidligere nævnt i denne tråd er det en fejl som ikke umiddelbart relaterer sig til det oprindelige spørgsmål: en markering af hvilke relaterede, som er en del af den lokalebase. Men hvis man ikke får materialer med over ved syndikering, kan man naturligvis heller ikke se, hvilke af dem, som tilhører den lokale base :)

Ovennævnte relaterer til syndikering af materialer som er placeret i Biblioteksbasen (jf. dump 4).

Problemer med syndikering af materialer som ligger i material list paragraph er beskrevet i et andet issue: https://platform.dandigbib.org/issues/5096 

Jeg har testet denne på upgrade-fbs med seneste release (7.x-6.2.1)
BPI service url: http://v13.bpi.inlead.dk

Jeg kan ikke genskabe fejlen, dvs. "Det er ikke længere muligt at hente en artikel fra BPI med tilføjede materialer og med det samme se, hvilke materialer ens eget bibliotek mangler i egen beholding. "
Jeg får en markering af materialer, der ikke findes i testbibliotekets beholdning. 
se vedlagte skærmdump

Jeg har endvidere også testet hvor jeg simulerer et biblitoek, der har materialerne i beholdning og konstateret at de kommer med, samt at teksten i feltet indhold også kommer med.
Se vedlagte skærmdump. 
 

Jeg vil mene vi kan lukke denne sag. 

Lukkes på baggrund af Gittes kommentarer. 

Ninna Rasmussen wrote:

Lukkes på baggrund af Gittes kommentarer. 

Genåbnes på baggrund af https://platform.dandigbib.org/issues/5239

Jeg har forsøgt at genskabe fejlen på upgrade-fbs:

Jeg har sat mijøet op med DDF´s BPI credentials, samt agency 999999.

a) Jeg søger BPI artiklen "Blixen for begyndere" frem og klikker på "udgiv", hvorved nyheden hentes ind på sitet i redigeringstilstand; jeg får dels en generel advarsel om, at der er objekter, der ikke findes, dels er de pågældende alle markerede med rødt i materialelisten. Se vedlagte skærmdump. 

b) jeg henter nu en anden artikel fra BPI. Denne gang er det en nyhed med flere typer paragraphs: Text paragraph, Text box paragraph, og Paragraph type: Carousel
Jeg redigeringsbilledet af nyheden får jeg øverst en liste over en lang række materialer, der ikke eksisterer (som agency 999999 ikke har i beholdning); men folder jeg de enkelte carousel paragraph´s ud, er der ingen markeringer her. 

c) jeg ændrer konfigs på upgrade så jeg simulerer Greve Bibliotek. og henter en test BPI nyhed med titeln "Booktok" ind fra Aarhus, med materialer, hvor "Show as material list" IKKE er valgt. Og denne gang optræder fejlen: jeg får ingen markering af, at der er et materiale, nemlig "The song of Achilles" som biblioteket ikke har; det kan man først se, da nyheden er publiceret, og der mangler såvel reserverknap samt beholdning. 

Arni har set på sagen og observerer følgende: 

"Jeg kan ikke rigtigt genskabe fejlen ud over et enkelt materiale "The song of Achilles (Engelsk)", som også drillede i Gittes test. Hver gang jeg prøver så kommer materialerne korrekt over. Jeg kan også se at de bliver overført korrekt i koden, har det rigtige format. Afhængigt af mine lokale indstillinger så bliver færre eller flere at materialerne fra Gittes testpost fra bpi markeret som ikke i lokal beholdning.

Mit bedste bud her og nu er at der er enkelt materialer som driller måske pga. forkert indexering eller lignende. "

Tove Andresen: Oplever I fortsat problemet for alle poster? 

Hej Gitte. Se Toves svar ovenfor. Har du et bud på næste step i denne sag - kan vi undersøge det på anden vis? 

Jeg kan godt genskabe fejlen fra #32 både på Egedals staging site og lokalt. Problemet skyldes at der bruges getObjectRequest til at hente posterne fra Opensearch og den ser ikke ud til at tage hensyn til på om biblioteket har posten eller ej. Den fanger ebøger og lignende som slet ikke er del af den søgeprofil som bruges men ikke poster som indgår i andre bibliotekers fysiske beholdning. De poster bliver returneret med getObjectRequest og ser derfor ud til at biblioteket har dem og derfor kommer der ingen advarsel.

Løsningen er at bruge en søgning i stedet så der filtreres for materialer som biblioteket ikke har. Det kræver en lille omskrivning af den metode som henter materialerne og checker om de er der.

Fikset i PR: https://github.com/ding2/ding2/pull/1875

Validate metoden bruger nu søgning til at checke at materiale existerer lokalt.

Hvis det er materialer lavet som en materiale liste så kommer der besked øverrst og rød kant udenom det enkelte materiale. Hvis det er lavet som paragraphs så kommer der en besked øverst om at det enkelte materialer ikke findes. Og der står: "Invalid material id: .." ind i selve feltet.

Lokalt hos mig får jeg nogle underlige warnings når jeg tilgår poster fra BPI som bruger paragraphs til materialevisning:
 

• Notice: Trying to get property of non-object i relation_get_endpoints() (linje 769 af /var/www/drupalvm/drupal/ddbcms/profiles/ding2/modules/contrib/relation/relation.module).
• Warning: Invalid argument supplied for foreach() i relation_get_endpoints() (linje 769 af /var/www/drupalvm/drupal/ddbcms/profiles/ding2/modules/contrib/relation/relation.module).
• Warning: array_shift() expects parameter 1 to be array, null given i ting_reference_field_widget_form() (linje 502 af /var/www/drupalvm/drupal/ddbcms/profiles/ding2/modules/ting_reference/ting_reference.module).
• Notice: Trying to get property of non-object i ting_reference_field_widget_form() (linje 503 af /var/www/drupalvm/drupal/ddbcms/profiles/ding2/modules/ting_reference/ting_reference.module).
• Notice: Trying to get property of non-object i ting_reference_field_widget_form() (linje 504 af /var/www/drupalvm/drupal/ddbcms/profiles/ding2/modules/ting_reference/ting_reference.module).

Dem får jeg også før jeg lavede den her rettelse så jeg tror ikke de er relateret. Og som sagt det her kommer kun lokalt så måske er det min lokale installation som har en fejl.

Reviewed. Jeg har et par enkelte kommentarer.

Pr opdateret.

Reviewed og godkendt.

Testet på upgrade-fbs med 

Jeg går ind under /admin/bpi, finder nyheden "Livet på kostskole" og klikker på den, så den bliver hentet ind. Til nyheden er vedhæftet flere materialer. 3 af dem er øbverst markeret således:
 

• The material 870970-basis:51694945 does not exist.
• The material 870970-basis:50710440 does not exist.
• The material 870970-basis:27476104 does not exist.

Jeg går ind under Paragraph type: Materials list - og klikker på "edit".

I materialelisten er de tre materialer præfixet med "Invalid material id:"

Så som redaktør kanman nu se, hvilke materialer ens eget bibliotek mangler i egen beholding, når man henter en nyhed ind fra BPI. Det er godt. 
MEN: jeg prøver nu at slette de tre søgninger på materialer, mit bibliotek ikke har. Og her sidder jeg så fast: 
jeg kan fint slette, så tre felter står tomme, men jeg får ikke lov til at gemme nyheden. Lige gyldigt hvad jeg gør kommer den ud med:

• Search field is required.
• Search field is required.
• Search field is required.
• Search field is required.

Så vi skal have en udvikler til at se på sagen igen. 

Endnu en test. Denne gang med en nyhed hvor materialerne er sat ind i fleterne under DATA WELL (i stedet for en paragraph af typen Materials list)

I den situation indeholde beskeden øverst på siden ikke info. hvilke materialer, der mangler i beholdning. Kun at der mangler nogen:

These materials doesn't exists.

Dog er materialet markeret med en rød firkant, når man scroller ned og ser listen. (se vedlagte pdf)

Det kan vi nok godt leve med. 

Nyt Pr lavet: https://github.com/ding2/ding2/pull/1904

Den gør det muligt at gemme nyheder hvor der er materialer som biblioteket ikke har. Hvis redaktøren åbner paragraphen med materialer så kommer der et tomt søgefelt nederst som gør at de ikke kan gemme. Løsningen på det er at kollapse paragraphen igen og så kan de godt gemme den.

reviewet og godkendt

Testet på upgrade-fbs med rel. 6.0.0-betA3

a) søger artiklen Livet på kostskole frem i BPI, og klikker på "preview".
a1) Bemærker at der ikke i preview længere vises "Related materials:" - selvom der faktisk er materialer vedhæftet. 

b) klikker på "syndicate" og får advarsler om at 3 materialer ikke er i bibliotekets beholdning :
 

• The material 870970-basis:51694945 does not exist.
• The material 870970-basis:50710440 does not exist.
• The material 870970-basis:27476104 does not exist.

c) artiklen hentes ind i "mit" cms i redigeringstilstand
c1) jeg klikker på "edit paragraph type materials" . Her står de tre materialer, der ikke er i beholdning, således i listen over vedhæftede materialer:
"Invalid material id: 870970-basis:51694945"

Nederst er et tomt søgefelt. Det er markeret som obligatorisk. Giver stadig den klassiske fejl "Search field is required." hvorved jeg ikke kan komme til at gemme nyheden. 

c2) hvis jeg derimod IKKE klikker på "edit paragraph type materials", og dermed lader den være klappet samme, kan jeg redigere i de andre paragraph-typer, tilknytte kategori og målgruppe og gemme. De materialer der ikke er i beholdning, slttes automatisk og der vises dermed ikke "tomme poster" overfor slutbruger.
Det tænker jeg godt vi kan leve med. Manualen bør opdateres på dette punkt.