Opened 8 years ago

Last modified 3 years ago

#15513 new defect

RangeSlider: value change from array to number block script

Reported by: Mathevet julien Owned by: dante
Priority: undecided Milestone: 1.15
Component: DojoX Form Version: 1.7.2
Keywords: Cc:
Blocked By: Blocking:

Description

I don't know if it's a browser bug. But sometimes value change from an array to number. And script launch an exception. It's hard to reproduce. In my code I seem to fix it with changing in _setValueAttr of RangeSlider the call of super FormValueWidget:

- FormValueWidget.prototype._setValueAttr.apply(this, arguments);
+ FormValueWidget.prototype._setValueAttr.apply(this, [actValue, priorityChange]);

_setValueAttr of RangeSlider overrides one of arguments from value to an array. And sometimes seems browser keep orginal value.

I tested only on Chrome

Change History (4)

comment:2 Changed 7 years ago by davidbo

I'm using the same dojo version (1.7.2) and also encountered the same issue. After debugging the problem, I fixed dojox\form\RangeSlider?.js at the end of _setValueAttr() (line 239).

instead of:
FormValueWidget?.prototype._setValueAttr.apply(this, arguments);

use:
var args = [ actValue, priorityChange, isMaxVal ];
FormValueWidget?.prototype._setValueAttr.apply(this, args);

comment:3 Changed 4 years ago by dylan

Milestone: tbd1.12

Will consider for 1.12.

comment:4 Changed 3 years ago by dylan

Milestone: 1.131.15

Ticket planning... move current 1.13 tickets out to 1.15 to make it easier to move tickets into the 1.13 milestone.

Note: See TracTickets for help on using tickets.