diff --git a/lib/galaxy/webapps/tool_shed/util/container_util.py b/lib/galaxy/webapps/tool_shed/util/container_util.py index feaf0409542..d7ad2c071b5 100644 --- a/lib/galaxy/webapps/tool_shed/util/container_util.py +++ b/lib/galaxy/webapps/tool_shed/util/container_util.py @@ -257,13 +257,12 @@ class Tool( object ): class ToolDependency( object ): """Tool dependency object""" - def __init__( self, id=None, name=None, version=None, type=None, install_dir=None, readme=None, installation_status=None, repository_id=None, + def __init__( self, id=None, name=None, version=None, type=None, readme=None, installation_status=None, repository_id=None, tool_dependency_id=None, is_orphan=None ): self.id = id self.name = name self.version = version self.type = type - self.install_dir = install_dir self.readme = readme self.installation_status = installation_status self.repository_id = repository_id @@ -948,12 +947,10 @@ def build_tool_dependencies_folder( trans, folder_id, tool_dependencies, label=' # Insert a header row. tool_dependency_id += 1 if trans.webapp.name == 'galaxy': - # Include the installation directory. tool_dependency = ToolDependency( id=tool_dependency_id, name='Name', version='Version', type='Type', - install_dir='Install directory', readme=None, installation_status='Installation status', repository_id=None, @@ -964,7 +961,6 @@ def build_tool_dependencies_folder( trans, folder_id, tool_dependencies, label=' name='Name', version='Version', type='Type', - install_dir=None, readme=None, installation_status=None, repository_id=None, @@ -995,7 +991,6 @@ def build_tool_dependencies_folder( trans, folder_id, tool_dependencies, label=' name=name, version=None, type=type, - install_dir=None, readme=None, installation_status=installation_status, repository_id=repository_id, @@ -1013,7 +1008,6 @@ def build_tool_dependencies_folder( trans, folder_id, tool_dependencies, label=' name = requirements_dict[ 'name' ] version = requirements_dict[ 'version' ] type = requirements_dict[ 'type' ] - install_dir = requirements_dict.get( 'install_dir', None ) repository_id = requirements_dict.get( 'repository_id', None ) td_id = requirements_dict.get( 'tool_dependency_id', None ) if trans.webapp.name == 'galaxy': @@ -1024,7 +1018,6 @@ def build_tool_dependencies_folder( trans, folder_id, tool_dependencies, label=' name=name, version=version, type=type, - install_dir=install_dir, readme=None, installation_status=installation_status, repository_id=repository_id, diff --git a/templates/webapps/tool_shed/repository/common.mako b/templates/webapps/tool_shed/repository/common.mako index de2df5b768c..8ecd38e33e8 100644 --- a/templates/webapps/tool_shed/repository/common.mako +++ b/templates/webapps/tool_shed/repository/common.mako @@ -888,11 +888,7 @@ <${cell_type}>${tool_dependency.type | h} <${cell_type}> %if trans.webapp.name == 'galaxy': - %if is_missing: - ${tool_dependency.installation_status | h} - %elif tool_dependency.install_dir: - ${tool_dependency.install_dir | h} - %endif + ${tool_dependency.installation_status | h} %else: %if row_is_header: ${tool_dependency.is_orphan | h} diff --git a/test/tool_shed/functional/test_1170_complex_prior_installation_required.py b/test/tool_shed/functional/test_1170_complex_prior_installation_required.py index b769031c6a9..1cc29bf3e34 100644 --- a/test/tool_shed/functional/test_1170_complex_prior_installation_required.py +++ b/test/tool_shed/functional/test_1170_complex_prior_installation_required.py @@ -63,7 +63,7 @@ class TestComplexPriorInstallation( ShedTwillTestCase ): uncompress_file=True, remove_repo_files_not_in_tar=False, commit_message='Uploaded matplotlib tool dependency tarball.', - strings_displayed=['orphan'], + strings_displayed=[ 'This repository currently contains a single file named tool_dependencies.xml' ], strings_not_displayed=[] ) def test_0010_create_numpy_repository( self ): @@ -89,7 +89,7 @@ class TestComplexPriorInstallation( ShedTwillTestCase ): uncompress_file=True, remove_repo_files_not_in_tar=False, commit_message='Uploaded numpy tool dependency tarball.', - strings_displayed=['orphan'], + strings_displayed=[ 'This repository currently contains a single file named tool_dependencies.xml' ], strings_not_displayed=[] ) def test_0015_create_complex_repository_dependency( self ):