Opened 11 years ago

Closed 11 years ago

#6310 closed defect (fixed)

dojo.attr() on functions doesn't erase old connect

Reported by: bill Owned by: alex
Priority: high Milestone: 1.1
Component: General Version: 1.0
Keywords: Cc:
Blocked By: Blocking:

Description

dojo.attr() connects the new function but doesn't disconnect the old function. See attr.html attached test case. Tested on IE6, FF3/mac and safari3.1 (mac).

This was originally filed under #6271 but I think it's a separate issue.

Given this bug and #6300 I suggest just removing the ability of dojo.attr() to handle functions altogether.

Attachments (1)

attr.html​ (876 bytes) - added by bill 11 years ago.

Download all attachments as: .zip

Change History (4)

Changed 11 years ago by bill

Attachment: attr.html​ added

comment:1 Changed 11 years ago by Adam Peller

Milestone: → 1.1
Priority: normal → high

comment:2 Changed 11 years ago by alex

Status: new → assigned

removing function setting in attr isn't useful or sane. Fixing it is.

comment:3 Changed 11 years ago by alex

Resolution: → fixed
Status: assigned → closed

(In [13207]) update to ensure that re-setting an event handler via dojo.attr() correctly handles dis-connection. misc size reductions. The case where dojo.connect() sets a handler and then dojo.attr() sets one will NOT be handled. Updates to tests. Fixes #6310. Fixes #6234. !strict

Note: See TracTickets for help on using tickets.