I understand the need behind your proposition but you want to insert
some behaviour for the view in the model.
This breaks the MVC separation.
I do a lot of the following:
if some_model.user
...
end
to check the presence of the association. Your proposition will break
this idiom which is IMHO heavily used in code.
Regards,
Pascal
Le 08.09.12 20:57, Kensodev a écrit :
This kind of response is precisely why I came here first and didn't
just invest the time and opened a pull request.
IMHO this should be supported in the frame work, and I will be more
then happy to work on it if you guys think it should go in.
I know Ruby can be used to override the relation.
I actually had a plan that the loading of the "Nil class" will be
automatically based on the relation name, but you can override it with
the :nil_class definition.
Anyway, let me know if you think it's viable.
On Friday, September 7, 2012 9:42:29 PM UTC+3, Aaron Patterson wrote:
On Fri, Sep 07, 2012 at 02:57:25PM -0300, Carlos Antonio da Silva
wrote:
> I believe you should be able to achieve the same just by
overriding the
> #user method in your class. I've commented on your last gist
example with
> an example code to make things more clear.
Agreed. We don't need any framework support for the null object
pattern. Just use Ruby.
--
Aaron Patterson
http://tenderlovemaking.com/
--
You received this message because you are subscribed to the Google
Groups "Ruby on Rails: Core" group.
To view this discussion on the web visit
https://groups.google.com/d/msg/rubyonrails-core/-/P9-vM1sUesMJ.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/rubyonrails-core?hl=en.
--
You received this message because you are subscribed to the Google Groups "Ruby on
Rails: Core" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/rubyonrails-core?hl=en.