Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Medienpool keine Rückmeldung bei Uploads von zu großen Dateien #5731

Open
danielplan opened this issue Jun 19, 2023 · 3 comments
Open

Medienpool keine Rückmeldung bei Uploads von zu großen Dateien #5731

danielplan opened this issue Jun 19, 2023 · 3 comments
Labels
Enhancement Improvements for existing features Mediapool "Mediapool"-Addon related things

Comments

@danielplan
Copy link

Description / Beschreibung
Wenn man im Medienpool versucht ein Bild hochzuladen bzw. auszutauschen, welches größer ist als die, in PHP.ini file festgelegte, maximale Dateigröße, wird keine Fehlermeldung angezeigt. Aus Usability Gründen wäre es vorteilhaft, wenn hier eine Fehlermeldung angezeigt wird, dass der Upload wegen zu großer Dateigröße nicht abgeschlossen wurde.

Affected versions / Verwendete Versionen
REDAXO: 5.15.1
PHP: 8.1.20
Database: MariaDB 10.5.19
AddOns: Medienpool

@skerbis
Copy link
Contributor

skerbis commented Jun 19, 2023

Das geschieht doch:

Bildschirmfoto 2023-06-19 um 11 44 41

@skerbis
Copy link
Contributor

skerbis commented Jun 23, 2023

@danielplan ?

@michael-kreatif
Copy link
Contributor

@skerbis Die Fehlermeldung kommt leider dann nicht, wenn post_max_size (ganze Body Request) kleiner als die hochgeladene Datei ist. Dann wird keine Meldung ausgegeben und die Seite neugeladen.

Der Grund liegt darin, dass dann die Request Variablen ($_POST, $_FILES) wie in der PHP Doku beschrieben (https://www.php.net/manual/en/ini.core.php#ini.post-max-size) leer sind und dadurch auch die Media Method in Zeile https://github.com/redaxo/redaxo/blob/main/redaxo/src/addons/mediapool/pages/upload.php#L17 ).

Wenn post_max größer als die Datei ist und upload_max_filesize, dann funktioniert die Fehlermeldung so wie bei dir im Screenshot.

Lösungsidee: Die PHP Empfehlung (https://stackoverflow.com/questions/2133652/how-to-gracefully-handle-files-that-exceed-phps-post-max-size) wäre, dass man in der Form Action URL zusätzlich einen Parameter mitgibt und dann kontrolliert, ob dieser Parameter gesetzt ist und die POST Variablen leer sind. Was meinst du?

@gharlan gharlan added Enhancement Improvements for existing features Mediapool "Mediapool"-Addon related things labels Jul 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improvements for existing features Mediapool "Mediapool"-Addon related things
Development

No branches or pull requests

4 participants