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