From 87ec822c343ca246bcea7369f271092991395702 Mon Sep 17 00:00:00 2001
From: Reedy <reedy@wikimedia.org>
Date: Fri, 20 Jul 2018 21:02:40 +0100
Subject: [PATCH] Explicitly mark Skin::makeVariablesScript() public

Remove else. If contains an unconditional return

Change-Id: I57a04339395d947dffbc3595d321106d4fd8256c
---
 includes/skins/Skin.php | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/includes/skins/Skin.php b/includes/skins/Skin.php
index b6de5108ac..ff2a9eafe9 100644
--- a/includes/skins/Skin.php
+++ b/includes/skins/Skin.php
@@ -401,15 +401,14 @@ abstract class Skin extends ContextSource {
 	 * @param string|null $nonce OutputPage::getCSPNonce()
 	 * @return string|WrappedString HTML
 	 */
-	static function makeVariablesScript( $data, $nonce = null ) {
+	public static function makeVariablesScript( $data, $nonce = null ) {
 		if ( $data ) {
 			return ResourceLoader::makeInlineScript(
 				ResourceLoader::makeConfigSetScript( $data ),
 				$nonce
 			);
-		} else {
-			return '';
 		}
+		return '';
 	}
 
 	/**
-- 
2.20.1