Skip to content

Commit 60f70e7

Browse files
committed
Make default_timezone a module instance variable
Follow rails/rails@c6e4dbe.
1 parent d998c49 commit 60f70e7

File tree

7 files changed

+18
-18
lines changed

7 files changed

+18
-18
lines changed

lib/active_record/connection_adapters/oracle_enhanced/jdbc_connection.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -145,9 +145,9 @@ def new_connection(config)
145145
end
146146

147147
# Set session time zone to current time zone
148-
if ActiveRecord::Base.default_timezone == :local
148+
if ActiveRecord.default_timezone == :local
149149
@raw_connection.setSessionTimeZone(time_zone)
150-
elsif ActiveRecord::Base.default_timezone == :utc
150+
elsif ActiveRecord.default_timezone == :utc
151151
@raw_connection.setSessionTimeZone("UTC")
152152
end
153153

@@ -498,13 +498,13 @@ def get_ruby_value_from_result_set(rset, i, type_name, get_lob_value = true)
498498
if dt = rset.getDATE(i)
499499
d = dt.dateValue
500500
t = dt.timeValue
501-
Time.send(Base.default_timezone, d.year + 1900, d.month + 1, d.date, t.hours, t.minutes, t.seconds)
501+
Time.send(ActiveRecord.default_timezone, d.year + 1900, d.month + 1, d.date, t.hours, t.minutes, t.seconds)
502502
else
503503
nil
504504
end
505505
when :TIMESTAMP, :TIMESTAMPTZ, :TIMESTAMPLTZ, :"TIMESTAMP WITH TIME ZONE", :"TIMESTAMP WITH LOCAL TIME ZONE"
506506
ts = rset.getTimestamp(i)
507-
ts && Time.send(Base.default_timezone, ts.year + 1900, ts.month + 1, ts.date, ts.hours, ts.minutes, ts.seconds,
507+
ts && Time.send(ActiveRecord.default_timezone, ts.year + 1900, ts.month + 1, ts.date, ts.hours, ts.minutes, ts.seconds,
508508
ts.nanos / 1000)
509509
when :CLOB
510510
get_lob_value ? lob_to_ruby_value(rset.getClob(i)) : rset.getClob(i)

lib/active_record/connection_adapters/oracle_enhanced/oci_connection.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -289,9 +289,9 @@ def create_time_with_default_timezone(value)
289289
end
290290
# code from Time.time_with_datetime_fallback
291291
begin
292-
Time.send(Base.default_timezone, year, month, day, hour, min, sec, usec)
292+
Time.send(ActiveRecord.default_timezone, year, month, day, hour, min, sec, usec)
293293
rescue
294-
offset = Base.default_timezone.to_sym == :local ? ::DateTime.local_offset : 0
294+
offset = ActiveRecord.default_timezone.to_sym == :local ? ::DateTime.local_offset : 0
295295
::DateTime.civil(year, month, day, hour, min, sec, offset)
296296
end
297297
end
@@ -343,9 +343,9 @@ def self.new_connection(config)
343343
conn.non_blocking = true if async
344344
conn.prefetch_rows = prefetch_rows
345345
conn.exec "alter session set cursor_sharing = #{cursor_sharing}" rescue nil if cursor_sharing
346-
if ActiveRecord::Base.default_timezone == :local
346+
if ActiveRecord.default_timezone == :local
347347
conn.exec "alter session set time_zone = '#{time_zone}'" unless time_zone.blank?
348-
elsif ActiveRecord::Base.default_timezone == :utc
348+
elsif ActiveRecord.default_timezone == :utc
349349
conn.exec "alter session set time_zone = '+00:00'"
350350
end
351351
conn.exec "alter session set current_schema = #{schema}" unless schema.blank?

lib/active_record/connection_adapters/oracle_enhanced/quoting.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ def _type_cast(value)
115115
case value
116116
when Type::OracleEnhanced::TimestampTz::Data, Type::OracleEnhanced::TimestampLtz::Data
117117
if value.acts_like?(:time)
118-
zone_conversion_method = ActiveRecord::Base.default_timezone == :utc ? :getutc : :getlocal
118+
zone_conversion_method = ActiveRecord.default_timezone == :utc ? :getutc : :getlocal
119119
value.respond_to?(zone_conversion_method) ? value.send(zone_conversion_method) : value
120120
else
121121
value

spec/active_record/connection_adapters/oracle_enhanced/connection_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ class ::Post < ActiveRecord::Base
232232
end
233233

234234
it "should respect default_timezone = :utc than time_zone setting" do
235-
# it expects that ActiveRecord::Base.default_timezone = :utc
235+
# it expects that ActiveRecord.default_timezone = :utc
236236
ActiveRecord::ConnectionAdapters::OracleEnhanced::Connection.create(CONNECTION_WITH_TIMEZONE_PARAMS)
237237
post = Post.create!
238238
created_at = post.created_at

spec/active_record/connection_adapters/oracle_enhanced/procedures_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ def raise_make_transaction_rollback
210210
expect(@employee.first_name).to eq("First")
211211
end
212212

213-
it "should not update record if nothing is changed and partial writes are enabled" do
213+
it "should not update record if nothing is changed and partial inserts are enabled" do
214214
TestEmployee.partial_writes = true
215215
@employee = TestEmployee.create(
216216
first_name: "First",

spec/active_record/connection_adapters/oracle_enhanced_data_types_spec.rb

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ class ::TestEmployee < ActiveRecord::Base
122122
end
123123

124124
after(:each) do
125-
ActiveRecord::Base.default_timezone = :utc
125+
ActiveRecord.default_timezone = :utc
126126
end
127127

128128
it "should assign ISO string to date column" do
@@ -170,7 +170,7 @@ class ::TestEmployee < ActiveRecord::Base
170170
end
171171

172172
it "should assign ISO time string to datetime column" do
173-
ActiveRecord::Base.default_timezone = :local
173+
ActiveRecord.default_timezone = :local
174174
@employee = TestEmployee.create(
175175
first_name: "First",
176176
last_name: "Last",
@@ -182,7 +182,7 @@ class ::TestEmployee < ActiveRecord::Base
182182
end
183183

184184
it "should assign NLS time string to datetime column" do
185-
ActiveRecord::Base.default_timezone = :local
185+
ActiveRecord.default_timezone = :local
186186
@employee = TestEmployee.create(
187187
first_name: "First",
188188
last_name: "Last",
@@ -205,7 +205,7 @@ class ::TestEmployee < ActiveRecord::Base
205205
end
206206

207207
it "should assign ISO date string to datetime column" do
208-
ActiveRecord::Base.default_timezone = :local
208+
ActiveRecord.default_timezone = :local
209209
@employee = TestEmployee.create(
210210
first_name: "First",
211211
last_name: "Last",
@@ -217,7 +217,7 @@ class ::TestEmployee < ActiveRecord::Base
217217
end
218218

219219
it "should assign NLS date string to datetime column" do
220-
ActiveRecord::Base.default_timezone = :local
220+
ActiveRecord.default_timezone = :local
221221
@employee = TestEmployee.create(
222222
first_name: "First",
223223
last_name: "Last",

spec/active_record/oracle_enhanced/type/timestamp_spec.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
include SchemaSpecHelper
55

66
before(:all) do
7-
ActiveRecord::Base.default_timezone = :local
7+
ActiveRecord.default_timezone = :local
88
ActiveRecord::Base.establish_connection(CONNECTION_WITH_TIMEZONE_PARAMS)
99
@conn = ActiveRecord::Base.connection
1010
schema_define do
@@ -28,7 +28,7 @@
2828

2929
after(:all) do
3030
@conn.drop_table :test_employees, if_exists: true
31-
ActiveRecord::Base.default_timezone = :utc
31+
ActiveRecord.default_timezone = :utc
3232
end
3333

3434
describe "/ TIMESTAMP WITH TIME ZONE values from ActiveRecord model" do

0 commit comments

Comments
 (0)