GeoPolygon - Fix wrong property passed to \count() (#1696)

ruflin · Nov 18, 2019 · e62898f · e62898f
1 parent d3dc5a7
commit e62898f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -50,6 +50,7 @@ All notable changes to this project will be documented in this file based on the
 
 ### Bugfixes
 
+* The `\Elastica\Query\GeoPolygon::count()` method now returns the count of points passed to the filter [#1696](https://github.com/ruflin/Elastica/pull/1696)
 * Always set the Guzzle `base_uri` to support connecting to multiple ES hosts. [#1618](https://github.com/ruflin/Elastica/pull/1618)
 * Properly handle underscore prefixes in options and bulk request metadata ([cf upstream](https://github.com/elastic/elasticsearch/issues/26886). [#1621](https://github.com/ruflin/Elastica/pull/1621)
 * Preserve zeros while doing float serialization to JSON. [#1635](https://github.com/ruflin/Elastica/pull/1635)

diff --git a/lib/Elastica/Query/GeoPolygon.php b/lib/Elastica/Query/GeoPolygon.php
@@ -56,6 +56,6 @@ public function toArray(): array
      */
     public function count(): int
     {
-        return \count($this->_key);
+        return \count($this->_points);
     }
 }
diff --git a/test/Elastica/Query/GeoPolygonTest.php b/test/Elastica/Query/GeoPolygonTest.php
@@ -46,6 +46,7 @@ public function testGeoPoint()
         // Only one point should be in polygon
         $points = [[16, 16], [16, 20], [20, 20], [20, 16], [16, 16]];
         $geoQuery = new GeoPolygon('location', $points);
+        $this->assertCount(\count($points), $geoQuery);
 
         $query = new Query(new MatchAll());
         $query->setPostFilter($geoQuery);