Project

General

Profile

Bug #3744

Huskelister i 29.1 (UDVIKLING AF RESCUE-LIST MODUL)

Added by Karsten Andersen 11 months ago. Updated about 2 months ago.

Status:
Closed
Priority:
Urgent
Assignee:
-
Target version:
-
Estimated time:
URL med eksempel:
Kategorier:
Min konto - Huskeliste

Description

Ved ikke om det bare er mig, men jeg har hidtil ikke haft problemer med huskelisten på mit personlige låner-login. Det er først efter jeg i dag har lagt 29.1 på vores dev, at den er forsvundet. På vores produktionsside optræder "Huskeliste" nu to gange. Den ene indeholder min huskeliste, mens den anden er tom. På dev-siden er der kun én huskeliste, og den er tom.

Eksperimentet med at oprette to lånere med samme navn er derimod faldet heldigt ud. Ingen problemer der. Deres respektive huskelister er ikke sammenblandede.

ding_listrescue.zip (2.54 KB) ding_listrescue.zip Rolf Madsen, 09/11/2018 11:18 AM
Skærmbillede 2018-10-15 12.57.49.png (111 KB) Skærmbillede 2018-10-15 12.57.49.png Slutbrugeren møder denne side Gitte Barlach, 10/15/2018 01:06 PM

History

#1 Updated by Rolf Madsen 11 months ago

  • Status changed from New to Needs analysis
  • Assignee set to Gitte Barlach
  • Target version set to Release 29-1 - Lister (7.x-4.4.0)

@Gitte, kan der være noget med konfigurationen af Openlist der driller her?

#2 Updated by Rolf Madsen 11 months ago

@Karsten, har I samme konfiguration (/admin/config/ding/provider/ting_openlist) på både jeres development- og produktionsmiljø af Openlist modulet?

#3 Updated by Rolf Madsen 11 months ago

Karsten skrev:

Rolf Madsen skrev:

@Karsten, har I samme konfiguration (/admin/config/ding/provider/ting_openlist) på både jeres development- og produktionsmiljø af Openlist modulet?

Ja, opsætningen er præcis det samme Openlist domain, Bibliotekskode og Openlist Præfix.

#4 Updated by Rolf Madsen 11 months ago

@Karsten jeg har fjernet billedet med præfixet da det ikke bør ligge på en offentlig side.

#5 Updated by Rolf Madsen 11 months ago

@Karsten kan du prøve at tjekke Openlist præfix tegn for tegn for at se om de er HELT ens?

Bare et enkelt blanktegn altså mellemrum til forskel er nok til at fejlen vil opstå.

#6 Updated by Karsten Andersen 11 months ago

Rolf Madsen skrev:

@Karsten kan du prøve at tjekke Openlist præfix tegn for tegn for at se om de er HELT ens?

Bare et enkelt blanktegn altså mellemrum til forskel er nok til at fejlen vil opstå.

Det er hermed tjekket. De er helt ens, og der er ingen blanktegn nogen steder.

#7 Updated by Gitte Barlach 11 months ago

  • Assignee changed from Gitte Barlach to Rasmus Høymann Laursen
  • Priority changed from Normal to Urgent

Hej Rasmus

Kan I kigge på denne hurtigst muligt?

En ting er at listerne ligger dobbelt på bibliotekets produktionssite. Det kan jo skyldes at det netop endnu ikke er opdateret til den release, der pt er i test.
Men det, der bekymrer er, at en given bruger, der tidligere ikke har haft problemer med sin liste nu pludselig ikke kan tilgå den jvf.:"På dev-siden er der kun én huskeliste, og den er tom"
Hvor er data blevet af?
Konfigurationerne af openlist provider på hhv prod.sitet og dev.sitet er ens iflg. Karsten.

 

#8 Updated by Gitte Barlach 11 months ago

  • Assignee changed from Rasmus Høymann Laursen to Karsten Andersen

Hej Karsten

ift. den tomme huskeliste: kan du finde en eller flere meget særprægede titler og sætte dem på den huskeliste, der er tom?
Gerne så specielle som muligt, altså en eller to titler som andre brugere med stor sansynlighed ikke har på deres lister?

fx 
https://www.fredensborgbibliotekerne.dk/ting/object/870970-basis%3A01420259
https://www.fredensborgbibliotekerne.dk/ting/object/870971-avis%3A83424478
https://www.fredensborgbibliotekerne.dk/ting/object/150071-ebog%3A555
https://www.fredensborgbibliotekerne.dk/ting/object/150010-master%3A22122677

