UKMNorge / UKMrapporter

0 stars 0 forks source link

excel må støtte visning av nummer #68

Closed mariusmandal closed 3 years ago

mariusmandal commented 4 years ago

Flyttet til 2021 for å sjekke om noen etterspør det

mariusmandal commented 4 years ago

Jeg mener dette bør på plass, det har bare vært litt tricky, og det er derfor det er utsatt. Skal man jobbe med denne informasjonen i excel, trenger man mest sannsynlig hvilket nummer i rekken innslaget er.

"Use-case": teknisk prøve-rapporten eksportert til excel. Må vise hvilket nummer innslaget er.

mariusmandal commented 3 years ago

Tatt inn i feature-sys

mariusmandal commented 3 years ago

Tatt inn i Sprint#3

mariusmandal commented 3 years ago

Gjenskape problemet

Setup

Feilen

I HTML-utgaven (den du ser på nå), ser du at innslagene har nummer foran seg, for å vise hvilken rekkefølge de har i hendelsen.

mariusmandal commented 3 years ago

Løsningsforslag

Beregnet tid: 4t

Legg til kolonnen "rekkefølge" i alle 3 ark i excel-filen

Disse to kunne vært tilnærming for å hente ut nummeret, da vi har to støtte-funksjoner som kan brukes:

  1. UKMNorge\Arrangement\Program\Hendelse::getNummer( Innslag $innslag ) Denne henter ut rekkefølgen ved å søke gjennom samlingen. Det er en teoretisk treg tilnærming, men jeg tror ikke det skal være et problem i produksjon, da vi sjeldent itererer gjennom mer enn 30 innslag

  2. UKMNorge\Innslag\Innslag::setAttr( $key, $value ) Du kan sette en attributt på et innslag-objekt. Dette følger objektet gjennom hele exec, men lagres ikke i minne eller database noe sted, så attributtene er kun tilgjengelig i ditt script

Optimal løsning Jeg har nå oppdatert UKMNorge/UKMapi, slik at dette er implementert nærmest mulig databasen. (internt bruker denne forslag 2 fra ovenfor)