tina4-python 3.11.10__tar.gz → 3.11.11__tar.gz

{tina4_python-3.11.10 → tina4_python-3.11.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tina4-python
-Version: 3.11.10
+Version: 3.11.11
 Summary: Tina4 for Python — 54 built-in features, zero dependencies
 Author-email: Andre van Zuydam <andrevanzuydam@gmail.com>
 License: MIT

{tina4_python-3.11.10 → tina4_python-3.11.11}/tina4_python/__init__.py

@@ -8,7 +8,7 @@ Tina4 Python v3.0 — Zero-dependency, lightweight web framework.
 
 One import, everything works.
 """
-__version__ = "3.11.10"
+__version__ = "3.11.11"
 
 # ── Route decorators ──
 from tina4_python.core.router import (  # noqa: E402, F401

{tina4_python-3.11.10 → tina4_python-3.11.11}/tina4_python/orm/fields.py

@@ -58,10 +58,41 @@ class Field:
                 raise ValueError(f"Field '{self.name}' is required")
             return self.default
 
-        # Type coercion
+        # Type coercion.
+        #
+        # `validate()` runs on BOTH the write path (user input from routes/forms)
+        # and the read path (rows coming back from the database driver). Drivers
+        # return values in different shapes per engine:
+        #
+        #   engine        datetime column     bool column    numeric column
+        #   --------      ----------------    -----------    --------------
+        #   SQLite        str                 int (0/1)      float / int
+        #   PostgreSQL    datetime            bool           Decimal
+        #   MySQL         datetime            int (0/1)      Decimal
+        #   MSSQL         datetime            bool           Decimal
+        #   Firebird      datetime            int (0/1)      Decimal
+        #
+        # So the rule is: if the driver already handed us the right type,
+        # accept it as-is. Otherwise coerce. This avoids the classic crash
+        # `datetime(datetime_instance)` that hit every PostgreSQL ORM read.
+        #
+        # Care is needed around `bool` being a subclass of `int` in Python —
+        # we handle those two paths explicitly before the generic fast path.
         try:
-            if self.field_type == bool and isinstance(value, int):
+            # BooleanField receiving an int (e.g. SQLite 0/1) → cast to bool
+            if self.field_type is bool and isinstance(value, int) and not isinstance(value, bool):
                 value = bool(value)
+            # IntegerField receiving a bool → cast to int (preserve legacy behaviour
+            # where True/False round-trip as 1/0 in numeric columns)
+            elif self.field_type is int and isinstance(value, bool):
+                value = int(value)
+            # DateTimeField receiving an ISO-8601 string (SQLite default) → parse
+            elif self.field_type is datetime and isinstance(value, str):
+                value = datetime.fromisoformat(value)
+            # Fast path: driver already handed us the correct type. Don't re-coerce —
+            # `datetime(datetime_instance)` etc. raises TypeError otherwise.
+            elif isinstance(value, self.field_type):
+                pass
             else:
                 value = self.field_type(value)
         except (TypeError, ValueError) as e: