Slightly improve lens info by using LensSpec tag instead of Lens tag

[mir07]

Fixes #234

Signed-off-by: Michael Rasmussen [email protected]

[d7415]

As there's already a hierarchy here, I'd just add any new tags to the list rather than replacing them. I'm not sure how likely it is for exiftool to return Lens but not LensSpec but if it's possible we should account for it. This would also allow us to include LensID in the list, as requested.

LensID isn't an official EXIF tag, but looks like it's a combination of others so I guess it's probably ok (as we're officially supporting exiftool anyway)

[d7415]

(I've added a reference to the issue in the description above so that it can be automatically closed on merge)

[mir07]

I have come to the same conclusion so I will prepare a patch to handle all possible lens info tags ordering from best to worst so:

1. LensID
2. LensSpec
3. Lens

[ildyria]

Also if such thing can also be added to Lychee-Laravel that would be nice. https://github.com/LycheeOrg/Lychee-laravel

[mir07]

I know but I have never been able to have a functioning Lychee-Laravel installation even when following the instructions to the letter. Without a functioning Lychee-Laravel installation providing patches is impossible.

[d7415]

Thanks @mir07 !