From 0a01d219e0a05e47280142b86101073f3ef9be6a Mon Sep 17 00:00:00 2001
From: Michael Tremer <michael.tremer@ipfire.org>
Date: Thu, 22 Apr 2010 15:29:05 +0200
Subject: [PATCH] Add option to show no banner at all.

---
 www/templates/modules/sidebar-banner.html | 10 ++++++----
 www/webapp/banners.py                     |  3 ++-
 2 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/www/templates/modules/sidebar-banner.html b/www/templates/modules/sidebar-banner.html
index 7a10b3b7..fb891fca 100644
--- a/www/templates/modules/sidebar-banner.html
+++ b/www/templates/modules/sidebar-banner.html
@@ -1,5 +1,7 @@
-<h4>{{ item.title }}</h4>
+{% if item %}
+	<h4>{{ item.title }}</h4>
 
-<a href="{{ item.link }}" {% if item.link.startswith("http://") %}target="_blank" {% end %}>
-	<img src="{{ static_url(item.uri) }}" border="0" alt="{{ _("Banner") }}" />
-</a>
+	<a href="{{ item.link }}" {% if item.link.startswith("http://") %}target="_blank" {% end %}>
+		<img src="{{ static_url(item.uri) }}" border="0" alt="{{ _("Banner") }}" />
+	</a>
+{% end %}
diff --git a/www/webapp/banners.py b/www/webapp/banners.py
index ba21ab99..65aeac8d 100644
--- a/www/webapp/banners.py
+++ b/www/webapp/banners.py
@@ -20,7 +20,8 @@ class Banners(object):
 			self.items.append(Item(**_stringify(item)))
 
 	def get(self):
-		return random.choice(self.items)
+		if self.items:
+			return random.choice(self.items)
 
 
 banners = Banners("banners.json")
-- 
2.47.3