Open Bug 221851 Opened 21 years ago Updated 2 years ago

Javascript changes <select> selection on Reload

Categories

(Core :: DOM: Forms, defect, P5)

Product:
Core
Component:
DOM: Forms
Platform:
x86
All
Type:
defect

Tracking

()

People

(Reporter: coffeebreaks, Unassigned)

References

(Blocks 1 open bug,
URL
)

Details

Attachments

(1 file, 1 obsolete file)

Reduced testcase #2
34.11 KB, text/html
Details 
Note: It might be a Mozilla bug or a web developer bug. 

The problem happens at the aforementionned URL where you are supposed to book a
flight for RyanAir.
Select London Stansted on the 1st drop down list and whatever you want on the
second one, submit the form and then come back. You will see that the page is
initially displayed correctly and then something, probably javascript, modifies
the selection of the second drop-down. From now on, if you do next/back many
times (without submitting) you will see a different element being selected. 

Happens with mozilla 1.5 (Gecko/20030925), doesn't with Opera 6.03. Tested on
GNU/Linux. Reproduced by Kowaslky from #mozillazine on mozilla 1.5rc2, firebird
and Netscape 7.1.

Opera 6.03 on Linux and IE 6.0 SP1 with all patches on windows do not reproduce
the problem.

Disabling javscript fixes the problem.

So it is either a mozilla javascript bug, or there is malfunctionning javascript
in the page, tailored to Gecko. I will try to investigate the javascript issue
to see if it is coming from us.
Some more info.
When I meant disable javascript, I mean disable it after you've made your
drop-down selections (as javascript is involved for these selections to work).

I also noticed that if I select the first entry in the second drop-down,
currently Aarhus when you have selected London Stanfeld in the first one, the
bug is not reproduced. Kowalski on #mozillazine found the same thing.

I looked at the javascript in the page and there's nothing that makes me thing
that this code is problematic. I think it is a strange mozilla bug.
Reloading the page changes (increments?) the index of the 2nd selection box, if
it doesn´t point to the first destination.

You don´t need to submit, just select a Origin, and a destination.
It doesn´t change for the first entry of the all destination list, AARHUS,
but that entry isn´t served for every origin, as there are shorter destiantion
lists for most of the origins.

Selecting the 2nd on the list, the list position got incremented with each reload,
Aberdeen->Alghero->Altenburg->Ancona->Barcelona->Bergerac->
But seen starting from Frankfurt, it doesn´t increment, but changes wildly, but
reproducable. Also changing with higher destinations from Stanstead.

It is not changing, when you set the Origin box back to Origin.

Browsersniffing seems to be fine, and Mozilla gets the IE_stylesheet, as
provided explicitely for mozilla.

Browsersniffing: http://www.bookryanair.com/skylights/css/ua.js
Stylesheet: http://www.bookryanair.com/skylights/css/moab_IE.css

Excerpts from the file:
The list changing function gets called twice,
in body onload, and that seems to be working, as you shortly see the original
destination after reload, then to be replaced by another one.
This may be done by the onchange call in the 1st input box.

I´m not a js programmer, so I don´t see more than described here.
Hope I´ll learn how to use venkman on bugs like this one.

<body bgcolor=#ffffff onLoad="changeOrigDestList(document.SBform.sector1_o, 
document.SBform.sector1_d, 'Destination', true, 1, '', '' ); 
change_sector_2_fields( '', true );

Origin box:
<select name='sector1_o' size=1
onChange="changeOrigDestList(document.SBform.sector1_o,
document.SBform.sector1_d, 'Destination', false, 1)" class='large' width=175>

Destination box:			
<select name='sector1_d' size=1  class='large' width=175>
			<option value='XXX' selected>Destination
Summary: Javscript changes selection → Javascript changes selection on Reload
Attached file Reduced testcase #1 (obsolete) — 

    
    

    
  

      
      
      
      

        Attached file
          
        Reduced testcase #2
        — 
      

    


  
Reduced testcase #1 has a typo; use testcase #2 instead

        Attachment #133090 -
        Attachment is obsolete: true

    
    

    
  
The key function is:

