These errors occur after the import.

* Importing (COMEDY) COMMERCIAL.mpeg
* Importing alabared.mpg
* Importing aua.mpg
* Importing basketball.mpg
* Importing biciklis.mpg

* file (COMEDY) COMMERCIAL.mpeg is not a valid acidfree element type ()
* warning: Division by zero in E:\Web Project\knw61\includes\image.inc on line 125.
* warning: imagecreatetruecolor() [function.imagecreatetruecolor]: Invalid image dimensions in E:\Web Project\knw61\includes\image.inc on line 225.
* warning: imagecopyresampled(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 226.
* warning: imagejpeg(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 300.
* warning: imagedestroy(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 229.
* warning: getimagesize() [function.getimagesize]: Read error! in E:\Web Project\knw61\includes\image.inc on line 89.
* warning: Division by zero in E:\Web Project\knw61\includes\image.inc on line 125.
* warning: imagecreatetruecolor() [function.imagecreatetruecolor]: Invalid image dimensions in E:\Web Project\knw61\includes\image.inc on line 225.
* warning: imagecopyresampled(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 226.
* warning: imagejpeg(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 300.
* warning: imagedestroy(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 229.
* warning: getimagesize() [function.getimagesize]: Read error! in E:\Web Project\knw61\includes\image.inc on line 89.
* warning: Division by zero in E:\Web Project\knw61\includes\image.inc on line 125.
* warning: imagecreatetruecolor() [function.imagecreatetruecolor]: Invalid image dimensions in E:\Web Project\knw61\includes\image.inc on line 225.
* warning: imagecopyresampled(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 226.
* warning: imagejpeg(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 300.
* warning: imagedestroy(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 229.
* warning: getimagesize() [function.getimagesize]: Read error! in E:\Web Project\knw61\includes\image.inc on line 89.
* warning: Division by zero in E:\Web Project\knw61\includes\image.inc on line 125.
* warning: imagecreatetruecolor() [function.imagecreatetruecolor]: Invalid image dimensions in E:\Web Project\knw61\includes\image.inc on line 225.
* warning: imagecopyresampled(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 226.
* warning: imagejpeg(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 300.
* warning: imagedestroy(): supplied argument is not a valid Image resource in E:\Web Project\knw61\includes\image.inc on line 229.
* warning: getimagesize() [function.getimagesize]: Read error! in E:\Web Project\knw61\includes\image.inc on line 89.

CommentFileSizeAuthor
#3 video_default_sizes.patch.txt3.01 KBvhmauery
#2 class_video.inc_0.patch2.29 KBxtony
#1 class_video.inc.patch2.32 KBxtony

Comments

xtony’s picture

xtony commented
StatusFileSize
new class_video.inc.patch2.32 KB

The problem causes by the video's width and height which are not set.

I added 2 more options to Video Manipulation section, default height and width.
The module will use these values as the default video dimension.

Please review this patch.

Thank you :)

xtony’s picture

xtony commented
StatusFileSize
new class_video.inc_0.patch2.29 KB

Try this patch, i have changed some strings in the patch.

vhmauery’s picture

vhmauery commented
Status: Active » Needs review
StatusFileSize
new video_default_sizes.patch.txt3.01 KB

try this patch -- it makes it so the size settings only show up if you are using 'none' as the thumbnail option.

xtony’s picture

xtony commented

I'm confusing. is this thumbnail or video size?
Why do it has to be hidden?

vhmauery’s picture

vhmauery commented

It has to be hidden because it doesn't make any sense for the mplayer or user-supplied thumbnail options. Just like mplayer path doesn't make sense for user-supplied thumbnails.

What you don't get here is that the video size gets automatically detected in the case of mplayer or user-supplied thumbnails. When you extract a frame from a video stream, the frame is the same dimensions as the video. That sets the video height and width. Then, that image gets resized down to the right size for the thumbnail.

xtony’s picture

xtony commented

Ok, i get it.

Thank you very much :)

vhmauery’s picture

vhmauery commented

So does the patch fix your problems? Or are you still seeing lots of errors. Frankly this fix wouldn't have been my first guess at fixing based on the errors. But if it fixes your problems, I will commit it.

xtony’s picture

xtony commented

Yes, everything is working fine now.

Thank you.

vhmauery’s picture

vhmauery commented
Status: Needs review » Fixed

fixed in CVS. Should be repackaged within 12 hours.

vhmauery’s picture

vhmauery commented
Status: Fixed » Closed (fixed)