tina4ruby 3.13.18 → 3.13.19

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0073552dcac0e437fa36de2038f6958cbdbcde9077ec2427d9fd1bf77242da0e
-  data.tar.gz: c5c667382a67031bef1d7f3646020984942079edb5de9c5084c135ce4be68e9d
+  metadata.gz: 93597b3b5afd4c56593d4c2d107b573790b68fbae4b0dd28f07d4f636a462381
+  data.tar.gz: c384790499ba27f16fc95efec1053a34c201d5444f742a9581167f6450418526
 SHA512:
-  metadata.gz: d7e9036bfe01826691e30382e35b8c4f70dfe2043602f41abae7a0e573993992a99f89e733b2b7dd74e4c2c2d6fe1c6be7faeb4d57d132d0be6c1d471f538afb
-  data.tar.gz: 7a4ec3bdd62b40c973e2fadbd52f205d92bc6fd2f0e18c1407a164a6cb27fcb03bffb50ec789851ac567105c1da81c6fbdcd00ece6fdbefef2411d1067f55610
+  metadata.gz: fd30dce3ddb8de6ec35fa4416f105b52df238951d723df7887ad24787a9e657bd04c90e904427dae4365faf602ef273cfb1716dcd655b15b0ac0caf45ecfa4d0
+  data.tar.gz: 1febfd89b40487edc5c51314d6bd5878ecd22244720959e58f54e56d90f2751d0543be833f4f0ddb2da51bcdee5ba8aa5d3c06d6723fe28df644ae4f6f2ef3db

data/lib/tina4/orm.rb

@@ -41,14 +41,32 @@ module Tina4
 
     class << self
       def db
-        # 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
+        # Resolution order:
+        #   1. @db is a Symbol/String → named connection from Tina4.databases
+        #      (bound via Tina4.bind_database(db, name:)). Raises a clear
+        #      error if that named connection was never registered.
+        #   2. @db is a Database/driver instance → use it directly.
+        #   3. Otherwise → global Tina4.database, else env-derived
+        #      auto-discovery (TINA4_DATABASE_URL). v3.13.12 wired this
+        #      fallback; before that auto_discover_db was never called.
+        case @db
+        when Symbol, String
+          name = @db.to_sym
+          Tina4.databases[name] || raise(
+            "Tina4 named database connection '#{@db}' is not registered for #{name}. " \
+            "Call Tina4.bind_database(db, name: #{@db.inspect}) before using this model."
+          )
+        when nil
+          Tina4.database || auto_discover_db
+        else
+          @db
+        end
       end
 
-      # Per-model database binding
+      # Per-model database binding.
+      #   self.db = some_database_instance   → use that connection
+      #   self.db = :analytics               → resolve a named connection
+      #                                         from Tina4.databases at access time
       def db=(database)
         @db = database
       end
@@ -476,8 +494,7 @@ module Tina4
       def auto_discover_db
         url = ENV["TINA4_DATABASE_URL"]
         return nil unless url
-        Tina4.database = Tina4::Database.new(url, username: ENV.fetch("TINA4_DATABASE_USERNAME", ""), password: ENV.fetch("TINA4_DATABASE_PASSWORD", ""))
-        Tina4.database
+        Tina4.bind_database(Tina4::Database.new(url, username: ENV.fetch("TINA4_DATABASE_USERNAME", ""), password: ENV.fetch("TINA4_DATABASE_PASSWORD", "")))
       end
 
       def find_by_filter(filter)
@@ -517,6 +534,15 @@ module Tina4
       @persisted = false
       @errors = []
       @relationship_cache = {}
+      # Accept a JSON object string (parity with Python/PHP/Node):
+      #   Widget.new('{"id":1,"name":"alpha"}')
+      attributes = JSON.parse(attributes) if attributes.is_a?(String)
+      # A single model is one record — reject an Array with a clear message.
+      if attributes.is_a?(Array)
+        raise ArgumentError,
+              "#{self.class}.new expects a Hash, keyword args, or a JSON object string " \
+              "for one record — got an Array. Map over the list to build many records."
+      end
       attributes.each do |key, value|
         setter = "#{key}="
         __send__(setter, value) if respond_to?(setter)

