From 5d9771c987b97e9701a1bfa1c040fbd0a9d2e665 Mon Sep 17 00:00:00 2001 From: pmishev Date: Thu, 11 Mar 2021 15:05:52 +0000 Subject: [PATCH] Allow null 'format' in DateRangeAggregation, as it is not required --- src/Aggregation/Bucketing/DateRangeAggregation.php | 9 +++++---- .../Aggregation/Bucketing/DateRangeAggregationTest.php | 2 +- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/src/Aggregation/Bucketing/DateRangeAggregation.php b/src/Aggregation/Bucketing/DateRangeAggregation.php index 3ca14e29..c5e3f4db 100644 --- a/src/Aggregation/Bucketing/DateRangeAggregation.php +++ b/src/Aggregation/Bucketing/DateRangeAggregation.php @@ -128,17 +128,18 @@ function ($v) { */ public function getArray() { - if ($this->getField() && $this->getFormat() && !empty($this->ranges)) { + if ($this->getField() && !empty($this->ranges)) { $data = [ - 'format' => $this->getFormat(), 'field' => $this->getField(), 'ranges' => $this->ranges, 'keyed' => $this->keyed, ]; - + if ($this->getFormat()) { + $data['format'] = $this->getFormat(); + } return $data; } - throw new \LogicException('Date range aggregation must have field, format set and range added.'); + throw new \LogicException('Date range aggregation must have field and range added.'); } /** diff --git a/tests/Unit/Aggregation/Bucketing/DateRangeAggregationTest.php b/tests/Unit/Aggregation/Bucketing/DateRangeAggregationTest.php index ca32f902..0e9ca38f 100644 --- a/tests/Unit/Aggregation/Bucketing/DateRangeAggregationTest.php +++ b/tests/Unit/Aggregation/Bucketing/DateRangeAggregationTest.php @@ -19,7 +19,7 @@ class DateRangeAggregationTest extends \PHPUnit\Framework\TestCase * Test if exception is thrown. * * @expectedException \LogicException - * @expectedExceptionMessage Date range aggregation must have field, format set and range added. + * @expectedExceptionMessage Date range aggregation must have field and range added. */ public function testIfExceptionIsThrownWhenNoParametersAreSet() {