Clean up DateRangeFilter - broken pills, and missing test

Author: lrljoe

src/Traits/Filters/HandlesPillsData.php

@@ -11,7 +11,7 @@ public function getPillDataForFilter(): array
         $filters = [];
 
         foreach ($this->getAppliedFiltersWithValuesForPills() as $filterKey => $value) {
-            if (! is_null($filter = $this->getFilterByKey($filterKey))) {
+            if (! is_null($filter = $this->getFilterByKey($filterKey)) && !$filter->isEmpty($filter->validate($value))) {
                 $filters[$filter->getKey()] = FilterPillData::make(
                     filterKey: $filter->getKey(),
                     customPillBlade: $filter->getCustomPillBlade() ?? null,

src/Traits/Filters/Helpers/FilterPillsHelpers.php

@@ -49,11 +49,13 @@ public function getAppliedFiltersWithValuesForPills(): array
             }
 
             $validatedValue = $filter->validate($item);
+
             if ($filter instanceof BooleanFilter) {
                 return ! ($filter->isEmpty($validatedValue));
-            } elseif ($validatedValue === null || $validatedValue === 'null') {
+            } elseif ($validatedValue === null || $validatedValue === 'null' || $filter->isEmpty($validatedValue)) {
                 return false;
             } elseif (is_array($validatedValue)) {
+                $filter->isEmpty($validatedValue);
                 if (array_key_exists(0, $validatedValue) && (is_null($validatedValue[0]) || $validatedValue[0] == 'null')) {
                     return false;
                 }

src/Views/Filters/DateRangeFilter.php

@@ -198,6 +198,10 @@ public function getFilterPillValue($value): array|string|bool|null
 
     public function isEmpty(array|string|null $value): bool
     {
+        if(is_null($value) || empty($value))
+        {
+            return true;
+        }
         $values = [];
         if (is_array($value)) {
             if (! isset($value['minDate']) || ! isset($value['maxDate'])) {

tests/Unit/Views/Filters/DateRangeFilterTest.php

@@ -435,4 +435,25 @@ public function test_check_if_can_get_locale(): void
         $this->assertSame('de', self::$filterInstance->getPillsLocale());
         $this->assertTrue(self::$filterInstance->hasPillsLocale());
     }
+
+    public function test_can_check_validation_rejects_invalid_values_array(): void
+    {
+        $missingStartDate = self::$filterInstance->validate([null,'2020-01-01']);
+        $missingEndDate = self::$filterInstance->validate(['2020-01-01', null]);
+        $missingBoth = self::$filterInstance->validate([null,null]);
+
+        $this->assertFalse($missingStartDate);
+        $this->assertFalse($missingEndDate);
+        $this->assertFalse($missingBoth);
+        $this->assertTrue(self::$filterInstance->isEmpty($missingStartDate));
+        $this->assertTrue(self::$filterInstance->isEmpty($missingEndDate));
+        $this->assertTrue(self::$filterInstance->isEmpty($missingBoth));
+    }
+
+    public function test_can_check_validation_rejects_broken_values_array(): void
+    {
+        $this->assertFalse(self::$filterInstance->validate(['minDate' => 'asdf', 'maxDate' => '2020-02-02']));
+    }
+
+
 }