These are relatively minor bugs and I know there are larger fish to fry right now, but I thought I would point them out anyway.

When uploading a video, and you have the Video Thumbnail Option set to "User Uploaded Thumbnails" and you forget to specify an jpeg to be uploaded, acidfree complains and throws a bunch of errors on a white screen. Idealy, and when you have time to code it, a warning to "Please Specify A Thumbnail Image" would be nice.

When you click 'Download" on an acidfree video node where the video is an .mp4, things go pretty wrong, and the file seems to get streamed into the browser pager as text and eventually your browser crashes. Regular .movs are fine however. I would guess this a simple mimetype fix.

Comments

vhmauery’s picture

vhmauery commented

The mp4 problem sounds like a server/browser issue, not an acidfree problem. Or possibly a drupal problem if you are having drupal serve ALL your files (check your admin/settings 'Download method'). This doesn't seem to happen on my machines.

I will keep the thumbnail thing in mind for future bughunts. Thanks.

B-Dot’s picture

B-Dot commented

Yes, it does seem like a browser issue (Safari 2.0.1 on Mac OS X 10.4.2). The download works in Firefox on the Mac and on Windows IE.

I do have my Download Method setting on "Private - Files transfered by Drupal". Right now, I have things wide open on my site, but at some point I will want to restrict downloads to members, so I assumed it was bettter to go with the "Private" setting right away, given the warning that appears on that page:

If you want any sort of access control on the downloading of files, this needs to be set to private. You can change this at any time, however all download URLs will change and there may be unexpected problems so it is not recommended.

I've had a look around Safari's settings but I can't see anything that might relate to why this browser doesn't prompt to save the file as the others do.

On the subject of general functionality in this area, might you consider incorporating some of the latest features of the Video Module that Luke Last has been implementing? Things like fields for file size, clip length, bitrate, as well as custom fields?

vhmauery’s picture

vhmauery commented

Maybe for a later revision. It would require a db structure change, which I am not willing to do until other things settle down a bit.

vhmauery’s picture

vhmauery commented
Status: Active » Closed (fixed)

I am going to close this in favour of allowing the users to change thumbnails on already existing nodes. There is a current postponed feature request open for this already. So follow that. Sometime in the future I will work that problem out...

http://drupal.org/node/34542 -- custom thumbnails.