kvolden wrote:

Solution found. The following line numbers are from Semantic Maps version 2.0.1.2:

In includes/SM_GeoCoordsValueDescription.php, change lines 63 and 64 from:

$conditions[] = "{$tableName}.$fieldNames[0] $comparator $lat";
$conditions[] = "{$tableName}.$fieldNames[1] $comparator $lon";

to:

$conditions[] = "{$tableName}.$fieldNames[1] $comparator $lat";
$conditions[] = "{$tableName}.$fieldNames[2] $comparator $lon";

This fixes the issue when querying for a match on a specific set of coordinates, like in the example in the opening post. It doesn't, however, fix the issue when doing a "within distance/proximity" query. To fix this, change lines 143 to 146 in includes/SM_AreaValueDescription.php from:

$conditions[] = "{$tableName}.$fieldNames[0] $smallerThen $north";
$conditions[] = "{$tableName}.$fieldNames[0] $biggerThen $south";
$conditions[] = "{$tableName}.$fieldNames[1] $smallerThen $east";
$conditions[] = "{$tableName}.$fieldNames[1] $biggerThen $west";

to:

$conditions[] = "{$tableName}.$fieldNames[1] $smallerThen $north";
$conditions[] = "{$tableName}.$fieldNames[1] $biggerThen $south";
$conditions[] = "{$tableName}.$fieldNames[2] $smallerThen $east";
$conditions[] = "{$tableName}.$fieldNames[2] $biggerThen $west";

For **** sake... That is some very bad abstraction going on in this code :)

Looks like you are right - thanks for finding the issue and suggesting a fix!

https://gerrit.wikimedia.org/r/#/c/51844/

Oh damn, I just realized I did not give you credits in the commit message. Sorry for that!

kvolden wrote:

No worries, man. :) I just needed the feature and thought I'd share what I found. No credits needed. ;)

Guess this also works https://gerrit.wikimedia.org/r/#/c/51845/1/RELEASE-NOTES

kvolden wrote:

Thanks. :) Could this be a duplicate of bug 42388, by the way?

Probably