Aggragations/Filter - fix for es1.2 when no sub aggregations

changes.txt

@@ -1,5 +1,8 @@
 CHANGES
 
+2014-05-27
+- Fix Aggragations/Filter to work with es v1.2.0
+
 2014-05-25
 - Added Guzzle transport as an alternative to the default Http transport #618
 - Added Elastica\ScanAndScroll Iterator (http://www.elasticsearch.org/guide/en/elasticsearch/guide/current/scan-scroll.html) #617

lib/Elastica/Aggregation/Filter.php

@@ -27,9 +27,15 @@ public function setFilter(AbstractFilter $filter)
      */
     public function toArray()
     {
-        return array(
-            "filter" => $this->getParam("filter"),
-            "aggs" => $this->_aggs
+        $array = array(
+            "filter" => $this->getParam("filter")
         );
+
+        if($this->_aggs)
+        {
+            $array['aggs'] = $this->_aggs;
+        }
+
+        return $array;
     }
 }

test/lib/Elastica/Test/Aggregation/FilterTest.php

@@ -54,10 +54,25 @@ public function testFilterAggregation()
 
         $query = new Query();
         $query->addAggregation($agg);
+
         $results = $this->_index->search($query)->getAggregation("filter");
         $results = $results['price']['value'];
 
         $this->assertEquals((5 + 8) / 2.0, $results);
     }
+
+    public function testFilterNoSubAggregation()
+    {
+        $agg = new Avg("price");
+        $agg->setField("price");
+
+        $query = new Query();
+        $query->addAggregation($agg);
+
+        $results = $this->_index->search($query)->getAggregation("price");
+        $results = $results['value'];
+
+        $this->assertEquals((5 + 8 + 1 + 3) / 4.0, $results);
+    }
 }