Formålet med øvelsen er at give B14 noget de kan identificere din liste og din bruger ud fra, når de kigger i OpenList databasen, 

#9 Updated by Karsten Andersen 11 months ago

Gitte Barlach skrev:

Hej Karsten

ift. den tomme huskeliste: kan du finde en eller flere meget særprægede titler og sætte dem på den huskeliste, der er tom?
Gerne så specielle som muligt, altså en eller to titler som andre brugere med stor sansynlighed ikke har på deres lister?

fx 
https://www.fredensborgbibliotekerne.dk/ting/object/870970-basis%3A01420259
https://www.fredensborgbibliotekerne.dk/ting/object/870971-avis%3A83424478
https://www.fredensborgbibliotekerne.dk/ting/object/150071-ebog%3A555
https://www.fredensborgbibliotekerne.dk/ting/object/150010-master%3A22122677

Formålet med øvelsen er at give B14 noget de kan identificere din liste og din bruger ud fra, når de kigger i OpenList databasen, 

Hej Gitte,

Det haver jeg nu gjort... Alle fire er tilføjet, vel at mærke på vores produktionssite. Ikke dev.

#10 Updated by Martin Dahl 11 months ago

Tak for input.

Jeg har fundet de lister der relaterer til denne huskeliste, men jeg ser ikke din tidligere huskeliste her.

Kan du sende mig fire materiale IDer fra den tidligere huskeliste så vi kan fremsøge den også?

 

#11 Updated by Karsten Andersen 11 months ago

Martin Dahl skrev:

Tak for input.

Jeg har fundet de lister der relaterer til denne huskeliste, men jeg ser ikke din tidligere huskeliste her.

Kan du sende mig fire materiale IDer fra den tidligere huskeliste så vi kan fremsøge den også?

 

Prøv med disse:

27928528

51044746

51392795

52407664

Hvis det kan hjælpe at få mit login, fortæller jeg det gerne i en privat besked. Der er ingen statshemmeligheder.

#12 Updated by Gitte Barlach 11 months ago

  • Assignee changed from Karsten Andersen to Martin Dahl

#13 Updated by Martin Dahl 11 months ago

  • Assignee changed from Martin Dahl to Karsten Andersen

@Karsten, tak for data.

Din huske liste, den har ca 30 elementer så vidt jeg kan se, herunder også 21264369 og 27269125?

Men jeg kan ikke gennemskue hvorfor du pludselig har to forskellige brugere.

Er den pågældende test udført på en frisk klon af jeres produktionsmiljø således at brugere også er overført?

#14 Updated by Gitte Barlach 11 months ago

  • Assignee changed from Karsten Andersen to Christian Vandel

Jeg har spurgt DBC om de kan oplyse om brugerdb også er overført. 

#15 Updated by Gitte Barlach 11 months ago

  • Assignee changed from Christian Vandel to Martin Dahl

Fra DBC/Christian Vandel:

-dev sites starter deres liv som rene kloner af prod-sitets database og files-katalog, udrullet på ny eller eksisterende kodebase – i dette tilfælde en ny kodebase. Så ja, brugerne er med.

#16 Updated by Karsten Andersen 11 months ago

Martin Dahl skrev:

@Karsten, tak for data.

Din huske liste, den har ca 30 elementer så vidt jeg kan se, herunder også 21264369 og 27269125?

Men jeg kan ikke gennemskue hvorfor du pludselig har to forskellige brugere.

Er den pågældende test udført på en frisk klon af jeres produktionsmiljø således at brugere også er overført?

Det var en helt frisk klon. Huskelister var det første jeg tjekkede efter at jeg havde oprettet mit dev-site. Da jeg her fandt en tom huskeliste, gik jeg tilbage til produktionssitet og fandt, at der her pludselig var to huskelister, hvoraf den ene var tom. Hvornår den tomme huskeliste er opstået og om det er sket i forbindelse med kloningen til dev, kan jeg ikke sige med sikkerhed, men det er for meget nylig, da jeg ellers ville have bemærket det.

Mht. to forskellige brugere, så har jeg min normale lånerprofil med et gammeldags lånerkortnummer, som jeg bruger til daglig. Derudover har jeg en bruger som er oprettet med NemID som test i forbindelse med den forrige opgrading af DDB CMS, og som jeg ikke har logget ind med siden dengang. Den ekstra huskeliste er opstået langt senere. De to forskellige brugere har dog helt samme navn, så kan det være problemet?

Det kunne være jeg skulle slette denne ekstra låner og prøve at lave et nyt dev-site og se, hvad der sker?

