jenkins-bot has submitted this change and it was merged.

Change subject: Use TitleMove hook to cleanup transcodes before move
......................................................................


Use TitleMove hook to cleanup transcodes before move

use new TitleMove hook added to core in Change-Id: Ic5e659abc

Bug: 53480
Change-Id: Ie61284b32c770f41495b357f57552694530ee834
---
M TimedMediaHandler.hooks.php
1 file changed, 2 insertions(+), 5 deletions(-)

Approvals:
  Parent5446: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/TimedMediaHandler.hooks.php b/TimedMediaHandler.hooks.php
index a681a54..8299df9 100644
--- a/TimedMediaHandler.hooks.php
+++ b/TimedMediaHandler.hooks.php
@@ -97,7 +97,7 @@
                $wgHooks['UploadComplete'][] = 
'TimedMediaHandlerHooks::checkUploadComplete';
 
                // When an image page is moved:
-               $wgHooks['TitleMoveComplete'][] = 
'TimedMediaHandlerHooks::checkTitleMoveComplete';
+               $wgHooks['TitleMove'][] = 
'TimedMediaHandlerHooks::checkTitleMove';
 
                // When image page is deleted so that we remove transcode 
settings / files.
                $wgHooks['FileDeleteComplete'][] = 
'TimedMediaHandlerHooks::onFileDeleteComplete';
@@ -256,7 +256,6 @@
                }
                return true;
        }
-
        /**
         * Handle moved titles
         *
@@ -266,11 +265,9 @@
         * @param $title Title
         * @param $newTitle Title
         * @param $user User
-        * @param $oldid int
-        * @param $newid int
         * @return bool
         */
-       public static function checkTitleMoveComplete( &$title, &$newTitle, 
&$user, $oldid, $newid ){
+       public static function checkTitleMove( $title, $newTitle, $user ){
                if( self::isTranscodableTitle( $title ) ){
                        // Remove all the transcode files and db states for 
this asset
                        // ( will be re-added the first time the asset is 
displayed with its new title )

-- 
To view, visit https://gerrit.wikimedia.org/r/83996
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie61284b32c770f41495b357f57552694530ee834
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/TimedMediaHandler
Gerrit-Branch: master
Gerrit-Owner: J <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: J <[email protected]>
Gerrit-Reviewer: Parent5446 <[email protected]>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to