Project

General

Profile

Bug #4407

Modulet subsearch_common ændrer agency+brøndprofil

Added by Tue Gaston 6 months ago. Updated 7 days ago.

Status:
Need more info
Priority:
Normal
Estimated time:
URL med eksempel:
Kategorier:
Søgning - Søgeresultat - hjemmeside

Description

Det er subsearch_common-modulet, der ændrer konfigurationen under /admin/config/opensearch/settings, således at bibliotekskoden og søgeprofilen pludselig er identisk med det, man har skrevet ind under sekundært agency og sekundær søgeprofil.

Det sker fordi modulet er sat til *midlertidigt* at udskifte brøndprofil og agency, mens den sekundære søgning kører. Men inden den ændrer dem, gemmer den de oprindelige indstillinger, og re-etablerer dem efter endt sekundær-søgning.

 

Altså:
1) Modulet lagrer oprindelig agency+brøndprofil

2) Modulet ændrer agency+brøndprofil

3) Modulet udfører secondary search

4) Modulet ændrer agency+brøndprofil tilbage til den oprindelige, som den lagrede I trin 1.

 

Problemet opstår, hvis først én bruger laver en søgning, der udløser secondary search, og så en anden bruger gør det samme, inden den første secondary search er afsluttet. Det er et vindue på få millisekunder, så det sker ikke tit – men når det sker, så lagrer modulet ikke den primære agency+brøndprofil, men den sekundære.

 

Så sker der nemlig flg.

1) Modulet lagrer oprindelig agency+brøndprofil pga søgning udløst af Bruger 1

2) Modulet ændrer agency+brøndprofil

3) Modulet aktiveres af Bruger 2, og lagrer agency+brøndprofil, dvs. IKKE den oprindelige, men den sekundære, som den jo nu midlertidigt er ændret til.

4) Modulet udfører secondary search for Bruger 1

5) Modulet ændrer agency+brøndprofil tilbage til den oprindelige, som den lagrede I trin 1.

6) Modulet udfører secondary search for Bruger 2

7) Modulet ændrer agency+brøndprofil tilbage til den SEKUNDÆRE, som den lagrede I trin 3.


Related issues

Related to DDB CMS - Feature #2241: Ændringer i Core -> introduktion af hooks som gør det muligt at løse #2173, #2174, #2240 og #2239Resolved (tag version)
Related to DDB CMS - Enhancement #2240: SAMLESAG - Forslag fra bibliotek.dk - subsearch_secondaryTechnical test
Related to DDB CMS - Enhancement #2174: SAMLESAG - Oversatte forslag - subsearch_translateTechnical test
Related to DDB CMS - Enhancement #2173: SAMLESAG - Mente-du-modul - subsearch_suggestionsTechnical test

History

#1 Updated by Tue Gaston 6 months ago

  • Related to Feature #2241: Ændringer i Core -> introduktion af hooks som gør det muligt at løse #2173, #2174, #2240 og #2239 added

#2 Updated by Tue Gaston 6 months ago

  • Related to Bug #3600: Genintroducér hooks til brug for "Mente du .. " og "Oversatte forslag" added

#3 Updated by Tue Gaston 6 months ago

  • Related to Enhancement #2240: SAMLESAG - Forslag fra bibliotek.dk - subsearch_secondary added

#4 Updated by Tue Gaston 5 months ago

  • Target version changed from Release 32 - Ballerup projekter to Release 32 - Ballerup projekter (Inlead)

#5 Updated by Tue Gaston 5 months ago

  • Status changed from Ready for development to Technical test
  • Assignee changed from Martin Cording to Tue Gaston

#6 Updated by Tue Gaston 5 months ago

  • Status changed from Technical test to Ready for development
  • Assignee changed from Tue Gaston to Martin Cording

#7 Updated by Tue Gaston 5 months ago

  • Status changed from Ready for development to Technical test
  • Assignee changed from Martin Cording to Tue Gaston

#8 Updated by Tue Gaston 5 months ago

  • Status changed from Technical test to Ready for development
  • Assignee changed from Tue Gaston to Martin Cording

#9 Updated by Martin Cording 5 months ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Martin Cording to Tue Gaston

#10 Updated by Tue Gaston 2 months ago

  • Related to deleted (Bug #3600: Genintroducér hooks til brug for "Mente du .. " og "Oversatte forslag")

#11 Updated by Tue Gaston 2 months ago

  • Related to Enhancement #2174: SAMLESAG - Oversatte forslag - subsearch_translate added

#12 Updated by Tue Gaston 2 months ago

  • Related to Enhancement #2173: SAMLESAG - Mente-du-modul - subsearch_suggestions added

#13 Updated by Tue Gaston 2 months ago

  • Status changed from Need more info to Technical test

#14 Updated by Tue Gaston 7 days ago

Jeg har netop slettet en række kommentarer, der fejlagtigt var havnet under dette issue, idet de omhandlede en urelateret problematik.
Problemet med ændring af agency/brøndprofil er ifølge email fra Martin Cording til mig d. 8/7 2019 løst "omskrivning af secondary-modul (#4407) - er færdig".

Men hvordan tester jeg den?

#15 Updated by Tue Gaston 7 days ago

  • Status changed from Technical test to Need more info
  • Assignee changed from Tue Gaston to Martin Cording

Also available in: Atom PDF