Previously, if a viewport was not contained in a request (for example, a non APL enabled request) the getViewportProfile() method would return an NPE. This is undesirable behavior, and has been updated to return a ViewportProfile.UNKNOWN_VIEWPORT_PROFILE in these cases.
Motivation and Context
User reported getting exceptions in a common flow that calls this method.
Testing
New and existing tests pass.
Screenshots (if appropriate)
Types of changes
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist
[x] My code follows the code style of this project
[ ] My change requires a change to the documentation
[ ] I have updated the documentation accordingly
[x] I have read the README document
[x] I have added tests to cover my changes
[x] All new and existing tests passed
License
[x] I confirm that this pull request can be released under the Apache 2 license
Description
Previously, if a viewport was not contained in a request (for example, a non APL enabled request) the
getViewportProfile()method would return an NPE. This is undesirable behavior, and has been updated to return aViewportProfile.UNKNOWN_VIEWPORT_PROFILEin these cases.Motivation and Context
User reported getting exceptions in a common flow that calls this method.
Testing
New and existing tests pass.
Screenshots (if appropriate)
Types of changes
Checklist
License