search

KelvinTegelaar / CIPP

CIPP is a M365 multitenant management solution
https://cyberdrain.com / https://cipp.app
GNU Affero General Public License v3.0
755 stars 4.42k forks source link

Complex filter "or" dosnt seem to work #2583

Closed kris6673 closed 1 week ago

kris6673 commented 1 week ago

Description

I am testing in the MFA report. In complex filters "or" dosnt seem to work, even if the code says it should "Complex: PerUser like enforced or disabled" only returns the enforced one and ignores the disabled ones. "Complex: PerUser like enforced or PerUser eq disabled" returns all users like no filtering has happened

Environment data

Self-hosted
Front and backend version: 5.8.5
Tried Tenant Cache Clear: false 
Tried Token Cache Clear: false
github-actions[bot] commented 1 week ago

Thank you for creating a bug. Please make sure your bug is indeed a unique case by checking current and past issues, and reading the complete documentation at https://docs.cipp.app/ If your bug is a known documentation issue, it will be closed without notice by a contributor. To confirm that this is not a bug found in the documentation, please copy and paste the following comment: "I confirm that I have checked the documentation thoroughly and believe this to be an actual bug."

Without confirming, your report will be closed in 24 hours. If you'd like this bug to be assigned to you, please comment "I would like to work on this please!".

kris6673 commented 1 week ago

I confirm that I have checked the documentation thoroughly and believe this to be an actual bug.

KelvinTegelaar commented 1 week ago

image Works fine here. :) Also using your filter:

image

Also: Enabled is going away in some time in the future, move users to enforced and filter on that. :)