Re: [Feature request] has_many option to generate additionnal has_many for each given scope.

[Julien Grillot]

Here is the PR: https://github.com/rails/rails/pull/28626

Does somebody have a thought on it?

[Joshua Flanagan]

The "before" version really doesn't seem that painful. Compared with the cost to the rails team to maintain the syntax, and the cost in understandability to other developers that happen upon your code. DSLs are only helpful when they are well understood by everyone that has to read them - something like this would be very rarely used, so every dev that encounters it in the wild will have to look up what "compose" means, (or try to figure out where published_posts is defined if they dont know the trick is to look at the posts has_many definition) to try and figure out the magic.

I think you're better off being more explicit in these cases.

[Kevin Deisz]

Probably a better first step would be to move this into a gem and then if it garners wide enough adaption making the case then.

--
You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rubyonrails-core+unsubscribe@googlegroups.com.
To post to this group, send email to rubyonrails-core@googlegroups.com.
Visit this group at https://groups.google.com/group/rubyonrails-core.
For more options, visit https://groups.google.com/d/optout.

--

Kevin D. Deisz

Localytics Software Engineer