yakamara / yform

YForm für REDAXO 5 – Formulare im Frontend und Backend mit Verwaltung von Datenbank-Tabellen.
MIT License
77 stars 55 forks source link

Upload - Json Config nicht intuitiv #1365

Open tbaddade opened 1 year ago

tbaddade commented 1 year ago

Ich wusste jetzt nicht, bei welchen Key ich die Fehlermeldung hinterlege, wenn eine nicht erlaubte Extension verwendet wird.

Bisher:

{
    "sizes":{
        "min":0,
        "max":15360000
    },
    "allowed_extensions": [
        "jpg",
        "zip"
    ],
    "disallowed_extensions":[
        "exe"
    ],
    "check":[
        "multiple_extensions",
        "zip_archive"
    ],
    "messages":{
        "min_error": "min_error_msg",
        "max_error": "max_error_msg",
        "type_error": "type_error_msg",
        "empty_error": "empty_error_msg",
        "system_error": "system_error_msg",
        "type_multiple_error": "type_multiple-msg",
        "zip-type_error": "zip-type_error-msg {0}",
        "type_zip_error": "type_zip_error-msg",
        "delete_file":"delete_file_msg"
    }
}

Folgende Idee ist während des Grübelns herausgekommen:

{
  "sizes": {
    "min": 0,
    "max": 15360000
  },
  "extensions": {
    "allowed": [
      "jpg",
      "zip"
    ],
    "disallowed": [
      "exe"
    ], 
    "check": [
      "multiple_extensions",
      "zip_archive"
    ],
  },
  "messages": {
    "error": {
      "sizes": {
        "min": "min_error_msg",
        "max": "max_error_msg"
      }, 
      "extensions": {
        "allowed": "allowed_error_msg",
        "disallowed": "disallowed_error_msg",
        "check": "check_error_msg"
      }
    }, 
    "success": {
      "deleted": "delete_file_msg"
    }
  }
}

Ob das jetzt intuitiver ist, lasse ich gern andere entscheiden. Auch wo diese type-Errors hingehören.

"empty_error":"empty_error_msg" würde ich komplett rausnehmen und über empty-Validate prüfen.

Ziel: Ich würde gern wegkommen vom, Value anlegen und erst einmal probieren wann was passiert/erscheint.

Was meinst du @dergel ?

dergel commented 1 year ago

finde ich gut.. Leider haben wir das im Moment noch anders und BC müsste dann auch eingebaut werden.

alxndr-w commented 1 year ago

1339