This project is archived and is in readonly mode.
Add :delegating option to belongs_to and has_one assoc macro
Working on a few projects I noticed that the targets for
delegate in my models
are usually other associations.
Just for kicks I tried to confirm that assumption
statistically.
Here is what I came up with
Results present the code samples found on GitHub and basically
say that
I'm not alone in that league and more than 80% of use cases of
delegate inside models is delegating to
association.
Attached patch makes it possible to write
class Invoice < ActiveRecord::Base
belongs_to :client, :delegating => [ :street, :city, :name ]
end
instead of
class Invoice < ActiveRecord::Base
belongs_to :client
delegate :street, :city, :name, :to => :client
end
-
Rohit Arondekar- Tag changed from activerecord associations, 3.x, patch to 3.x, activerecord, associations, patch
-
-
Nathan Kleyn+1, this seems like a really useful addition. I've had this same style of code littering my models for too long. =]
-
Santiago Pastorino- State changed from new to open
This issue has been automatically marked as stale because it has not been commented on for at least three months.
The resources of the Rails core team are limited, and so we are asking for your help. If you can still reproduce this error on the 3-0-stable branch or on master, please reply with all of the information you have about it and add "[state:open]" to your comment. This will reopen the ticket for review. Likewise, if you feel that this is a very important feature for Rails to include, please reply with your explanation so we can consider it.
Thank you for all your contributions, and we hope you will understand this step to focus our efforts where they are most helpful.
-
Santiago Pastorino
- State changed from open to stale