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

Author: deguif

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)

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);
     }
 }

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);