Skip to content

Commit 9d8eaf0

Browse files
committed
Removed unecessary (at this time) method.
Added fix for new add_joins! arguments.
1 parent 94128bf commit 9d8eaf0

File tree

1 file changed

+8
-22
lines changed

1 file changed

+8
-22
lines changed

lib/active_record/connection_adapters/sqlserver_adapter.rb

Lines changed: 8 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,10 @@ def self.construct_finder_sql_for_association_limiting(options, join_dependency)
5959
is_distinct = !options[:joins].blank? || include_eager_conditions?(options) || include_eager_order?(options)
6060

6161
sql = "SELECT #{table_name}.#{connection.quote_column_name(primary_key)} FROM #{table_name} "
62-
62+
6363
if is_distinct
6464
sql << join_dependency.join_associations.collect(&:association_join).join
65-
add_joins!(sql, options, scope)
65+
add_joins!(sql, options[:joins], scope)
6666
end
6767

6868
add_conditions!(sql, options[:conditions], scope)
@@ -85,20 +85,6 @@ def self.construct_finder_sql_for_association_limiting(options, join_dependency)
8585
return sanitize_sql(sql)
8686
end
8787

88-
89-
# Fix SQL Server
90-
def self.add_order!(sql, order, scope = :auto)
91-
scope = scope(:find) if :auto == scope
92-
scoped_order = scope[:order] if scope
93-
94-
order = [order, scoped_order].delete_if(&:nil?).join(', ')
95-
included_fields = order.gsub(/\s(asc|desc)/i, '')
96-
97-
if order != ''
98-
sql.gsub!(/(.+?) FROM/, "\\1, #{included_fields} FROM")
99-
sql << " ORDER BY #{order}"
100-
end
101-
end # self.add_order
10288
end # class Base
10389

10490
module ConnectionAdapters
@@ -395,15 +381,15 @@ def insert_sql(sql, name = nil, pk = nil, id_value = nil, sequence_name = nil)
395381
end
396382

397383
def update_sql(sql, name = nil)
398-
autoCommiting = @connection["AutoCommit"]
384+
auto_commiting = @connection["AutoCommit"]
399385
begin
400-
begin_db_transaction if autoCommiting
386+
begin_db_transaction if auto_commiting
401387
execute(sql, name)
402-
affectedRows = select_value("SELECT @@ROWCOUNT AS AffectedRows")
403-
commit_db_transaction if autoCommiting
404-
affectedRows
388+
affected_rows = select_value("SELECT @@ROWCOUNT AS AffectedRows")
389+
commit_db_transaction if auto_commiting
390+
affected_rows
405391
rescue
406-
rollback_db_transaction if autoCommiting
392+
rollback_db_transaction if auto_commiting
407393
raise
408394
end
409395
end

0 commit comments

Comments
 (0)