[baderaParticipant]

Dear Hristo

Thanks for the answer. Primarily, I would like to prevent from execution of javascript. I.e. if a jqxGrid gets ‘‘ as data (because probably a user enter this as his firstname and there is a grid showing all users firstname), javascript shall not be executed. So I wonder if it is correct, if we do the filtering on input data in the backend – it is not the backend nor other clients, which gets trouble with such input – it is just the browser. So I think it would be best practise to do the sanitizing before displaying the content.
But of course, I would appreciate if I could do it in the backend – it would be simpler than making e.g. regarding jqxGrid for each column a custom renderer which sanitize the data to display. On the other hand, it would be great, if jqWidgets would give an option to enable such sanitizing.

According your answer, I think that jqWidgets do not currently have such option – so I like to ask you what is state of the art – how do professionals solve it? Do they on backend or in browser?

Thanks for your help!
Best regards,
– badera

[baderaParticipant]

I would still be pleased for an answer.

[baderaParticipant]

Thanks Peter.

You wrote: or add custom CSS classes when you create it in order to style it.

How can I do this properly? As I pointed out, we cannot just set the class in the html tag, because it is removed during creation of the widget. So I need to do it afterwards with JQuery I think. But what is the clean way to know, when the widget has been created? There is no ready event / callback or similar.

– badera

[baderaParticipant]

Hello Hristo

Thank you very much. It works like a charm!
Best regards,
– badera

[baderaParticipant]

Dear Peter

Thank you very much. beginUpdate and endUpdate definitely does the trick.
I also though about that prior writing this question here, and I tried it; but it did not work. I found the reason: I also had an expandAll before selecting. And this is even with beginUpdate and endUpdate a problem. There is somthing not proper, I think.

demo:
https://www.jseditor.io/?key=treegrid-select-multiple-rows-very-slow-2

Why does this piece of code still takes > 15 seconds:

    $scope.selectWithProblem = function () {
        
        $scope.treeGridSettings.apply('beginUpdate');
        $scope.treeGridSettings.apply('expandAll');             // < before selecting
        for (var i = 100; i < 300; i++)
            $scope.treeGridSettings.apply('selectRow', i);
        $scope.treeGridSettings.apply('endUpdate');
    }

And this one not:

    $scope.selectWithoutProblem = function () {
        
        $scope.treeGridSettings.apply('beginUpdate');
        for (var i = 100; i < 300; i++)
            $scope.treeGridSettings.apply('selectRow', i);
        $scope.treeGridSettings.apply('expandAll');             // < after selecting
        $scope.treeGridSettings.apply('endUpdate');
    }

Why this big difference? By the way the differences even exists, if there are already all rows expanded or even, if there are all rows on same level (i.e. no hierarchy)

Best regards,
– badera

[baderaParticipant]

OK, so please explain me, why in the following example $scope.treeGrid.clearSelection() throws a JS exception!

(Open F12 Tools – you would not see it in the JS Editor console)
https://www.jseditor.io/?key=treegrid-clearselection-instance-object-error

This snippet works:

    $scope.select = function () {
         $scope.treeGrid.selectRow(1);
     }
 

But this one not:

    $scope.clear = function () {
         $scope.treeGrid.clearSelection();
     }
 

what is wrong?
– badera

[baderaParticipant]

Dear Hristo
I have shared it and saved again.

[baderaParticipant]

With Are these two changes the only two? I mean similar default value changes. Are there others?

[baderaParticipant]

Hello Hristo

Please take a look at this example I modified:
https://www.jseditor.io/?key=treegrid-update-data-error

Here, you will see two problems:
1) what I tried to describe in previous posts: after second click to “Click” button, you will get JS exceptions – please open F12 tools to see them; they are not displayed in your Console of {JS Editor}.

2) Filter clear icon (“cross”) is no more displayed after reassigning data:
– Enter something to filter in the filter input and click on the maginfier -> the cancel filter button is correctly shown beside magnifier
– Click on Button “Click”
-> cancel filter button is no more displayed and “Search:” is moved to top…. OK, we can manually clear the text again in the input the remove filter.

I hope I was now able to tell my problems. Can you confirm to fix these in the next release?

Here is a demo video for completeness:

Thanks.
– badera

[baderaParticipant]

Dear Hristo

I try it again…
1) I use angular 1. So I do not like to use the jquery way just for this.
2) Do you agree that my example in the initial post SHOULD work? Why does it not? Is it a bug?

[baderaParticipant]

I also remarked that if we call updateBoundData (as shown in this demo http://jsfiddle.net/jqwidgets/7J2qP/ – but using angular 1) the same exception happens.
What can I do? I should bring out a relase next week containing such a treegrid but I need to update the data…

[baderaParticipant]

Dear Hristo

Well, with “refresh” I do mean reassign new data, not only redraw the grid. So if you look in my code abouve, I already do what you advised me: “Create a new source with a new DataAdapter and set to the TreeGrid”. Ok, this example assigns again the same data. In reality, the records would be different.
So why does the code fail? In my opinion, I do exactly what you advice me. Whats wrong?

[baderaParticipant]

Thanks Peter for the feedback. I feared to get this answer.
So this would worth to be added to the feature requets. Not?

[baderaParticipant]

You can just take the demo at
http://www.jqwidgets.com/jquery-widgets-demo/demos/jqxeditor/index.htm

With Firefox 45.0.2 (Win64), I have the described behaviour; I also updated to version 49.0.1 on a different PC, same bad behaviour.
With Chrome, it seems to be OK.

I can make a video, if needed.

[baderaParticipant]

No answer?

[Author]

Posts