Nabavka: broj računa, datum računa, PDV iznos + KPR backfill za stare nabavke

internal/db/repository.go

@@ -78,6 +78,8 @@ type PdvKprRepository interface {
 	Obrisi(ctx context.Context, id int64) error
 	// ObrisiPoIzvoru briše zapise vezane za dati izvor (npr. pri brisanju nabavke)
 	ObrisiPoIzvoru(ctx context.Context, izvor string, izvorID int64) error
+	// PostojiZaIzvor vraća true ako postoji bar jedan zapis za dati izvor i izvorID
+	PostojiZaIzvor(ctx context.Context, izvor string, izvorID int64) (bool, error)
 }
 
 // NivelacijaRepository definiše operacije nad evidencijom promene prodajnih cena

internal/db/sqlite/nabavka.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"database/sql"
 	"fmt"
+	"time"
 
 	"ntech/internal/model"
 )
@@ -24,6 +25,7 @@ func (r *NabavkaRepo) Lista(ctx context.Context) ([]model.NabavkaSaDetaljem, err
 		SELECT
 			n.id, n.dobavljac_id, n.napomena, n.ukupno, n.metod_raspodele,
 			n.stornirano, n.razlog_storniranja, n.datum,
+			n.broj_racuna, n.datum_racuna, n.pdv_iznos,
 			COALESCE(d.naziv, '') AS dobavljac_naziv
 		FROM nabavke n
 		LEFT JOIN dobavljaci d ON n.dobavljac_id = d.id
@@ -37,11 +39,12 @@ func (r *NabavkaRepo) Lista(ctx context.Context) ([]model.NabavkaSaDetaljem, err
 	for redovi.Next() {
 		var n model.NabavkaSaDetaljem
 		var dobavljacID sql.NullInt64
-		var napomena, metod, razlogStorniranja sql.NullString
+		var napomena, metod, razlogStorniranja, brojRacuna, datumRacuna sql.NullString
 
 		err := redovi.Scan(
 			&n.ID, &dobavljacID, &napomena, &n.Ukupno, &metod,
 			&n.Stornirano, &razlogStorniranja, &n.Datum,
+			&brojRacuna, &datumRacuna, &n.PdvIznos,
 			&n.DobavljacNaziv,
 		)
 		if err != nil {
@@ -54,6 +57,12 @@ func (r *NabavkaRepo) Lista(ctx context.Context) ([]model.NabavkaSaDetaljem, err
 		n.Napomena = napomena.String
 		n.MetodRaspodele = metod.String
 		n.RazlogStorniranja = razlogStorniranja.String
+		n.BrojRacuna = brojRacuna.String
+		if datumRacuna.Valid && datumRacuna.String != "" {
+			if t, e := time.Parse("2006-01-02", datumRacuna.String[:10]); e == nil {
+				n.DatumRacuna = &t
+			}
+		}
 
 		rezultat = append(rezultat, n)
 	}
@@ -65,14 +74,16 @@ func (r *NabavkaRepo) Lista(ctx context.Context) ([]model.NabavkaSaDetaljem, err
 func (r *NabavkaRepo) DohvatiID(ctx context.Context, id int64) (*model.Nabavka, error) {
 	var n model.Nabavka
 	var dobavljacID sql.NullInt64
-	var napomena, metod, razlogStorniranja sql.NullString
+	var napomena, metod, razlogStorniranja, brojRacuna, datumRacuna sql.NullString
 
 	err := r.db.QueryRowContext(ctx, `
 		SELECT id, dobavljac_id, napomena, ukupno, metod_raspodele,
-		       stornirano, razlog_storniranja, datum
+		       stornirano, razlog_storniranja, datum,
+		       broj_racuna, datum_racuna, pdv_iznos
 		FROM nabavke WHERE id = ?`, id).Scan(
 		&n.ID, &dobavljacID, &napomena, &n.Ukupno, &metod,
 		&n.Stornirano, &razlogStorniranja, &n.Datum,
+		&brojRacuna, &datumRacuna, &n.PdvIznos,
 	)
 	if err != nil {
 		return nil, fmt.Errorf("ntech: NabavkaRepo.DohvatiID: %w", err)
@@ -84,6 +95,12 @@ func (r *NabavkaRepo) DohvatiID(ctx context.Context, id int64) (*model.Nabavka,
 	n.Napomena = napomena.String
 	n.MetodRaspodele = metod.String
 	n.RazlogStorniranja = razlogStorniranja.String
+	n.BrojRacuna = brojRacuna.String
+	if datumRacuna.Valid && datumRacuna.String != "" {
+		if t, e := time.Parse("2006-01-02", datumRacuna.String[:10]); e == nil {
+			n.DatumRacuna = &t
+		}
+	}
 
 	return &n, nil
 }
@@ -163,9 +180,10 @@ func (r *NabavkaRepo) Kreiraj(ctx context.Context, n *model.Nabavka, stavke []mo
 
 	// upisujemo zaglavlje nabavke
 	rezultat, err := tx.ExecContext(ctx, `
-		INSERT INTO nabavke (dobavljac_id, napomena, ukupno, metod_raspodele)
-		VALUES (?, ?, ?, ?)`,
+		INSERT INTO nabavke (dobavljac_id, napomena, ukupno, metod_raspodele, broj_racuna, datum_racuna, pdv_iznos)
+		VALUES (?, ?, ?, ?, ?, ?, ?)`,
 		nullInt64(n.DobavljacID), nullString(n.Napomena), ukupno, nullString(n.MetodRaspodele),
+		nullString(n.BrojRacuna), nullDateString(n.DatumRacuna), n.PdvIznos,
 	)
 	if err != nil {
 		return 0, fmt.Errorf("ntech: NabavkaRepo.Kreiraj: insert nabavka: %w", err)

internal/db/sqlite/pdv_evidencija.go

@@ -292,3 +292,15 @@ func (r *PdvKprRepo) ObrisiPoIzvoru(ctx context.Context, izvor string, izvorID i
 	}
 	return nil
 }
+
+// PostojiZaIzvor vraća true ako postoji bar jedan KPR zapis za dati izvor i izvorID
+func (r *PdvKprRepo) PostojiZaIzvor(ctx context.Context, izvor string, izvorID int64) (bool, error) {
+	var n int
+	err := r.db.QueryRowContext(ctx,
+		"SELECT COUNT(*) FROM pdv_kpr WHERE izvor = ? AND izvor_id = ?", izvor, izvorID,
+	).Scan(&n)
+	if err != nil {
+		return false, fmt.Errorf("ntech: PdvKprRepo.PostojiZaIzvor: %w", err)
+	}
+	return n > 0, nil
+}

internal/db/sqlite/utils.go

@@ -46,3 +46,12 @@ func nullTime(v *time.Time) sql.NullTime {
 	}
 	return sql.NullTime{Time: *v, Valid: true}
 }
+
+// nullDateString upisuje *time.Time kao "YYYY-MM-DD" string ili NULL —
+// koristiti za DATE kolone gde sql.NullTime nije kompatibilan s driver-om
+func nullDateString(v *time.Time) sql.NullString {
+	if v == nil {
+		return sql.NullString{}
+	}
+	return sql.NullString{String: v.Format("2006-01-02"), Valid: true}
+}

internal/handler/nabavka.go

@@ -7,6 +7,7 @@ import (
 	"net/http"
 	"strconv"
 	"strings"
+	"time"
 
 	"ntech/internal/db"
 	"ntech/internal/db/sqlite"
@@ -32,6 +33,7 @@ type PodaciFormeNabavke struct {
 	Kategorije  []model.Kategorija // za dropdown u modalu novog artikla
 	Marza       string             // podrazumevana marža (%) za kalkulaciju
 	PdvObveznik bool               // da li firma obračunava PDV (utiče na prodajnu cenu u kalkulaciji)
+	Danas       string             // format "2006-01-02" — default za datum računa
 	Greska      string
 }
 
@@ -44,6 +46,7 @@ type PodaciDetaljiNabavke struct {
 	UkupanTrosak   float64
 	DobavljacNaziv string
 	Sacuvano       bool
+	Greska         string
 }
 
 // artikalUJSON pretvara listu artikala u template.JS vrednost bezbednu za umetanje u <script> tag
@@ -140,6 +143,7 @@ func (h *Handler) NovaNabavka(w http.ResponseWriter, r *http.Request) {
 		Kategorije:     kategorije,
 		Marza:          vrednostIliDefault(podesavanja, "kalkulacija_marza", "20"),
 		PdvObveznik:    h.modulUkljucen(r.Context(), "pdv"),
+		Danas:          time.Now().Format("2006-01-02"),
 	})
 }
 
@@ -155,6 +159,14 @@ func (h *Handler) SacuvajNabavku(w http.ResponseWriter, r *http.Request) {
 	}
 
 	nabavka, stavke, troskovi, greska := parseFormuNabavke(r)
+	if greska == "" && h.modulUkljucen(r.Context(), "pdv") {
+		switch {
+		case nabavka.DobavljacID == nil:
+			greska = "PDV evidencija je uključena — dobavljač je obavezan da bi KPR imao ispravan PIB."
+		case nabavka.BrojRacuna == "":
+			greska = "PDV evidencija je uključena — broj računa dobavljača je obavezan za KPR."
+		}
+	}
 	if greska != "" {
 		podesavanja, _ := sqlite.DohvatiSvaPodesavanja(r.Context(), h.DB)
 		artikli, _ := h.Artikli.Lista(r.Context(), db.ArtikalFilter{})
@@ -172,6 +184,7 @@ func (h *Handler) SacuvajNabavku(w http.ResponseWriter, r *http.Request) {
 			Kategorije:     kategorije,
 			Marza:          vrednostIliDefault(podesavanja, "kalkulacija_marza", "20"),
 			PdvObveznik:    h.modulUkljucen(r.Context(), "pdv"),
+			Danas:          time.Now().Format("2006-01-02"),
 			Greska:         greska,
 		})
 		return
@@ -203,6 +216,7 @@ func (h *Handler) SacuvajNabavku(w http.ResponseWriter, r *http.Request) {
 			}
 		}
 		nabavka.ID = id
+		nabavka.Datum = time.Now()
 		kpr := model.KprIzNabavke(nabavka, naziv, pib, mesto, stavkePdv)
 		if _, e := h.PdvKprRepo.Kreiraj(r.Context(), &kpr); e != nil {
 			slog.Error("auto-upis u KPR nije uspeo", "nabavka_id", id, "error", e)
@@ -284,6 +298,7 @@ func (h *Handler) DetaljiNabavke(w http.ResponseWriter, r *http.Request) {
 
 	nabavka, err := h.NabavkeRepo.DohvatiID(r.Context(), id)
 	if err != nil {
+		slog.Error("DetaljiNabavke: DohvatiID greška", "id", id, "error", err)
 		http.Error(w, "Nabavka nije pronađena", http.StatusNotFound)
 		return
 	}
@@ -322,6 +337,10 @@ func (h *Handler) DetaljiNabavke(w http.ResponseWriter, r *http.Request) {
 	for _, t := range troskovi {
 		ukupanTrosak += t.Iznos
 	}
+	var greska string
+	if r.URL.Query().Get("greska") == "storno" {
+		greska = "Storniranje nije uspelo. Nabavka je možda već stornirana."
+	}
 	podaci := PodaciDetaljiNabavke{
 		PodaciStranice: ps,
 		Nabavka:        *nabavka,
@@ -330,6 +349,7 @@ func (h *Handler) DetaljiNabavke(w http.ResponseWriter, r *http.Request) {
 		UkupanTrosak:   ukupanTrosak,
 		DobavljacNaziv: dobavljacNaziv,
 		Sacuvano:       r.URL.Query().Get("sacuvano") == "1",
+		Greska:         greska,
 	}
 
 	h.renderujTemplate(w, "nabavka_detalji", podaci)
@@ -354,7 +374,7 @@ func (h *Handler) StornoNabavke(w http.ResponseWriter, r *http.Request) {
 
 	if err := h.NabavkeRepo.Storno(r.Context(), id, razlog, &k.ID); err != nil {
 		slog.Error("greška pri storniranju nabavke", "error", err)
-		http.Redirect(w, r, "/nabavke/"+strconv.FormatInt(id, 10), http.StatusSeeOther)
+		http.Redirect(w, r, "/nabavke/"+strconv.FormatInt(id, 10)+"?greska=storno", http.StatusSeeOther)
 		return
 	}
 	// stornirana nabavka ne ulazi u PDV — ukloni vezani auto-KPR zapis
@@ -378,37 +398,47 @@ func parseFormuNabavke(r *http.Request) (model.Nabavka, []model.StavkaNabavke, [
 		}
 	}
 	nabavka.Napomena = strings.TrimSpace(r.FormValue("napomena"))
+	nabavka.BrojRacuna = strings.TrimSpace(r.FormValue("broj_racuna"))
+
+	if datStr := strings.TrimSpace(r.FormValue("datum_racuna")); datStr != "" {
+		if t, e := time.Parse("2006-01-02", datStr); e == nil {
+			nabavka.DatumRacuna = &t
+		}
+	}
+
+	if pdvStr := strings.TrimSpace(r.FormValue("pdv_iznos")); pdvStr != "" {
+		if v, e := strconv.ParseFloat(pdvStr, 64); e == nil && v > 0 {
+			nabavka.PdvIznos = v
+		}
+	}
 
 	// paralelni nizovi stavki
 	artikalIDovi := r.Form["artikal_id[]"]
 	kolicine := r.Form["kolicina[]"]
 	cene := r.Form["cena_po_komadu[]"]
 
-	if len(artikalIDovi) == 0 {
-		return nabavka, nil, nil, "Nabavka mora imati najmanje jednu stavku."
-	}
-
-	if len(artikalIDovi) != len(kolicine) || len(artikalIDovi) != len(cene) {
-		return nabavka, nil, nil, "Greška u podacima forme — broj stavki nije ispravan."
-	}
-
 	var stavke []model.StavkaNabavke
 	for i := range artikalIDovi {
-		artikalID, err := strconv.ParseInt(strings.TrimSpace(artikalIDovi[i]), 10, 64)
+		// preskoči red koji korisnik nije popunio
+		artikalIDStr := strings.TrimSpace(artikalIDovi[i])
+		if artikalIDStr == "" {
+			continue
+		}
+		artikalID, err := strconv.ParseInt(artikalIDStr, 10, 64)
 		if err != nil || artikalID <= 0 {
-			return nabavka, nil, nil, "Neispravan artikal u stavci."
+			continue
 		}
-
-		kolicina, err := strconv.Atoi(strings.TrimSpace(kolicine[i]))
-		if err != nil || kolicina <= 0 {
-			return nabavka, nil, nil, "Količina mora biti pozitivan broj."
+		var kolicina int
+		if i < len(kolicine) {
+			kolicina, _ = strconv.Atoi(strings.TrimSpace(kolicine[i]))
 		}
-
-		cena, err := strconv.ParseFloat(strings.TrimSpace(cene[i]), 64)
-		if err != nil || cena < 0 {
-			return nabavka, nil, nil, "Cena mora biti pozitivan broj."
+		if kolicina <= 0 {
+			kolicina = 1
+		}
+		var cena float64
+		if i < len(cene) {
+			cena, _ = strconv.ParseFloat(strings.TrimSpace(cene[i]), 64)
 		}
-
 		stavke = append(stavke, model.StavkaNabavke{
 			ArtikalID:    artikalID,
 			Kolicina:     kolicina,
@@ -416,6 +446,10 @@ func parseFormuNabavke(r *http.Request) (model.Nabavka, []model.StavkaNabavke, [
 		})
 	}
 
+	if len(stavke) == 0 {
+		return nabavka, nil, nil, "Nabavka mora imati najmanje jednu stavku."
+	}
+
 	// zavisni troškovi (opcioni, paralelni nizovi); prazni redovi se preskaču
 	naziviT := r.Form["trosak_naziv[]"]
 	iznosiT := r.Form["trosak_iznos[]"]

internal/handler/pdv_kpr.go

@@ -1,6 +1,8 @@
 package handler
 
 import (
+	"fmt"
+	"log/slog"
 	"net/http"
 	"strings"
 	"time"
@@ -157,6 +159,70 @@ func (h *Handler) SacuvajPdvKpr(w http.ResponseWriter, r *http.Request) {
 	http.Redirect(w, r, "/pdv/kpr?sacuvano=1", http.StatusSeeOther)
 }
 
+// KprBackfillNabavke kreira KPR zapise za sve aktivne nabavke koje ih nemaju.
+// Namenjen za jednokratno popunjavanje starih podataka.
+func (h *Handler) KprBackfillNabavke(w http.ResponseWriter, r *http.Request) {
+	if _, ok := h.zahtevajDozvolu(w, r, "pdv.dodaj"); !ok {
+		return
+	}
+	ctx := r.Context()
+
+	nabavke, err := h.NabavkeRepo.Lista(ctx)
+	if err != nil {
+		http.Error(w, "Greška pri učitavanju nabavki", http.StatusInternalServerError)
+		return
+	}
+
+	var kreirano, preskoceno int
+	for _, nd := range nabavke {
+		if nd.Stornirano {
+			continue
+		}
+		// preskoči ako već ima KPR zapis
+		postoji, _ := h.PdvKprRepo.PostojiZaIzvor(ctx, "nabavka", nd.ID)
+		if postoji {
+			preskoceno++
+			continue
+		}
+
+		stavke, err := h.NabavkeRepo.DohvatiStavke(ctx, nd.ID)
+		if err != nil {
+			slog.Error("backfill KPR: greška pri čitanju stavki", "nabavka_id", nd.ID, "error", err)
+			continue
+		}
+		var stavkePdv []model.NabavkaStavkaPdv
+		for _, s := range stavke {
+			var stopa float64
+			if a, e := h.Artikli.DohvatiID(ctx, s.ArtikalID); e == nil {
+				stopa = a.PdvStopa
+			}
+			stavkePdv = append(stavkePdv, model.NabavkaStavkaPdv{
+				Osnovica: float64(s.Kolicina) * s.CenaPoKomadu,
+				PdvStopa: stopa,
+			})
+		}
+		naziv, pib, mesto := "Nepoznat dobavljač", "", ""
+		if nd.DobavljacID != nil {
+			if d, e := h.DobavljaciRepo.DohvatiID(ctx, *nd.DobavljacID); e == nil {
+				naziv, pib, mesto = d.Naziv, d.PIB, d.Mesto
+			}
+		}
+		nabavka := nd.Nabavka
+		if nabavka.Datum.IsZero() {
+			nabavka.Datum = time.Now()
+		}
+		kpr := model.KprIzNabavke(nabavka, naziv, pib, mesto, stavkePdv)
+		if _, e := h.PdvKprRepo.Kreiraj(ctx, &kpr); e != nil {
+			slog.Error("backfill KPR: greška pri kreiranju zapisa", "nabavka_id", nd.ID, "error", e)
+			continue
+		}
+		kreirano++
+	}
+
+	middleware.SetFlash(w, r, h.DB, "uspeh", fmt.Sprintf("Backfill završen: %d novih KPR zapisa, %d preskočeno.", kreirano, preskoceno))
+	http.Redirect(w, r, "/pdv/kpr", http.StatusSeeOther)
+}
+
 // ObrisiPdvKpr briše zapis iz KPR
 func (h *Handler) ObrisiPdvKpr(w http.ResponseWriter, r *http.Request) {
 	if _, ok := h.zahtevajDozvolu(w, r, "pdv.obrisi"); !ok {

internal/model/nabavka.go

@@ -15,6 +15,9 @@ type Nabavka struct {
 	Stornirano        bool
 	RazlogStorniranja string
 	Datum             time.Time
+	BrojRacuna        string     // broj računa dobavljača (za KPR BrojDokumenta)
+	DatumRacuna       *time.Time // datum prometa sa računa dobavljača (za KPR DatumPrometa)
+	PdvIznos          float64    // stvarni PDV sa računa; 0 = koristiti aproksimaciju iz stope
 }
 
 // NabavkaTrosak je jedna stavka zavisnih troškova nabavke (npr. prevoz, carina)

internal/model/pdv_evidencija.go

@@ -195,16 +195,28 @@ type NabavkaStavkaPdv struct {
 	PdvStopa float64
 }
 
-// KprIzNabavke gradi KPR zapis iz nabavke. PDV se izvodi iz stope artikla po stavci
-// (⚠ aproksimacija: nabavna cena = osnovica bez PDV; stvaran PDV sa računa dobavljača može
-// se razlikovati). Grupiše po stopi (20→opšta, 10→posebna, ostalo→oslobođena nabavka).
-// Broj dokumenta je sintetički ("NAB-<id>") jer nabavka ne čuva broj računa dobavljača.
+// KprIzNabavke gradi KPR zapis iz nabavke.
+// Ako nabavka ima BrojRacuna, koristi ga kao BrojDokumenta; inače sintetički "NAB-<id>".
+// Ako nabavka ima DatumRacuna, koristi ga kao DatumPrometa; inače datum nabavke.
+// Ako nabavka ima PdvIznos > 0, koristi ga direktno (stvarni PDV sa računa dobavljača)
+// raspoređen proporcionalno po stavkama; inače aproksimacija iz stope artikla.
 func KprIzNabavke(nabavka Nabavka, dobavljacNaziv, dobavljacPib, dobavljacMesto string, stavke []NabavkaStavkaPdv) PdvKpr {
 	id := nabavka.ID
+
+	brojDok := fmt.Sprintf("NAB-%d", id)
+	if nabavka.BrojRacuna != "" {
+		brojDok = nabavka.BrojRacuna
+	}
+
+	datumPrometa := nabavka.Datum
+	if nabavka.DatumRacuna != nil {
+		datumPrometa = *nabavka.DatumRacuna
+	}
+
 	k := PdvKpr{
-		DatumPrometa:   nabavka.Datum,
+		DatumPrometa:   datumPrometa,
 		DatumKnjizenja: nabavka.Datum,
-		BrojDokumenta:  fmt.Sprintf("NAB-%d", id),
+		BrojDokumenta:  brojDok,
 		DobavljacNaziv: dobavljacNaziv,
 		DobavljacPib:   dobavljacPib,
 		DobavljacMesto: dobavljacMesto,
@@ -212,19 +224,47 @@ func KprIzNabavke(nabavka Nabavka, dobavljacNaziv, dobavljacPib, dobavljacMesto
 		Izvor:          "nabavka",
 		IzvorID:        &id,
 	}
-	for _, s := range stavke {
-		pdv := s.Osnovica * s.PdvStopa / 100
-		switch s.PdvStopa {
-		case 20:
-			k.OsnovicaOpsta += s.Osnovica
-			k.PdvOpsta += pdv
-		case 10:
-			k.OsnovicaPosebna += s.Osnovica
-			k.PdvPosebna += pdv
-		default:
-			k.OslobodenNabavka += s.Osnovica
+
+	// ako je unesen stvarni PDV sa računa, raspodeli ga proporcionalno po stavkama
+	if nabavka.PdvIznos > 0 {
+		var ukupnaOsnovica float64
+		for _, s := range stavke {
+			ukupnaOsnovica += s.Osnovica
+		}
+		for _, s := range stavke {
+			var udeo float64
+			if ukupnaOsnovica > 0 {
+				udeo = s.Osnovica / ukupnaOsnovica
+			}
+			pdv := math.Round(nabavka.PdvIznos*udeo*100) / 100
+			switch s.PdvStopa {
+			case 20:
+				k.OsnovicaOpsta += s.Osnovica
+				k.PdvOpsta += pdv
+			case 10:
+				k.OsnovicaPosebna += s.Osnovica
+				k.PdvPosebna += pdv
+			default:
+				k.OslobodenNabavka += s.Osnovica
+			}
+			k.Ukupno += s.Osnovica + pdv
+		}
+	} else {
+		// aproksimacija: PDV iz stope artikla
+		for _, s := range stavke {
+			pdv := s.Osnovica * s.PdvStopa / 100
+			switch s.PdvStopa {
+			case 20:
+				k.OsnovicaOpsta += s.Osnovica
+				k.PdvOpsta += pdv
+			case 10:
+				k.OsnovicaPosebna += s.Osnovica
+				k.PdvPosebna += pdv
+			default:
+				k.OslobodenNabavka += s.Osnovica
+			}
+			k.Ukupno += s.Osnovica + pdv
 		}
-		k.Ukupno += s.Osnovica + pdv
 	}
 	return k
 }