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)
Change History (3)
Changed 15 years ago by osimons
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].
Use warnings instead of TracError for editing the build configuration with recipe.