Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Eigenschaften in Mitglied Query #220

Merged
merged 2 commits into from
May 29, 2024

Conversation

JohannMaierhofer
Copy link

@JohannMaierhofer JohannMaierhofer commented Apr 27, 2024

Das Query der Mitglieder hat nicht funktioniert für die Verknüpfung "und" bei den Eigenschaften.
Die Implementierung für "oder" iteriert über alle Eigenschaften und schaut ob es mindestens einen Match gibt. Das passt.
Die Implementierung für "und" iteriert über alle Gruppen und schaut ob es in jeder Gruppe einen Match gibt. Das funktioniert aber nicht wenn eine Gruppe mehrere Eigenschaften hat weil es dem Code reicht wenn es für eine davon einen Match gibt. Es muss aber für alle sein. Also muss man statt >0 auf Gleichheit mit der Anzahl der Eigenschaften prüfen.
Man könnte sich auch die Schleife über die Gruppen sparen und wie bei "oder" direkt über die Eigenschaften iterieren.
Ich habe den Query Code korrigiert damit er auch "und" korrekt ausführt.

@JohannMaierhofer JohannMaierhofer marked this pull request as draft April 27, 2024 17:24
@JohannMaierhofer JohannMaierhofer marked this pull request as ready for review April 27, 2024 17:33
@dippeal dippeal added the fix This will fix a bug label May 4, 2024
@willuhn willuhn merged commit 63ac307 into openjverein:master May 29, 2024
@JohannMaierhofer JohannMaierhofer deleted the fix_eigenschaftenquery branch May 30, 2024 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix This will fix a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants