Project

General

Profile

Bug #1732

Manglende statistik på ordsøgning i 2016 i Webtrends

Added by Niels Frandsen over 3 years ago. Updated about 3 years ago.

Status:
Resolved (tag version)
Priority:
Normal
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Administration - Systemkonfiguration

Description

Fra 2016 har vi ikke fået leveret data på "On-Site Search Phrases".
Det ønsker vi gerne hurtigst muligt løst - og om muligt inklusiv historikken fra 1.1.2016.
Vi har haft forelagt sagen hos DBC, som giver følgende svar:

Fra vores side af (DBC) er den eneste opgave mht. statistik at aktivere webtrends-modulet og indsætte kodeord, hvilket er gjort - så det er nok en sag for DDB sagssystemet eller KPI-index.

drush @ballerup.ddbcms.dk vget webtrends
webtrends_dcs_id: "dcs222zaakgxvq9qp08uhcpap_7y8h"
webtrends_domain: "ballerup.dk"
webtrends_onsitedoms: "ballerup.dk"

History

#1 Updated by Steen Larsen over 3 years ago

Kunne det være oversættelsen den er gal med?

Original:
Search result (!count !string)
Oversat:
Søgeresultat (!count !string)

Ballerup (eksempel fra søgning):
Søgeresultat (165 hits)

Den korrekte burde være
Søgeresultat (165 hits)

En der har adgang til oversættelse ( admin/config/regional/translate/translate) kan søge på "data-js-count"
Findes den - så check oversættelsen
Findes den ikke - så skyldes fejlen noget andet.

#2 Updated by Simon Holt over 3 years ago

Det er højst sandsynligt det. Vi har haft en anden sag, hvor det var den oversættelse der drillede og har hjulpet flere andre biblioteker med samme profil.

Det er også en noget skrøbelig løsning og har et forslag til en løsning, hvor det bliver håndteret på serveren i stedet, der ikke er afhængig af et en oversættelse.

Vi får lige lavet et PR snarest muligt.

#3 Updated by Simon Holt over 3 years ago

Det er højst sandsynligt det. Vi har haft en anden sag, hvor det var den oversættelse der drillede og har hjulpet flere andre biblioteker med samme problem.

Det er også en noget skrøbelig løsning og har et forslag til en løsning, hvor det bliver håndteret på serveren i stedet, der ikke er afhængig af en oversættelse.

Jeg får lige lavet et PR snarest muligt.

#4 Updated by Michael Anker over 3 years ago

For hulan da også
der stod
Søgeresultat (!count hits)

som nu er rettet til
Søgeresultat (!count !string)

Tusind tak for hjælpen. Spændt på at se om vi har dejlig statistik i morgen :-)

Simon. Er PR et Pull Request for så ville det da være dejligt at få fikset den fejl, så den ikke optræder for andre der ikke kender forbindelsen med oversættelse og så webstatistikken.

#5 Updated by Rolf Madsen over 3 years ago

  • Status changed from New to Ready for development
  • Assignee set to Simon Holt
  • Target version set to DDB CMS 2016 2. opgradering

Tak for hurtig og som altid kyndig hjælp Simon! :-)

Jeg tillader mig at assigne dette issue til dig.

#6 Updated by Michael Anker over 3 years ago

Og det virker, så nu er det problem fikset for os, men en central løsning ville nok være at foretrække så andre ikke kommer til at oversætte lidt for meget :-)

#7 Updated by Simon Holt over 3 years ago

Jeg er i gang med en løsning der sætter Webtrends onsite-search metatags op på serveren i stedet for at håndtere det i Javascript på klienten:

https://help.webtrends.com/en/Analytics9Admin/onsite_search_parameters.html

Så skal Webtrends Javascript nok selv sørge for at oss-parametrene bliver tracket korrekt.

Jeg havde egentlig en løsning klar, som vi har brugt på vejlebib, men det viser sig at den ikke kan bruges mere når der søges i indhold på hjemmesiden, da vi er gået over til Search API. Så skal lige have fundet en løsning på det.

#8 Updated by Steen Larsen over 3 years ago

Ok, den eksisterende løsning blev vist indført pga søgning tidligere blev udført via ajax?
Men det er ligegyldigt - hvis du kan få en løsning med metatags til at fungere, hvor der også tages hensyn til f.eks. caching er det jo fint nok.

