Ruby on Rails

Comments and changes to this ticket

• 

  nerdrew October 23rd, 2008 @ 04:22 PM

  test

• 

  nerdrew June 7th, 2008 @ 01:11 AM

  I can't attach files without an error, so I've uploaded it here:

  http://www.nerdputer.com/test_ra...

• 

  Greg Campbell June 7th, 2008 @ 01:52 AM

  The same is true for a has_one association with a condition.

  I have:

  class Contact < ActiveRecord::Base
    has_one :primary_email, :class_name => 'ContactEmail', :conditions => {:primary_email => true}
  end
  
  Contact.find(:first).primary_email #=> #<ContactEmail ... >
  Contact.find(:first, :include => :primary_email) #=> error
  

  The error in question is ActiveRecord::StatementInvalid: Mysql::Error: Unknown column 'contacts.primary_email' in 'where clause': SELECT `contact_emails`.* FROM `contact_emails` WHERE (`contact_emails`.contact_id IN (1) AND (`contacts`.`primary_email` = 1))

• 

  Joshua Peek September 18th, 2008 @ 03:49 PM

  • → Tag cleared.
  • → State changed from “new” to “stale”
• 

  nerdrew September 18th, 2008 @ 08:03 PM

  Why is this stale now? I'm still seeing the same bug (I think) in edge. It could be that I'm using the conditions hash wrong, but I don't know how else I'd do it. My understanding is that the conditions hash of a has_many should be properties on the associated model. So, in the example above, the {:end_date => nil} should apply to the employment model, not the company model.

• 

  Tarmo Tänav September 18th, 2008 @ 09:24 PM

  stale only means that the bug is not moving anywhere, either no patch for a long time or the patch no longer applies.

• 

  nerdrew September 19th, 2008 @ 03:21 AM

  Can someone test this for me? I have some custom ActiveRecord improvements (nested condition hashes and includes), but it seems to work for me.

  @@@ruby

  class <<ActiveRecord::Base

  private

  def find_associated_records(ids, reflection, preload_options)

  options = reflection.options
  table_name = reflection.klass.quoted_table_name
  
  if !options[:conditions].is_a?(Hash) && !preload_options[:conditions].is_a?(Hash)
    if interface = reflection.options[:as]
      conditions = "#{reflection.klass.quoted_table_name}.#{connection.quote_column_name "#{interface}_id"} #{in_or_equals_for_ids(ids)} and #{reflection.klass.quoted_table_name}.#{connection.quote_column_name "#{interface}_type"} = '#{self.base_class.sti_name}'"
    else
      foreign_key = reflection.primary_key_name
      conditions = "#{reflection.klass.quoted_table_name}.#{foreign_key} #{in_or_equals_for_ids(ids)}"
    end
  
    conditions << append_conditions(options, preload_options)
    conditions = [conditions, ids]
  else
    conditions = if interface = reflection.options[:as]
      {"#{interface}_id" => ids, "#{interface}_type" => self.base_class.sti_name}
    else
      {reflection.primary_key_name => ids}
    end
  
    conditions.merge!(options[:conditions]) if options[:conditions]
    conditions.merge!(preload_options[:conditions]) if preload_options[:conditions]
  end
  
  reflection.klass.find(:all,
                        :select => (preload_options[:select] || options[:select] || "#{table_name}.*"),
                        :include => preload_options[:include] || options[:include],
                        :conditions => conditions,
                        :joins => options[:joins],
                        :group => preload_options[:group] || options[:group],
                        :order => preload_options[:order] || options[:order])
  
  

  end

  end #adding to ActiveRecord::Base

• 

  nerdrew September 19th, 2008 @ 03:22 AM

  I wish there was a way to edit comments...

  Here it is again.

  
  
  class <<ActiveRecord::Base
  
    private
  
    def find_associated_records(ids, reflection, preload_options)
      options = reflection.options
      table_name = reflection.klass.quoted_table_name
  
      if !options[:conditions].is_a?(Hash) && !preload_options[:conditions].is_a?(Hash)
        if interface = reflection.options[:as]
          conditions = "#{reflection.klass.quoted_table_name}.#{connection.quote_column_name "#{interface}_id"} #{in_or_equals_for_ids(ids)} and #{reflection.klass.quoted_table_name}.#{connection.quote_column_name "#{interface}_type"} = '#{self.base_class.sti_name}'"
        else
          foreign_key = reflection.primary_key_name
          conditions = "#{reflection.klass.quoted_table_name}.#{foreign_key} #{in_or_equals_for_ids(ids)}"
        end
  
        conditions << append_conditions(options, preload_options)
        conditions = [conditions, ids]
      else
        conditions = if interface = reflection.options[:as]
          {"#{interface}_id" => ids, "#{interface}_type" => self.base_class.sti_name}
        else
          {reflection.primary_key_name => ids}
        end
  
        conditions.merge!(options[:conditions]) if options[:conditions]
        conditions.merge!(preload_options[:conditions]) if preload_options[:conditions]
      end
  
      reflection.klass.find(:all,
                            :select => (preload_options[:select] || options[:select] || "#{table_name}.*"),
                            :include => preload_options[:include] || options[:include],
                            :conditions => conditions,
                            :joins => options[:joins],
                            :group => preload_options[:group] || options[:group],
                            :order => preload_options[:order] || options[:order])
    end
  
  end #adding to ActiveRecord::Base