On May 23, 12:59 am, Marnen Laibow-Koser <[email protected]> wrote:

> > module Enumerable
> >   def my_compress
> >    ....
>
> > ... but when I visit the web page in question I get "No method
> > 'my_compress' for Array ..."
>
> > I'm probably doing something fundamentally wrong - what could/should I
> > be doing?
>
> You should be putting your model logic in the model, not in the
> controller.
>

on top of that, by putting it inside the controller like that you've
created a new module called LookupController::Enumerable rather than
extending Enumerable. If I were you I'd keep extensions to core
classes somewhere in lib.

Fred

>
>
> > thanks
>
> Best,
> --
> Marnen Laibow-Koserhttp://www.marnen.org
> [email protected]
> --
> Posted viahttp://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 
> athttp://groups.google.com/group/rubyonrails-talk?hl=en.

-- 
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