Commit cefdf53794689f654e189488fd349fd9c6a7064d
1 parent
88fa6068
Exists in
master
and in
112 other branches
Remove restriction about derived parameter position in the download grid + Fix a…
… bug with parameter deletion
Showing
1 changed file
with
4 additions
and
3 deletions
Show diff stats
js/app/views/DownloadUI.js
... | ... | @@ -105,7 +105,7 @@ Ext.define('amdaUI.DownloadUI', { |
105 | 105 | } |
106 | 106 | |
107 | 107 | var r = Ext.create('amdaModel.DownloadParam', {paramid: paramId, type: type, dim1: dim1, dim2: dim2, template_args: {} }); |
108 | - var pos = paramId.indexOf('ws_',0) == -1 ? 0 :this.paramGrid.store.getCount(); | |
108 | + var pos = this.paramGrid.store.getCount(); | |
109 | 109 | this.paramGrid.store.insert(pos,r); |
110 | 110 | this.paramGrid.getView().refresh(); |
111 | 111 | if (!isLeaf || needArgs) |
... | ... | @@ -276,11 +276,12 @@ Ext.define('amdaUI.DownloadUI', { |
276 | 276 | |
277 | 277 | |
278 | 278 | actionItem: function(grid, cell, cellIndex, record, row, recordIndex, e){ |
279 | - this.paramGrid.getView().select(row); | |
280 | 279 | if (cellIndex == 3) |
281 | 280 | grid.getStore().remove(record); |
282 | - else if (cellIndex == 2) | |
281 | + else if (cellIndex == 2) { | |
282 | + this.paramGrid.getView().select(row); | |
283 | 283 | this.editParameterArgs(record); |
284 | + } | |
284 | 285 | }, |
285 | 286 | |
286 | 287 | editParameterArgs: function(record) { |
... | ... |