Rails Ruby on Rails

Comments and changes to this ticket

• You flagged this item as spam.
  

  Roger May 23rd, 2008 @ 11:01 PM

  to test this patch apply, then add a syntax error to a view, and go to that view. It should have a red filename now [dark red].

• You flagged this item as spam.
  

  Roger May 23rd, 2008 @ 11:35 PM

  This version of the patch [v2] now adds coloration for the line on which the error occurred.

  • better_looking_error_message2.diff 2.7 KB

• You flagged this item as spam.
  

  Roger May 23rd, 2008 @ 11:40 PM

  v3 takes out an extra line I had added in there by mistake

  • better_looking_error_message3.diff 3.7 KB

• You flagged this item as spam.
  

  Roger May 24th, 2008 @ 12:20 AM

  v4 combine all commits into a single commit, add better colors. This is a nice patch for being able to instantly see which line your code erred on.

  • better_looking_error_message4.diff 1.6 KB

• You flagged this item as spam.
  

  Adam S May 24th, 2008 @ 01:31 PM

  +1

  Errors can definitely be improved in Rails (see mod_rails errors for an example), these changes don't go a long way, but also don't hurt.

• 

  Pratik June 17th, 2008 @ 05:24 PM

  • Assigned user set to “Pratik”
  • State changed from “new” to “incomplete”
  • Milestone cleared.

  I tried it and it was very difficult to see the red color. I think you should discuss this change in rails core mailing list. There might be other similar places where we can improve error reporting.

• You flagged this item as spam.
  

  Roger June 20th, 2008 @ 07:22 PM

  I almost wonder if it wouldn't be even more helpful to include more with the error messages. Like an inspection of the controller's local variables. My thought is to [eventually] post some prototype error pages so we can decide on a reasonable color scheme. mod_rails style might work, too. They're nice.

  Thanks.

  -R

• You flagged this item as spam.
  

  Roger January 11th, 2009 @ 05:39 AM

  • Tag set to “edge, patch”

  At least with MRI, it appears possible to snapshot local variables and parameters as to what they were passed in--might make for a nicer output.

• 

  Prem Sichanugrist (sikachu) January 22nd, 2010 @ 08:52 AM

  • State changed from “incomplete” to “stale”