diff options
Diffstat (limited to 'src/Wallabag/CoreBundle/Form/Type/EntryFilterType.php')
-rw-r--r-- | src/Wallabag/CoreBundle/Form/Type/EntryFilterType.php | 36 |
1 files changed, 28 insertions, 8 deletions
diff --git a/src/Wallabag/CoreBundle/Form/Type/EntryFilterType.php b/src/Wallabag/CoreBundle/Form/Type/EntryFilterType.php index cd4d3490..3c597b5d 100644 --- a/src/Wallabag/CoreBundle/Form/Type/EntryFilterType.php +++ b/src/Wallabag/CoreBundle/Form/Type/EntryFilterType.php | |||
@@ -36,17 +36,26 @@ class EntryFilterType extends AbstractType | |||
36 | $builder | 36 | $builder |
37 | ->add('readingTime', NumberRangeFilterType::class, [ | 37 | ->add('readingTime', NumberRangeFilterType::class, [ |
38 | 'apply_filter' => function (QueryInterface $filterQuery, $field, $values) { | 38 | 'apply_filter' => function (QueryInterface $filterQuery, $field, $values) { |
39 | $value = $values['value']; | 39 | $lower = $values['value']['left_number'][0]; |
40 | $upper = $values['value']['right_number'][0]; | ||
41 | |||
42 | $min = (int) ($lower * $this->user->getConfig()->getReadingSpeed()); | ||
43 | $max = (int) ($upper * $this->user->getConfig()->getReadingSpeed()); | ||
40 | 44 | ||
41 | if (null === $value['left_number'][0] || null === $value['right_number'][0]) { | 45 | if (null === $lower && null === $upper) { |
46 | // no value? no filter | ||
42 | return; | 47 | return; |
48 | } elseif (null === $lower && null !== $upper) { | ||
49 | // only lower value is defined: query all entries with reading LOWER THAN this value | ||
50 | $expression = $filterQuery->getExpr()->lte($field, $max); | ||
51 | } elseif (null !== $lower && null === $upper) { | ||
52 | // only upper value is defined: query all entries with reading GREATER THAN this value | ||
53 | $expression = $filterQuery->getExpr()->gte($field, $min); | ||
54 | } else { | ||
55 | // both value are defined, perform a between | ||
56 | $expression = $filterQuery->getExpr()->between($field, $min, $max); | ||
43 | } | 57 | } |
44 | 58 | ||
45 | $min = (int) ($value['left_number'][0] * $this->user->getConfig()->getReadingSpeed()); | ||
46 | $max = (int) ($value['right_number'][0] * $this->user->getConfig()->getReadingSpeed()); | ||
47 | |||
48 | $expression = $filterQuery->getExpr()->between($field, $min, $max); | ||
49 | |||
50 | return $filterQuery->createCondition($expression); | 59 | return $filterQuery->createCondition($expression); |
51 | }, | 60 | }, |
52 | 'label' => 'entry.filters.reading_time.label', | 61 | 'label' => 'entry.filters.reading_time.label', |
@@ -87,6 +96,18 @@ class EntryFilterType extends AbstractType | |||
87 | ->add('isStarred', CheckboxFilterType::class, [ | 96 | ->add('isStarred', CheckboxFilterType::class, [ |
88 | 'label' => 'entry.filters.starred_label', | 97 | 'label' => 'entry.filters.starred_label', |
89 | ]) | 98 | ]) |
99 | ->add('isUnread', CheckboxFilterType::class, [ | ||
100 | 'label' => 'entry.filters.unread_label', | ||
101 | 'apply_filter' => function (QueryInterface $filterQuery, $field, $values) { | ||
102 | if (false === $values['value']) { | ||
103 | return; | ||
104 | } | ||
105 | |||
106 | $expression = $filterQuery->getExpr()->eq('e.isArchived', 'false'); | ||
107 | |||
108 | return $filterQuery->createCondition($expression); | ||
109 | }, | ||
110 | ]) | ||
90 | ->add('previewPicture', CheckboxFilterType::class, [ | 111 | ->add('previewPicture', CheckboxFilterType::class, [ |
91 | 'apply_filter' => function (QueryInterface $filterQuery, $field, $values) { | 112 | 'apply_filter' => function (QueryInterface $filterQuery, $field, $values) { |
92 | if (false === $values['value']) { | 113 | if (false === $values['value']) { |
@@ -101,7 +122,6 @@ class EntryFilterType extends AbstractType | |||
101 | ]) | 122 | ]) |
102 | ->add('language', ChoiceFilterType::class, [ | 123 | ->add('language', ChoiceFilterType::class, [ |
103 | 'choices' => array_flip($this->repository->findDistinctLanguageByUser($this->user->getId())), | 124 | 'choices' => array_flip($this->repository->findDistinctLanguageByUser($this->user->getId())), |
104 | 'choices_as_values' => true, | ||
105 | 'label' => 'entry.filters.language_label', | 125 | 'label' => 'entry.filters.language_label', |
106 | ]) | 126 | ]) |
107 | ; | 127 | ; |