Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Dateiverwaltung prüft beim Umbenennen Länge des Dateinamens ohne Dateiendung #8472

Closed
Mynyx opened this issue Sep 10, 2016 · 8 comments
Closed

Dateiverwaltung prüft beim Umbenennen Länge des Dateinamens ohne Dateiendung #8472

Mynyx opened this issue Sep 10, 2016 · 8 comments
Assignees
@leofeyer
Labels
defect
Milestone
3.5.17

Comments

@Mynyx
Copy link
Contributor

Mynyx commented Sep 10, 2016 •
edited
Loading

Wenn man in der Dateiverwaltung eine Datei bearbeitet und den Namen ändert, dann wird bei der Prüfung der Länge des Namens die Dateiendung nicht berücksichtigt. So kann man für den Namen 255 Zeichen eingeben, obwohl Name und Dateiendung gemeinsam nur 255 Zeichen lang sein dürften.
@leofeyer
Copy link
Member

@contao/developers Wie lösen wir das? Sollen wir maxlength einfach auf 245 setzen?

@Aybee
Copy link
Contributor

Aybee commented Sep 12, 2016

Wo findet denn die Prüfung statt? Kommen wir an dieser Stelle nicht an den Suffix ran?

@ausi
Copy link
Member

ausi commented Sep 12, 2016

IMO sollten wir die maxlength um die Länge des suffixes kürzen, sofern einfach möglich.

@leofeyer leofeyer added this to the 3.5.17 milestone Sep 19, 2016
@leofeyer leofeyer self-assigned this Sep 19, 2016
leofeyer added a commit that referenced this issue Sep 19, 2016
@leofeyer
Reduce the maximum field length by the file extension length (see #8472…
db56e8d 
…).
@leofeyer
Copy link
Member

Behoben in db56e8d.

@Mynyx
Copy link
Contributor Author

Mynyx commented Sep 19, 2016

Wäre es nicht notwendig, für den Punkt im Dateinamen noch ein Zeichen abzuziehen?

@leofeyer
Copy link
Member

Nein, denn der Punkt muss ja auch mit gespeichert werden.

@fritzmg
Copy link
Contributor

fritzmg commented Sep 19, 2016

Ich denke das meint er ja. Aber ich schätze in $dc->extension ist der Punkt bereits enthalten?

@leofeyer
Copy link
Member

Ja.

leofeyer added a commit that referenced this issue Sep 21, 2016
@leofeyer
Reduce the maximum field length by the file extension length (see #8472…
a788a72 
…).
jsonn pushed a commit to jsonn/pkgsrc that referenced this issue Sep 24, 2016
Update contao42 to 4.2.4. This is a leaf package.
872b72d 
### 4.2.4 (2016-09-21)

 * Handle special character passwords in the "close account" module (see contao/core#8455).
 * Handle broken SVG files in the Image and File class (see contao/core#8470).
 * Reduce the maximum field length by the file extension length (see contao/core#8472).
 * Fall back to the field name if there is no label (see contao/core#8461).
 * Do not assume NULL by default for binary fields (see contao/core#8477).
 * Correctly render the diff view if not the latest version is active (see contao/core#8481).
 * Update the list of countries and languages (see contao/core#8453).
 * Correctly set up the MooTools CDN URL (see contao/core#8458).
 * Also check the URL length when determining the search URL (see contao/core#8460).
 * Only regenerate the session ID upon login.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@leofeyer leofeyer

Labels
defect
Projects
None yet
Milestone
3.5.17
Development

No branches or pull requests
5 participants
@ausi @Aybee @leofeyer @Mynyx @fritzmg