Project

General

Profile

Actions

Feature #12630

closed

Input: date[time]: min / max values

Added by Carsten Rose almost 3 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Carsten Rose
Target version:
Start date:
27.05.2022
Due date:
% Done:

100%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

Description

Mitunter ist es gewuenscht fuer ein Datumsfeld min/max Werte setzen zu koennen.

Aktuell werden input Elemente fuer Datumsfelder nicht type='date' sondern type='text' ausgeliefert.

Grund: die Kompatibilitaet / gleiches Verhalten ueber alle Browser ist besser.
Nachteil: die Browser internen Datepicker werden nicht benutzt. Min / Max kann nicht funktionieren.

Aktuell hat QFQ keinen Datepicker. Wenn ein Datepicker integriert wird, koennte der min/max Vergleich z.B. clientseitig via JS integriert werden.

Serverseitig ist der Check bereits implementiert.


Files


Related issues

Related to QFQ - Feature #10096: JQuery / Bootstrap: DatePickerClosedCarsten Rose11.02.202005.01.2022

Actions
Related to QFQ - Bug #14302: Form-Field Dokumentation: datetime / extra brokenClosedEnis Nuredini10.06.202217.06.2022

Actions
Related to QFQ - Bug #14303: datetime broken with pickerClosedEnis Nuredini10.06.2022

Actions
Related to QFQ - Bug #14245: Form Save Btn bleibt disabled wenn Datumsfeld über Datepicker geändertClosedEnis Nuredini27.05.2022

Actions
Related to QFQ - Bug #14791: dateTimePicker: AbstractBuildForm.php / Line: 3172 / Undefined offset: 2ClosedCarsten Rose26.09.2022

Actions
Actions #1

Updated by Carsten Rose almost 3 years ago

Actions #2

Updated by Carsten Rose over 2 years ago

Neu: By default wird der Browserdatepicker aktiviert.
Systemconfig & FE: datepicker=no|browser|jquery

Actions #3

Updated by Carsten Rose about 2 years ago

  • Assignee changed from Carsten Rose to Enis Nuredini
  • Target version changed from next3 to 355

Hallo Enis

Wie ist denn der aktueller Stand bzgl. min/max?

Die Option FE: datepicker=no|browser|jquery mit Default auf `jquery` finde ich richtig gut.

Actions #4

Updated by Enis Nuredini about 2 years ago

  • Status changed from New to ToDo
Actions #5

Updated by Enis Nuredini about 2 years ago

  • Status changed from ToDo to Feedback
  • Assignee changed from Enis Nuredini to Carsten Rose

Über Parameter besteht die Möglichkeit den Jquery datepicker zu deaktivieren: datepicker = no. Min und max Date Abfrage funktioniert bei beiden Varianten.

Actions #6

Updated by Carsten Rose almost 2 years ago

  • Status changed from Feedback to New
  • Assignee changed from Carsten Rose to Enis Nuredini
  • Beim Merge mit 'develop' gibt es Konflikte. Bitte loesen.
  • Die Einstellung, welcher datetimepicker verwendet wird, sollte pro Typo3 Instanz, pro Form und pro FormElement moeglich sein (vergleichbar mit 'dateFormat') - das war mit `Systemconfig & FE: datepicker=no|browser|jquery` gemeint.
  • Aktuell scheint 'browser' nicht implementiert zu sein. Wenn der Aufwand vertretbar ist bitte implementieren. In dem Falls sollte der HTML Tag Typ dann wirklich date/time/dateteim sein (und nicht text).
  • Sollten wir anstelle von 'no|browser|jquery' die Werte 'no|browser|qfq' nehmen? Beschluss : 'no|browser|qfq'
  • Bitte Konstanten immer via Constants.php definieren. Niemals direkt a la `formElement['datepicker']`. Hier also z.B. FE_DATE_PICKER.
  • Der Name `datepicker` sollte, analog zu `dateFormat` in Camel Case geschrieben werden: `datePicker`
  • Darauf achten das wenn datepicker nicht definiert ist, ein Default gesetzt ist. Aktuell knallt es wenn der Parameterwert 'datepicker' fehlt - das Problem loest sich wenn wenn sauber 'config > form > formelement' implementiert ist.
  • Sollten wir anstelle von `datePicker` besser `dateTimePickerType` schreiben? Das ist spezifischer. By default wird es in der Systemconfig gesetzt (=jquery oder qfq) und man muss es nirgendwo angeben - in dem Fall stoert der laengere Name nicht und er ist selbstsprechender. Beschluss : dateTimePickerType
  • Bitte Dokumentation anpassen. Aktuell ist zwar `datepicker=no` in der Doku enthalten, aber nicht erklaert welche Optionen noch existieren.
Actions #7

Updated by Carsten Rose almost 2 years ago

  • Tracker changed from Support to Feature
Actions #8

Updated by Carsten Rose almost 2 years ago

  • Related to Bug #14302: Form-Field Dokumentation: datetime / extra broken added
Actions #9

Updated by Carsten Rose almost 2 years ago

  • Related to Bug #14303: datetime broken with picker added
Actions #10

Updated by Enis Nuredini almost 2 years ago

  • Status changed from New to ToDo
Actions #11

Updated by Enis Nuredini almost 2 years ago

  • Status changed from ToDo to In Progress
Actions #12

Updated by Enis Nuredini almost 2 years ago

  • Status changed from In Progress to Closed
Actions #13

Updated by Carsten Rose over 1 year ago

  • Target version changed from 355 to 22.8.0
Actions #14

Updated by Carsten Rose over 1 year ago

  • Related to Bug #14245: Form Save Btn bleibt disabled wenn Datumsfeld über Datepicker geändert added
Actions #15

Updated by Carsten Rose over 1 year ago

Ist es implementiert?

Sobald ich bei einem Datetime Feld folgendes eingebe:

min = 2022-08-01 12:00:00
max = 2022-09-01 12:00:00

Funktioniert der Datetime Picker nicht mehr - er oeffnet sich nicht mehr. In der JS Console gibt es einen Fehler:

Actions #16

Updated by Enis Nuredini over 1 year ago

Implementation war bei min/max nur für das deutsche Format gedacht. Im neuen Branch F12630_datepicker_min_max_bug werden beide Formate unterstützt. Die oben genannten Testdaten wurden getestet.

Actions #17

Updated by Enis Nuredini over 1 year ago

  • Assignee changed from Enis Nuredini to Carsten Rose
Actions #18

Updated by Carsten Rose over 1 year ago

  • Status changed from Feedback to Closed
  • Target version changed from 355 to 385
Actions #19

Updated by Carsten Rose over 1 year ago

  • Target version changed from 385 to 22.9.2
Actions #20

Updated by Carsten Rose over 1 year ago

  • Related to Bug #14791: dateTimePicker: AbstractBuildForm.php / Line: 3172 / Undefined offset: 2 added
Actions #21

Updated by Carsten Rose over 1 year ago

  • Target version changed from 22.9.2 to 22.10.0
Actions

Also available in: Atom PDF