177055 - Editing Form select options with the Selection List Properties dialog causes options to be deleted

Status: RESOLVED FIXED

(SeaMonkey :: Composer, defect)

Description

[Jason Jenkins]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20021019
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20021019

Editing a form with select options with the composer tool "Selection List
Properties" causes options to be removed.  For example I have a form with a
selection list that contains the options A, B, C, and D. If I use the dialog
Selection List Properties and add the options E and F, then click ok, then look
at the source code I would find that the only options listed now are A,C,E.  It
appears to remove every other option.  On another note if I just open the
Selection List Properties and click cancel or the x to close the window it does
not exhibit this mysterious behavior.

Reproducible: Always

Steps to Reproduce:
1.Edit a form that already contains select options of A,B,C,D and then add E and F.
2.Click ok

Actual Results:  
Options B,D, and F have been deleted.  It appears to remove every other option

Expected Results:  
For it to have added options E and F to the list; so I would then have the
options of A,B,C,D,E, and F.

Comment 1

[Jason Jenkins]

Attachment: A test file containing a form with select options of A,B,C, and D

When I edit this file in composer and use the Selection List Properties dialog
to add or remove options from the list it will delete every other option.

Comment 2

[Kathleen :Brade]

reassign to new placeholder

Comment 3

[[e-mail]]

Actually what I see with the latest builds is that the options get reversed :-(

Comment 4

[[e-mail]]

Attachment: Fix an uninitialized variable

Comment 5

[Kathleen :Brade]

Comment on attachment 115014 [details] [diff] [review]
Fix an uninitialized variable

r=brade

Comment 6

[[e-mail]]

Comment on attachment 115014 [details] [diff] [review]
Fix an uninitialized variable

simple patch that could be slotted in whenever you choose...

Comment 7

[Asa Dotzler [:asa]]

Comment on attachment 115014 [details] [diff] [review]
Fix an uninitialized variable

a=asa (on behalf of drivers) for checkin to 1.3

Comment 8

[[e-mail]]

Fix checked in to trunk.

Comment 9

[[e-mail]]

Fix checked in.