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.

Sign up for GitHub

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

Jump to bottom

Counter widget sends double requests, the first one has viewport as undefined #10674

Closed
1 task done
MV88 opened this issue Nov 13, 2024 · 1 comment · Fixed by #10683 or #10705
Closed
1 task done

Counter widget sends double requests, the first one has viewport as undefined #10674

MV88 opened this issue Nov 13, 2024 · 1 comment · Fixed by #10683 or #10705
Assignees
@rowheat02 @ElenaGallo
Labels
Accepted bug Widgets
Milestone
2024.02.02

Comments

@MV88
Copy link
Contributor

MV88 commented Nov 13, 2024

Description

WpsCounter triggers a wrong request

How to reproduce

  • open a map with a counter widget only
  • inspect requests

Expected Result

to not see a first request with undefined in the payload
Current Result

two requests are sent
one has a payload geom as undefined
the other seems ok

  • 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

Other useful information
@MV88 MV88 added this to the 2024.02.02 milestone Nov 13, 2024
@tdipisa tdipisa assigned rowheat02 and unassigned tdipisa Nov 15, 2024
@rowheat02 rowheat02 mentioned this issue Nov 18, 2024
Merged
12 tasks
@rowheat02 rowheat02 linked a pull request Nov 18, 2024 that will close this issue
Counter widget sends double requests, the first one has viewport as undefined #10674 #10683
Merged
12 tasks
offtherailz pushed a commit that referenced this issue Nov 27, 2024
@rowheat02
Counter widget sends double requests, the first one has viewport as u…
a174c99 
…ndefined #10674 (#10683)
@tdipisa tdipisa added the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Nov 27, 2024
@offtherailz
Copy link
Member

note: the "undefined" filter is a more general problem, not related with counter widget (it appears also in WFS request of feature grid and in general whenever an and filter is used.

Here I opened a separate issue for the undefined filter problem #10695

@ElenaGallo ElenaGallo self-assigned this Dec 2, 2024
rowheat02 added a commit to rowheat02/MapStore2 that referenced this issue Dec 2, 2024
@rowheat02
Counter widget sends double requests, the first one has viewport as u…
33bfd9a 
…ndefined geosolutions-it#10674 (geosolutions-it#10683)
@rowheat02 rowheat02 mentioned this issue Dec 2, 2024
Merged
@rowheat02 rowheat02 linked a pull request Dec 2, 2024 that will close this issue
[Backport 2024.02.xx] - Counter widget sends double requests, the first one has viewport as undefined #10674 #10683 #10705
Merged
tdipisa pushed a commit that referenced this issue Dec 2, 2024
@rowheat02
Counter widget sends double requests, the first one has viewport as u…
739af9e 
…ndefined #10674 (#10683) (#10705)
@tdipisa tdipisa removed the BackportNeeded Commits provided for an issue need to be backported to the milestone's stable branch label Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rowheat02 rowheat02

@ElenaGallo ElenaGallo

Labels
Accepted bug Widgets
Projects
None yet
Milestone
2024.02.02
5 participants
@offtherailz @tdipisa @MV88 @rowheat02 @ElenaGallo