From ccb3f9d748c9b22ac70c7424edd757e1c5c7ed2f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Dalibor=20Markovi=C4=87?= Date: Tue, 23 Jun 2026 02:35:49 +0200 Subject: [PATCH] =?UTF-8?q?Servis:=20garancija=20kao=20trajanje=20u=20dani?= =?UTF-8?q?ma=20+=20Tehni=C4=8Dar=20=E2=86=92=20Serviser?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Garancija se u detaljima prikazuje kao trajanje na srpskom (npr. „1 mesec i 15 dana", „2 meseca", „20 dana"); helper garancijaTekst + srpska deklinacija - Nova kolona garancija_dana (trajanje od završetka); garancija_do ostaje za fazu Završeno - Modal: textbox „Trajanje garancije (dana)" umesto kalendara; Podrazumevano i Bez garancije - Nova ruta/handler AzurirajGarancijaDana; forma izmene čuva trajanje (ne briše) - Podešavanja → Servis: rok garancije prebačen iz meseci u dane (servis_garancija_dana, default 60) - „Garancija do" uklonjena iz forme Izmeni nalog (kod arhiviran u docs/garancija-buduca-logika.md); SacuvajIzmenaNaloga čuva garanciju iz baze da je auto-snimanje ne pregazi - Prikazni tekst „Tehničar" → „Serviser" svuda (identifikatori Tehnicar/tehnicar_id netaknuti) --- cmd/ntech/main.go | 1 + docs/garancija-buduca-logika.md | 66 ++++++++++ internal/db/repository.go | 1 + internal/db/sqlite/servis.go | 38 ++++-- internal/db/sqlite/utils.go | 9 ++ internal/handler/kes.go | 2 + internal/handler/podesavanja.go | 14 +- internal/handler/servis.go | 120 +++++++++++++++--- internal/model/servis.go | 1 + migrations/072_servis_garancija_dana.sql | 4 + .../stranice/podesavanja_servis.html | 11 +- web/templates/stranice/servis_detalji.html | 30 ++--- web/templates/stranice/servis_forma.html | 31 +---- web/templates/stranice/servis_otpremnica.html | 4 +- web/templates/stranice/servis_predracun.html | 2 +- web/templates/stranice/servis_stampa.html | 4 +- 16 files changed, 255 insertions(+), 83 deletions(-) create mode 100644 docs/garancija-buduca-logika.md create mode 100644 migrations/072_servis_garancija_dana.sql diff --git a/cmd/ntech/main.go b/cmd/ntech/main.go index 6792e5e..a896961 100644 --- a/cmd/ntech/main.go +++ b/cmd/ntech/main.go @@ -360,6 +360,7 @@ func main() { r.With(ntechmw.RequireDozvola(h.DozvoleRepo.ImaDozvolu, "servis.pregled")).Get("/servis/{id}/predracun", h.StampaPredracuna) r.With(doz("servis.izmeni")).Post("/servis/{id}/status", h.PromeniStatus) r.With(doz("servis.izmeni")).Post("/servis/{id}/garancija", h.AzurirajGaranciju) + r.With(doz("servis.izmeni")).Post("/servis/{id}/garancija-dana", h.AzurirajGarancijaDana) r.With(doz("servis.izmeni")).Post("/servis/{id}/predvidjen-datum", h.AzurirajPredvidjenDatum) r.With(doz("servis.izmeni")).Post("/servis/{id}/tehnicar", h.AzurirajTehnicar) r.With(doz("servis.izmeni")).Post("/servis/{id}/napomena-klijentu", h.SacuvajNapomenuKlijentu) diff --git a/docs/garancija-buduca-logika.md b/docs/garancija-buduca-logika.md new file mode 100644 index 0000000..9c139ac --- /dev/null +++ b/docs/garancija-buduca-logika.md @@ -0,0 +1,66 @@ +# Garancija — buduća logika (arhiva koda) + +## Plan (zašto) + +Trenutno se „Garancija do" unosi kao konkretan datum. To nema smisla dok je uređaj +tek primljen (status **Primljeno**) — popravka još nije ni počela, pa ni datum +garancije nije poznat. + +**Buduća logika:** +- Garancija se izražava kao **broj dana od datuma završetka** (podesivo, kao i sada + preko `servis_garancija_meseci` / mogući novi `servis_garancija_dana`). +- Dok je nalog u radu (Primljeno, U popravci…), klijentu se prikazuje samo + **podrazumevana garancija** kao informacija (npr. „garancija: 12 meseci od završetka"). +- **Pravi datum garancije** (`garancija_do`) upisuje se tek kad nalog pređe u + **Završeno** → `datum_zavrsetka + garantni period`. + +Zbog toga je „Garancija do" uklonjena iz forme **Izmeni nalog** (vidi commit), a +ovde je arhiviran kod da se kasnije ponovo upotrebi / prilagodi. + +## Arhivirani kod — „Garancija do" blok iz `servis_forma.html` (Izmeni nalog) + +```html +{{if .Izmena}} + +
+ +
+ + +
+ +
+{{end}} +``` + +## Backend koji OSTAJE (i dalje koristi modal „Garancija do" u detaljima) + +Nije uklonjeno — ovo su delovi koje treba znati pri ponovnoj upotrebi: + +- `internal/handler/servis.go`: + - `defaultGarancija(datumPrijema, podesavanja)` — prijem + `servis_garancija_meseci`. + Za buduću logiku zameniti osnovicu `datum_zavrsetka` umesto `datum_prijema`. + - `garancijaPrePrijema(garancija, prijem)` — validacija da garancija nije pre prijema. + - `AzurirajGaranciju` (ruta `POST /servis/{id}/garancija`) — snima `garancija_do`, + prazno / „bez garancije" → NULL. + - U `parseFormuNaloga`: blok koji čita `garancija_do` + `bez_garancije`. + - `BezGarancije` flag (čita NULL stanje iz baze) i `GarancijaDefault` (prijem + meseci) + u `PodaciDetaljiNaloga` / `PodaciFormeNaloga`. +- `internal/db/sqlite/servis.go`: `AzurirajGaranciju` repo metoda; kolona `garancija_do`. +- Podešavanje: `servis_garancija_meseci` (Podešavanja → Servis). + +## TODO kad se vraća + +1. Dodati prikaz „podrazumevana garancija" (informativno) na nalogu u statusu Primljeno. +2. Pri prelasku u **Završeno** automatski izračunati `garancija_do = datum_zavrsetka + period`. +3. Razmotriti period u danima umesto/pored meseci. diff --git a/internal/db/repository.go b/internal/db/repository.go index 193d0dd..c245796 100644 --- a/internal/db/repository.go +++ b/internal/db/repository.go @@ -192,6 +192,7 @@ type ServisRepository interface { Izmeni(ctx context.Context, n *model.ServisniNalog) error AzurirajStatus(ctx context.Context, id int64, status string) error AzurirajGaranciju(ctx context.Context, id int64, garancijaDo *time.Time) error + AzurirajGarancijaDana(ctx context.Context, id int64, dana *int) error AzurirajPredvidjenDatum(ctx context.Context, id int64, predvidjenDatum *time.Time) error AzurirajTehnicar(ctx context.Context, id int64, tehnicarID *int64) error AzurirajNapomenuKlijentu(ctx context.Context, id int64, tekst string) error diff --git a/internal/db/sqlite/servis.go b/internal/db/sqlite/servis.go index 710952f..e62cb09 100644 --- a/internal/db/sqlite/servis.go +++ b/internal/db/sqlite/servis.go @@ -57,7 +57,7 @@ func (r *ServisRepo) Lista(ctx context.Context, pretraga, status string) ([]mode SELECT sn.id, sn.klijent_id, sn.tehnicar_id, sn.broj_naloga, sn.uredjaj, sn.serijski_broj, sn.opis_kvara, sn.trazene_nadogradnje, sn.status, sn.cena_od, sn.cena_do, sn.cena_konacna, - sn.avans, sn.napomena, sn.garancija_do, sn.datum_prijema, sn.datum_zavrsetka, sn.predvidjen_datum, + sn.avans, sn.napomena, sn.garancija_do, sn.garancija_dana, sn.datum_prijema, sn.datum_zavrsetka, sn.predvidjen_datum, sn.ostecenja, sn.pin_uredjaja, sn.pribor, sn.napomena_klijentu, sn.javni_token, COALESCE(kp.naziv, '') AS klijent_naziv FROM servisni_nalozi sn @@ -104,7 +104,7 @@ func (r *ServisRepo) DohvatiID(ctx context.Context, id int64) (*model.ServisniNa SELECT id, klijent_id, tehnicar_id, broj_naloga, uredjaj, serijski_broj, opis_kvara, trazene_nadogradnje, status, cena_od, cena_do, cena_konacna, - avans, napomena, garancija_do, datum_prijema, datum_zavrsetka, predvidjen_datum, + avans, napomena, garancija_do, garancija_dana, datum_prijema, datum_zavrsetka, predvidjen_datum, ostecenja, pin_uredjaja, pribor, napomena_klijentu, javni_token FROM servisni_nalozi WHERE id = ?`, id) @@ -127,14 +127,14 @@ func (r *ServisRepo) Kreiraj(ctx context.Context, n *model.ServisniNalog) (int64 rezultat, err := r.db.ExecContext(ctx, ` INSERT INTO servisni_nalozi (klijent_id, tehnicar_id, broj_naloga, uredjaj, serijski_broj, opis_kvara, trazene_nadogradnje, - status, cena_od, cena_do, cena_konacna, avans, napomena, garancija_do, datum_zavrsetka, predvidjen_datum, + status, cena_od, cena_do, cena_konacna, avans, napomena, garancija_do, garancija_dana, datum_zavrsetka, predvidjen_datum, ostecenja, pin_uredjaja, pribor, datum_prijema, javni_token) - VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`, + VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`, nullInt64(n.KlijentID), nullInt64(n.TehnicarID), n.BrojNaloga, n.Uredjaj, nullString(n.SerijskiBroj), n.OpisKvara, n.TrazeneNadogradnje, n.Status, nullFloat64(n.CenaOd), nullFloat64(n.CenaDo), nullFloat64(n.CenaKonacna), nullFloat64(n.Avans), nullString(n.Napomena), - nullTime(n.GarancijaDo), nullTime(n.DatumZavrsetka), nullTime(n.PredvidjenDatum), + nullTime(n.GarancijaDo), nullInt(n.GarancijaDana), nullTime(n.DatumZavrsetka), nullTime(n.PredvidjenDatum), nullString(n.Ostecenja), nullString(n.PinUredjaja), nullString(n.Pribor), n.DatumPrijema, token, ) @@ -156,7 +156,7 @@ func (r *ServisRepo) DohvatiJavniToken(ctx context.Context, token string) (*mode SELECT id, klijent_id, tehnicar_id, broj_naloga, uredjaj, serijski_broj, opis_kvara, trazene_nadogradnje, status, cena_od, cena_do, cena_konacna, - avans, napomena, garancija_do, datum_prijema, datum_zavrsetka, predvidjen_datum, + avans, napomena, garancija_do, garancija_dana, datum_prijema, datum_zavrsetka, predvidjen_datum, ostecenja, pin_uredjaja, pribor, napomena_klijentu, javni_token FROM servisni_nalozi WHERE javni_token = ?`, token) @@ -173,12 +173,12 @@ func (r *ServisRepo) Izmeni(ctx context.Context, n *model.ServisniNalog) error { UPDATE servisni_nalozi SET klijent_id = ?, tehnicar_id = ?, uredjaj = ?, serijski_broj = ?, opis_kvara = ?, trazene_nadogradnje = ?, status = ?, cena_od = ?, cena_do = ?, cena_konacna = ?, - avans = ?, napomena = ?, garancija_do = ?, datum_zavrsetka = ?, predvidjen_datum = ?, + avans = ?, napomena = ?, garancija_do = ?, garancija_dana = ?, datum_zavrsetka = ?, predvidjen_datum = ?, ostecenja = ?, pin_uredjaja = ?, pribor = ? WHERE id = ?`, nullInt64(n.KlijentID), nullInt64(n.TehnicarID), n.Uredjaj, nullString(n.SerijskiBroj), n.OpisKvara, n.TrazeneNadogradnje, n.Status, nullFloat64(n.CenaOd), nullFloat64(n.CenaDo), nullFloat64(n.CenaKonacna), - nullFloat64(n.Avans), nullString(n.Napomena), nullTime(n.GarancijaDo), nullTime(n.DatumZavrsetka), nullTime(n.PredvidjenDatum), + nullFloat64(n.Avans), nullString(n.Napomena), nullTime(n.GarancijaDo), nullInt(n.GarancijaDana), nullTime(n.DatumZavrsetka), nullTime(n.PredvidjenDatum), nullString(n.Ostecenja), nullString(n.PinUredjaja), nullString(n.Pribor), n.ID, ) @@ -229,6 +229,19 @@ func (r *ServisRepo) AzurirajGaranciju(ctx context.Context, id int64, garancijaD return nil } +// AzurirajGarancijaDana postavlja trajanje garancije u danima (od završetka radova). +// dana == nil ili 0 → bez garancije. +func (r *ServisRepo) AzurirajGarancijaDana(ctx context.Context, id int64, dana *int) error { + _, err := r.db.ExecContext(ctx, + "UPDATE servisni_nalozi SET garancija_dana = ? WHERE id = ?", + nullInt(dana), id, + ) + if err != nil { + return fmt.Errorf("ntech: ServisRepo.AzurirajGarancijaDana: %w", err) + } + return nil +} + // AzurirajPredvidjenDatum postavlja ili briše ručni override predviđenog datuma popravke. // predvidjenDatum == nil → vraća se na izvedeni default (prijem + rok iz podešavanja). func (r *ServisRepo) AzurirajPredvidjenDatum(ctx context.Context, id int64, predvidjenDatum *time.Time) error { @@ -251,7 +264,7 @@ func (r *ServisRepo) AzurirajPredvidjenDatum(ctx context.Context, id int64, pred return nil } -// AzurirajTehnicar postavlja ili uklanja dodeljenog tehničara na nalogu. +// AzurirajTehnicar postavlja ili uklanja dodeljenog servisera na nalogu. // tehnicarID == nil → nedodeljen. func (r *ServisRepo) AzurirajTehnicar(ctx context.Context, id int64, tehnicarID *int64) error { _, err := r.db.ExecContext(ctx, @@ -359,11 +372,12 @@ func scanNalog(scan func(...any) error, n *model.ServisniNalog, klijentNaziv *st var serijskiBroj, napomena, ostecenja, pinUredjaja, pribor, napomenaKlijentu, javniToken sql.NullString var cenaOd, cenaDo, cenaKonacna, avans sql.NullFloat64 var garancijaDo, datumZavrsetka, predvidjenDatum sql.NullTime + var garancijaDana sql.NullInt64 args := []any{ &n.ID, &klijentID, &tehnicarID, &n.BrojNaloga, &n.Uredjaj, &serijskiBroj, &n.OpisKvara, &n.TrazeneNadogradnje, &n.Status, &cenaOd, &cenaDo, &cenaKonacna, - &avans, &napomena, &garancijaDo, &n.DatumPrijema, &datumZavrsetka, &predvidjenDatum, + &avans, &napomena, &garancijaDo, &garancijaDana, &n.DatumPrijema, &datumZavrsetka, &predvidjenDatum, &ostecenja, &pinUredjaja, &pribor, &napomenaKlijentu, &javniToken, } @@ -417,6 +431,10 @@ func scanNalog(scan func(...any) error, n *model.ServisniNalog, klijentNaziv *st v := predvidjenDatum.Time n.PredvidjenDatum = &v } + if garancijaDana.Valid { + v := int(garancijaDana.Int64) + n.GarancijaDana = &v + } n.JavniToken = javniToken.String return nil diff --git a/internal/db/sqlite/utils.go b/internal/db/sqlite/utils.go index b61935c..5ff9a3e 100644 --- a/internal/db/sqlite/utils.go +++ b/internal/db/sqlite/utils.go @@ -20,6 +20,15 @@ func nullInt64(v *int64) sql.NullInt64 { return sql.NullInt64{Int64: *v, Valid: true} } +// nullInt pretvara *int pokazivač u sql.NullInt64 — +// koristi se za opciona celobrojna polja (npr. trajanje garancije u danima) +func nullInt(v *int) sql.NullInt64 { + if v == nil { + return sql.NullInt64{} + } + return sql.NullInt64{Int64: int64(*v), Valid: true} +} + // nullFloat64 pretvara *float64 pokazivač u sql.NullFloat64 — // koristi se za opciona numerička polja kao što su cene func nullFloat64(v *float64) sql.NullFloat64 { diff --git a/internal/handler/kes.go b/internal/handler/kes.go index bbc1090..8d50737 100644 --- a/internal/handler/kes.go +++ b/internal/handler/kes.go @@ -68,6 +68,8 @@ var sablonskeFunkcije = template.FuncMap{ }, // telefon formatira srpski broj telefona radi lakšeg čitanja: "0641234567" → "064 123 4567" "telefon": formatirajTelefon, + // garancijaTekst prikazuje trajanje garancije (dani) na srpskom: 45 → „1 mesec i 15 dana" + "garancijaTekst": garancijaTekst, // json serijalizuje vrednost u JSON za ugradnju u