Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #4051 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

[PATCH] Slider.setValue( ) needs sanity check

Reported by: jeff@ionicenterprise.com Assigned to: Rails
Priority: high Milestone:
Component: script.aculo.us Version: 1.0.0
Severity: major Keywords: slider
Cc:

Description

If the developer calls Slider.setValue() without a second argument (the handle index), the Slider.activeHandle and Slider.activeHandleIdx get set to null. If the user then clicks on the track to set the slider position, Slider.startDrag() calls Position.cumulativeOffset(null) which is invalid.

To eliminate this possibility, I recommend the following change to slider.js:

  setValue: function(sliderValue, handleIdx){
    if(!this.active) {
-      this.activeHandle    = this.handles[handleIdx];
-      this.activeHandleIdx = handleIdx;
+      this.activeHandleIdx = handleIdx || 0;
+      this.activeHandle    = this.handles[activeHandleIdx];
      this.updateStyles();
    }

Attachments

slider_js.diff (0.5 kB) - added by jeff@ionicenterprise.com on 03/02/06 20:45:50.
provides sanity check for setValue function

Change History

03/02/06 20:30:33 changed by anonymous

  • keywords set to slider.
  • summary changed from [PATCH] Slider.setValue( ) needs sanity check to Slider.setValue( ) needs sanity check.

03/02/06 20:45:50 changed by jeff@ionicenterprise.com

  • attachment slider_js.diff added.

provides sanity check for setValue function

03/02/06 20:47:28 changed by anonymous

  • summary changed from Slider.setValue( ) needs sanity check to [PATCH] Slider.setValue( ) needs sanity check.

patch provided...

uh,

this.activeHandleIdx

04/18/06 12:24:42 changed by matthias <matthias@knallgrau.at>

That also fixes bug#3226 and bug#4765 for me.

04/18/06 12:28:06 changed by matthias <matthias@knallgrau.at>

  • priority changed from normal to high.
  • severity changed from normal to major.

because this bug causes several browser to crash (especially IE will end up in an endless loop) this bug should be fixed ASAP - thanks

06/09/06 10:49:13 changed by madrobby

  • status changed from new to closed.
  • resolution set to fixed.

(In [4448]) script.aculo.us: Fix an endless loop with sliders, fixes #3226, #4051, #4765 [thx jeff]

07/23/06 11:28:16 changed by anonymous

www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www www