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

Fehler: Bildresize fehlgeschlagen (original) #66

Open
tbaddade opened this issue Feb 25, 2021 · 4 comments
Open

Fehler: Bildresize fehlgeschlagen (original) #66

tbaddade opened this issue Feb 25, 2021 · 4 comments

Comments

@tbaddade
Copy link
Member

Ich erhalte nach einem Upload diesen Fehler

Bildresize fehlgeschlagen (original)

Was mich jetzt wundert, warum will er überhaupt resizen?
Weder in den Einstellungen ist die Checkbox "Bildverkleinerung standardmäßig aktiviert" noch beim Upload selbst "übergroße Bilder vor Upload verkleinern" angehakt.

@IngoWinter
Copy link
Member

Kannst du das reproduzieren? Liegt es an einer Datei oder kommt das generell bei Bildern?

Die Fehlermeldung kommt von der internen PHP Resize Funktion des Uploaders, die eigentlich deaktiviert ist.

@tbaddade
Copy link
Member Author

tbaddade commented Feb 25, 2021

Liegt es an einer Datei oder kommt das generell bei Bildern?

Bei zwei Bilder ist das Original kaputt, bei zwei Anderen war eine Seite > 17.000px. Bei 1.600 Bilder recht wenig, aber man muss die hochgeladenen Bildern zum Schluss durchscrollen und auf Fehlermeldungen überprüfen. Vielleicht könnte man diese ggf. auch sammeln und am Ende anzeigen lassen?

Was mich nur allgemein irritiert ist, das überhaupt resize angewendet wird. Zumindest die zwei Bilder mit 17.000px hätten hochgeladen werden können.

Jetzt kommt noch bei 5 Bildern diese Meldung

Datei zu klein

@IngoWinter
Copy link
Member

IngoWinter commented Feb 25, 2021

Es ist kein resize der fehlschlägt, sondern die serverseitige Bilddrehung des Originals.
https://github.com/FriendsOfREDAXO/uploader/blob/master/lib/uploader_upload_handler.php#L1133

Wegen einer schwer in Worte zu fassenden Kombination schwer zu beschreibendener exif/browser/canvas/autorotate Gedönse
blueimp/jQuery-File-Upload#3530 (comment)
ist es erforderlich, manche Bilddateien serverseitig zu rotieren. Und dafür sind 17.000px wohl zu groß.

Um das abzufangen müsste man vor dem Resize checken, ob der Server das kann
https://github.com/FriendsOfREDAXO/uploader/blob/master/lib/uploader_upload_handler.php#L1119

Fehlermeldung "Datei zu klein" hatte ich noch nie. Die Mindestgröße ist hier mit 1 festgelegt
https://github.com/FriendsOfREDAXO/uploader/blob/master/lib/uploader_upload_handler.php#L119
Ich denke weniger geht nicht :)

@tbaddade
Copy link
Member Author

Die Fehlermeldung ist hier nur irreführend. Das Bild wurde dennoch, vermutlich, im Original hochgeladen. Eventuell kann man die Info mit ein wenig mehr Text und eher als Warnung denn als Fehler darstellen.

Selbst die zwei defekten Bilder lagen zum Schluss im Medienpool.

Wegen den zu kleinen Bildern. Ich vermute, dass es am Browser selbst lag. Nach 1600 Dateien kam auch die Info wegen Ressourcenverbrauch. Selbst ein normaler Download wurde verweigert. :)

Was für mich hier im Issue bleibt:
Fehler und Warnungen einsammeln und am unteren Ende, wo der Prozessbalken läuft, gesammelt ausgeben.

Danke für das AddOn.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants