Itvisors / mendix-DataTables

DataTables.net wrapper for Mendix
Apache License 2.0
4 stars 3 forks source link

Search causes error with Desktop Modeler 7.23.0 #8

Closed gitsno closed 5 years ago

gitsno commented 5 years ago

After upgrading Desktop Modeler from 7.22.2 to 7.23.0 the search filters do not work with the DataTables widget. In the demo app for example, if you open the Search filters 1 page and select a filter value - when you click the Refresh button the following error occurs:

Trying to unsubscribe using an invalid subscription handle Error: Trying to unsubscribe using an invalid subscription handle at new t (http://localhost:8080/mxclientsystem/mxui/mxui.js?636855768487379877:20:19212) at t.Data.unsubscribe (http://localhost:8080/mxclientsystem/mxui/mxui.js?636855768487379877:79:28030) at http://localhost:8080/widgets/DataTables/widget/DataTables.js?636855768487379877:903:29 at Object.forEach (http://localhost:8080/mxclientsystem/mxui/mxui.js?636855768487379877:5:30384) at E.(anonymous function).O._resetRowObjectSubscriptions (http://localhost:8080/widgets/DataTables/widget/DataTables.js?636855768487379877:902:27) at E.(anonymous function).O._getData (http://localhost:8080/widgets/DataTables/widget/DataTables.js?636855768487379877:932:18) at jQuery.fn.init. (http://localhost:8080/mxclientsystem/mxui/mxui.js?636855768487379877:5:26619) at _fnBuildAjax (http://localhost:8080/widgets/DataTables/lib/jquery.dataTables.js?636855768487379877:3926:27) at _fnAjaxUpdate (http://localhost:8080/widgets/DataTables/lib/jquery.dataTables.js?636855768487379877:3951:4) at _fnDraw (http://localhost:8080/widgets/DataTables/lib/jquery.dataTables.js?636855768487379877:3414:40)

mgroeneweg commented 5 years ago

Hi, it is strange that this started with this release. Did you open a ticket with Mendix as well? I would like their opinion on this issue. There is nothing in the release notes about a change in subscription handlers in the client API.

gitsno commented 5 years ago

Yes, I have opened a ticket with Mendix.

mgroeneweg commented 5 years ago

Great! Please post any updates here.

gitsno commented 5 years ago

Mendix have indicated that they have fixed the issue and that the fix should go out in the next release so I'll verify once that release is available.

mgroeneweg commented 5 years ago

That´s fast!

Thanks for the update.

gitsno commented 5 years ago

The issue is not fixed in 7.23.1, but Mendix indicated in the ticket that it should be fixed in 7.23.2

mgroeneweg commented 5 years ago

Thanks for letting me know. I have a ticket for which a fix scheduled for that release as well, probably due to the performance issue that surfaced with 7.23.0. That one will also be in 7.23.2

mgroeneweg commented 5 years ago

New version is available in the appstore. Mendix release notes had a known issue which helped me fix this.