There is an error in the code that sets the time for extracting the thumbnails. I think it is a custom modification of the PHPVideoToolkit made for the Drupal version, because it is not present in the original code available in the project repository.

In lines 1855-1856 it reads:

<?php
$this->addCommand('-ss', $this->formatTimecode($seconds - 5, $timecode_format, '%hh:%mm:%ss:%ms', $frames_per_second), TRUE);
$this->addCommand('-ss', $this->formatTimecode(5, $timecode_format, '%hh:%mm:%ss:%ms', $frames_per_second), FALSE);
?>

The timecode format is wrong. Milliseconds must use a dot "." as separator. The correct code is:

<?php
$this->addCommand('-ss', $this->formatTimecode($seconds - 5, $timecode_format, '%hh:%mm:%ss.%ms', $frames_per_second), TRUE);
$this->addCommand('-ss', $this->formatTimecode(5, $timecode_format, '%hh:%mm:%ss.%ms', $frames_per_second), FALSE);
?>

You owe me a beer for the time wasted debugging this ;)

CommentFileSizeAuthor
#1 video-invalid_timecode-1907504-1.patch1.09 KBJvE
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JvE’s picture

Version: 7.x-2.9 » 7.x-2.x-dev
Component: Code » Video Transcoding
Status: Active » Needs review
FileSize
1.09 KB

Nasty bug. Wasted some of my time too.

The ffmpeg error message (for reference): "Invalid duration specification for ss"

Jorrit’s picture

Status: Needs review » Fixed

Thanks a lot for finding this! Next time you're in The Netherlands, I'll buy you a beer :)

marcusn’s picture

Next time you are in Finland I will buy you two beers.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.