Fix #2491. Avoid wrong coordinate changes with openlayers map click

[offtherailz]

Description

The openlayers map simulate feature click inside his singleclick event. The problem was that the point was moved everytime a vector layer is clicked, not only when the layer had handleClickOnLayer set to true.
For the future (and all the maps) I think we should avoid this and pass in the click event the feature clicked, instead of changing the clicked point. This is easier now because the mouse click event is now managed separately from the feature info.

Issues

• Fix Openlayers moves marker when click on vector data #2491

Please check if the PR fulfills these requirements

• The commit message follows our guidelines: https://github.com/geosolutions-it/MapStore2/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce?

• Bugfix

What is the current behavior?
See #2491

What is the new behavior?
The marker is in the correct point.
The behavior is the original when handleClickOnLayer is true.

Does this PR introduce a breaking change?

• No

[coveralls]

Coverage increased (+0.006%) to 80.748% when pulling 8252235 on offtherailz:i2491 into d8d5b60 on geosolutions-it:master.