Skip to content

Commit

Permalink
Fixes grid columns not being set properly when the first query/find e…
Browse files Browse the repository at this point in the history
…xecuted.
  • Loading branch information
tmcgee committed Apr 1, 2016
1 parent 9ac78e5 commit 36fadd7
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 2 deletions.
13 changes: 11 additions & 2 deletions widgets/AttributesTable/Table.js
Original file line number Diff line number Diff line change
Expand Up @@ -112,13 +112,22 @@ define([
this.createGrid();
this.setToolbarButtons();

var options = {
queryOptions: this.queryOptions,
findOptions: this.findOptions,
gridOptions: this.gridOptions,
featureOptions: this.featureOptions,
symbolOptions: this.symbolOptions,
toolbarOptions: this.toolbarOptions
};

if (this.queryParameters && (this.queryParameters.url || this.queryParameters.layerID)) {
this.own(aspect.after(this, 'startup', lang.hitch(this, function () {
this.executeQueryTask();
this.executeQueryTask(options);
})));
} else if (this.findOptions && this.findOptions.url) {
this.own(aspect.after(this, 'startup', lang.hitch(this, function () {
this.executeFindTask();
this.executeFindTask(options);
})));
}
},
Expand Down
1 change: 1 addition & 0 deletions widgets/AttributesTable/_FindTaskMixin.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@ define([
},

executeFindTask: function (options) {
this.getConfiguration(options);
if (this.executingQuery === true) {
return;
}
Expand Down
1 change: 1 addition & 0 deletions widgets/AttributesTable/_QueryTaskMixin.js
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,7 @@ define([
},

executeQueryTask: function (options) {
this.getConfiguration(options);
if (this.executingQuery === true) {
return;
}
Expand Down

0 comments on commit 36fadd7

Please sign in to comment.