Prefer activerecord_class.default_timezone when available

Schema#default_timezone read the global ActiveRecord.default_timezone
(or ActiveRecord::Base.default_timezone for AR < 7.0) unconditionally,
ignoring the configured @connection.activerecord_class. If a caller
wired plsql up with an ActiveRecord subclass that overrides
default_timezone, the override was silently dropped.

Check the connection's activerecord_class first and call
default_timezone on it when the class responds to it. AR 5.x – 7.x
expose default_timezone on ActiveRecord::Base (and therefore on all
subclasses), which honors any subclass override. AR 8.0 removed the
class-level accessor, so in that case fall back to the module-level
ActiveRecord.default_timezone.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: yahonda

lib/plsql/schema.rb

@@ -99,9 +99,12 @@ def default_timezone
         @original_schema.default_timezone
       else
         @default_timezone ||
-          # Use ActiveRecord default_timezone when ActiveRecord connection is used
-          (@connection && @connection.activerecord_class &&
-            (ActiveRecord.respond_to?(:default_timezone) ? ActiveRecord.default_timezone : ActiveRecord::Base.default_timezone)) ||
+          # Use ActiveRecord default_timezone when ActiveRecord connection is used,
+          # preferring the connection's activerecord_class so a subclass override
+          # (available in AR < 8.0) is honored before falling back to the
+          # module-level accessor (AR 7.0+; the only one in AR 8.0+).
+          (@connection && (ar_class = @connection.activerecord_class) &&
+            (ar_class.respond_to?(:default_timezone) ? ar_class.default_timezone : ActiveRecord.default_timezone)) ||
           # default to local timezone
           :local
       end