Merge pull request #822 from rails-sqlserver/enable-frozen-strings-part-2

Enable frozen strings - part 2

Author: wpolicarpo

CHANGELOG.md

@@ -16,6 +16,7 @@
 - [#783](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/783) Update test matrix
 - [#820](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/820) Enable frozen strings for tests
 - [#821](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/821) Enable frozen strings - part 1
+- [#822](https://github.com/rails-sqlserver/activerecord-sqlserver-adapter/pull/822) Enable frozen strings - part 2
 
 #### Added
 

lib/active_record/connection_adapters/sqlserver/type/big_integer.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer
       module Type
         class BigInteger < Integer
 
           def sqlserver_type
-            'bigint'.freeze
+            "bigint"
           end
 
         end

lib/active_record/connection_adapters/sqlserver/type/binary.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer
@@ -9,8 +11,9 @@ def type
           end
 
           def sqlserver_type
-            'binary'.tap do |type|
-              type << "(#{limit})" if limit
+            "binary".yield_self do |type|
+              type += "(#{limit})" if limit
+              type
             end
           end
 

lib/active_record/connection_adapters/sqlserver/type/boolean.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer
       module Type
         class Boolean < ActiveRecord::Type::Boolean
 
           def sqlserver_type
-            'bit'.freeze
+            "bit"
           end
 
         end

lib/active_record/connection_adapters/sqlserver/type/char.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer
@@ -15,8 +17,9 @@ def serialize(value)
           end
 
           def sqlserver_type
-            'char'.tap do |type|
-              type << "(#{limit})" if limit
+            "char".yield_self do |type|
+              type += "(#{limit})" if limit
+              type
             end
           end
 

lib/active_record/connection_adapters/sqlserver/type/data.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer

lib/active_record/connection_adapters/sqlserver/type/date.rb

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer
       module Type
         class Date < ActiveRecord::Type::Date
 
           def sqlserver_type
-            'date'.freeze
+            'date'
           end
 
           def serialize(value)

lib/active_record/connection_adapters/sqlserver/type/datetime.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer
@@ -7,16 +9,15 @@ class DateTime < ActiveRecord::Type::DateTime
           include TimeValueFractional
 
           def sqlserver_type
-            'datetime'.freeze
+            "datetime"
           end
 
           def serialize(value)
             value = super
             return value unless value.acts_like?(:time)
-            datetime = value.to_s(:_sqlserver_datetime).tap do |v|
-              fraction = quote_fractional(value)
-              v << ".#{fraction}"
-            end
+            
+            datetime = "#{value.to_s(:_sqlserver_datetime)}.#{quote_fractional(value)}"
+
             Data.new datetime, self
           end
 
@@ -43,7 +44,7 @@ def fast_string_to_time(string)
           end
 
           def fast_string_to_time_format
-            "#{::Time::DATE_FORMATS[:_sqlserver_datetime]}.%N".freeze
+            "#{::Time::DATE_FORMATS[:_sqlserver_datetime]}.%N"
           end
         end
       end

lib/active_record/connection_adapters/sqlserver/type/datetime2.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer

lib/active_record/connection_adapters/sqlserver/type/datetimeoffset.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ActiveRecord
   module ConnectionAdapters
     module SQLServer