Problem/Motivation

I came across this issue because we also use the optional_end_date module. With both that and this module enabled, and content with an empty end-date field, we were missing results in our overview.

Proposed resolution

Add extra condition groups to handle the empty end date.

CommentFileSizeAuthor
#2 3313436-2.patch1.82 KBwilfred waltman

Comments

Wilfred Waltman created an issue. See original summary.

wilfred waltman’s picture

wilfred waltman commented
StatusFileSize
new 3313436-2.patch1.82 KB

Created a patch to handle at least a part of the problem, in the addConditionGroupsByActiveItem method.

Probably something like this also needs to be added in the addConditionGroupsByActiveItemWithOverlap method but I'm not sure.

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented

Hey Wilfred, thanks a lot for reporting this issue and also for investigating and providing a fix.
I'll need to find some time to investigate and test this. I'll probably will need to add some test coverage for this.

tim-diels’s picture

tim-diels
He/Him
Primary language Dutch
Location Belgium 🇧🇪
commented
Status: Active » Needs review

Setting this to Needs Review as there is a patch.