From ca83d978086cdc4f3c2b58ee5b4feb7c8d439c34 Mon Sep 17 00:00:00 2001 From: River Tarnell Date: Sat, 2 Jul 2005 22:39:22 +0000 Subject: [PATCH] avoid breaking on php5 --- includes/Skin.php | 41 ++++++++++++++++++++--------------------- 1 file changed, 20 insertions(+), 21 deletions(-) diff --git a/includes/Skin.php b/includes/Skin.php index 140e6ade87..65b16d31dc 100644 --- a/includes/Skin.php +++ b/includes/Skin.php @@ -430,34 +430,33 @@ END; # get a big array of the parents tree $parenttree = $wgTitle->getParentCategoryTree(); - # Render the array as a serie of links - # Need to give skin cause $this is undefined at this level - function walkThrough ($tree, &$skin) { - $return = ''; - foreach($tree as $element => $parent) { - if(empty($parent)) { - # element start a new list - $return .= '
'; - } else { - # grab the others elements - $return .= walkThrough($parent, $skin); - } - # add our current element to the list - $eltitle = Title::NewFromText($element); - if(!empty($parent)) $return .= ' > '; - $return .= $skin->makeLinkObj( $eltitle, $eltitle->getText() ) ; - } - return $return; - } - # Skin object passed by reference cause it can not be # accessed under the method subfunction walkThrough. - $s .= walkThrough($parenttree, $this); + $s .= Skin::drawCategoryBrowser($parenttree, $this); } return $s; } + # Render the array as a serie of links + function drawCategoryBrowser ($tree, &$skin) { + $return = ''; + foreach ($tree as $element => $parent) { + if (empty($parent)) { + # element start a new list + $return .= '
'; + } else { + # grab the others elements + $return .= Skin::drawCategoryBrowser($parent, $skin); + } + # add our current element to the list + $eltitle = Title::NewFromText($element); + if (!empty($parent)) $return .= ' > '; + $return .= $skin->makeLinkObj( $eltitle, $eltitle->getText() ) ; + } + return $return; + } + function getCategories() { $catlinks=$this->getCategoryLinks(); if(!empty($catlinks)) { -- 2.20.1