From ce127d5e84cbdaa97c85c6e33c6b374609609c11 Mon Sep 17 00:00:00 2001 From: John Chilton Date: Mon, 5 Aug 2013 23:50:04 -0500 Subject: [PATCH] Fix tool.requires_login behvaior broken with dynamic tool box filters (sorry again Dan and thanks for the bug report). --- lib/galaxy/tools/filters/__init__.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/lib/galaxy/tools/filters/__init__.py b/lib/galaxy/tools/filters/__init__.py index a99a94d7454..11b62334784 100644 --- a/lib/galaxy/tools/filters/__init__.py +++ b/lib/galaxy/tools/filters/__init__.py @@ -25,8 +25,7 @@ class FilterFactory( object ): """ filters = self.default_filters.copy() - if not trans.user: - filters[ "tool" ].append( _requires_login ) + filters[ "tool" ].append( _handle_requires_login ) if kwds.get( "trackster", False ): filters[ "tool" ].append( _has_trackster_conf ) @@ -59,8 +58,8 @@ def _not_hidden( context, tool ): return not tool.hidden -def _requires_login( context, tool ): - return not tool.require_login +def _handle_requires_login( context, tool ): + return not tool.require_login or context.trans.user def _has_trackster_conf( context, tool ):