Hi Mr Farook.
Yes what you proposed works perfectly (it is pretty much the same thing I was doing, except for the fact that you are doing it within the "onComplete" callback while I was proposing to do it directly within the "processQuery" function, which would help reduce the number of data to work with for other operations like paging.
Query : for filtering with already dataset.
Please confirm filtering is also working same like for above searching sample?
A: Yes it also works for filtering, simpy by changing the code a bit and "or'ing" the requestType check like so:
var newData = [];
var gridData;
function load() {
gridData = document.getElementById("Grid").ej2_instances[0].dataSource;
}
function action(args) {
if (args.requestType == "searching" || args.requestType == "filtering") {
if (!args.searchString && !args.currentFilteringColumn) {
this.dataSource = gridData;
}
else {
for (var i = 0; i < args.rows.length; i++) {
newData.push(args.rows[i].data);
}
this.dataSource = newData;
newData = [];
}
}
}
The only thing that differs in the way I implemented it was that I was personally not replacing the original dataSource property of the DataManager.
Instead, I created a new variable (dataSourceFiltered) which held the filtered/searched dataset and which I was using for the "processQuery" function instead of the original dataSource whenever the request type was anything else than "filtering" or "searching".
In the end, the result is pretty much the same, except for the fact that your proposed alternative kind of "pollutes" the global scope, which could be seen as undesirable. If you could add a new property to the dataManager which would keep the reference to the original dataset (such as with the gridData variable in the code you posted), that would be awesome in my opinion!
Then you could provide API endpoints like "getFilteredRecords()", which in your implementation would simply return the updated "dataSource" and another one like "getOriginalRecords()" or equivalent, which would return the content of "gridData" above.
Thanks for your time and hope you will implement it in the solution for everyone to take profit of the performance improvement for operations such as paging!
P.S: Sorry for the formatting, I don't know why but the editor does not respect the text format I entered at all and I have no menu displayed (top bar) when editing the response... so sorry for the look, but the content should be fine :) )
