Merb::RenderMixin#set_status refers to non-existent RESPONSE_CODES constant
François Beausoleil
francois at teksol.info
Thu Dec 13 10:49:34 EST 2007
#set_status
http://merb.devjavu.com/browser/trunk/lib/merb/mixins/render.rb#L270
refers to a non-existent constant. I tried, while creating #359, to
make things OK, but I failed. I started by using STATUS_CODES instead,
then used #[] instead of calling a method named STATUS_CODES, then I
found out that during the call to #inherited
http://merb.devjavu.com/browser/trunk/lib/merb/exceptions.rb#L111, the
STATUS constant isn't defined (yet). I tried to add a #status method to
Base, but even that failed because the class didn't have the constant
during the call to #inherited.
So, I'm rather stuck. It's probably a good idea to make the exception
subclasses have a method, rather than a constant, to return the correct
status code value.
What's the consensus ? Should I start work on this ? What effect would
this have on the rest of the code ?
Bye !
François Beausoleil
http://blog.teksol.info/
More information about the Merb-devel
mailing list