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

Changeset 1220

Show
Ignore:
Timestamp:
04/19/05 05:24:50 (3 years ago)
Author:
david
Message:

Fixed stray comma when using eager loading and ordering together from has_many associations #1143

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/activerecord/CHANGELOG

    r1216 r1220  
     1*SVN* 
     2 
     3* Fixed stray comma when using eager loading and ordering together from has_many associations #1143 
     4 
     5 
    16*1.10.0* (19th April, 2005) 
    27 
  • trunk/activerecord/lib/active_record/associations/has_many_association.rb

    r1210 r1220  
    6868        else 
    6969          options[:conditions] = @finder_sql + (options[:conditions] ? " AND #{options[:conditions]}" : "") 
    70           options[:order] = options[:order] ? "#{options[:order]}, #{@options[:order]}" : @options[:order] 
     70 
     71          if options[:order] && @options[:order] 
     72            options[:order] = "#{options[:order]}, #{@options[:order]}" 
     73          elsif @options[:order] 
     74            options[:order] = @options[:order] 
     75          end 
     76 
    7177          @association_class.find(args.size == 1 ? args.first : args, options) 
    7278        end 
  • trunk/activerecord/test/associations_go_eager_test.rb

    r1201 r1220  
    2626 
    2727  def test_loading_from_an_association 
    28     posts = @david.posts.find(:all, :include => :comments
     28    posts = @david.posts.find(:all, :include => :comments, :order => "posts.id DESC"
    2929    assert_equal 2, posts.first.comments.size 
    3030  end 
  • trunk/activerecord/test/fixtures/post.rb

    r1132 r1220  
    11class Post < ActiveRecord::Base 
    22  belongs_to :author 
    3   has_many   :comments 
     3  has_many   :comments, :order => "body" 
    44  has_and_belongs_to_many :categories 
    55end