Edgewall Software
Modify

Opened 15 years ago

Closed 15 years ago

#413 closed defect (fixed)

Use warnings instead of error when posting forms with mistakes

Reported by: osimons Owned by: osimons
Priority: major Milestone: 0.6
Component: General Version: dev
Keywords: Cc:
Operating System: BSD

Description

Errors with posted forms should note warnings and re-populate forms with errors instead of throwing TracError and hope that the user can hit 'back' with data intact. This should also support multiple warnings as feedback for a single POST instead of try-and-try-again with error pages.

Attachments (1)

t413-use_warnings-r656.diff (14.8 KB) - added by osimons 15 years ago.
Use warnings instead of TracError for editing the build configuration with recipe.

Download all attachments as: .zip

Change History (3)

Changed 15 years ago by osimons

Use warnings instead of TracError for editing the build configuration with recipe.

comment:1 Changed 15 years ago by osimons

attachment:t413-use_warnings-r656.diff uses warnings to notify user of any errors, and repopulates the form for continued edit - including faulty fields. It provides a much nicer user experience for the often error-prone work of tweaking configuration.

Patch turned out quite large due to the abundance of tests that got affected by the change... Some ~10 tests are updated to handle and check warnings and notices.

comment:2 Changed 15 years ago by osimons

  • Resolution set to fixed
  • Status changed from new to closed

Patch updated to latest trunk, and committed in [705].

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain osimons.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.