From 10a2cb03f3a32cb217c68d8ec4e6dde4756e76ae Mon Sep 17 00:00:00 2001
From: Jeremy Goecks <jeremy.goecks@emory.edu>
Date: Tue, 18 Jun 2013 12:43:38 -0400
Subject: [PATCH] Return empty string rather than None because Mako functions
 write by default; add comments as well.

---
 templates/refresh_frames.mako | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/templates/refresh_frames.mako b/templates/refresh_frames.mako
index efac9aaae68..97c97599b59 100644
--- a/templates/refresh_frames.mako
+++ b/templates/refresh_frames.mako
@@ -1,9 +1,11 @@
 ## Include JavaScript code to refresh Galaxy application frames as needed.
 <%def name="handle_refresh_frames()">
+    ## If no refresh frames, print nothing.
     <%
-        if not refresh_frames: return
+        if not refresh_frames: return ''
     %>
 
+    ## Write JavaScript to refresh specified frames.
     <script type="text/javascript">
     %if 'everything' in refresh_frames:
         parent.location.href="${h.url_for( controller='root' )}";