better

cmd/cart/mutation_set_delivery.go

@@ -39,63 +39,58 @@ import (
 //   - Variable delivery pricing (based on weight, distance, provider, etc.)
 //   - Validation of provider codes
 //   - Multi-currency delivery pricing
-func init() {
-	RegisterMutation[messages.SetDelivery](
-		"SetDelivery",
-		func(g *CartGrain, m *messages.SetDelivery) error {
-			if m == nil {
-				return fmt.Errorf("SetDelivery: nil payload")
-			}
-			if m.Provider == "" {
-				return fmt.Errorf("SetDelivery: provider is empty")
-			}
 
-			withDelivery := g.ItemsWithDelivery()
-			targetItems := make([]int, 0)
+func SetDelivery(g *CartGrain, m *messages.SetDelivery) error {
+	if m == nil {
+		return fmt.Errorf("SetDelivery: nil payload")
+	}
+	if m.Provider == "" {
+		return fmt.Errorf("SetDelivery: provider is empty")
+	}
 
-			if len(m.Items) == 0 {
-				// Use every item currently without a delivery
-				targetItems = append(targetItems, g.ItemsWithoutDelivery()...)
-			} else {
-				// Validate explicit list
-				for _, id64 := range m.Items {
-					id := int(id64)
-					found := false
-					for _, it := range g.Items {
-						if it.Id == id {
-							found = true
-							break
-						}
-					}
-					if !found {
-						return fmt.Errorf("SetDelivery: item id %d not found", id)
-					}
-					if slices.Contains(withDelivery, id) {
-						return fmt.Errorf("SetDelivery: item id %d already has a delivery", id)
-					}
-					targetItems = append(targetItems, id)
+	withDelivery := g.ItemsWithDelivery()
+	targetItems := make([]int, 0)
+
+	if len(m.Items) == 0 {
+		// Use every item currently without a delivery
+		targetItems = append(targetItems, g.ItemsWithoutDelivery()...)
+	} else {
+		// Validate explicit list
+		for _, id64 := range m.Items {
+			id := int(id64)
+			found := false
+			for _, it := range g.Items {
+				if it.Id == id {
+					found = true
+					break
 				}
 			}
-
-			if len(targetItems) == 0 {
-				return fmt.Errorf("SetDelivery: no eligible items to attach")
+			if !found {
+				return fmt.Errorf("SetDelivery: item id %d not found", id)
 			}
+			if slices.Contains(withDelivery, id) {
+				return fmt.Errorf("SetDelivery: item id %d already has a delivery", id)
+			}
+			targetItems = append(targetItems, id)
+		}
+	}
 
-			// Append new delivery
-			g.mu.Lock()
-			g.lastDeliveryId++
-			newId := g.lastDeliveryId
-			g.Deliveries = append(g.Deliveries, &CartDelivery{
-				Id:          newId,
-				Provider:    m.Provider,
-				PickupPoint: m.PickupPoint,
-				Price:       4900, // TODO: externalize pricing
-				Items:       targetItems,
-			})
-			g.mu.Unlock()
+	if len(targetItems) == 0 {
+		return fmt.Errorf("SetDelivery: no eligible items to attach")
+	}
 
-			return nil
-		},
-		WithTotals(),
-	)
+	// Append new delivery
+	g.mu.Lock()
+	g.lastDeliveryId++
+	newId := g.lastDeliveryId
+	g.Deliveries = append(g.Deliveries, &CartDelivery{
+		Id:          newId,
+		Provider:    m.Provider,
+		PickupPoint: m.PickupPoint,
+		Price:       4900, // TODO: externalize pricing
+		Items:       targetItems,
+	})
+	g.mu.Unlock()
+
+	return nil
 }