Opened 11 years ago

Closed 10 years ago

#5504 closed defect (wontfix)

[patch][needs cla] dojox.grid.editors.Select doesn't use values

Reported by: guest Owned by: Bryan Forbes
Priority: high Milestone: future
Component: DojoX Grid Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Nathan Toone)

In a grid, when I use a dojox.grid.editors.Select, the value returned for the cell is always the label. To retrieve the value, the following modification should be done in dojoxgrid\_dataeditors.js:

dojo.declare("dojox.grid.editors.Select", dojox.grid.editors.Input, {
...
getValue: function(inRowIndex){
   var n = this.getNode(inRowIndex);
	
   if(n){
      var i = n.selectedIndex;
      var o = n.options[i];
      var v = this.values[i];
      return this.cell.returnIndex ? i : v || o.value || o.innerHTML;
   }
}
...

Change History (6)

comment:1 Changed 11 years ago by Adam Peller

Component: GeneralDojoX Grid
Description: modified (diff)
Owner: changed from anonymous to sorvell

please state your name and whether you've filed a CLA. Also, it's best to file patch files. Thanks.

comment:2 Changed 11 years ago by dylan

Milestone: 1.2
Summary: dojox.grid.editors.Select doesn't use values[patch][needs cla] dojox.grid.editors.Select doesn't use values

comment:3 Changed 11 years ago by Bryan Forbes

Owner: changed from sorvell to Bryan Forbes

comment:4 Changed 11 years ago by bill

Milestone: 1.21.3

comment:5 Changed 10 years ago by Bryan Forbes

Milestone: 1.3future

comment:6 Changed 10 years ago by Nathan Toone

Description: modified (diff)
Resolution: wontfix
Status: newclosed

Clearing abandoned bug.

Note: See TracTickets for help on using tickets.