From 86ddf120d7eebdf2d38c6bd13353a9244f62b818 Mon Sep 17 00:00:00 2001 From: Mikael Nordfeldth Date: Mon, 21 Apr 2014 12:33:41 +0200 Subject: [PATCH] Better event name (creating thumbnail _source_) --- EVENTS.txt | 2 +- lib/mediafile.php | 2 +- plugins/VideoThumbnails/VideoThumbnailsPlugin.php | 8 ++++---- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/EVENTS.txt b/EVENTS.txt index 705a761358..4d1215f6e0 100644 --- a/EVENTS.txt +++ b/EVENTS.txt @@ -1441,7 +1441,7 @@ OtherAccountProfiles: Hook to add account profiles to a user account profile blo text: text for the profile image: mini image for the profile -CreateFileImageThumbnail: Hook to create image thumbnail source from a File +CreateFileImageThumbnailSource: Hook to create image thumbnail source from a File - $file: MediaFile object with related metadata - &$imgPath: Path to image file which can be used as source for our thumbnail algorithm. - $media: MIME media type ('image', 'video', 'audio' etc.) diff --git a/lib/mediafile.php b/lib/mediafile.php index 64bc039f57..1791b1a698 100644 --- a/lib/mediafile.php +++ b/lib/mediafile.php @@ -120,7 +120,7 @@ class MediaFile $imgPath = null; $media = common_get_mime_media($this->mimetype); - if (Event::handle('CreateFileImageThumbnail', array($this, &$imgPath, $media))) { + if (Event::handle('CreateFileImageThumbnailSource', array($this, &$imgPath, $media))) { switch ($media) { case 'image': $imgPath = $this->getPath(); diff --git a/plugins/VideoThumbnails/VideoThumbnailsPlugin.php b/plugins/VideoThumbnails/VideoThumbnailsPlugin.php index 6621d0b104..42f45571eb 100644 --- a/plugins/VideoThumbnails/VideoThumbnailsPlugin.php +++ b/plugins/VideoThumbnails/VideoThumbnailsPlugin.php @@ -41,12 +41,12 @@ class VideoThumbnailsPlugin extends Plugin { /* * This function should only extract an image from the video stream - * and disregard any scaling aspects in the resulting file, since - * this will be handled in the core thumbnail algorithm. + * and disregard any cropping or scaling in the resulting file, as + * that will be handled in the core thumbnail algorithm. */ - public function onCreateFileImageThumbnail(MediaFile $file, &$imgPath, $media=null) + public function onCreateFileImageThumbnailSource(MediaFile $file, &$imgPath, $media=null) { - // This might accidentally pass application/ogg videos. + // The calling function might accidentally pass application/ogg videos. // If that's a problem, let's fix it in the calling function. if ($media !== 'video') { return true;