Pål Bergström wrote: > I've made a simple cms. I have a table with page content. In the view I > make a call for a helper that get's the data: > > View: > > <%= content(the_id) %> > > In the application helper: > > def content(c) > @content = Pagecontent.find(c).content > end > > This works great. Any problems with this, like security?
Yeah, you're totally breaking MVC. The query should be in the controller. > Any other way > to do it this simple? Put the query in the controller. Assign it to a variable. Use the variable in the view. Just as simple. No helper necessary. No breaking MVC. 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 -~----------~----~----~----~------~----~------~--~---
