Ticket #610 (closed defect: fixed)

Opened 7 months ago

Last modified 3 months ago

[PATCH] Exceptions from parsing a page are suppressed in development mode

Reported by: jonleighton Assigned to: seancribbs
Priority: minor Milestone: 0.6.5-Chisel
Component: core Keywords:
Cc: j@jonathanleighton.com

Description

While writing an extension I created a tag which caused an exception. Unfortunately the PageContext class rescues from exceptions unless RAILS_ENV == "testing", so I couldn't get much information on the problem. My patch allows exceptions to happen unless RAILS_ENV == "production".

I've deleted some tests from the test directory and made sure they are implemented as specs. Hope that's okay (I assume the general idea is to move towards RSpec).

Attachments

radiant_page_context_exceptions_in_development.diff (4.1 kB) - added by jonleighton on 02/10/08 16:52:45.

Change History

02/10/08 16:52:45 changed by jonleighton

  • attachment radiant_page_context_exceptions_in_development.diff added.

02/23/08 12:09:34 changed by seancribbs

  • owner set to seancribbs.
  • status changed from new to assigned.
  • milestone set to 0.6.5-Chisel.

02/23/08 12:15:19 changed by seancribbs

  • status changed from assigned to closed.
  • resolution set to fixed.

Resolved in [757]. Thank you!

05/29/08 17:30:17 changed by airforce1

thanks favorite air force ones