Search and Filter cannot be used at the same time
Permalink Browser Info Environment
I'm submitting as regular general discussion since my client didn't give us the access to their project.
Anyway, we're using 5.6.0.0.0.5.7.
Select Filter Search doesn't work when date/time filter is used at the same time..
Situation:
- We have a page list of an event listing
- It has select search to narrow down the event category (which is set as select attribute)
- It has event date. We wanted to set the filter "is today or after" so that the past event page won't show up.
When the date/time filter is set, the search doesn't really work.
Thanks
Anyway, we're using 5.6.0.0.0.5.7.
Select Filter Search doesn't work when date/time filter is used at the same time..
Situation:
- We have a page list of an event listing
- It has select search to narrow down the event category (which is set as select attribute)
- It has event date. We wanted to set the filter "is today or after" so that the past event page won't show up.
When the date/time filter is set, the search doesn't really work.
Thanks
Type: | Discussion |
---|---|
Status: | In Progress |
OK Fixed
For the bug fix that I've been doing I deserve for a couple licenses. :p
For the bug fix that I've been doing I deserve for a couple licenses. :p
diff --git a/packages/skybluesofa_page_list_plus/models/page_list_plus.php b/packages/skybluesofa_page_list_plus/models/page_list_plus.php index 74cc938..f785d30 100755 --- a/packages/skybluesofa_page_list_plus/models/page_list_plus.php +++ b/packages/skybluesofa_page_list_plus/models/page_list_plus.php @@ -261,6 +261,8 @@ class PageListPlus } else { $currentValue = $this->collectionObject->getCollectionAttributeValue($attributeKey); $handle = 'ak_' . $attributeKey->akHandle; + + $isDate = false; if ($attributeKey->atHandle == "date_time") { $isDate = true; $db = Loader::db(); -- 1.8.4.2
Dave,
I got excited and it was a bit too early to post my previous patch.
I should have put $isDate = false a bit earlier to cover the standard property.
I think you should also do one for 5.7
I've attached two files for PL+
I got excited and it was a bit too early to post my previous patch.
I should have put $isDate = false a bit earlier to cover the standard property.
I think you should also do one for 5.7
I've attached two files for PL+
I've run the debug mode.
This is the default query.
Default query
Then, this is when I ran the search.
With search query
You can see that 'ak_campaign_product' is outputting DATE format where it supposed to be just a keyword.