new rule: "leavemodal" looks for LEAVE triggers with RETURN NO-APPLY
Thu, 2007-08-02 20:49 — jurjen
| Project: | Prolint Issue Tracker |
| Component: | Rules |
| Category: | feature request |
| Priority: | normal |
| Assigned: | jurjen |
| Status: | closed |
Jump to:
Description
Submitted by Tim Townsend
"return no-apply" in a leave trigger is bad because creates a modal UI, where the user isn't free to enter data in the order they choose. 99% of the time it can be fixed by just removing the line. You need to have code later (e.g. at save time) to validate the data anyway, since you can't guarantee that the userwill put focus in every field.
- Login to post comments
Comments
#1
code submitted by Tim, committed to the Subversion repository as revision 332. Will be in the next Prolint release, but you can already check it out from Subversion if you want it now.
#2