(Issue #16) we used to call matlab.internal.webwindowmanager.instance.findAllWebwindows() in aboutJSLibs to test for existing webwindows, in order to avoid creating new ones when it is not required. It turns out that in some edge cases, the list can contain webwindow handles that are not associated with a uifigure (e.g. a login dialog), which don't have the Dojo library loaded, and subsequently fail when we attempt to invoke the library.
(Issue #17) The fact that getWidgetList returns a slightly different input was overcome by filtering the list better, to the same state that was working in previous versions. This fixed the demo.
sco1
commented
6 years ago
Description
matlab.internal.webwindowmanager.instance.findAllWebwindows()inaboutJSLibsto test for existing webwindows, in order to avoid creating new ones when it is not required. It turns out that in some edge cases, the list can contain webwindow handles that are not associated with auifigure(e.g. a login dialog), which don't have the Dojo library loaded, and subsequently fail when we attempt to invoke the library.getWidgetListreturns a slightly different input was overcome by filtering the list better, to the same state that was working in previous versions. This fixed the demo.Motivation and Context
Fixes: https://github.com/StackOverflowMATLABchat/mlapptools/issues/16 , https://github.com/StackOverflowMATLABchat/mlapptools/issues/17.
How Has This Been Tested?
aboutJSLibs()was executed manually in the conditions that previously caused an error, and it worked. (18a)TableDemowas run manually, clicking the button and observing the desired results. (17b, 18a)Types of changes