Project

General

Profile

Feature #5112

FormElement Check Type = allbut nicht kompatibel mit Encode = Specialchar

Added by Elias Villiger almost 3 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
14.12.2017
Due date:
% Done:

100%

Estimated time:
Discuss:

Description

Wenn ein FormElement auf check type "allbut" gesetzt ist, sind dadurch Zeichen wie ' und " erlaubt, ein & ist aber z.B. nicht erlaubt.

Wenn nun encode auf specialchar steht und ein ' oder " in das Feld eingegeben wird das Zeichen zum Speichern in seine HTML-Repräsentation umgewandelt, also z.B. ". Diese Zeichenfolge wird aber aufgrund dem & Zeichen nicht akzeptiert, d.h. der Text wird nicht gespeichert. Eine Meldung wird aber nicht angezeigt, wie bei anderen unzulässigen Eingaben.

1) Man könnte das Problem entschärfen, indem man eine Fehlermeldung "unzulässige Zeichen" anzeigt -> macht aber wenig Sinn, weil ja der User gar kein unzulässiges Zeichen eingegeben hat.

2) Das Feld encode wird bei Auswahl von checkType allbut automatisch ausgeblendet (dynamic update) und auf "none" gesetzt.

3) Bei Auswahl der Kombination checkType=allbut und encode=specialchar wird im FormEditor rechts eine Warnung angezeigt, dass diese Kombination bei gewissen Eingaben nicht kompatibel ist.

Incompatibility Warning.PNG (106 KB) Incompatibility Warning.PNG Elias Villiger, 12.10.2018 17:08
1754

Associated revisions

Revision 844a9d80 (diff)
Added by Elias Villiger about 2 years ago

F5112 - Add incompatibility warning for encode specialchar and checkType allbut - refs #5112, closes #5112

Revision 072c1960 (diff)
Added by Carsten Rose about 2 years ago

F5112 - Add incompatibility warning for encode specialchar and checkType allbut… - change: no warning, allow '&'. This makes more sense than throwing a warning and forcing the use to switch to encoding='NONE'.

History

#1 Updated by Elias Villiger almost 3 years ago

  • Description updated (diff)

#2 Updated by Elias Villiger almost 3 years ago

  • Description updated (diff)

#3 Updated by Carsten Rose over 2 years ago

  • Target version set to next3

#4 Updated by Elias Villiger about 2 years ago

1754

Vorschlag als Screenshot (und Merge Request)

#5 Updated by Elias Villiger about 2 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

#6 Updated by Carsten Rose about 2 years ago

  • Target version changed from next3 to 18.10.2

Also available in: Atom PDF