VlaamseKunstcollectie / Imagehub

A IIIF Presentation API compliant aggregator and web service
GNU General Public License v3.0
1 stars 2 forks source link

bug: fix array to string conversion when trying to output manifest count #26

Closed Hobbesball closed 5 years ago

Hobbesball commented 5 years ago

https://github.com/VlaamseKunstcollectie/Imagehub/blob/98d864e140b098f71ec2cd3507a12785230e880e/src/AppBundle/Imagehub/Command/GenerateManifestsCommand.php#L756

On the above offending line you try to output how many manifests were generated in a log. In this log you call on the array that holds all the manifests. This causes Symfony to throw an error: array to string conversion.

Solution: initialize a new variable , e.g. 'manifestcount', and count the amount of members in the $manifests array. Call on the $manifestcount variable in your log.

Kitania commented 5 years ago

Fixed in d33c99d468fadaa7f5f903a9ff3ba9015372069f

Simply forgot to use count($manifests) instead of $manifests.