From 30ae3c595b6b2a5000207df5e2a357ff9f8a7f12 Mon Sep 17 00:00:00 2001
From: David Drury <david@drury.me.uk>
Date: Fri, 14 Feb 2014 13:23:16 +0000
Subject: Validation & Bug fix

The closing </div> for <div id="medialist-page"> is within an if
statement (if ($search)) which is false on initial page load. Apart from
a validation error this causes the page structure to be wrong and the
footer to be incorrectly styled
---
 medialist.php | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/medialist.php b/medialist.php
index ca04d34c48..dde4c19ec6 100644
--- a/medialist.php
+++ b/medialist.php
@@ -27,7 +27,8 @@ require_once WT_ROOT.'includes/functions/functions_edit.php';
 require_once WT_ROOT.'includes/functions/functions_print_facts.php';
 
 $controller = new WT_Controller_Page();
-$controller->setPageTitle(WT_I18N::translate('Media objects'));
+$controller->setPageTitle(WT_I18N::translate('Media objects'))
+			->pageHeader();
 
 $search = WT_Filter::get('search');
 $sortby = WT_Filter::get('sortby', 'file|title', 'title');
@@ -65,8 +66,6 @@ $medialist = WT_Query_Media::mediaList(
 	$filter
 );
 
-$controller->pageHeader();
-
 ?>
 <div id="medialist-page"><h2><?php echo $controller->getPageTitle(); ?></h2>
 
@@ -311,6 +310,7 @@ if ($search) {
 		echo '</tr></tbody>';
 		echo '</table>';
 	}
-  echo '</div>
-  </div>';
+  echo '</div>';
 }
+echo '</div>';
+
-- 
cgit v1.3