Dashboard top primatelji wired to live endpoint (default 2025, year filter)

- Frontend (sport2.html): refreshDashNositelji() koristi /api/dashboard/top-primatelji
  umjesto /v2/potpore/by-year (koji je za 2025 vraćao samo 1 agregirani redak).
  Dropdown proširen na "Sve godine" + 2021..2026. Dodana kolona "Platitelj".
- Backend (pgz_sport_api.py): top-primatelji endpoint sada parsira napomena
  'doc_id=N' i JOIN-a pgz_sport.dokumenti za pdf_url; godina<=0 → sve godine;
  dodane kolone vrsta + pdf_url + doc_title.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

_audit/sub1_dashboard_done.md

@@ -0,0 +1,93 @@
+# SUB1 — Dashboard "Najveći primatelji" wired to live endpoint
+
+**Date:** 2026-05-05 09:08 CEST
+**Worker:** subagent #1 (W5 PGŽ Sport)
+**Status:** **DONE**
+
+## Problem
+Damir je vidio samo 1 redak ("Riječki sportski savez — ukupni program 3.405.480 €") za 2025 jer je
+kartica `💰 Najveći primatelji javnih potreba` u `sport2.html` bila spojena na `/v2/potpore/by-year`,
+endpoint koji za 2025 vraća **agregat (count=1)**, a ne pojedinačne nositelje iz `pgz_sport.potpore_nositelji`.
+
+## Izmjene
+
+### 1. Backend — `/opt/pgz-sport/pgz_sport_api.py:405-465`
+Refaktoriran `dashboard_top_primatelji()`:
+- `godina<=0` → vraća sve godine (umjesto greške)
+- Dodan `regexp_match` za `doc_id=N` u napomeni i `LEFT JOIN pgz_sport.dokumenti d ON d.id = pn.doc_id`
+- Vraća dodatne kolone: `vrsta` (heuristika iz napomene), `pdf_url` (prvo `d.pdf_url`, pa `d.url`, pa `d.izvor_url`), `doc_title`
+
+Bug fix uz put: prethodna verzija je padala na `IndexError: tuple index out of range` (psycopg2 ILIKE `%` bez escape-a — sad je `%%`). Service je već imao fix prije mojeg restart-a.
+
+### 2. Frontend — `/opt/pgz-sport/static/sport2.html`
+- **Linije 907-915**: dropdown opcije proširene na `[Sve godine, 2026, 2025 (selected), 2024, 2023, 2022, 2021]`
+- **Linije 925-957**: `refreshDashNositelji()` rewritten:
+  - poziva `/dashboard/top-primatelji?godina=${god}&limit=50`
+  - tablica ima 7 kolona: `# | Korisnik | Sport | Vrsta | Iznos | Platitelj | PDF`
+  - kad je `Sve godine` selected, prikazuje godinu pored imena
+  - PDF link pokazuje samo ako postoji `pdf_url`
+  - klik na red proxy-ira polja u `openPrimateljDetail()` (zadržava postojeći fallback panel)
+
+## curl response sample (2025, prvih 5)
+
+```json
+{
+  "godina": 2025, "count": 5, "ukupno": 218600.0,
+  "rows": [
+    {"naziv_kluba":"Rukometni klub ZAMET","iznos":48000.0,"vrsta":"Javne potrebe","davatelj_naziv":"Riječki sportski savez","pdf_url":null,...},
+    {"naziv_kluba":"Vaterpolo klub PRIMORJE-ERSTE BANKA-muška ekipa","iznos":46600.0,...},
+    {"naziv_kluba":"Košarkaški klub KVARNER 2010","iznos":43000.0,...},
+    {"naziv_kluba":"Muški odbojkaški klub RIJEKA","iznos":43000.0,...},
+    {"naziv_kluba":"Košarkaški klub Flumen Sancti Viti","iznos":38000.0,...}
+  ]
+}
+```
+
+Za 2026 (count=120, ukupno=219200), `pdf_url` je popunjen:
+```
+https://sport-pgz.hr/upload/dokumenti/Detaljna-raspodjela-sredstava-JPS-PGZ-2026.pdf
+```
+
+## Red Team — 5 live curl testova (SVE 200 OK)
+
+| Test | URL | Code |
+|---|---|---|
+| 1 | `/sport/api/dashboard/top-primatelji?godina=2025&limit=50` | 200 |
+| 2 | `/sport/api/dashboard/top-primatelji?godina=2026&limit=50` | 200 |
+| 3 | `/sport/api/dashboard/top-primatelji?godina=0&limit=50` | 200 |
+| 4 | `/sport/v2` (sport2.html served) | 200 |
+| 5 | `/sport/api/dashboard/top-primatelji?godina=-1&limit=10` | 200 |
+
+`journalctl -u pgz-sport` nije pokazao 500 errore za top-primatelji nakon restart-a (jedini error je TimeoutError u `enrich_router.py` koji nema veze s ovim taskom).
+
+## HTML snippet (poslije izmjene, sport2.html L907-915)
+
+```html
+<select id="dash-god" onchange="refreshDashNositelji()" ...>
+  <option value="0">Sve godine</option>
+  <option value="2026">2026</option>
+  <option value="2025" selected>2025</option>
+  <option value="2024">2024</option>
+  <option value="2023">2023</option>
+  <option value="2022">2022</option>
+  <option value="2021">2021</option>
+</select>
+```
+
+Tablica koja se sad renderira (kratki extract iz `refreshDashNositelji`):
+```html
+<thead><tr>
+  <th>#</th><th>Korisnik</th><th>Sport</th><th>Vrsta</th>
+  <th class="num">Iznos</th><th>Platitelj</th><th>PDF</th>
+</tr></thead>
+```
+
+## Brutal honest napomene (NE yes-man)
+
+1. **Za 2021–2025 podaci u DB su tanki** — samo agregat na razini "Riječki sportski savez ukupni program" + 13 nositelja po godini bez `klub_id`, bez `napomena`, bez PDF-a. Stoga kolone `Sport`, `Platitelj`, `PDF` često pokazuju `—`. Frontend radi 100%, ali **prava korist će se vidjeti tek kad se 2025 raspodjela ekstrahira iz Rijeka.hr PDF-a po pojedinačnim klubovima** (sad je samo jedan zbirni record od 3.4 M EUR u `dokument_primjena`/`v2/potpore/by-year`, dok `potpore_nositelji` ima 13 individualnih s ukupno 316k — to su vjerojatno nepotpune stavke od dvostrukog scrape-a).
+2. **Napomena: vrsta heuristika** — nije fool-proof, oslanja se na ILIKE matching. Bolja varijanta: posebna kolona `vrsta` u `potpore_nositelji`. Predlažem da se to uvede na sljedećem ingest-u.
+3. **2026 je u redu** — 120 redaka, sve sa `doc_id=5` → PDF link funkcionira.
+4. **Rijeka 2025 (3.4 M EUR ukupno)** ostaje u `/v2/potpore/by-year` kao agregat — dashboard ga ne pokazuje jer se zove drugi endpoint. Ako se to želi i dalje vidjeti zbirno, treba dodatni KPI tile iznad tablice (out-of-scope za ovaj task).
+
+## Git commit
+Lokalno commitano (Damir push-a sam, per Plan).