data/lib/tina4/response.rb

@@ -80,6 +80,7 @@ module Tina4
     # Matches Python __call__ / PHP __invoke / Node response() pattern.
     def call(data = nil, status_code = 200, content_type = nil)
       @status_code = status_code
+      data = jsonable(data)
       if content_type
         @headers["content-type"] = content_type
         @body = data.to_s
@@ -96,10 +97,29 @@ module Tina4
     def json(data, status_or_opts = nil, status: nil)
       @status_code = status || (status_or_opts.is_a?(Integer) ? status_or_opts : 200)
       @headers["content-type"] = JSON_CONTENT_TYPE
+      data = jsonable(data)
       @body = data.is_a?(String) ? data : JSON.generate(data)
       self
     end
 
+    # Normalise domain objects into JSON-serialisable structures so handlers can
+    # `response.(model)` / `response.json(model)` without calling .to_h by hand:
+    #
+    #   response.json(user)            # ORM model      -> Hash
+    #   response.json(User.all)        # Array<ORM>      -> Array<Hash>
+    #   response.json(db.fetch(sql))   # DatabaseResult  -> Array<Hash>
+    #
+    # Plain Hash / Array / String pass through unchanged (Array members that are
+    # models are still converted).
+    def jsonable(data)
+      return data.to_h if data.is_a?(Tina4::ORM)
+      return data.records if data.is_a?(Tina4::DatabaseResult)
+      return data.map { |item| item.is_a?(Tina4::ORM) ? item.to_h : item } if data.is_a?(Array)
+
+      data
+    end
+    private :jsonable
+
     def html(content, status_or_opts = nil, status: nil)
       @status_code = status || (status_or_opts.is_a?(Integer) ? status_or_opts : 200)
       @headers["content-type"] = HTML_CONTENT_TYPE

data/lib/tina4/seeder.rb

@@ -373,7 +373,7 @@ module Tina4
 
     db = Tina4.database
     unless db
-      Tina4::Log.error("Seeder: No database connection. Set Tina4.database first.")
+      Tina4::Log.error("Seeder: No database connection. Call Tina4.bind_database(db) first.")
       return 0
     end
 

data/lib/tina4/version.rb

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

data/lib/tina4.rb

@@ -120,7 +120,26 @@ module Tina4
   BANNER
 
   class << self
-    attr_accessor :root_dir, :database
+    attr_accessor :root_dir
+    attr_reader :database
+
+    # Bind a database connection.
+    #   bind_database(db)                  → sets the global default (Tina4.database)
+    #   bind_database(db, name: :analytics) → registers a named connection
+    # A model with `self.db = :analytics` resolves from this named registry;
+    # otherwise models fall back to the global default / TINA4_DATABASE_URL.
+    def bind_database(db, name: nil)
+      if name.nil?
+        @database = db
+      else
+        (@databases ||= {})[name.to_sym] = db
+      end
+      db
+    end
+
+    # Named connection registry. bind_database(db, name:) populates it;
+    # models with a Symbol/String `self.db` resolve against it.
+    def databases = (@databases ||= {})
 
     def print_banner(host: "0.0.0.0", port: 7147, server_name: nil)
       # TINA4_SUPPRESS — short-circuit ALL banner output for headless / CI runs.
@@ -450,7 +469,7 @@ module Tina4
       db_url = ENV["TINA4_DATABASE_URL"]
       if db_url && !db_url.empty?
         begin
-          @database = Tina4::Database.new(db_url)
+          bind_database(Tina4::Database.new(db_url))
           Tina4::Log.info("Database connected: #{db_url.sub(/:[^:@]+@/, ':***@')}")
         rescue => e
           Tina4::Log.error("Database connection failed: #{e.message}")

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tina4ruby
 version: !ruby/object:Gem::Version
-  version: 3.13.18
+  version: 3.13.19
 platform: ruby
 authors:
 - Tina4 Team