fix: strict mode nested function adal-angular

AzureAD / azure-activedirectory-library-for-js

The code for ADAL.js and ADAL Angular has been moved to the MSAL.js repo. Please open any issues or PRs at the link below.

https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/maintenance/adal-angular

Apache License 2.0

627 stars 374 forks source link

fix: strict mode nested function adal-angular #820

Closed Izzmo closed 5 years ago

Izzmo commented 5 years ago

When using the adal-angular library within PhantomJS and strict mode (and in my case Chisel Web Accessibility tool), it throws an error keeping the AngularJS app from running. Moving in the function declaration to an inline call fixes this.

Tests

Unfortunately, there is no way to test this from what I can tell. If you think you can, please let me know and I'll update.

msftclas commented 5 years ago

All CLA requirements met.

m4m4m4 commented 5 years ago

UglifyJS also renders an error due to these nested functions