Vitalii Khustochka's profile

Vitalii Khustochka commented at 5:11 AM
Looks like this happens because reorder_flag was removed. Named scope defined with except(:order).order(xxx) now has no difference with the one defined with ord...
Vitalii Khustochka commented at 3:34 AM

Typo in the description: "Instead there is a test named test_named_scope_overwrites_default but I think this name is incorrect ..."

test_named_scope_appends_def...
Vitalii Khustochka created the ticket at 3:30 AM
Method reorder that existed before was removed in master and deprecated in 3.0.3 (https://github.com/rails/rails/commit/e0b76d6151821527f16b3f163abde.... Instea...

- State changed from new to resolved
by Vitalii Khustochka at 7:37 AM
(from [b75fca9e57423eec81384ff3e35f38fd4f7fe47f]) Added reorder delegation for ActiveRecord::Base(to be able to overwrite the default_scope ordering in the name...

Added reorder delegation for ActiveRe... was committed by Vitalii Khustochka at 7:37 AM

Vitalii Khustochka commented at 12:31 PM
- Title changed from Impossible to overwrite the default scope ordering in the named scope to Impossible to overwrite the default scope ordering in the named scope using relation syntax
by Vitalii Khustochka at 12:28 PM
- Tag set to arel rails3, order, patch, scope
by Vitalii Khustochka at 12:23 PM

Thus I have added reorder to ActiveRecord::Base delegations. Also I have added the test.

Please review the patch and comment if I have done something the wrong ...

Vitalii Khustochka created the ticket at 12:15 PM

Assuming we have

  class DeveloperOrderedBySalary < ActiveRecord::Base
    self.table_name = 'developers'
    default_scope :order => 'salary DESC'
    scope :...