Closed Bug 14626 Opened 21 years ago Closed 20 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: 20 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.