tina4ruby 3.13.53 → 3.13.54

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79a09ae5dffe2688af3e75a98dcf999929a1c560f3d8afbdc874590d6dfca275
-  data.tar.gz: 92d0d0b44f0e103a3f8ac894c620b075446bab88040d865095e7b9c23ea421fe
+  metadata.gz: cd2180527148137de7bd985ed3951aefcae89d9fee30032de3c34434b3cc209e
+  data.tar.gz: 78e32b6e6021f24c455bb002339a59c54a8a134a1b219493c6dc994d66ebaff5
 SHA512:
-  metadata.gz: 128df90a6f281fefb62fc1e49e3075c52474d2c9f70cddc992e1c68a083b8dff38e9c6d014ffec0019736719f49bc2f80cb93eb2f3dff2b3deb336845db1f0e2
-  data.tar.gz: c579de40838336727fd25581e8d4db17bd3c449034e4805ba29a8215be60f5344c7608152c6c21cc7258396525e1bc5aee86b12ec4927de62f1745f39797601d
+  metadata.gz: 8f422139294a6d51b34507f116dd21621e5299829b02f7ec613083bd2bc9d051a4c832c01dab8120978708fa637302b35b6b05d2f463a188f91dfc9d89d2879b
+  data.tar.gz: 96ebc4a47de1674a828dc5913c4586fc82170f0c04159724d69a1cd6a99d15596bc5f217aa76ed4e119675559fc48d7b22cbd71eccd10e231de32d1ab2bc61da

data/lib/tina4/migration.rb

@@ -229,19 +229,34 @@ module Tina4
       end
     end
 
+    # Return a result row keyed by lower-case symbols. Firebird returns rows keyed
+    # by UPPERCASE string column names while SQLite/others return lower-case symbol
+    # keys, so the tracking-table reads below are normalised to one shape. Without
+    # this, `row[:migration_name]` is nil on Firebird and every applied migration
+    # is treated as pending and re-run.
+    def normalize_row(row)
+      return {} if row.nil?
+
+      source = row.respond_to?(:to_h) ? row.to_h : row
+      source.each_with_object({}) { |(k, v), h| h[k.to_s.downcase.to_sym] = v }
+    end
+
     def completed_migrations
       result = @db.fetch("SELECT migration_name FROM #{TRACKING_TABLE} WHERE passed = 1 ORDER BY id")
-      result.map { |r| r[:migration_name] }
+      result.map { |r| normalize_row(r)[:migration_name] }
     end
 
     def completed_migrations_with_batch
       result = @db.fetch("SELECT id, migration_name, batch FROM #{TRACKING_TABLE} WHERE passed = 1 ORDER BY id")
-      result.map { |r| { id: r[:id], migration_name: r[:migration_name], batch: r[:batch] } }
+      result.map do |r|
+        row = normalize_row(r)
+        { id: row[:id], migration_name: row[:migration_name], batch: row[:batch] }
+      end
     end
 
     def next_batch_number
-      result = @db.fetch_one("SELECT MAX(batch) as max_batch FROM #{TRACKING_TABLE} WHERE passed = 1")
-      (result && result[:max_batch] ? result[:max_batch].to_i : 0) + 1
+      result = normalize_row(@db.fetch_one("SELECT MAX(batch) as max_batch FROM #{TRACKING_TABLE} WHERE passed = 1"))
+      (result[:max_batch] ? result[:max_batch].to_i : 0) + 1
     end
 
     def pending_migrations
@@ -479,12 +494,22 @@ module Tina4
           next
         end
 
-        # Statement delimiter — only reached outside blocks/comments/strings.
+        # Statement delimiter — only reached outside blocks/comments/strings. A
+        # SET TERM directive switches the active terminator and is consumed
+        # (never emitted); any other completed statement is collected.
         if !delimiter.empty? && sql[i, dlen] == delimiter
+          i += dlen
           stmt = current.strip
-          statements << stmt unless stmt.empty?
           current = +""
-          i += dlen
+          unless stmt.empty?
+            new_term = parse_set_term(stmt)
+            if new_term
+              delimiter = new_term
+              dlen = delimiter.length
+            else
+              statements << stmt
+            end
+          end
           next
         end
 
@@ -492,11 +517,30 @@ module Tina4
         i += 1
       end
 
+      # Trailing statement (may not end with a delimiter). A trailing SET TERM
+      # directive is a no-op — consume it, don't emit it.
       stmt = current.strip
-      statements << stmt unless stmt.empty?
+      statements << stmt unless stmt.empty? || parse_set_term(stmt)
       statements
     end
 
+    # Return the new terminator from a `SET TERM <new> <current>` directive, or
+    # nil when +statement+ is not a SET TERM directive.
+    #
+    # SET TERM is a script-level directive (recognised by isql and other
+    # InterBase/Firebird tooling, not run by the engine) that changes the
+    # terminator separating statements. Recognising it lets a statement whose own
+    # body contains the default ";" terminator — a trigger, stored procedure or
+    # EXECUTE BLOCK — be kept intact rather than split on those inner ";". The
+    # terminator may be more than one character (e.g. "!!").
+    #
+    # @param statement [String] a single, already-stripped statement
+    # @return [String, nil] the new terminator, or nil when not a SET TERM directive
+    def parse_set_term(statement)
+      m = statement.strip.match(/\ASET\s+TERM\s+(\S+)\z/i)
+      m && m[1]
+    end
+
     def execute_sql_file(file)
       sql = File.read(file)
       statements = split_sql_statements(sql)

data/lib/tina4/version.rb

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

metadata

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