Closed Bug 14626 Opened 26 years ago Closed 26 years ago

rules API change needed to include "handled"

Categories

(Core :: DOM: Editor, defect, P3)

Product:
Core
Component:
DOM: Editor
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: buster, Assigned: mozeditor)

Details

The closing half of the rules sandwich (DidDoAction()) only gets called when WillDoAction() didn't cancel the operation. However, the cancel boolena returned from WillDoAction() has an overloaded meaning. It could mean we decided we can't do anything, and want to cancel. It could mean we decided we had to handle the even tentirely in the rules code and want to cancel the default processing. But in one of those cases you want to call DidDoAction() and in one you don't. Not enough information. I would like to change the rules api to have WillDoAction return an additional boolean: handled. this would be distinct from cancel in the obvious way.
Status: NEW → ASSIGNED
Target Milestone: M12
accepting bug / m12
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
fixed. i guess i'll have to verify as well.
Status: RESOLVED → VERIFIED
v'fied
You need to log in before you can comment on or make changes to this bug.