Reedy has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/322608

Change subject: Fix comment blocks
......................................................................

Fix comment blocks

Change-Id: Ia08f110aa47ce5067547daef5d241f98d9e2c727
---
M TimedMediaHandler.hooks.php
1 file changed, 50 insertions(+), 8 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/TimedMediaHandler 
refs/changes/08/322608/1

diff --git a/TimedMediaHandler.hooks.php b/TimedMediaHandler.hooks.php
index 199b317..712ac8f 100644
--- a/TimedMediaHandler.hooks.php
+++ b/TimedMediaHandler.hooks.php
@@ -9,9 +9,14 @@
 
 class TimedMediaHandlerHooks {
 
-       // Register TimedMediaHandler namespace IDs
-       // These are configurable due to Commons history: T123823
-       // These need to be before registerhooks due to: T123695
+       /**
+        * Register TimedMediaHandler namespace IDs
+        * These are configurable due to Commons history: T123823
+        * These need to be before registerhooks due to: T123695
+        *
+        * @param array $list
+        * @return bool
+        */
        public static function addCanonicalNamespaces( array &$list ) {
                global $wgEnableLocalTimedText, $wgTimedTextNS;
                if ( $wgEnableLocalTimedText ) {
@@ -28,9 +33,15 @@
                return true;
        }
 
-       // At some point these should be registered in extension.json
-       // But for now we register them dynamically, because they are config 
dependent,
-       // while we have two players
+
+       /**
+        * At some point these should be registered in extension.json
+        * But for now we register them dynamically, because they are config 
dependent,
+        * while we have two players
+        *
+        * @param ResourceLoader $resourceLoader
+        * @return bool
+        */
        public static function resourceLoaderRegisterModules( &$resourceLoader 
) {
                $baseExtensionResource = [
                        'localBasePath' => __DIR__,
@@ -189,13 +200,17 @@
                return true;
        }
 
-       // Register TimedMediaHandler Hooks
+       /**
+        * Register TimedMediaHandler Hooks
+        *
+        * @return bool
+        */
        public static function register() {
                global $wgHooks, $wgJobClasses, 
$wgJobTypesExcludedFromDefaultQueue, $wgMediaHandlers,
                $wgResourceModules, $wgExcludeFromThumbnailPurge,
                $wgFileExtensions, $wgTmhEnableMp4Uploads, 
$wgExtensionAssetsPath,
                $wgMwEmbedModuleConfig, $wgEnableLocalTimedText, 
$wgTmhFileExtensions,
-               $wgTmhTheoraTwoPassEncoding, $wgTmhWebPlayer, 
$wgWikimediaJenkinsCI;
+               $wgTmhTheoraTwoPassEncoding, $wgWikimediaJenkinsCI;
 
                // set config for parser tests
                if ( isset( $wgWikimediaJenkinsCI ) && $wgWikimediaJenkinsCI  
=== true ) {
@@ -443,6 +458,10 @@
                return true;
        }
 
+       /**
+        * @param SkinTemplate $sktemplate
+        * @param array $links
+        */
        public static function onSkinTemplateNavigation( SkinTemplate 
&$sktemplate, array &$links ) {
                if ( self::isTimedMediaHandlerTitle( $sktemplate->getTitle() ) 
) {
                        $ttTitle = Title::makeTitleSafe( NS_TIMEDTEXT, 
$sktemplate->getTitle()->getDBkey() );
@@ -554,6 +573,7 @@
                }
                return true;
        }
+
        /**
         * Handle moved titles
         *
@@ -732,6 +752,10 @@
                return true;
        }
 
+       /**
+        * @param DatabaseUpdater $updater
+        * @return bool
+        */
        public static function checkSchemaUpdates( DatabaseUpdater $updater ) {
                $base = __DIR__;
 
@@ -750,6 +774,10 @@
                return true;
        }
 
+       /**
+        * @param array $qp
+        * @return bool
+        */
        public static function onwgQueryPages( $qp ) {
                $qp[] = [ 'SpecialOrphanedTimedText', 'OrphanedTimedText' ];
                return true;
@@ -780,6 +808,12 @@
                return true;
        }
 
+       /**
+        * @param $hash
+        * @param User $user
+        * @param $forOptions
+        * @return bool
+        */
        public static function changePageRenderingHash( &$hash, User $user, 
&$forOptions ) {
                if ( self::activePlayerMode() === 'videojs' ) {
                        if ( $user->getOption( 'tmh-videojs' ) === '1' ) {
@@ -789,6 +823,11 @@
                }
        }
 
+       /**
+        * @param $user
+        * @param $prefs
+        * @return bool
+        */
        public static function onGetBetaFeaturePreferences( $user, &$prefs ) {
                global $wgTmhUseBetaFeatures, $wgExtensionAssetsPath;
 
@@ -813,6 +852,9 @@
                return true;
        }
 
+       /**
+        * @return string
+        */
        public static function activePlayerMode() {
                global $wgTmhWebPlayer, $wgTmhUseBetaFeatures, $wgUser;
                $context = new RequestContext();

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia08f110aa47ce5067547daef5d241f98d9e2c727
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/TimedMediaHandler
Gerrit-Branch: master
Gerrit-Owner: Reedy <[email protected]>

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

Reply via email to