mike-jumper opened a new pull request #463: GUACAMOLE-930: Remove unnecessary $scope.$apply(). URL: https://github.com/apache/guacamole-client/pull/463 As noted on #453, a `[$rootScope:infdig]` error occurs on recent builds from git which prevents the language selection field from rendering. This is a regression from [GUACAMOLE-630](https://issues.apache.org/jira/browse/GUACAMOLE-630), specifically from commit 93ba19ac26a3509878023886eee0140bcf3face9, which changed the way that the injectors for fields are produced. This change removes the `$apply()` call entirely, as it is unnecessary. Promises returned by the `$q` and `$http` services, including the promise returned by `languageService.getLanguages()`, will implicitly `$apply()` when needed. It's not fully clear why this was not a problem before this commit. It may be that the way field injectors were produced in the past somehow did not satisfy the conditions which would normally result in an automatic `$apply()` for promises returned by `$http`.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