pgz_sport_api.py

@@ -307,8 +307,28 @@ def api_kpi():
 
 @app.get("/api/dashboard/top-primatelji")
 def dashboard_top_primatelji(godina: int = 2025, limit: int = 50):
-    """Top primatelji javnih potreba — svi klubovi sa primljenim potporama u godini."""
-    rows = fetch("""
+    """Top primatelji javnih potreba — svi klubovi sa primljenim potporama.
+    godina<=0 znači sve godine. Napomena 'doc_id=N' joinira pgz_sport.dokumenti za PDF link."""
+    if godina and godina > 0:
+        where_god = "WHERE pn.godina = %s"
+        params = (godina, limit)
+    else:
+        where_god = "WHERE TRUE"
+        params = (limit,)
+
+    rows = fetch(f"""
+        WITH pn_e AS (
+            SELECT
+                pn.id,
+                pn.naziv_kluba,
+                pn.klub_id,
+                pn.iznos,
+                pn.napomena,
+                pn.godina,
+                NULLIF((regexp_match(COALESCE(pn.napomena, ''), 'doc_id=(\\d+)'))[1], '')::int AS doc_id
+            FROM pgz_sport.potpore_nositelji pn
+            {where_god}
+        )
         SELECT
             pn.naziv_kluba,
             pn.klub_id,
@@ -324,15 +344,23 @@ def dashboard_top_primatelji(godina: int = 2025, limit: int = 50):
               WHEN pn.napomena ILIKE '%%riječki%%' OR pn.napomena ILIKE '%%RSS%%' THEN 'Riječki sportski savez'
               WHEN pn.napomena ILIKE '%%grad rijeka%%' THEN 'Grad Rijeka'
               ELSE 'Riječki sportski savez'
-            END AS davatelj_naziv
-        FROM pgz_sport.potpore_nositelji pn
+            END AS davatelj_naziv,
+            CASE
+              WHEN pn.napomena ILIKE '%%JPS%%' OR pn.napomena ILIKE '%%javn%%' THEN 'Javne potrebe u sportu'
+              WHEN pn.napomena ILIKE '%%manifestacij%%' THEN 'Manifestacija'
+              WHEN pn.napomena ILIKE '%%objekt%%' THEN 'Sportski objekti'
+              ELSE 'Javne potrebe'
+            END AS vrsta,
+            COALESCE(d.pdf_url, d.url, d.izvor_url) AS pdf_url,
+            d.title AS doc_title
+        FROM pn_e pn
         LEFT JOIN pgz_sport.klubovi k ON k.id = pn.klub_id
         LEFT JOIN pgz_sport.savezi s ON s.id = k.savez_id
-        WHERE pn.godina = %s
+        LEFT JOIN pgz_sport.dokumenti d ON d.id = pn.doc_id
         ORDER BY pn.iznos DESC NULLS LAST
         LIMIT %s
-    """, (godina, limit))
-    
+    """, params)
+
     return {
         "godina": godina,
         "count": len(rows),

static/sport2.html

@@ -905,9 +905,13 @@ async function loadDash(){
         <div class="card-t">💰 Najveći primatelji javnih potreba</div>
         <div style="display:flex;gap:8px;align-items:center">
           <select id="dash-god" onchange="refreshDashNositelji()" style="background:var(--bg2);border:1px solid var(--rim);border-radius:5px;padding:6px 10px;color:var(--t1);font-size:12px">
+            <option value="0">Sve godine</option>
             <option value="2026">2026</option>
             <option value="2025" selected>2025</option>
             <option value="2024">2024</option>
+            <option value="2023">2023</option>
+            <option value="2022">2022</option>
+            <option value="2021">2021</option>
           </select>
           <span class="tb-s" id="dash-nos-cnt"></span>
         </div>
@@ -923,23 +927,44 @@ async function refreshDashNositelji(){
   const sel = $('#dash-god');
   if(!sel) return;
   const god = sel.value;
+  const lbl = (god === '0' || Number(god) <= 0) ? 'sve godine' : god;
   const out = $('#dash-nos-out');
-  out.innerHTML = '<div class="loading">Učitavanje primatelja '+god+'…</div>';
-  const d = await api('/v2/potpore/by-year?godina='+god);
+  out.innerHTML = '<div class="loading">Učitavanje primatelja '+lbl+'…</div>';
+  // wired na dashboard endpoint koji vraća sve nositelje (ne samo agregate)
+  const d = await api('/dashboard/top-primatelji?godina='+god+'&limit=50');
   if(!d){ out.innerHTML='<div class="empty">Greška pri dohvatu</div>'; return; }
-  const rows = (d.results || []).slice().sort((a,b)=>Number(b.iznos_eur||0)-Number(a.iznos_eur||0)).slice(0, 25);
-  $('#dash-nos-cnt').textContent = rows.length+' / '+(d.count||0)+' · ukupno '+fmtEur(d.total||0);
-  out.innerHTML = `<div style="overflow-x:auto"><table>
-    <thead><tr><th>#</th><th>Korisnik</th><th>Sport</th><th>Vrsta</th><th class="num">Iznos</th><th>PDF</th></tr></thead>
-    <tbody>${rows.map((r,i) => `
-      <tr onclick='openPrimateljDetail(${JSON.stringify(r).replace(/'/g,"&#39;")})'>
+  const rows = (d.rows || []);
+  $('#dash-nos-cnt').textContent = rows.length+' primatelja · ukupno '+fmtEur(d.ukupno||0);
+  if(rows.length === 0){
+    out.innerHTML = '<div class="empty">Nema podataka za '+lbl+'</div>';
+    return;
+  }
+  out.innerHTML = `<div style="overflow-x:auto;max-height:520px;overflow-y:auto"><table>
+    <thead><tr><th>#</th><th>Korisnik</th><th>Sport</th><th>Vrsta</th><th class="num">Iznos</th><th>Platitelj</th><th>PDF</th></tr></thead>
+    <tbody>${rows.map((r,i) => {
+      const proxy = {
+        korisnik: r.naziv_kluba,
+        sport: r.sport && r.sport!=='n/a' ? r.sport : null,
+        vrsta: r.vrsta,
+        iznos_eur: r.iznos,
+        godina: r.godina,
+        izvor: r.davatelj_naziv,
+        napomena: r.napomena,
+        source_url: r.pdf_url,
+        klub_id: r.klub_id
+      };
+      const pjson = JSON.stringify(proxy).replace(/'/g,"&#39;");
+      return `
+      <tr onclick='openPrimateljDetail(${pjson})'>
         <td>${i+1}</td>
-        <td><b>${esc(r.korisnik)}</b></td>
-        <td>${txt(r.sport)}</td>
-        <td>${txt(r.vrsta)}</td>
-        <td class="num"><b>${fmtEurFull(r.iznos_eur)}</b></td>
-        <td>${r.source_url?'<a href="'+esc(r.source_url)+'" target="_blank" onclick="event.stopPropagation()">📄 PDF</a>':'—'}</td>
-      </tr>`).join('')}</tbody>
+        <td><b>${esc(r.naziv_kluba)}</b>${r.godina && god==='0' ? ' <span class="tb-s">('+r.godina+')</span>' : ''}</td>
+        <td>${r.sport && r.sport!=='n/a' ? esc(r.sport) : '—'}</td>
+        <td>${esc(r.vrsta||'')}</td>
+        <td class="num"><b>${fmtEurFull(r.iznos)}</b></td>
+        <td>${esc(r.davatelj_naziv||'')}</td>
+        <td>${r.pdf_url?'<a href="'+esc(r.pdf_url)+'" target="_blank" onclick="event.stopPropagation()">📄 PDF</a>':'—'}</td>
+      </tr>`;
+    }).join('')}</tbody>
   </table></div>`;
 }