#17 Updated by Martin Dahl 11 months ago

  • Assignee changed from Martin Dahl to Karsten Andersen

@Karsten, tak for feedback.

Lad os endelig ikke blande andre use cases ind i denne tråd, hvor vi stadig forsøger at udrede hvorfor dine lister på en eksisterende bruger er forskellige i produktion og staging.

Kan jeg bede dig i begge af de to versioner at klikke ind på redigering af din bruger,  og sende mig hvad der står i browserens URL. Så kan afgøre om det er samme user id der er tale om, for det lader til at det muligvis ikke er tilfældet.

#18 Updated by Karsten Andersen 11 months ago

Martin Dahl skrev:

@Karsten, tak for feedback.

Lad os endelig ikke blande andre use cases ind i denne tråd, hvor vi stadig forsøger at udrede hvorfor dine lister på en eksisterende bruger er forskellige i produktion og staging.

Kan jeg bede dig i begge af de to versioner at klikke ind på redigering af din bruger,  og sende mig hvad der står i browserens URL. Så kan afgøre om det er samme user id der er tale om, for det lader til at det muligvis ikke er tilfældet.

Der står da bare hhv. https://www.fredensborgbibliotekerne.dk/user/me/edit og https://fredensborg-dev.ddbcms.dk/user/me/edit

... eller ser jeg et forkert sted?

#19 Updated by Martin Dahl 11 months ago

Arh, ok. Klart. Kan du istedet inspecte siden på begge versioner, og sende hvad der står af classes i Body objektet. Ligesom vedhæftet.

'

/m

 

#20 Updated by Karsten Andersen 11 months ago

Martin Dahl skrev:

Arh, ok. Klart. Kan du istedet inspecte siden på begge versioner, og sende hvad der står af classes i Body objektet. Ligesom vedhæftet.

'

/m

 

Her er fra produktionsside:

<body class="html not-front logged-in no-sidebars page-user page-user- page-user-60 page-user-edit i18n-da has-dynamic-background page-panels">

Og fra dev:

<body class="html not-front logged-in no-sidebars page-user page-user- page-user-60 page-user-edit i18n-da has-dynamic-background page-panels">

De er med andre ord ens...

#21 Updated by Martin Dahl 11 months ago

  • Assignee changed from Karsten Andersen to Gitte Barlach

Tak for inputs fra alle,.

Analysen af kryds-kontaminerede brugere (Case B) viser følgende:

1514 lister svarende til 250-500 brugere som aktivt har benyttet openlist inden for det sidste år, kan være påvirket af kryds-kontaminering skyldet inkorrekt staging og/eller andre følgefejl.

Af disse brugere forventes en del at være testbrugere som er oprettet i forbindelse med diverse tests under skift af OpenList URL

Og af de resterende har ikke alle en videre aktivitet i listerne.

Det forventes således at kun få bliver berørt når nyeste RC idriftsættes.

De berørte brugere der måtte savne elementer kan vi servicere med at bede dem gå til en særlig URL når de er logget ind i systemet.

Dette rescue modul kan skrives i een samlet fil så den er let at merge.

Modulet kan matche elementer fra lokale lister og indsætte ikke-dubletter i de korrekte online OpenList lister.

#22 Updated by Gitte Barlach 11 months ago

  • Subject changed from Huskelister i 29.1 to Huskelister i 29.1 (UDVIKLING AF RESCUE-LIST MODUL)
  • Status changed from Needs analysis to Ready for development
  • Assignee changed from Gitte Barlach to Rasmus Høymann Laursen

Hej Rasmus
I må meget gerne gå igang med denne.

#23 Updated by Rasmus Høymann Laursen 11 months ago

  • Assignee changed from Rasmus Høymann Laursen to Philip Birk-Jensen

#24 Updated by Rasmus Høymann Laursen 11 months ago

  • Assignee changed from Philip Birk-Jensen to Martin Dahl

#25 Updated by Christel Krabbenhøft 11 months ago

  • Target version changed from Release 29-1 - Lister (7.x-4.4.0) to Release 29-2 - Bugfixes (B14)

#26 Updated by Martin Dahl 10 months ago

  • Assignee changed from Martin Dahl to Gitte Barlach

Vi har sendt en zip for at kunne teste sagerne på Karstens staging miljø inden vi sender PR. Er den blevet testet?

#27 Updated by Rolf Madsen 10 months ago

Ding_listresque modul vedhæftet.

