Closed
Bug 337139
Opened 20 years ago
Closed 20 years ago
select1, hooking up a send to ev:xforms-select only works the first time
Categories
(Core Graveyard :: XForms, defect, P1)
Core Graveyard
XForms
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: allan, Assigned: aaronr)
References
()
Details
(Keywords: fixed1.8.0.5, fixed1.8.1)
Attachments
(2 files)
If I hook up a send to an xforms-select on a xforms:select1, it only triggers the submission the first time. On subsequent selects I get this error:
Error: elm.parentNode has no properties
Source File: chrome://xforms/content/select1.xml
Line: 942
| Reporter | ||
Comment 1•20 years ago
|
||
| Reporter | ||
Comment 2•20 years ago
|
||
Possibly a dupe of 332292?
| Reporter | ||
Comment 3•20 years ago
|
||
Aparently the selected element does not always have a parent node? Checking for that makes the dispatchEvent() work.
It does not fix the testcase 100%, because the xforms-select event is sent before the node value is changed, so the result gotten from the server is "lagging". But it might be correct event sequencing? Dunno.
Attachment #223042 -
Flags: review?(doronr)
Comment 4•20 years ago
|
||
Comment on attachment 223042 [details] [diff] [review]
Crude fix
ugh, you really should figure out why elm has no parentNode - you probably have a reference to an element no longer in the DOM.
Check is ok for a temp fix though (0.6)
Attachment #223042 -
Flags: review?(doronr) → review+
| Reporter | ||
Comment 5•20 years ago
|
||
Comment on attachment 223042 [details] [diff] [review]
Crude fix
something's wrong alright, but temp fix. for 0.6, please.
Attachment #223042 -
Flags: review?(Olli.Pettay)
Comment 6•20 years ago
|
||
Comment on attachment 223042 [details] [diff] [review]
Crude fix
Could you put the whole
method inside CDATA.
& is just too ugly,
& is much better
With that r=me
Attachment #223042 -
Flags: review?(Olli.Pettay) → review+
| Reporter | ||
Comment 7•20 years ago
|
||
Fixed on trunk w/smaug's comment fixed
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch
| Reporter | ||
Comment 8•20 years ago
|
||
Filed follow up bug 339062.
| Reporter | ||
Updated•20 years ago
|
Keywords: fixed1.8.1
| Reporter | ||
Updated•20 years ago
|
Keywords: fixed1.8.0.5
| Reporter | ||
Updated•20 years ago
|
Whiteboard: xf-to-branch
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•