Latest 1.3 release throws ArgumentException when importing PDF
First off, thanks for the package. I used it to save quite a bit of time importing a bunch of PDFs. Along the way however, I ran into some errors that I thought be helpful to share:
The latest 1.3 release throws an ArgumentException when importing a PDF as it attempts to erroneously generate a thumbnail from a PDF. Therefore, multi import fails (without notification) as it successfully imports the first item, but then fails to generate a thumbnail from it, in turn bringing the entire process to a halt.
I downloaded the source from CodePlex, but you have the GenerateThumbnail logic commented out. Built that version and uploaded and PDFs now import fine (although there's some UI things that are changed and glitchy - two upload buttons, a close link that doesn't work, etc. - that comes from it being in the trunk).
I would have been tempted to make some changes and improvements to submit to you, but unfortunately, there are no tags in the tag directory, only the trunk. Moreover, the DLL for 1.3 says version 1.0. Might I suggest that you tag each release when you release it and also version the DLL?
Otherwise, again, thanks for the package (and the quick response to my previous request concerning the directory tree not showing) - it's very much appreciated!
Ted
(Umbraco 4.5.2, IIS7, ASP.NET 4.0, Integrated Pipeline)
I'm moving this out to it's own project on CodePlex using Mercurial for source control. I will update the project page with details hopefully later on today.
In the meantime I have tagged v1.3 in the current project and also committed a fix for the issue you have encountered with PDF's, there is now a check in place that the media is an image before attempting thumbnail generation.
If you want to build from the source and let me know how you get on then great, if not I will attempt to get an amended package out to you in the next 24 hours.
Wow. Fast response. All of that sounds good. The only thing is I'm an hour away from heading out on the road for vacation so won't get to it any time soon. The existing build works fine for now, although it'll be nice to get the UI cleaned up later.
Latest 1.3 release throws ArgumentException when importing PDF
First off, thanks for the package. I used it to save quite a bit of time importing a bunch of PDFs. Along the way however, I ran into some errors that I thought be helpful to share:
Otherwise, again, thanks for the package (and the quick response to my previous request concerning the directory tree not showing) - it's very much appreciated!
Ted
(Umbraco 4.5.2, IIS7, ASP.NET 4.0, Integrated Pipeline)
Hi Ted,
I'm moving this out to it's own project on CodePlex using Mercurial for source control. I will update the project page with details hopefully later on today.
In the meantime I have tagged v1.3 in the current project and also committed a fix for the issue you have encountered with PDF's, there is now a check in place that the media is an image before attempting thumbnail generation.
If you want to build from the source and let me know how you get on then great, if not I will attempt to get an amended package out to you in the next 24 hours.
Simon
Simon,
Wow. Fast response. All of that sounds good. The only thing is I'm an hour away from heading out on the road for vacation so won't get to it any time soon. The existing build works fine for now, although it'll be nice to get the UI cleaned up later.
Thanks!
Ted
is working on a reply...