Merge branch 'feature/prodaja-propusti'

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/prodaja.go

@@ -51,8 +51,8 @@ func (r *ProdajaRepo) Lista(ctx context.Context, pretraga string) ([]model.Proda
 	args := []any{}
 
 	if pretraga != "" {
-		upit += " AND pn.broj_naloga LIKE ?"
-		args = append(args, "%"+pretraga+"%")
+		upit += " AND (pn.broj_naloga LIKE ? OR kp.naziv LIKE ? OR pn.napomena LIKE ?)"
+		args = append(args, "%"+pretraga+"%", "%"+pretraga+"%", "%"+pretraga+"%")
 	}
 
 	upit += " ORDER BY pn.datum DESC"
@@ -122,10 +122,13 @@ func (r *ProdajaRepo) DohvatiID(ctx context.Context, id int64) (*model.ProdajniN
 // DohvatiStavke vraća stavke prodaje sa nazivima artikala i PDV podacima za dati nalog
 func (r *ProdajaRepo) DohvatiStavke(ctx context.Context, nalogID int64) ([]model.StavkaProdajeSaArtiklom, error) {
 	redovi, err := r.db.QueryContext(ctx, `
-		SELECT sp.id, sp.nalog_id, sp.artikal_id, sp.kolicina, sp.cena_po_komadu, sp.ukupno,
-		       sp.pdv_stopa, sp.pdv_iznos, sp.cena_bez_pdv, a.naziv, a.jedinica_mere
+		SELECT sp.id, sp.nalog_id, sp.artikal_id, sp.kolicina, sp.cena_po_komadu,
+		       sp.popust_procenat, sp.ukupno,
+		       sp.pdv_stopa, sp.pdv_iznos, sp.cena_bez_pdv,
+		       a.naziv, a.jedinica_mere, COALESCE(k.naziv, '') AS kategorija_naziv
 		FROM stavke_prodaje sp
 		JOIN artikli a ON a.id = sp.artikal_id
+		LEFT JOIN kategorije k ON k.id = a.kategorija_id
 		WHERE sp.nalog_id = ?
 		ORDER BY sp.id`, nalogID)
 	if err != nil {
@@ -138,9 +141,9 @@ func (r *ProdajaRepo) DohvatiStavke(ctx context.Context, nalogID int64) ([]model
 		var s model.StavkaProdajeSaArtiklom
 		err := redovi.Scan(
 			&s.ID, &s.NalogID, &s.ArtikalID, &s.Kolicina,
-			&s.CenaPoKomadu, &s.Ukupno,
+			&s.CenaPoKomadu, &s.PopustProcenat, &s.Ukupno,
 			&s.PdvStopa, &s.PdvIznos, &s.CenaBezPdv,
-			&s.ArtikalNaziv, &s.JedinicaMere,
+			&s.ArtikalNaziv, &s.JedinicaMere, &s.KategorijaNaziv,
 		)
 		if err != nil {
 			return nil, fmt.Errorf("ntech: ProdajaRepo.DohvatiStavke: scan: %w", err)
@@ -215,12 +218,17 @@ func (r *ProdajaRepo) Kreiraj(ctx context.Context, n *model.ProdajniNalog, stavk
 			pdvIznos = cenaBezPdv * s.PdvStopa / 100
 		}
 
-		ukupnoStavke := float64(s.Kolicina) * s.CenaPoKomadu
+		// popust umanjuje cenu po komadu pre računanja ukupnog i PDV-a
+		cenaPoslePopusta := s.CenaPoKomadu
+		if s.PopustProcenat > 0 {
+			cenaPoslePopusta = cenaPoslePopusta * (1 - s.PopustProcenat/100)
+		}
+		ukupnoStavke := float64(s.Kolicina) * cenaPoslePopusta
 		_, err = tx.ExecContext(ctx, `
 			INSERT INTO stavke_prodaje
-				(nalog_id, artikal_id, kolicina, cena_po_komadu, ukupno, pdv_stopa, pdv_iznos, cena_bez_pdv)
-			VALUES (?, ?, ?, ?, ?, ?, ?, ?)`,
-			nalogID, s.ArtikalID, s.Kolicina, s.CenaPoKomadu, ukupnoStavke,
+				(nalog_id, artikal_id, kolicina, cena_po_komadu, popust_procenat, ukupno, pdv_stopa, pdv_iznos, cena_bez_pdv)
+			VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)`,
+			nalogID, s.ArtikalID, s.Kolicina, s.CenaPoKomadu, s.PopustProcenat, ukupnoStavke,
 			s.PdvStopa, pdvIznos, cenaBezPdv,
 		)
 		if err != 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/handler/prodaja.go

@@ -62,18 +62,32 @@ type PodaciStampeProdaje struct {
 	PIB          string
 }
 
-// artikalUJSONSaCenom pretvara listu artikala u template.JS vrednost sa prodajnom cenom, PDV stopom i stanjem
+// artikalUJSONSaCenom pretvara listu artikala u template.JS vrednost sa prodajnom cenom, PDV stopom, kategorijom i stanjem
 func artikalUJSONSaCenom(artikli []model.ArtikalSaKategorijom) template.JS {
 	type stavka struct {
-		ID       int64   `json:"id"`
-		Naziv    string  `json:"naziv"`
-		Cena     float64 `json:"cena"`
-		PdvStopa float64 `json:"pdv_stopa"`
-		Kolicina int     `json:"kolicina"`
+		ID              int64   `json:"id"`
+		Naziv           string  `json:"naziv"`
+		Sifra           string  `json:"sifra"`
+		Barkod          string  `json:"barkod"`
+		Cena            float64 `json:"cena"`
+		CenaSaPdv       float64 `json:"cena_sa_pdv"`
+		PdvStopa        float64 `json:"pdv_stopa"`
+		Kolicina        int     `json:"kolicina"`
+		KategorijaNaziv string  `json:"kategorija_naziv"`
 	}
 	lista := make([]stavka, 0, len(artikli))
 	for _, a := range artikli {
-		lista = append(lista, stavka{ID: a.ID, Naziv: a.Naziv, Cena: a.ProdajnaCena, PdvStopa: a.PdvStopa, Kolicina: a.Kolicina})
+		lista = append(lista, stavka{
+			ID:              a.ID,
+			Naziv:           a.Naziv,
+			Sifra:           a.Sifra,
+			Barkod:          a.Barkod,
+			Cena:            a.ProdajnaCena,
+			CenaSaPdv:       a.CenaSaPdv,
+			PdvStopa:        a.PdvStopa,
+			Kolicina:        a.Kolicina,
+			KategorijaNaziv: a.KategorijaNaziv,
+		})
 	}
 	b, _ := json.Marshal(lista)
 	return template.JS(b)
@@ -186,7 +200,11 @@ func (h *Handler) SacuvajProdaju(w http.ResponseWriter, r *http.Request) {
 
 	var ukupno float64
 	for _, s := range stavke {
-		ukupno += float64(s.Kolicina) * s.CenaPoKomadu
+		cenaPoslePopusta := s.CenaPoKomadu
+		if s.PopustProcenat > 0 {
+			cenaPoslePopusta = cenaPoslePopusta * (1 - s.PopustProcenat/100)
+		}
+		ukupno += float64(s.Kolicina) * cenaPoslePopusta
 	}
 	nalog.Ukupno = ukupno
 
@@ -394,9 +412,10 @@ func (h *Handler) StampaProdaje(w http.ResponseWriter, r *http.Request) {
 	h.renderujStandalone(w, "prodaja_stampa", podaci)
 }
 
-// ObrisiProdaju prima POST zahtev, vraća stanje na magacin i briše nalog
+// ObrisiProdaju prima POST zahtev i stornira nalog umesto fizičkog brisanja.
+// Jednom izdat račun ne sme da nestane — umesto brisanja koristi se storno.
 func (h *Handler) ObrisiProdaju(w http.ResponseWriter, r *http.Request) {
-	if _, ok := h.zahtevajDozvolu(w, r, "prodaja.obrisi"); !ok {
+	if _, ok := h.zahtevajDozvolu(w, r, "prodaja.storno"); !ok {
 		return
 	}
 	k := middleware.KorisnikIzKonteksta(r.Context())
@@ -406,9 +425,8 @@ func (h *Handler) ObrisiProdaju(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	if err := h.ProdajaRepo.Obrisi(r.Context(), id, &k.ID); err != nil {
-		http.Error(w, "Greška pri brisanju naloga", http.StatusInternalServerError)
-		return
+	if err := h.ProdajaRepo.Storno(r.Context(), id, "administrativno uklanjanje", &k.ID); err != nil {
+		slog.Error("greška pri uklanjanju naloga", "error", err)
 	}
 	// ukloni vezani auto-KIR zapis (ako ga je ova prodaja kreirala)
 	if err := h.PdvKirRepo.ObrisiPoIzvoru(r.Context(), "prodaja", id); err != nil {
@@ -438,6 +456,7 @@ func parseFormuProdaje(r *http.Request) (model.ProdajniNalog, []model.StavkaProd
 	kolicine := r.Form["kolicina[]"]
 	cene := r.Form["cena_po_komadu[]"]
 	pdvStope := r.Form["pdv_stopa[]"]
+	popusti := r.Form["popust[]"]
 
 	if len(artikalIDovi) == 0 {
 		return nalog, nil, "Prodaja mora imati najmanje jednu stavku."
@@ -469,11 +488,17 @@ func parseFormuProdaje(r *http.Request) (model.ProdajniNalog, []model.StavkaProd
 			pdvStopa, _ = strconv.ParseFloat(strings.TrimSpace(pdvStope[i]), 64)
 		}
 
+		var popust float64
+		if i < len(popusti) {
+			popust, _ = strconv.ParseFloat(strings.TrimSpace(popusti[i]), 64)
+		}
+
 		stavke = append(stavke, model.StavkaProdaje{
-			ArtikalID:    artikalID,
-			Kolicina:     kolicina,
-			CenaPoKomadu: cena,
-			PdvStopa:     pdvStopa,
+			ArtikalID:      artikalID,
+			Kolicina:       kolicina,
+			CenaPoKomadu:   cena,
+			PdvStopa:       pdvStopa,
+			PopustProcenat: popust,
 		})
 	}
 

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
 }

internal/model/prodaja.go

@@ -17,15 +17,16 @@ type ProdajniNalog struct {
 
 // StavkaProdaje predstavlja jednu liniju (artikal) unutar prodaje
 type StavkaProdaje struct {
-	ID           int64
-	NalogID      int64
-	ArtikalID    int64
-	Kolicina     int
-	CenaPoKomadu float64
-	Ukupno       float64
-	PdvStopa     float64
-	PdvIznos     float64
-	CenaBezPdv   float64
+	ID             int64
+	NalogID        int64
+	ArtikalID      int64
+	Kolicina       int
+	CenaPoKomadu   float64
+	PopustProcenat float64 // rabat u procentima (npr. 10 = 10% popusta)
+	Ukupno         float64
+	PdvStopa       float64
+	PdvIznos       float64
+	CenaBezPdv     float64
 }
 
 // UkupnoBezPdv vraća ukupno bez PDV-a za stavku (CenaBezPdv × količina)
@@ -42,6 +43,7 @@ type ProdajniNalogSaDetaljem struct {
 // StavkaProdajeSaArtiklom je stavka prodaje sa nazivom artikla — za prikaz u detaljima
 type StavkaProdajeSaArtiklom struct {
 	StavkaProdaje
-	ArtikalNaziv string
-	JedinicaMere string
+	ArtikalNaziv    string
+	JedinicaMere    string
+	KategorijaNaziv string
 }