Sharagoz -- wrote:
>>> If it is literally something as simple as MyArray.all I believe there
>>> is nothing wrong with calling the model direct from the view.

But you are wrong.  The view should never, ever, ever touch the 
database.

>>>
>> 
>> It's dirty, horrible, bad form, breaks the separation of layers...
> Beware of the MVC police Colin, this suggestion will certanly not get 
> good housekeeping seal of approval :D
> 
> I agree through. I'm not gonna add a before filter just to set 
> MyArray.all into a class variable. I'd rather call it directly and claim 
> to be pragmatic.

Claim all you like.  The fact is that in MVC architecture, database 
queries don't belong in the view.  A before_filter is the proper place 
for this.

Best,
--
Marnen Laibow-Koser
http://www.marnen.org
[email protected]
-- 
Posted via http://www.ruby-forum.com/.

-- 
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Talk" 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-talk?hl=en.

Reply via email to