dan/ffmpeg-mappable-media
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

We add a missing parameter to the ffmpeg command

Browse source
This commit is contained in:
Romain Biard 2016-11-16 12:43:40 -03:00
commit 18845d406e
2 changed files with 10 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
src/FFMpeg/Media/Video.php
View file

@ -185,10 +185,12 @@ class Video extends Audio
/**
* Gets the waveform of the video.
*
* @param integer $width
* @param integer $height
* @return Waveform
*/
public function waveform()
public function waveform($width = 640, $height = 120)
{
return new Waveform($this, $this->driver, $this->ffprobe);
return new Waveform($this, $this->driver, $this->ffprobe, $width, $height);
}
}

7
src/FFMpeg/Media/Waveform.php
View file

@ -23,11 +23,15 @@ class Waveform extends AbstractMediaType
{
/** @var Video */
private $video;
private $width;
private $height;
public function __construct(Video $video, FFMpegDriver $driver, FFProbe $ffprobe)
public function __construct(Video $video, FFMpegDriver $driver, FFProbe $ffprobe, $width, $height)
{
parent::__construct($video->getPathfile(), $driver, $ffprobe);
$this->video = $video;
$this->width = $width;
$this->height = $height;
}
/**
@ -79,6 +83,7 @@ class Waveform extends AbstractMediaType
*/
$commands = array(
'-i', 'input', '-filter_complex',
'showwavespic=s='.$this->width.'x'.$this->height,
'-frames:v', '1',
$this->pathfile
);