Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maps will crash (RxJS error) on identify with some WMTS layers #8724

Closed
1 task done
ale-cristofori opened this issue Oct 24, 2022 · 1 comment
Closed
1 task done

Comments

@ale-cristofori
Copy link
Contributor

ale-cristofori commented Oct 24, 2022

Description

Related support issue is #2292
This bug seems to happen exclusively with client data and an example geostory map with client data can be found here

When having a WMTS layer loaded in the TOC

  1. performing a click identify on the WMTS overlay layer will crash the map
    recording_1

  2. clicking on an area where an overlay layer is not there will trigger a ReduxRJs error, blocking all subsequent interactions between user and map, see below
    recording_2

How to reproduce

  • Open the geostory map linked above
  • Click to identify on the overlay WMTS layer (osm_roads)
  • The application will crash
  • Re-load the map at point 1 (React error in console)
  • Click to identify on an area where there is no overlay data
  • The pop-up identify result will appear with a spinner showing but no results will show (RxJS error in console)

Expected Result

  • Open the geostory map linked above
  • Click to identify on the overlay WMTS layer (osm_roads)
  • A pop-up will appear with the identify results for the overlay layer
  • Click to identify on an area where there is no overlay data
  • A pop-up warning of no results will appear on the map

Current Result

  1. performing a click identify on the WMTS overlay layer will crash the map.

  2. clicking on an area where an overlay layer is not there will trigger a Redux RJs error, blocking all subsequent interactions between user and map.

  • Not browser related
Browser info (use this site: https://www.whatsmybrowser.org/ for non expert users)
Browser Affected Version
Internet Explorer
Edge
Chrome
Firefox
Safari
alex-fko added a commit to alex-fko/MapStore2 that referenced this issue Oct 24, 2022
offtherailz pushed a commit that referenced this issue Oct 26, 2022
@ElenaGallo ElenaGallo self-assigned this Oct 26, 2022
@tdipisa tdipisa added this to the 2022.02.01 milestone Oct 26, 2022
alex-fko added a commit to alex-fko/MapStore2 that referenced this issue Oct 26, 2022
…on` parameter; geosolutions-it#8724 WMTS request identify flow fixes (geosolutions-it#8725)

(cherry picked from commit 88c7ad8)
alex-fko added a commit to alex-fko/MapStore2 that referenced this issue Oct 26, 2022
…on` parameter; geosolutions-it#8724 WMTS request identify flow fixes (geosolutions-it#8725)

(cherry picked from commit 88c7ad8)
offtherailz pushed a commit that referenced this issue Nov 2, 2022
offtherailz pushed a commit that referenced this issue Nov 2, 2022
…8724 WMTS request identify flow fixes (#8725) (#8733)

(cherry picked from commit 88c7ad8)
@ale-cristofori
Copy link
Contributor Author

The associated PR and backport for this issue have been merged, this issue can be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants