Reedy has uploaded a new change for review.
https://gerrit.wikimedia.org/r/67451
Change subject: Fixup mixed leading whitespace
......................................................................
Fixup mixed leading whitespace
Minor comment fixes
Change-Id: Iaf5ab35f0c0513a906531cda29bdadf71b4574b9
---
M GuidedTourHooks.php
M includes/ResourceLoaderGuidedTourSiteStylesModule.php
2 files changed, 15 insertions(+), 9 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/GuidedTour
refs/changes/51/67451/1
diff --git a/GuidedTourHooks.php b/GuidedTourHooks.php
index 754f774..2630a4c 100644
--- a/GuidedTourHooks.php
+++ b/GuidedTourHooks.php
@@ -92,14 +92,14 @@
* @param OutputPage $out output page
* @param string $tourName the tour name, such as 'gettingstarted'
*
- * @return true if a module was added, false otherwise
+ * @return bool true
*/
private static function addTour( $out, $tourName ) {
global $wgResourceModules;
// Exclude '-' because MediaWiki message keys use it as a
separator after the tourname.
// Exclude '.' because module names use it as a separator.
- if ( $tourName !== NULL && strpbrk( $tourName, '-.' ) === FALSE
) {
+ if ( $tourName !== null && strpbrk( $tourName, '-.' ) === false
) {
$tourModuleName = "ext.guidedTour.tour.$tourName";
if ( isset ( $wgResourceModules[$tourModuleName] ) ) {
// Add the tour itself for extension-defined
tours.
@@ -135,7 +135,6 @@
* @return bool true in all cases
*/
public static function onBeforePageDisplay( $out, $skin ) {
- global $wgResourceModules;
// test for tour enabled in url first
$request = $out->getRequest();
$queryTourName = $request->getVal( self::TOUR_PARAM );
@@ -175,12 +174,16 @@
* @param array &$files test files
* @return bool always true
*/
- public static function onUnitTestsList( &$files ) {
- $testDir = __DIR__ . '/tests';
- $files = array_merge( $files, glob( "$testDir/*Test.php" ) );
- return true;
- }
+ public static function onUnitTestsList( &$files ) {
+ $testDir = __DIR__ . '/tests';
+ $files = array_merge( $files, glob( "$testDir/*Test.php" ) );
+ return true;
+ }
+ /**
+ * @param $redirectParams array
+ * @return bool
+ */
public static function onRedirectSpecialArticleRedirectParams(
&$redirectParams ) {
array_push( $redirectParams, self::TOUR_PARAM, 'step' );
diff --git a/includes/ResourceLoaderGuidedTourSiteStylesModule.php
b/includes/ResourceLoaderGuidedTourSiteStylesModule.php
index 67bbe20..a61c1da 100644
--- a/includes/ResourceLoaderGuidedTourSiteStylesModule.php
+++ b/includes/ResourceLoaderGuidedTourSiteStylesModule.php
@@ -23,7 +23,10 @@
);
}
- // Load custom CSS after all other Guiders and GuidedTour CSS.
+ /**
+ * Load custom CSS after all other Guiders and GuidedTour CSS.
+ * @return Array
+ */
public function getDependencies() {
return array(
'ext.guidedTour.styles',
--
To view, visit https://gerrit.wikimedia.org/r/67451
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iaf5ab35f0c0513a906531cda29bdadf71b4574b9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/GuidedTour
Gerrit-Branch: master
Gerrit-Owner: Reedy <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
