https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=44103 --- Comment #8 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Created attachment 211607 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=211607&action=edit Bug 44103: (follow-up) Similar change for alternative AjaxUpload in upload-images.js Main goal of this patch ia adding the same check around JSON.parse. Note that this AjaxUpload is a slightly changed copy/paste. Using this opportunity to rename this AjaxUpload to AjaxImageUpload to make a distinction between the two. I added a FIXME for the error processing in cbUpload. The logic used is the same as in a few other js files, but is just wrong. Correcting that is outside the scope of this bug. Opened a new bug. Test plan: Upload an image with upload-images that is larger than the max request body. Verify that it does not silently fail but shows the error: Upload status: Failed - (AJAX_FAILURE) Note: This message should be enhanced on a follow-up report. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> -- You are receiving this mail because: You are watching all bug changes.