function changeOrigDestList(origBox, destBox, labelDest, fromOnload,  
                            whichSector, labelOpenJaw, preferredDest)
475 {
476     if (!document.images)
477     {
478         return;
479     }
480 
481     var origIndex    = eval(origBox.selectedIndex)
482     var destIndex    = eval(destBox.selectedIndex)
483     var orig         = origBox.options[ origIndex ].value
484     var dest         = destBox.options[ destIndex ].value
485     var initDest    = 0
486 
487 
488     if (orig != "XXX")
489     {
490         orig = eval(orig)
491         destBox.length = 1
492         var newDest = new Option(labelDest)
493         destBox.options[0] = newDest
494         destBox.options[0].value = "XXX"
495 
496         for (var i = 0; i < orig.length-1; i++)
497         {
498             destBox.length += 1
499             destBox.options[destBox.length - 1] = new Option( 
                                eval('s' + orig[i]) + " (" + orig[i] + ")" )
500 
501             destBox.options[destBox.length - 1].value = orig[i]
502 
503             if ( dest == destBox.options[destBox.length - 1].value )
504             {
505                 initDest = destBox.length - 1
506             }
507         }
508 
509 
510         if (fromOnload == true)
511         {
512             origBox.selectedIndex = origIndex
513             destBox.selectedIndex = destIndex
514         }
515         else
516         {
517             destBox.selectedIndex = initDest
518         }
519 
520     }
521     else
522     {
523         refillDestList( destBox, labelDest, labelOpenJaw);
524     }
525 
526 
527 }

    
    

    
  
The function is being called from two places:

1. From the onLoad handler of the body element:

<body onLoad="changeOrigDestList(document.SBform.sector1_o,   
                                 document.SBform.sector1_d,
                                'Destination', true, 1, '', '' )">


2. From the onChange handler of the first drop-down:

<select name='sector1_o' onChange="changeOrigDestList(document.SBform.sector1_o, 
                                                      document.SBform.sector1_d,
                                                      'Destination', false, 1)">



The two drop-down lists are named "sector1_o", "sector1_d", respectively.
In the function, these two lists are passed in as |origBox| and |destBox|. 
Notice the function stores the selectedIndex of |destBox| on this line:


482     var destIndex    = eval(destBox.selectedIndex)


Then notice |destBox| is recreated from scratch. For example, if we
have selected "London Stansted" in |origBox|, the funtions loops through
a linked list |aSTN| defined earlier in the file as a 69-element array:


           ["AAR", "AHO", "AOC", "AOI", etc. etc. ]


Then the |selectedIndex|of |destBox| is set back to its original value on 
this line, if we fall into the (fromOnload == true) branch of the function:


513          destBox.selectedIndex = destIndex

or here, otherwise:

517          destBox.selectedIndex = initDest


Note that |initDest| has a rather more complicated definition above.
This is what happens if (fromOnload == false), i.e. the function is
being called from the onChange handler of the first drop-down ...

    
    

    
  
It is clear there is nothing here involving JavaScript Engine.
Reassigning to DOM Level 0 for further triage. 

I don't know enough about the interplay between the <body> onLoad
handler and the <select> onChange handler to comment further -
Component: JavaScript Engine → DOM Level 0

    
    

    
  
setting default owner, QA -
Assignee: rogerl → dom_bugs
QA Contact: PhilSchwartau → ashshbhatt

    
    

    
  
IE6 seems to behave well on the reduced testcase, although I do get
different results depending on whether I access the testcase from the
Bugzilla server or as a saved local file:

Bugzilla server: IE6 reloads both drop-downs to their initial default values
     Local file: IE6 preserves the chosen values in both drop-downs upon reload

I am using "Reload" in each case (not Shift+Reload), yet IE6 seems to
show this difference in behaviors. Each behavior is acceptable, however,
and there is no sign of the jumping around that Mozilla is doing.
Blocks: 389601
Assignee: general → nobody
QA Contact: ashshbhatt → general

    
    

    
  
https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven't been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.
Priority: -- → P5

    
  
Component: DOM: Core & HTML → DOM: Forms
Summary: Javascript changes selection on Reload → Javascript changes <select> selection on Reload

    
  
Severity: minor → S4

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: