From 0a48ad6308986ef125adf850422320b1231bd3f8 Mon Sep 17 00:00:00 2001
From: Kai_WMDE <kai.nissen@wikimedia.de>
Date: Tue, 10 Nov 2015 14:53:07 +0100
Subject: [PATCH] applied coding conventions

---
 inc/wikibox.class.php | 52 ++++++++++++++++++++++++++-----------------
 1 file changed, 31 insertions(+), 21 deletions(-)

diff --git a/inc/wikibox.class.php b/inc/wikibox.class.php
index dbd43b6..c3024c1 100644
--- a/inc/wikibox.class.php
+++ b/inc/wikibox.class.php
@@ -1,48 +1,58 @@
 <?php
-require_once("wikirip.inc.php");
+require_once 'wikirip.inc.php';
 
 class WikiBox extends WikiRip {
 	var $test = null;
 	var $testText = null;
 
-	function __construct($url, $cache = null, $cachedir = null) {
-		parent::__construct($url, $cache, $cachedir);
+	function __construct( $url, $cache = null, $cachedir = null ) {
+		parent::__construct( $url, $cache, $cachedir );
 	}
 
 	function pick_page( $listPage, $purge = false, $testPage = false ) {
-		$list = $this->rip_page($listPage, 'raw', $purge);
-		if (!$list && !$testPage) return false;
+		$list = $this->rip_page( $listPage, 'raw', $purge );
+		if ( !$list && !$testPage ) {
+			return false;
+		}
 
-		$list = preg_replace('@<!--.*?-->@s', '', $list);
-		if (!preg_match_all('/^\*+.*\[\[(.+?)( *\|.*?)?\]\]/m', $list, $mm, PREG_SET_ORDER)) $mm = array();
-		if (!$mm && !$testPage) return  false;
+		$list = preg_replace( '@<!--.*?-->@s', '', $list );
+		if ( !preg_match_all( '/^\*+.*\[\[(.+?)( *\|.*?)?\]\]/m', $list, $mm, PREG_SET_ORDER ) ) {
+			$mm = array();
+		}
+		if ( !$mm && !$testPage) {
+			return false;
+		}
 
 		$list = array();
 
-		foreach ($mm as $m) {
-			$list[] = WikiRip::normalizeTitle($m[1]);
+		foreach ( $mm as $m ) {
+			$list[] = WikiRip::normalizeTitle( $m[1] );
 		}
 
 		if ( $testPage ) {
-			$page = WikiRip::normalizeTitle("Web:{$testPage}");
+			$page = WikiRip::normalizeTitle( "Web:{$testPage}" );
 			$this->cache_duration = 0; //disable cache for testing
-			$html = $this->rip_page($page, 'render', true);
-			if (!$html) $html = "<i class='error'>page not found: ".htmlspecialchars($page)."</i>";
-			else if (!in_array($page, $list) && !preg_match('/wikibox-test/i', $html)) $html = "<i class='error'>inactive feature page not marked with \"wikibox-test\": ".htmlspecialchars($page)."</i>";
+			$html = $this->rip_page( $page, 'render', true );
+			if ( !$html ) {
+				$html = '<i class="error">page not found: ' . htmlspecialchars( $page ) . '</i>';
+			} else if ( !in_array( $page, $list ) && !preg_match( '/wikibox-test/i', $html ) ) {
+				$html = '<i class="error">inactive feature page not marked with "wikibox-test": ' . htmlspecialchars( $page ) . '</i>';
+			}
 		} else {
-			$i = mt_rand(0, count($list)-1 );
+			$i = mt_rand( 0, count( $list ) - 1 );
 			$page = $list[$i];
 				
-			$html = $this->rip_page($page, 'render', $purge);
-			if (!$html) return false;
+			$html = $this->rip_page( $page, 'render', $purge );
+			if ( !$html ) {
+				return false;
+			}
 		}
 
-		if ($html) {
-			$html = preg_replace('!(<(span|div)[^<>]*>)?\s*BEGIN_NORIP.*?END_NORIP\s*(</\2 *>)?!', '', $html);
-			$html = preg_replace('!(<(span|div)[^<>]*wikibox-test[^<>]*>).*(</\2 *>)!', '', $html);
+		if ( $html ) {
+			$html = preg_replace( '!(<(span|div)[^<>]*>)?\s*BEGIN_NORIP.*?END_NORIP\s*(</\2 *>)?!', '', $html );
+			$html = preg_replace( '!(<(span|div)[^<>]*wikibox-test[^<>]*>).*(</\2 *>)!', '', $html );
 		}
 
 		return $html;
 	}
 }
-