Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
We have a Drupal 9 site where users can upload photos to albums using the plupload widget. After upload, images are resized to max 800x600 with JPGs quality 75%. Unfortunately, we keep running into WSOD due to timeouts (as previously reported in this issue).
After much testing on both Pantheon and Platform.sh, the bottleneck seems to be the host's PHP setting of MAX_EXECUTION_TIME. For Pantheon, this is hard capped at 59 seconds, while Platform.sh lets you increase it to 300 seconds.
We've tried to resize and compress all images in Photoshop to max 800x600 at 70% quality, but even uploading 300 of these images (totalling about 25MB) often results in a WSOD. Strangely enough, all images usually do seem to end up in the album after a while.
Steps to reproduce
- Create a free Pantheon site
- Install Drupal 9
- Install and enable the Album Photos contrib module
- Go to
/admin/config/media/photosand enable Plupload and set maximum image resolution to 800x600 - Create a new album and add a large number of photos (50 to 500 depending on size)
- Plupload will start uploading the files one by one
- Once Plupload finishes uploading all files, click 'Confirm Upload'
- On a Pantheon site, after exactly 59 seconds, you will see a WSOD with
Application did not respond in time. (Results may vary depending on your server capacity and settings.)
Proposed resolution
We're hoping that integrating Drupal's Batch API (when using plupload) will improve the process of uploading large amounts of images.
Remaining tasks
- Determine how to integrate Drupal's batch API in this module
- Make the necessary changes
- Test and review
- Commit!
User interface changes
Probably none
API changes
Probably none
Data model changes
Probably none
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | photos-plupload-batch-3347017-2.patch | 13.84 KB | Nathaniel |
Comments
Comment #2
Nathaniel CreditAttribution: Nathaniel commentedStarted some code for this. Not thoroughly tested, but seems to work.
Comment #3
tintoWow, that's fast! Thanks @nathaniel!
I've tested patch #2 on a fresh Drupal install - both on my local machine and on a new Pantheon site - and it seems to work like a charm on both sites. Here's the combination of modules/versions that I used:
So looking good here!
Comment #4
FrontmobeI have applied the patch provided and can confirm that the Plupload uploads now work flawlessly with the batch API!
I tested this locally as well as remotely with a high number of large files, combined with short PHP timeouts configured. The new pluploadSubmitBatch() and batchFinished() functions in PhotosUploadForm.php work and the integration with batch API like progress messages work as expected. No errors show up in the php error log when using the module with the patch applied.
I had to manually apply the patch to Photos module 6.0.2, because the patch is created against de dev version of the module. Any chance of maybe tagging a new release once this patch is confirmed to work ok?
Many thanks for this nice extension of the module, using batch API for bulk uploads really increases stability.
Comment #5
Nathaniel CreditAttribution: Nathaniel commentedSounds good to me!
Tested "Maximum number of photos per album" and that still works as expected. I'll prep a new release.
Comment #7
Nathaniel CreditAttribution: Nathaniel commented