Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #1160 (closed 1: fixed)

Opened 3 years ago

Last modified 2 years ago

Patty

Reported by: wayne_hearn@dell.com Assigned to: David
Priority: 1 Milestone: 1
Component: 1 Version: 1
Severity: 1 Keywords: Patty
Cc: Patty

Description

When order_by is assigned a nil value the existing code throws this error: NoMethodError (undefined method `[]' for nil:NilClass):

Here is our fix: 

def add_limit_with_offset!(sql, limit, offset)
     order_by = sql.include?("ORDER BY") ? get_order_by(sql.sub(/.*ORDER\sBY./, "")) : nil
     sqlString = "SELECT * FROM ( SELECT TOP #{limit} * FROM ( SELECT TOP #{limit + offset}"
     if order_by.nil?
          sql.gsub!(/SELECT/i, "#{sqlString}")<<" ) AS tmp1 ) AS tmp2"
     else
          sql.gsub!(/SELECT/i, "#{sqlString}")<<" ) AS tmp1 ORDER BY #{order_by[1]} ) AS tmp2 ORDER BY #{order_by[0]}"
     end
end

Attachments

sqlserver_adapter.rb (12.2 kB) - added by wayne_hearn@dell.com on 04/20/05 19:36:50.
sqlserver_adapter

Change History

04/20/05 19:36:50 changed by wayne_hearn@dell.com

  • attachment sqlserver_adapter.rb added.

sqlserver_adapter

04/21/05 01:28:10 changed by anonymous

  • summary changed from sqlserver_adpater add_limit_with_offset! breaks with nil order by to [PATCH] sqlserver_adpater add_limit_with_offset! breaks with nil order by.

06/03/05 08:14:21 changed by dkulis@latnet.lv

Unfortunately the pacth breaks paginator functionality - all pages start with the 1st record...

Probably in case ORDER BY is ommited, it is possible to order by all columns?

06/17/05 11:16:59 changed by mail@robertbrook.com

Unhappily I can confirm this behaviour after installing the patch. All pages start with the first record.

07/01/05 17:19:34 changed by david

  • status changed from new to closed.
  • resolution set to fixed.

Fixed by #1569

08/17/05 09:42:40 changed by anonymous

Add/Change #1160 ([PATCH] sqlserver_adpater add_limit_with_offset! breaks with nil order by

07/23/06 11:28:32 changed by anonymous

www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www

08/02/06 13:14:16 changed by anonymous

5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j 5i5j