- Issue created by @zviryatko
- πΊπΈUnited States ceithamer728
I was having the same problem, none of the grouped filters would work. You can add them in the view admin but they all resulted in an error when trying to actually filter by them in the form.
zviryatko's patch didn't work for me. Not sure if he's on a different version but the code looked different.
I'm on version 1.0.1.
Attached a patch that resolved the issue for me on our version. I did similar thing as zviryatko and just added this check at the top of DateBase::acceptExposedInput()
if ($this->isAGroup()) { return parent::acceptExposedInput($input); }
- πΊπ¦Ukraine zviryatko
thanks @ceithamer728, actually I've tried to include the case to use default parent method when grouping is enabled, but it looks like not all cases. Can you give more context what grouping filters you have, or upload screenshot of views filter configuration, will try to add some tests that covers both.
- Status changed to Postponed: needs info
about 1 month ago 11:14pm 26 June 2025 - π¨π¦Canada joelpittet Vancouver
Is this issue trying to solve a similar problem?
π TypeError: array_key_exists(): Argument #2 ($array) must be of type array, string given Active - πΊπ¦Ukraine zviryatko
@joelpittet, partly yes, when using a grouped filter, the expose identifier doesn't exist. The second problem is that original numeric filter rules must be applied since they contain logic for grouped conditions.
- π¨π¦Canada joelpittet Vancouver
@zviryatko Thanks for the clarification! Would you be open to helping move the issue forward by turning these patches into Merge Requests? That would make it easier for us to review and discuss the code more effectively. Also, feel free to join in that issues discussion and review it's MR to help move it forward.
- πΊπ¦Ukraine zviryatko
@joelpittet, oh, sorry, my patch was actually an improvement of #3385419, I think it's better to close this one, and I'll add my changes there.
- πΊπ¦Ukraine zviryatko
zviryatko β changed the visibility of the branch 3472713-exposed-filter-group to hidden.
- π¨π¦Canada joelpittet Vancouver
Itβs definitely beneficial to keep things separate if they truly are distinct. If this can be resolved independently, Iβd recommend keeping it here. Though if they are tied at the hip, then keeping them in the same issue is preferred.
- πΊπ¦Ukraine zviryatko
They are connected; it's still about the grouping option which does not work correctly in general. Both changes are related to the same problem, this change covers another aspect.