lambda-erp 0.6.20__tar.gz → 0.6.22__tar.gz

{lambda_erp-0.6.20 → lambda_erp-0.6.22}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lambda-erp
-Version: 0.6.20
+Version: 0.6.22
 Summary: Core ERP logic - accounting, sales, purchasing, inventory
 Author: TORUS INVESTMENTS AG
 License-Expression: Apache-2.0

{lambda_erp-0.6.20 → lambda_erp-0.6.22}/api/errors.py

@@ -1,5 +1,7 @@
 """Map lambda_erp exceptions to HTTP responses."""
 
+import re
+
 from fastapi import FastAPI, Request
 from fastapi.responses import JSONResponse
 from lambda_erp.exceptions import (
@@ -14,6 +16,56 @@ from lambda_erp.exceptions import (
 )
 
 
+# --- DB integrity violations -------------------------------------------------
+# A unique/foreign-key violation raised by the driver (psycopg on Postgres,
+# sqlite3 on SQLite) has no handler below, so it fell through to FastAPI's opaque
+# 500 "Internal Server Error" — which a form may show as a stray red line or
+# swallow entirely (e.g. "saving the UID does nothing, no warning"). Map it to a
+# clear 409 with the colliding field/value so the UI can actually explain it.
+
+_PG_KEY_RE = re.compile(r"Key \(([^)]+)\)=\(([^)]*)\)")
+_SQLITE_UNIQUE_RE = re.compile(r"UNIQUE constraint failed: [^.]+\.([^\s,]+)")
+
+
+def _integrity_error_types() -> list:
+    """The driver IntegrityError classes that are actually installed (SQLite
+    always; psycopg only with the [postgres] extra). Import defensively so a
+    SQLite-only deployment doesn't need psycopg."""
+    types: list = []
+    try:
+        import sqlite3
+        types.append(sqlite3.IntegrityError)
+    except Exception:
+        pass
+    for mod_name in ("psycopg", "psycopg2"):
+        try:
+            mod = __import__(mod_name)
+            err = getattr(mod, "IntegrityError", None) or getattr(
+                getattr(mod, "errors", None), "IntegrityError", None
+            )
+            if err is not None:
+                types.append(err)
+        except Exception:
+            pass
+    return types
+
+
+def _friendly_integrity(msg: str) -> str:
+    m = _PG_KEY_RE.search(msg or "")
+    if m:
+        return f"A record with {m.group(1)} = “{m.group(2)}” already exists."
+    m = _SQLITE_UNIQUE_RE.search(msg or "")
+    if m:
+        return f"A record with this {m.group(1)} already exists."
+    if "foreign key" in (msg or "").lower():
+        return "This links to a record that doesn't exist."
+    return "A record with these details already exists (a unique field is duplicated)."
+
+
+async def _integrity_handler(request: Request, exc: Exception):
+    return JSONResponse(status_code=409, content={"detail": _friendly_integrity(str(exc))})
+
+
 def register_exception_handlers(app: FastAPI):
 
     @app.exception_handler(ValueError)
@@ -54,3 +106,6 @@ def register_exception_handlers(app: FastAPI):
     @app.exception_handler(InsufficientFunds)
     async def insufficient_funds_error(request: Request, exc: InsufficientFunds):
         return JSONResponse(status_code=422, content={"detail": str(exc)})
+
+    for _exc_type in _integrity_error_types():
+        app.add_exception_handler(_exc_type, _integrity_handler)

{lambda_erp-0.6.20 → lambda_erp-0.6.22}/api/routers/masters.py

@@ -319,9 +319,12 @@ def search_masters(master_type: str, q: str = "", _user: dict = _viewer):
     if not q:
         return db.get_all(doctype, filters=_with_active_filter(db, doctype), fields=["name", name_field], limit=10)
 
+    # Case-INSENSITIVE: on Postgres plain LIKE is case-sensitive, so "plus medica"
+    # would miss "Plus Medica AG". LOWER(...) LIKE LOWER(...) is portable (SQLite
+    # + Postgres) and matches how the document search (_search_clause) works.
     rows = db.sql(
         f'SELECT name, "{name_field}" FROM "{doctype}" '
-        f'WHERE {active_prefix}(name LIKE ? OR "{name_field}" LIKE ?) LIMIT 10',
+        f'WHERE {active_prefix}(LOWER(name) LIKE LOWER(?) OR LOWER("{name_field}") LIKE LOWER(?)) LIMIT 10',
         [f"%{q}%", f"%{q}%"],
     )
     return rows

{lambda_erp-0.6.20 → lambda_erp-0.6.22}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "lambda-erp"
-version = "0.6.20"
+version = "0.6.22"
 description = "Core ERP logic - accounting, sales, purchasing, inventory"
 readme = "README.md"
 license = "Apache-2.0"