tina4ruby 3.13.9 → 3.13.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c247ddb7851f9f5e0f8cfb8a7a74a4e7690cd9fe606d5d32f7603aa9547aff1d
-  data.tar.gz: 75d979c50632935abe4c4864d3c0f28065b3ad87e2f13a81db999f04379fcfb8
+  metadata.gz: 303058fcf9f267e5034b659e75be74b7a5fba69740968063b7370ff046452c8c
+  data.tar.gz: 993d93df6a783588a4b6c33bd8737a522c5dba96aafefb9dcc701bc6a30eb6c0
 SHA512:
-  metadata.gz: ed1956eb82a7ce8242fcbb6c95b14613d72a6931c62bda8bf2e6944f7f77f61e838f40f0a2aeac031f8392d4230407129e2d0edd958c2ab4db2ecc5831fb0570
-  data.tar.gz: a8fed6c6985d6f717c083c977ef26df48ddf55af67816e4595918ef4183a5d78c3cea7c98e0849a8b1407cf5ef34f67b69840de0465d4b086561c2fa2ab3e1ac
+  metadata.gz: a0c1c3f7a454e6cee7b4512290b8b0cf3de3c1683d1f797f64cc7efc8caa5e1cd6a245a0d96351f151b878b49e57e8a2461c6f120151b0f1bcc660d0f1dcdd0c
+  data.tar.gz: 0d4eb1114071ca5173b36b73ce1af9c705e23a69d8aff1b45b335b1ba5587ca9fee4825b6dba1f9cacdab5cde01e2c64e77d73bd70950a7f976f64f3e779c583

data/lib/tina4/database.rb

@@ -82,6 +82,21 @@ module Tina4
       "odbc" => "Tina4::Drivers::OdbcDriver"
     }.freeze
 
+    # v3.13.12 — strip trailing `;` from user SQL before the framework
+    # wraps it with COUNT(*) subqueries or appends LIMIT/OFFSET. Without
+    # this, ``"SELECT * FROM t;"`` produces ``"SELECT * FROM t; LIMIT 100
+    # OFFSET 0"`` — a syntax error on every engine. Internal semicolons
+    # (inside string literals, between meaningful statements) are left
+    # alone; drivers reject those if multi-statement isn't supported.
+    def self.strip_trailing_semicolons(sql)
+      return sql if sql.nil? || sql.empty?
+      stripped = sql.rstrip
+      while stripped.end_with?(";")
+        stripped = stripped[0..-2].rstrip
+      end
+      stripped
+    end
+
     # Static factory — cross-framework consistency: Database.create(url)
     def self.create(url, username: "", password: "", pool: nil)
       new(url, username: username.empty? ? nil : username,
@@ -245,7 +260,11 @@ module Tina4
     #   rows.each { |row| puts row["name"] }
     #
     # Returns [] (not nil) when no rows match.
-    def fetch_all(sql, params = [], limit: 100, offset: nil)
+    #
+    # v3.13.12: default `limit` is **nil** (no truncation) — the method
+    # name says fetch_all, so it returns all matching rows. Pre-v3.13.12
+    # silently truncated to 100. Pass an explicit `limit:` to cap.
+    def fetch_all(sql, params = [], limit: nil, offset: nil)
       fetch(sql, params, limit: limit, offset: offset).records
     end
 
@@ -253,6 +272,13 @@ module Tina4
       offset ||= 0
       drv = current_driver
 
+      # v3.13.12: strip trailing `;` so the driver's apply_limit
+      # (which appends "LIMIT N OFFSET M") doesn't produce
+      # "SELECT * FROM t; LIMIT 100 OFFSET 0" — a syntax error
+      # on every engine. Also helps any COUNT(*) FROM (sql)
+      # subqueries downstream survive a user-supplied semicolon.
+      sql = Tina4::Database.strip_trailing_semicolons(sql)
+
       effective_sql = sql
       # Skip appending LIMIT if SQL already has one
       has_limit = sql.upcase.split("--")[0].include?("LIMIT")
@@ -279,6 +305,7 @@ module Tina4
     end
 
     def fetch_one(sql, params = [])
+      sql = Tina4::Database.strip_trailing_semicolons(sql)
       if @cache_enabled
         key = cache_key(sql + ":ONE", params)
         cached = cache_get(key)

data/lib/tina4/orm.rb

@@ -18,7 +18,11 @@ module Tina4
 
     class << self
       def db
-        @db || Tina4.database
+        # v3.13.12: implicit binding from TINA4_DATABASE_URL.
+        # Resolution: per-class @db → global Tina4.database → env-derived
+        # auto-discovery. Pre-v3.13.12 this fell through to nil — the
+        # helper auto_discover_db existed but was never called.
+        @db || Tina4.database || auto_discover_db
       end
 
       # Per-model database binding
@@ -302,13 +306,27 @@ module Tina4
       def create_table
         return true if db.table_exists?(table_name)
 
+        # v3.13.11 (BooleanField parity): pick each engine's native
+        # bool type where it's reliable. SQLite has no native bool;
+        # Firebird's driver round-trip for native BOOLEAN is uneven —
+        # both stay on INTEGER. PG/MySQL/MSSQL use their native types
+        # so Python ``True``/Ruby ``true`` bind cleanly without
+        # ``operator does not exist: boolean = integer`` errors.
+        engine = (db.respond_to?(:get_database_type) ? db.get_database_type : "").to_s.downcase
+        bool_sql = case engine
+                   when "postgres", "postgresql" then "BOOLEAN"
+                   when "mysql" then "BOOLEAN" # alias for TINYINT(1)
+                   when "mssql", "sqlserver" then "BIT"
+                   else "INTEGER" # sqlite, firebird, odbc, anything else
+                   end
+
         type_map = {
           integer: "INTEGER",
           string: "VARCHAR(255)",
           text: "TEXT",
           float: "REAL",
           decimal: "REAL",
-          boolean: "INTEGER",
+          boolean: bool_sql,
           date: "DATE",
           datetime: "DATETIME",
           timestamp: "TIMESTAMP",
@@ -419,10 +437,18 @@ module Tina4
         setter = "#{key}="
         __send__(setter, value) if respond_to?(setter)
       end
-      # Set defaults
+      # Set defaults.
+      # v3.13.11 (issue #50.1): when the default is a Proc/lambda
+      # (``default: -> { Time.now }``), call it per-instance so
+      # per-row timestamps actually differ. Class objects are
+      # excluded — ``default: Integer`` is almost never intended
+      # to mean ``Integer.new`` (and Integer has no zero-arg
+      # constructor anyway).
       self.class.field_definitions.each do |name, opts|
         if __send__(name).nil? && opts[:default]
-          __send__("#{name}=", opts[:default])
+          d = opts[:default]
+          d = d.call if d.respond_to?(:call) && !d.is_a?(Class)
+          __send__("#{name}=", d)
         end
       end
     end

data/lib/tina4/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Tina4
-  VERSION = "3.13.9"
+  VERSION = "3.13.12"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tina4ruby
 version: !ruby/object:Gem::Version
-  version: 3.13.9
+  version: 3.13.12
 platform: ruby
 authors:
 - Tina4 Team
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-06-10 00:00:00.000000000 Z
+date: 2026-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack