Maximum number of photos per album

Hi Nathaniel,

Thanks for your quick feedback, I think that would be a great way to add the functionality of setting a maximum number of photos per album.

I attached an elementary patch that updates the PhotosUpload service and validation for the photos_image entity form, like you mentioned. To keep things simple, I set $album_max_size in the code for now but setting that value in the module config at /admin/config/media/photos under "Album limit" definitely sounds like the way to go.

One thing that still puzzles me is that although I include "return FALSE;" in the service, the image still is created as a Drupal file on the file system. The image is not included in the album (due to the validation in the form), but returning FALSE when $photocount exceeds the maximum number of photos per album does not prevent the creation of the Drupal file. Do you have any suggestions on how to solve this?

Especially when using plupload to upload a large number of photos, this would otherwise mean that still many Drupal files would be created even though the limit is exceeded.

Have a good weekend!

Hi Nathaniel,

I can confirm the functionality in the patch works perfectly! The config under /admin/config/media/photos -> "Album limit" is clear and the added messages are too. Uploading more images than allowed now also does not result in Drupal files being created for the excess images, which keeps things tidy. This works when using Plupload or using the default file widgets on the upload form.

From a functionality standpoint I would say this is now complete, thank you for working on this so quickly!

Cheers,
Marco