Se evt. #1731 om hvordan facetter skal sendes til KPI - der er dog ikke noget løsningsforslag i den sag endnu.

#9 Updated by Simon Holt over 3 years ago

https://github.com/ding2/ding2/pull/252

Sætter OSS metatags op på serveren ved at implementere relevante hooks i stedet for den gamle metode, hvor det blev håndteret i Javascript på klienten.

Har også tilføjet et prefix, så man kan differentiere i mellem om brugeren søger i brønden eller på hjemmesiden:

Søgnnig i brønd på "harry potter": "ting_search:harry potter"
Søgnnig på hjemmeside på "biblioteker": "node:biblioteker"

Angående håndtering af facetter; jeg laver et PR der bygger på dette med et løsningsforslag i #1731

#10 Updated by Rolf Madsen about 3 years ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Simon Holt to Jesper Kristensen

#11 Updated by Jesper Kristensen about 3 years ago

  • Status changed from Needs code review to Need more info
  • Assignee changed from Jesper Kristensen to Simon Holt

Så de to PRs 252 og 253 har de samme ændringer?

Hvis det er rigtig, vil du så ikke lukke den ene af dem?

#12 Updated by Simon Holt about 3 years ago

Hej Jesper

Ville gerne holde tingene adskilt da denne sag handler om at søgestatistikken kan ødelægges af en forkert oversættelse og den anden #1731 omhandler tracking af facetter i søgestatistikken.

Så PR 253 bygger ovenpå og indeholder alle commits fra 252, men også et ekstra commit, der tilføjer det med facetterne.

Beklager hvis det ikke var den mest hensigtsmæssige måde at gøre det på.

#13 Updated by Rolf Madsen about 3 years ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Simon Holt to Jesper Kristensen

#14 Updated by Simon Holt about 3 years ago

Inden i merger denne her skylder jeg lige at forklare:

Den her sag handlede egentlig kun om, at søgestatistikken kan blive ødelagt af oversættelse. Men kom til at snige lidt ekstra funktionalitet ind i PR: Jeg mente det ville være relevant at se, om brugerne søgte på hjemmesiden eller i brønden direkte i søgestatistikken. Så derfor har jeg tilføjet det som et prefix til søgetermen. Altså "ting_search:[søge term]" og "node: [søge term]".

Efter diskussionen i #1731 er jeg kommet i tvivl om det er det rigtige at gøre? Og hvis det er, burde vi måske skrive noget mere sigende end ting_search og node (kunne måske være konfigurerbar). Under alle omstændigheder skal I i hver fald lige tage stilling til dette før I approver og merger.

#15 Updated by Rolf Madsen about 3 years ago

Opsummering

Løsningen på #1732 ligger i https://github.com/ding2/ding2/pull/252 og skal med i DDB CMS 2016 2. opgradering.

https://github.com/ding2/ding2/pull/253 skal lukkes, og Simon skal lave et nyt PR for #1731, som kun indeholder rettelserne til statistik for facetterne. #1731 afklares efterfølgende og tages med i en senere release.

Hvis det ikke er korrekt forstået må du lige sige til Simon! :-)

#16 Updated by Rolf Madsen about 3 years ago

  • Subject changed from Manglende statistik på ordsøgning i 2016 to Manglende statistik på ordsøgning i 2016 i Webtrends

#17 Updated by Jørgen Nielsen about 3 years ago

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

reviewet og godkendt

#18 Updated by Simon Holt about 3 years ago

@Rolf

Vi kan godt bare lade løsningen til #1731 (PR 253) være, hvis altså ikke den forstyrrer?

Når løsningen her bliver merget (PR 252) vil PR 253 kun afvige fra master, med det ene commit der tilføjer facetterne.

#19 Updated by Rolf Madsen about 3 years ago

@Simon, det er ikke noget jeg har forudsætningerne for at afgøre.
Hvis Core team ikke har nogen indvendinger så har jeg heller ikke. :-)

#20 Updated by Jesper Kristensen about 3 years ago

  • Status changed from Reviewed to Technical test

#21 Updated by Rolf Madsen about 3 years ago

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

Når jeg tjekker view-source:https://upgrade-alma.ddbcms.dk/search/ting/harry%20potter ser jeg:

Søgeresultater (522 Resultater)

Hermed testet og godkendt.

Also available in: Atom PDF