#28 Updated by Rolf Madsen 10 months ago

  • Status changed from Ready for development to Technical test
  • Assignee changed from Gitte Barlach to Karsten Andersen
  • Target version changed from Release 29-2 - Bugfixes (B14) to Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0)

@Karsten, når du kommer tilbage fra ferie så tag lige fat i os når du er klar til at teste med den bruger der mangler sin liste.

#29 Updated by Gitte Barlach 9 months ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Karsten Andersen to Rasmus Høymann Laursen

Hej Martin 

Jeg har bedt Jesper K. om at udføre code review af Ding_listresque modulet. Dernæst er tanken at Aarhus afprøver det ift. en bestemt bruger, der har henvendt sig, da han har mistet sin huskeliste efter seneste opgradering.

I må meget gerne lave et PR til core i denne sag. Så vi har mulighed for at køre en normal review proces via Git.  (@Jesper: dermed ikke være sagt at modulet skal merges til core lige nu)

#30 Updated by Rasmus Høymann Laursen 9 months ago

  • Assignee changed from Rasmus Høymann Laursen to Martin Dahl

#31 Updated by Gitte Barlach 9 months ago

vi har lagt modulet på stg.aakb.dk; det er lidt råt i kanten set ift at en slutbruger skal anvende funktionen. 

så opsummeret:

a) vi vil meget gerne have et PR til Core, så Jesper har et sted at komme af med kommentarer efter code review
b) vi vil meget gerne have dokumentation (som minimum inline i koden) på hvad modulet gør, samt en forklaring vi kan give til hhv webredaktioner og slutbrugere
c) UX ift. slutbrugeren: som det er nu kommer brugeren ind på en side, hvor vi viser et array (se skærmdump samt nedenfor); siden skal gentænkes så den er egnet til en slutbruger
d) vi har godt nok en konkret bruger her i Aarhus der mangler sin huskeliste. Men vi tør ikke rigtigt give ham url´en og bede ham klikke på "proceed" da vi er meget usikre på, hvad der så sker, samt om noget kan reddes/genskabes, hvis det går galt. 

 

Array
(
    [0] => stdClass Object
        (
            [id] => 902713
            [title] => Materialer jeg har bedømt
            [type] => ratings
        )

    [1] => stdClass Object

 

#32 Updated by Martin Dahl 9 months ago

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

#33 Updated by Gitte Barlach 9 months ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#34 Updated by Jesper Kristensen 9 months ago

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

Review med kommentar til små ændringer.

#35 Updated by Stefan Søndervang 9 months ago

Hej Martin. Vi du ikke kigge på denne?

#36 Updated by Martin Dahl 9 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Martin Dahl to Stefan Søndervang

#37 Updated by Stefan Søndervang 9 months ago

  • Assignee changed from Stefan Søndervang to Jesper Kristensen

#38 Updated by Jesper Kristensen 9 months ago

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

Være venlig at gennemgå PR'er og svare på alle stille spørgsmål

#39 Updated by Gitte Barlach 9 months ago

Hej Martin 

Er der mon nyt i denne sag?

#40 Updated by Gitte Barlach 9 months ago

hej Martin 
Endnu en bruger har mistet sin huskeliste. Kan I tage Jer af de sidste ting her i sagen jvf. kommentar 38, så vi kan få modulet afprøvet sammen med pågældende bruger?

#41 Updated by Martin Dahl 9 months ago

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

#42 Updated by Jesper Kristensen 9 months ago

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

Reviewed og afventer release

#43 Updated by Gitte Barlach 9 months ago

  • Status changed from Technical test to Reviewed

#44 Updated by Kasper Garnæs 8 months ago

  • Status changed from Reviewed to Technical test

Merged.

#45 Updated by Christel Krabbenhøft 6 months ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Rolf Madsen
  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0) to Release 31 - bugfixes

Albertslund Bibliotek har testet på en bruger med følgende resultat: "Brugeren fik ikke sin gamle liste tilbage, men mistede til gengæld sin nye liste."

Rolf, vil du tilføje den listen over sager, der skal prioriteres ifm. OpenList.

 

#46 Updated by Christel Krabbenhøft 3 months ago

  • Status changed from Reviewed - Needs info/rework to Closed
  • Assignee deleted (Rolf Madsen)

Da vi desværre ikke har haft nogle successer med med rescue-modulet, lukker vi sagen for nu efter antagelse om, at de ramte brugere har fundet alternative løsninger. 

#47 Updated by Christel Krabbenhøft about 2 months ago

  • Target version deleted (Release 31 - bugfixes)

Also available in: Atom PDF