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

Highlighting missing from select feature tool - MS4W specific #107

Closed
markvolzlyoncounty opened this issue Sep 28, 2015 · 5 comments
Closed
Assignees
Labels
Milestone

Comments

@markvolzlyoncounty
Copy link

Hello,

After using select features, I noticed that the buffer and feature results highlights are missing from the map. This may be MS4W specific as the demo works fine.

p.s. ms4w is selecting the correct parcels on the results tab.

@tchaddad
Copy link
Contributor

tchaddad commented Dec 4, 2015

Hi Mark, I think this is the ticket for the issue that Jeff just commented on on the mailing list:
https://lists.osgeo.org/pipermail/geomoose-users/2015-December/005813.html
Can you try his solution and report back and close this issue if his solution works for you?

@markvolzlyoncounty
Copy link
Author

With a couple changes to the query I can now see highlights as expected in the selection results!

Changes in c:\ms4w\apps\geomoose2\php\highlight.php

Line 47
Replace: DATA 'features'
With: DATA 'select wkt_geometry from features'

Line 66
Replace: DATA 'selection'
With: DATA 'select area, wkt_geometry from selection'

Thanks Jeff and Tanya!

@tchaddad
Copy link
Contributor

So for the record, Jeff's fix is actually supposed to be applied to this file (not the PHP file):

https://github.com/geomoose/geomoose-services/blob/master/php/select/highlight.map

@tchaddad tchaddad reopened this Dec 10, 2015
@tchaddad
Copy link
Contributor

The ticket should probably remain open until the next review of closeable tickets (in case the fix impacts any other items).

@theduckylittle theduckylittle added this to the 2.9 milestone Dec 10, 2015
@theduckylittle theduckylittle self-assigned this Dec 10, 2015
@theduckylittle
Copy link
Member

Especially since no one actually updated the code, it's relatively important that we left it open.

Code changes here: geomoose/geomoose-services@f63c0e9

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

No branches or pull requests

3 participants