Objekte mit Formular wählen und filtern (QGIS 3.22/3.28)

[ackmar]

Das Filtern zwischen zwei Datümer (Typ QDate) ergibt falsches Ergebnis.
("Ausstellungsdatum">='01.01.2022' AND "Ausstellungsdatum"<='31.12.2022')

Wird wahrscheinlich nicht als Datum erkannt und funktioniert leider nur per Korrektur mit folgendem Ausdruck.
("Ausstellungsdatum">=to_date('2022-01-01') AND "Ausstellungsdatum"<=to_date('2022-12-31'))

[signedav]

Ja, ich denke das ist so. Ich glaube zwar, dass dies gehen würde:
("Ausstellungsdatum">='2022-01-01' AND "Ausstellungsdatum"<='2022-12-31')

Dennoch ist das to:date stabiler. Dort kannst du auch das Format angeben so:

("Ausstellungsdatum">=to_date('01.01.2022','dd.MM.yyyy') AND "Ausstellungsdatum"<=to_date('31.12.2022','dd.MM.yyyy')

[ackmar]

Leider filtert diese Formel keine Daten und gibt wieder den gesamten Inhalt aus :
("Ausstellungsdatum">='2022-01-01' AND "Ausstellungsdatum"<='2022-06-30')

Kann man dies nicht als Bug deklarieren? Der User erwartet natürlich aus dem oben definierten Filter eine korrekte Auswertung und nicht erst bei der nachträglichen Modifizierung in der unteren Ausdruckszeile.

[signedav]

Du übergibst halt einen String. Ich dachte das "amerikanische" Format wird irgendwie erkannt aber offenbar nicht. Ein Bug ist es meiner Meinung nach nicht. Aber natürlich schon etwas umständlich so.

[signedav]

Du benutzt die Filterfunktion im GUI und die generiert eine "falsche" Expression.

Sorry, jetzt begreif ichs. Habs am Handy nicht gesehen (Nachteil, wenn man beim Frühstück am Handy Issue trackt. Response time top - Qualität flop. Wenigstens gratis ;-))

Ja, das scheint mir schon ein Bug zu sein.

[ackmar]

Ich hätte es auch klarer formulieren können. Das tut Eurer super Qualität nichts ab.

Könntest Du bitte an Romedi eine Aufwandschätzung für das Bug Fixing mitteilen. Je nach Grösse des Aufwandes würden wir den Bug gerne beheben lassen.

Vorab besten Dank.

[signedav]

Also das hier würde ja gehen "Ausstellungsdatum">='2021-09-19' AND "Ausstellungsdatum"<='2022-12-19'.

Das heisst irgendwo hier https://github.com/qgis/QGIS/blob/master/src/gui/editorwidgets/qgsdatetimesearchwidgetwrapper.cpp#L86-L97 müssten wir das DateFormat mitgeben. Etwa so:

return fieldName + "='" + v.toString(Qt::ISODate) + '\'';

Vielleicht ist es das schon. Allerdings wird hier von QVariant konvertiert. Ist das immer QDateTime? 🤔 Ich mach mal eine Schätzung.

[signedav]

Btw. hier der Issue auf QGIS qgis/QGIS#43177

[ackmar]

Gegenüber dem QDate wird es mit QDateTime etwas komplexer. Die Zeit oder und Zeitzone ist im nachstehenden Filter nicht unterstützt.

Bsp. ("test">='2022-12-19 05:00:00' AND "test"<='2022-12-21 20:00:00')

[signedav]

Kannst du das testen: ("test">='2022-12-19T05:00:00' AND "test"<='2022-12-21T20:00:00')

[ackmar]

So würde es klappen.

[signedav]

Hab mir das gerade codemässig angeschaut und etwas bemerkt, dass mir vorhin noch nicht aufgefallen ist. Der Wert wird immer in das Feldformat konvertiert. Was hast du dort eingetragen?

Sofern das Feld ein QDate ist, sollte dort eigentlich immer yyyy-MM-dd drin stehen. Kannst du das mal prüfen @ackmar ?

[signedav]

Zu meiner Info:

Das Format wird bereits ermittelt. Und zwar hier https://github.com/qgis/QGIS/blob/master/src/gui/editorwidgets/qgsdatetimesearchwidgetwrapper.cpp#L49-L59 Anhand des genannten Settings. Ich werd mal noch in die Implementierung dieses Settings schauen. Eventuell ist es eher für den Fall, dass das Feld kein spezifischer Datumstyp hat...

[signedav]

Dieses Setting ist aber wahrlich verwirrend. Ein PR sollte diese Verwirrung lösen und Fehlkonfigurationen vermeiden.

qgis/QGIS#52063

[ackmar]

Jetzt bin ich total verwirrt. In meinem Beispiel sieht das Setting folgendermassen aus:

Das Feldformat und die Customanzeige sind hier identisch und wurden aus einem SQL mit folgender Umformung definiert:
to_char(e.erstellt_am :: DATE, 'dd.MM.yyyy') as "Erstellungsdatum"

Als User würde ich in diesem Beispiel mindestens das Format erwarten, in dem es ursprünglich definiert ist, aber hier muss trotzdem das amerikanische Format für den Filter angewendet werden.

Wird die angebliche Lösung für 3.30 auch auf 3.28 backported?

[signedav]

Ich denke nicht dass es gebackportet wird, da es nur hilft Fehlkonfigurationen zu vermeiden.

Aber dein Fall hier scheint noch etwas anderes zu benötigen. Ich schau mal weiter und melde mich.

[signedav]

Habn noch was gefunden

Habe ich ein date column in der SQL Tabelle, lädt mir QGIS 3.28 das schweizer Format in die Settings (anhand meiner Systemsprache):

Dies ist gefixt worden (vor zwei Wochen) für 3.30 qgis/QGIS#51471

Hier wird das technische Datumsformat beibehalten und die Anzeige gemäss Systemsprache angepasst:

Ich denke, das würde auch dein Problem lösen. Dennoch schau ich noch dein SQL Case an...

[signedav]

Dein SQL Case kann ich hingegen nicht reproduzieren:

Ich denke aber dass wir folgendermassen weiterfahren können:

1. Konfiguriere dein Field Format zu "yyyy-MM-dd" die Anzeige kannst du auf dem schweizer Format lassen. Filter sollten nun funktionieren.
2. Zukünftig (ab 3.30) wird nun auch das korrekte Field Format beim erstellen des Layers genommen.

Falls euch ein Backport wichtig ist, könnte ich das mal anfragen. Allerdings würde ich gerne darauf verzichten, da der Bug nicht zu grossen Problemen führt. Ausserdem wäre die Config deines aktuellen Projektes ja falsch gesetzt. Die müsste man auch mit dem Fix noch umkonfigurieren...

[ackmar]

@signedav
Es funktioniert wie Du im Abschnitt 1 beschrieben hast. Wir verzichten auf die Anfrage mit dem Backport und schauen dann mit der künftigen LTR-Version im nächsten Jahr. Die Sache muss daher nicht mehr weiterverfolgt werden.

Herzlichen Dank für deine Abklärungen und deinen Service.

[ZsanettMed]

kein ZOHO ticket, warten auf die Version 3.30