fix: header "Next update" visar fel datum (första istället för andra tisdagen) #12

New Issue

2026-05-22T09:50:11+02:00

shcizo commented

2026-05-22 09:50:11 +02:00

Beskrivning

Header visar fel datum för "Next update". Den ska visa andra tisdagen i månaden (PDGA:s officiella publiceringscykel, se CLAUDE.md), men visar istället första tisdagen i nästa månad.

Exempel (idag 2026-05-22):

Korrekt nästa update: 2026-06-09 (andra tisdagen i juni — maj månads andra tisdag var 12 maj, redan passerad)
Header visar: Tue 2 Jun (första tisdagen i juni)

Felet är konsekvent ~7 dagar.

Teknisk orsak

Det finns två separata implementationer av "nästa update-datum":

Korrekt: getNextPDGAUpdateDate() i src/services/rating-calculator.js:40-71 — hanterar både "andra tisdagen i innevarande månad om vi är före" och "nästa månads andra tisdag om vi passerat". Används av rating-beräkningen.
Fel (duplikat): computeNextUpdate() i src/services/topbar-service.js:22-31 — tar alltid första tisdagen i nästa månad. Används av header.

Header anropar getTopbarLocals() (src/services/topbar-service.js:33-41) som returnerar nextUpdate till views/partials/topbar.ejs:24.

Förslag till lösning

Ta bort duplikatet computeNextUpdate() och låt topbar-service.js använda den centrala getNextPDGAUpdateDate() från rating-calculator.js. Behåll formateringen ("Tue 13 Jun") som en liten formatter i topbar-service.js.

Filer som påverkas:

src/services/topbar-service.js — importera getNextPDGAUpdateDate, ersätt computeNextUpdate med en formaterings-wrapper
src/services/rating-calculator.js — exportera getNextPDGAUpdateDate om inte redan exporterad

Scope

Endast datumlogiken i header. Ingen UI-/styling-ändring.
Ingen ändring av getNextPDGAUpdateDate() själv (används redan korrekt i rating-beräkningen).

## Beskrivning Header visar fel datum för "Next update". Den ska visa **andra tisdagen i månaden** (PDGA:s officiella publiceringscykel, se `CLAUDE.md`), men visar istället **första tisdagen i nästa månad**. **Exempel (idag 2026-05-22):** - Korrekt nästa update: **2026-06-09** (andra tisdagen i juni — maj månads andra tisdag var 12 maj, redan passerad) - Header visar: `Tue 2 Jun` (första tisdagen i juni) Felet är konsekvent ~7 dagar. ## Teknisk orsak Det finns två separata implementationer av "nästa update-datum": 1. **Korrekt:** `getNextPDGAUpdateDate()` i `src/services/rating-calculator.js:40-71` — hanterar både "andra tisdagen i innevarande månad om vi är före" och "nästa månads andra tisdag om vi passerat". Används av rating-beräkningen. 2. **Fel (duplikat):** `computeNextUpdate()` i `src/services/topbar-service.js:22-31` — tar alltid första tisdagen i *nästa* månad. Används av header. Header anropar `getTopbarLocals()` (`src/services/topbar-service.js:33-41`) som returnerar `nextUpdate` till `views/partials/topbar.ejs:24`. ## Förslag till lösning Ta bort duplikatet `computeNextUpdate()` och låt `topbar-service.js` använda den centrala `getNextPDGAUpdateDate()` från `rating-calculator.js`. Behåll formateringen (`"Tue 13 Jun"`) som en liten formatter i `topbar-service.js`. **Filer som påverkas:** - `src/services/topbar-service.js` — importera `getNextPDGAUpdateDate`, ersätt `computeNextUpdate` med en formaterings-wrapper - `src/services/rating-calculator.js` — exportera `getNextPDGAUpdateDate` om inte redan exporterad ## Scope - Endast datumlogiken i header. Ingen UI-/styling-ändring. - Ingen ändring av `getNextPDGAUpdateDate()` själv (används redan korrekt i rating-beräkningen).

shcizo added the bug label 2026-05-22 09:50:11 +02:00

shcizo referenced this issue from a commit

2026-05-22 09:57:45 +02:00

fix: header "Next update" uses second Tuesday (closes #12)

shcizo referenced a pull request that will close this issue

2026-05-22 09:58:01 +02:00

fix: header "Next update" uses second Tuesday (closes #12) #13

shcizo closed this issue

2026-05-22 10:00:42 +02:00

shcizo referenced this issue from a commit

2026-05-22 10:00:42 +02:00

Merge pull request 'fix: header "Next update" uses second Tuesday (closes #12)' (#13) from fix/header-next-update-date into main