[Fusionforge-commits] r8857 - trunk/gforge/plugins/mediawiki/mediawiki-skin

Thorsten Glaser mirabilos at libremir.placard.fr.eu.org
Tue Feb 23 18:01:47 CET 2010


Author: mirabilos
Date: 2010-02-23 18:01:47 +0100 (Tue, 23 Feb 2010)
New Revision: 8857

Modified:
   trunk/gforge/plugins/mediawiki/mediawiki-skin/FusionForge.php
Log:
<ul></ul> is invalid, there must be at least one <li>, so count


Modified: trunk/gforge/plugins/mediawiki/mediawiki-skin/FusionForge.php
===================================================================
--- trunk/gforge/plugins/mediawiki/mediawiki-skin/FusionForge.php	2010-02-23 15:50:21 UTC (rev 8856)
+++ trunk/gforge/plugins/mediawiki/mediawiki-skin/FusionForge.php	2010-02-23 17:01:47 UTC (rev 8857)
@@ -180,16 +180,22 @@
 	<div class="portlet" id="p-personal">
 		<h5><?php $this->msg('personaltools') ?></h5>
 		<div class="pBody">
-			<ul>
-<?php 			foreach($this->data['personal_urls'] as $key => $item) { ?>
+<?php			$put_ul = false;
+ 			foreach($this->data['personal_urls'] as $key => $item) {
+				if (!$put_ul) {
+					echo "\t\t\t<ul>\n";
+					$put_ul = true;
+				} ?>
 				<li id="<?php echo Sanitizer::escapeId( "pt-$key" ) ?>"<?php
 					if ($item['active']) { ?> class="active"<?php } ?>><a href="<?php
 				echo htmlspecialchars($item['href']) ?>"<?php echo $skin->tooltipAndAccesskey('pt-'.$key) ?><?php
 				if(!empty($item['class'])) { ?> class="<?php
 				echo htmlspecialchars($item['class']) ?>"<?php } ?>><?php
 				echo htmlspecialchars($item['text']) ?></a></li>
-<?php			} ?>
-			</ul>
+<?php			}
+			if ($put_ul) {
+				echo "\t\t\t</ul>\n";
+			} ?>
 		</div>
 	</div>
 	<div class="portlet" id="p-logo">




More information about the Fusionforge-commits mailing list