- galaxy-commits - lists.galaxyproject.org

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 29 May '13

29 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/7b2c4fff40c0/ Changeset: 7b2c4fff40c0 Branch: next-stable User: carlfeberhard Date: 2013-05-29 22:25:01 Summary: HDA.get_api_value: remove history_id as tuple Affected #: 1 file diff -r 05674ed7665cbd7a1012d8b31b728cdbf52a3dbd -r 7b2c4fff40c0682bdb64a5fe8049df15cf836541 lib/galaxy/model/__init__.py --- a/lib/galaxy/model/__init__.py +++ b/lib/galaxy/model/__init__.py @@ -1684,7 +1684,7 @@ misc_blurb = hda.blurb ) if hda.history is not None: - rval['history_id'] = hda.history.id, + rval['history_id'] = hda.history.id rval[ 'peek' ] = to_unicode( hda.display_peek() ) for name, spec in hda.metadata.spec.items(): https://bitbucket.org/galaxy/galaxy-central/commits/93f6e07db854/ Changeset: 93f6e07db854 User: carlfeberhard Date: 2013-05-29 22:26:01 Summary: merge next-stable Affected #: 1 file diff -r 13797aac2667307106edeaba3076f04abd96e60d -r 93f6e07db8549411be10ba5385ba4cf9fab0c046 lib/galaxy/model/__init__.py --- a/lib/galaxy/model/__init__.py +++ b/lib/galaxy/model/__init__.py @@ -1684,7 +1684,7 @@ misc_blurb = hda.blurb ) if hda.history is not None: - rval['history_id'] = hda.history.id, + rval['history_id'] = hda.history.id rval[ 'peek' ] = to_unicode( hda.display_peek() ) for name, spec in hda.metadata.spec.items(): Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 29 May '13

29 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/05674ed7665c/ Changeset: 05674ed7665c Branch: next-stable User: greg Date: 2013-05-29 22:18:27 Summary: Fix styles for automated tool test results containers in the tool shed. Affected #: 2 files diff -r eb524e42fc65eb69274857bcaf85030f0edd32f0 -r 05674ed7665cbd7a1012d8b31b728cdbf52a3dbd lib/galaxy/webapps/tool_shed/api/repositories.py --- a/lib/galaxy/webapps/tool_shed/api/repositories.py +++ b/lib/galaxy/webapps/tool_shed/api/repositories.py @@ -33,7 +33,7 @@ @web.expose_api_anonymous def get_ordered_installable_revisions( self, trans, name, owner, **kwd ): """ - GET /api/repository/get_ordered_installable_revisions + GET /api/repositories/get_ordered_installable_revisions :param name: the name of the Repository :param owner: the owner of the Repository @@ -59,7 +59,7 @@ @web.expose_api_anonymous def get_repository_revision_install_info( self, trans, name, owner, changeset_revision, **kwd ): """ - GET /api/repository/get_repository_revision_install_info + GET /api/repositories/get_repository_revision_install_info :param name: the name of the Repository :param owner: the owner of the Repository diff -r eb524e42fc65eb69274857bcaf85030f0edd32f0 -r 05674ed7665cbd7a1012d8b31b728cdbf52a3dbd templates/webapps/tool_shed/repository/common.mako --- a/templates/webapps/tool_shed/repository/common.mako +++ b/templates/webapps/tool_shed/repository/common.mako @@ -395,13 +395,23 @@ <%def name="render_failed_test( failed_test, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( failed_test.id ) %> + <style type="text/css"> + #failed_test_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="readme_table"> + <table id="failed_test_table"><tr><td bgcolor="#FFFFCC"><b>Tool id:</b> ${failed_test.tool_id | h}</td></tr><tr><td><b>Tool version:</b> ${failed_test.tool_id | h}</td></tr><tr><td><b>Test:</b> ${failed_test.test_id | h}</td></tr> @@ -485,14 +495,14 @@ encoded_id = trans.security.encode_id( invalid_tool_dependency.id ) %><style type="text/css"> - #readme_table{ table-layout:fixed; - width:100%; - overflow-wrap:normal; - overflow:hidden; - border:0px; - word-break:keep-all; - word-wrap:break-word; - line-break:strict; } + #invalid_td_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } </style><tr class="datasetRow" %if parent is not None: @@ -500,7 +510,9 @@ %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - ${ invalid_tool_dependency.error | h } + <table id="invalid_td_table"> + <tr><td>${ invalid_tool_dependency.error | h }</td></tr> + </table></td></tr><% @@ -511,13 +523,23 @@ <%def name="render_missing_test_component( missing_test_component, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( missing_test_component.id ) %> + <style type="text/css"> + #missing_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="readme_table"> + <table id="missing_table"><tr><td bgcolor="#FFFFCC"><b>Tool id:</b> ${missing_test_component.tool_id | h}</td></tr><tr><td><b>Tool version:</b> ${missing_test_component.tool_version | h}</td></tr><tr><td><b>Tool guid:</b> ${missing_test_component.tool_guid | h}</td></tr> @@ -632,13 +654,23 @@ <%def name="render_tool_dependency_installation_error( installation_error, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( installation_error.id ) %> + <style type="text/css"> + #td_install_error_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="td_installation_errors"> + <table id="td_install_error_table"><tr bgcolor="#FFFFCC"><th>Type</th><th>Name</th><th>Version</th></tr> @@ -660,13 +692,23 @@ <%def name="render_repository_installation_error( installation_error, pad, parent, row_counter, row_is_header=False, is_current_repository=False )"><% encoded_id = trans.security.encode_id( installation_error.id ) %> + <style type="text/css"> + #rd_install_error_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="repository_installation_errors"> + <table id="rd_install_error_table"> %if not is_current_repository: <tr bgcolor="#FFFFCC"><th>Tool shed</th><th>Name</th><th>Owner</th><th>Changeset revision</th> @@ -691,13 +733,23 @@ <%def name="render_passed_test( passed_test, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( passed_test.id ) %> + <style type="text/css"> + #passed_tests_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="readme_table"> + <table id="passed_tests_table"><tr><td bgcolor="#FFFFCC"><b>Tool id:</b> ${passed_test.tool_id | h}</td></tr><tr><td><b>Tool version:</b> ${passed_test.tool_id | h}</td></tr><tr><td><b>Test:</b> ${passed_test.test_id | h}</td></tr> https://bitbucket.org/galaxy/galaxy-central/commits/13797aac2667/ Changeset: 13797aac2667 User: greg Date: 2013-05-29 22:18:54 Summary: Merged from next-stable Affected #: 2 files diff -r 49cc4a899f2ff5853d6c4f4c70ef9063df2f5b1b -r 13797aac2667307106edeaba3076f04abd96e60d lib/galaxy/webapps/tool_shed/api/repositories.py --- a/lib/galaxy/webapps/tool_shed/api/repositories.py +++ b/lib/galaxy/webapps/tool_shed/api/repositories.py @@ -33,7 +33,7 @@ @web.expose_api_anonymous def get_ordered_installable_revisions( self, trans, name, owner, **kwd ): """ - GET /api/repository/get_ordered_installable_revisions + GET /api/repositories/get_ordered_installable_revisions :param name: the name of the Repository :param owner: the owner of the Repository @@ -59,7 +59,7 @@ @web.expose_api_anonymous def get_repository_revision_install_info( self, trans, name, owner, changeset_revision, **kwd ): """ - GET /api/repository/get_repository_revision_install_info + GET /api/repositories/get_repository_revision_install_info :param name: the name of the Repository :param owner: the owner of the Repository diff -r 49cc4a899f2ff5853d6c4f4c70ef9063df2f5b1b -r 13797aac2667307106edeaba3076f04abd96e60d templates/webapps/tool_shed/repository/common.mako --- a/templates/webapps/tool_shed/repository/common.mako +++ b/templates/webapps/tool_shed/repository/common.mako @@ -395,13 +395,23 @@ <%def name="render_failed_test( failed_test, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( failed_test.id ) %> + <style type="text/css"> + #failed_test_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="readme_table"> + <table id="failed_test_table"><tr><td bgcolor="#FFFFCC"><b>Tool id:</b> ${failed_test.tool_id | h}</td></tr><tr><td><b>Tool version:</b> ${failed_test.tool_id | h}</td></tr><tr><td><b>Test:</b> ${failed_test.test_id | h}</td></tr> @@ -485,14 +495,14 @@ encoded_id = trans.security.encode_id( invalid_tool_dependency.id ) %><style type="text/css"> - #readme_table{ table-layout:fixed; - width:100%; - overflow-wrap:normal; - overflow:hidden; - border:0px; - word-break:keep-all; - word-wrap:break-word; - line-break:strict; } + #invalid_td_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } </style><tr class="datasetRow" %if parent is not None: @@ -500,7 +510,9 @@ %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - ${ invalid_tool_dependency.error | h } + <table id="invalid_td_table"> + <tr><td>${ invalid_tool_dependency.error | h }</td></tr> + </table></td></tr><% @@ -511,13 +523,23 @@ <%def name="render_missing_test_component( missing_test_component, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( missing_test_component.id ) %> + <style type="text/css"> + #missing_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="readme_table"> + <table id="missing_table"><tr><td bgcolor="#FFFFCC"><b>Tool id:</b> ${missing_test_component.tool_id | h}</td></tr><tr><td><b>Tool version:</b> ${missing_test_component.tool_version | h}</td></tr><tr><td><b>Tool guid:</b> ${missing_test_component.tool_guid | h}</td></tr> @@ -632,13 +654,23 @@ <%def name="render_tool_dependency_installation_error( installation_error, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( installation_error.id ) %> + <style type="text/css"> + #td_install_error_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="td_installation_errors"> + <table id="td_install_error_table"><tr bgcolor="#FFFFCC"><th>Type</th><th>Name</th><th>Version</th></tr> @@ -660,13 +692,23 @@ <%def name="render_repository_installation_error( installation_error, pad, parent, row_counter, row_is_header=False, is_current_repository=False )"><% encoded_id = trans.security.encode_id( installation_error.id ) %> + <style type="text/css"> + #rd_install_error_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="repository_installation_errors"> + <table id="rd_install_error_table"> %if not is_current_repository: <tr bgcolor="#FFFFCC"><th>Tool shed</th><th>Name</th><th>Owner</th><th>Changeset revision</th> @@ -691,13 +733,23 @@ <%def name="render_passed_test( passed_test, pad, parent, row_counter, row_is_header=False )"><% encoded_id = trans.security.encode_id( passed_test.id ) %> + <style type="text/css"> + #passed_tests_table{ table-layout:fixed; + width:100%; + overflow-wrap:normal; + overflow:hidden; + border:0px; + word-break:keep-all; + word-wrap:break-word; + line-break:strict; } + </style><tr class="datasetRow" %if parent is not None: parent="${parent}" %endif id="libraryItem-${encoded_id}"><td style="padding-left: ${pad+20}px;"> - <table class="grid" id="readme_table"> + <table id="passed_tests_table"><tr><td bgcolor="#FFFFCC"><b>Tool id:</b> ${passed_test.tool_id | h}</td></tr><tr><td><b>Tool version:</b> ${passed_test.tool_id | h}</td></tr><tr><td><b>Test:</b> ${passed_test.test_id | h}</td></tr> Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 29 May '13

29 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/eb524e42fc65/ Changeset: eb524e42fc65 Branch: next-stable User: natefoo Date: 2013-05-29 22:16:11 Summary: Update pgcleanup.py for dialect name change. Affected #: 1 file diff -r f3bb3c093851c7b07b8d24bfcfa30e986298fb89 -r eb524e42fc65eb69274857bcaf85030f0edd32f0 scripts/cleanup_datasets/pgcleanup.py --- a/scripts/cleanup_datasets/pgcleanup.py +++ b/scripts/cleanup_datasets/pgcleanup.py @@ -108,7 +108,7 @@ args = url.translate_connect_args() args.update(url.query) - assert url.get_dialect().name == 'postgres', 'This script can only be used with PostgreSQL.' + assert url.get_dialect().name == 'postgresql', 'This script can only be used with PostgreSQL.' self.conn = psycopg2.connect(**args) https://bitbucket.org/galaxy/galaxy-central/commits/49cc4a899f2f/ Changeset: 49cc4a899f2f User: natefoo Date: 2013-05-29 22:16:24 Summary: merge next-stable. Affected #: 1 file diff -r 07a135ce18a7e40009fe93654ce46030ff213799 -r 49cc4a899f2ff5853d6c4f4c70ef9063df2f5b1b scripts/cleanup_datasets/pgcleanup.py --- a/scripts/cleanup_datasets/pgcleanup.py +++ b/scripts/cleanup_datasets/pgcleanup.py @@ -108,7 +108,7 @@ args = url.translate_connect_args() args.update(url.query) - assert url.get_dialect().name == 'postgres', 'This script can only be used with PostgreSQL.' + assert url.get_dialect().name == 'postgresql', 'This script can only be used with PostgreSQL.' self.conn = psycopg2.connect(**args) Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 29 May '13

29 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/f3bb3c093851/ Changeset: f3bb3c093851 Branch: next-stable User: Dave Bouvier Date: 2013-05-29 21:24:52 Summary: Make the functional test results RSS feed output properly formatted data. Affected #: 1 file diff -r a6f688dade808b0e16903fbb50186ad9656c0d1c -r f3bb3c093851c7b07b8d24bfcfa30e986298fb89 lib/galaxy/webapps/tool_shed/controllers/repository.py --- a/lib/galaxy/webapps/tool_shed/controllers/repository.py +++ b/lib/galaxy/webapps/tool_shed/controllers/repository.py @@ -1468,25 +1468,39 @@ trans.model.Repository.table.c.user_id == user.id ) ): if not metadata_row.tool_test_results: continue + current_repository_errors = [] + tool_dependency_errors = [] + repository_dependency_errors = [] + description_lines = [] # Per the RSS 2.0 specification, all dates in RSS feeds must be formatted as specified in RFC 822 # section 5.1, e.g. Sat, 07 Sep 2002 00:00:01 UT time_tested = metadata_row.time_last_tested.strftime( '%a, %d %b %Y %H:%M:%S UT' ) repository = metadata_row.repository # Generate a citable URL for this repository with owner and changeset revision. repository_citable_url = suc.url_join( tool_shed_url, 'view', user.username, repository.name, metadata_row.changeset_revision ) - title = 'Functional test results for changeset revision %s of %s' % ( metadata_row.changeset_revision, repository.name ) passed_tests = len( metadata_row.tool_test_results.get( 'passed_tests', [] ) ) failed_tests = len( metadata_row.tool_test_results.get( 'failed_tests', [] ) ) missing_test_components = len( metadata_row.tool_test_results.get( 'missing_test_components', [] ) ) - description = '%d tests passed, %d tests failed, %d tests missing test components.' % \ - ( passed_tests, failed_tests, missing_test_components ) + installation_errors = metadata_row.tool_test_results.get( 'installation_errors', [] ) + if installation_errors: + tool_dependency_errors = installation_errors.get( 'tool_dependencies', [] ) + repository_dependency_errors = installation_errors.get( 'repository_dependencies', [] ) + current_repository_errors = installation_errors.get( 'current_repository', [] ) + description_lines.append( '%d tests passed, %d tests failed, %d tests missing test components.' % \ + ( passed_tests, failed_tests, missing_test_components ) ) + if current_repository_errors: + description_lines.append( '\nThis repository did not install correctly. ' ) + if tool_dependency_errors or repository_dependency_errors: + description_lines.append( '\n%d tool dependencies and %d repository dependencies failed to install. ' % \ + len( tool_dependency_errors ), len( repository_dependency_errors ) ) + title = 'Revision %s of %s' % ( metadata_row.changeset_revision, repository.name ) # The guid attribute in an RSS feed's list of items allows a feed reader to choose not to show an item as updated # if the guid is unchanged. For functional test results, the citable URL is sufficiently unique to enable # that behavior. functional_test_results.append( dict( title=title, guid=repository_citable_url, link=repository_citable_url, - description=description, + description='\n'.join( description_lines ), pubdate=time_tested ) ) trans.response.set_content_type( 'application/rss+xml' ) return trans.fill_template( '/rss.mako', https://bitbucket.org/galaxy/galaxy-central/commits/07a135ce18a7/ Changeset: 07a135ce18a7 User: Dave Bouvier Date: 2013-05-29 21:25:38 Summary: Merge with next-stable. Affected #: 1 file diff -r e3455a0928824b8ef0fb8f9337e3b0a1d194d2ed -r 07a135ce18a7e40009fe93654ce46030ff213799 lib/galaxy/webapps/tool_shed/controllers/repository.py --- a/lib/galaxy/webapps/tool_shed/controllers/repository.py +++ b/lib/galaxy/webapps/tool_shed/controllers/repository.py @@ -1458,25 +1458,39 @@ trans.model.Repository.table.c.user_id == user.id ) ): if not metadata_row.tool_test_results: continue + current_repository_errors = [] + tool_dependency_errors = [] + repository_dependency_errors = [] + description_lines = [] # Per the RSS 2.0 specification, all dates in RSS feeds must be formatted as specified in RFC 822 # section 5.1, e.g. Sat, 07 Sep 2002 00:00:01 UT time_tested = metadata_row.time_last_tested.strftime( '%a, %d %b %Y %H:%M:%S UT' ) repository = metadata_row.repository # Generate a citable URL for this repository with owner and changeset revision. repository_citable_url = suc.url_join( tool_shed_url, 'view', user.username, repository.name, metadata_row.changeset_revision ) - title = 'Functional test results for changeset revision %s of %s' % ( metadata_row.changeset_revision, repository.name ) passed_tests = len( metadata_row.tool_test_results.get( 'passed_tests', [] ) ) failed_tests = len( metadata_row.tool_test_results.get( 'failed_tests', [] ) ) missing_test_components = len( metadata_row.tool_test_results.get( 'missing_test_components', [] ) ) - description = '%d tests passed, %d tests failed, %d tests missing test components.' % \ - ( passed_tests, failed_tests, missing_test_components ) + installation_errors = metadata_row.tool_test_results.get( 'installation_errors', [] ) + if installation_errors: + tool_dependency_errors = installation_errors.get( 'tool_dependencies', [] ) + repository_dependency_errors = installation_errors.get( 'repository_dependencies', [] ) + current_repository_errors = installation_errors.get( 'current_repository', [] ) + description_lines.append( '%d tests passed, %d tests failed, %d tests missing test components.' % \ + ( passed_tests, failed_tests, missing_test_components ) ) + if current_repository_errors: + description_lines.append( '\nThis repository did not install correctly. ' ) + if tool_dependency_errors or repository_dependency_errors: + description_lines.append( '\n%d tool dependencies and %d repository dependencies failed to install. ' % \ + len( tool_dependency_errors ), len( repository_dependency_errors ) ) + title = 'Revision %s of %s' % ( metadata_row.changeset_revision, repository.name ) # The guid attribute in an RSS feed's list of items allows a feed reader to choose not to show an item as updated # if the guid is unchanged. For functional test results, the citable URL is sufficiently unique to enable # that behavior. functional_test_results.append( dict( title=title, guid=repository_citable_url, link=repository_citable_url, - description=description, + description='\n'.join( description_lines ), pubdate=time_tested ) ) trans.response.set_content_type( 'application/rss+xml' ) return trans.fill_template( '/rss.mako', Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 29 May '13

29 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/a6f688dade80/ Changeset: a6f688dade80 Branch: next-stable User: natefoo Date: 2013-05-29 19:55:08 Summary: Make PBKDF2 password hashing conditional, enable by default. Affected #: 3 files diff -r 886a722b0aef6560320c1ddf7382e6207731f49f -r a6f688dade808b0e16903fbb50186ad9656c0d1c lib/galaxy/app.py --- a/lib/galaxy/app.py +++ b/lib/galaxy/app.py @@ -62,7 +62,8 @@ self.config.database_engine_options, database_query_profiling_proxy = self.config.database_query_profiling_proxy, object_store = self.object_store, - trace_logger=self.trace_logger ) + trace_logger=self.trace_logger, + use_pbkdf2=self.config.get_bool( 'use_pbkdf2', True ) ) # Manage installed tool shed repositories. self.installed_repository_manager = tool_shed.galaxy_install.InstalledRepositoryManager( self ) # Create an empty datatypes registry. diff -r 886a722b0aef6560320c1ddf7382e6207731f49f -r a6f688dade808b0e16903fbb50186ad9656c0d1c lib/galaxy/model/__init__.py --- a/lib/galaxy/model/__init__.py +++ b/lib/galaxy/model/__init__.py @@ -62,6 +62,7 @@ class User( object, APIItem ): + use_pbkdf2 = True """ Data for a Galaxy user or admin and relations to their histories, credentials, and roles. @@ -87,7 +88,10 @@ """ Set user password to the digest of `cleartext`. """ - self.password = galaxy.security.passwords.hash_password( cleartext ) + if User.use_pbkdf2: + self.password = galaxy.security.passwords.hash_password( cleartext ) + else: + self.password = new_secure_hash( text_type=cleartext ) def check_password( self, cleartext ): """ diff -r 886a722b0aef6560320c1ddf7382e6207731f49f -r a6f688dade808b0e16903fbb50186ad9656c0d1c lib/galaxy/model/mapping.py --- a/lib/galaxy/model/mapping.py +++ b/lib/galaxy/model/mapping.py @@ -2008,12 +2008,14 @@ # Let this go, it could possibly work with db's we don't support log.error( "database_connection contains an unknown SQLAlchemy database dialect: %s" % dialect ) -def init( file_path, url, engine_options={}, create_tables=False, database_query_profiling_proxy=False, object_store=None, trace_logger=None ): +def init( file_path, url, engine_options={}, create_tables=False, database_query_profiling_proxy=False, object_store=None, trace_logger=None, use_pbkdf2=True ): """Connect mappings to the database""" # Connect dataset to the file path model.Dataset.file_path = file_path # Connect dataset to object store model.Dataset.object_store = object_store + # Use PBKDF2 password hashing? + model.User.use_pbkdf2 = use_pbkdf2 # Load the appropriate db module load_egg_for_url( url ) # Should we use the logging proxy? https://bitbucket.org/galaxy/galaxy-central/commits/e3455a092882/ Changeset: e3455a092882 User: natefoo Date: 2013-05-29 19:55:22 Summary: merge next-stable. Affected #: 3 files diff -r 35ac562e7e81f232b99c2c377d415b6aeab538cb -r e3455a0928824b8ef0fb8f9337e3b0a1d194d2ed lib/galaxy/app.py --- a/lib/galaxy/app.py +++ b/lib/galaxy/app.py @@ -62,7 +62,8 @@ self.config.database_engine_options, database_query_profiling_proxy = self.config.database_query_profiling_proxy, object_store = self.object_store, - trace_logger=self.trace_logger ) + trace_logger=self.trace_logger, + use_pbkdf2=self.config.get_bool( 'use_pbkdf2', True ) ) # Manage installed tool shed repositories. self.installed_repository_manager = tool_shed.galaxy_install.InstalledRepositoryManager( self ) # Create an empty datatypes registry. diff -r 35ac562e7e81f232b99c2c377d415b6aeab538cb -r e3455a0928824b8ef0fb8f9337e3b0a1d194d2ed lib/galaxy/model/__init__.py --- a/lib/galaxy/model/__init__.py +++ b/lib/galaxy/model/__init__.py @@ -62,6 +62,7 @@ class User( object, APIItem ): + use_pbkdf2 = True """ Data for a Galaxy user or admin and relations to their histories, credentials, and roles. @@ -87,7 +88,10 @@ """ Set user password to the digest of `cleartext`. """ - self.password = galaxy.security.passwords.hash_password( cleartext ) + if User.use_pbkdf2: + self.password = galaxy.security.passwords.hash_password( cleartext ) + else: + self.password = new_secure_hash( text_type=cleartext ) def check_password( self, cleartext ): """ diff -r 35ac562e7e81f232b99c2c377d415b6aeab538cb -r e3455a0928824b8ef0fb8f9337e3b0a1d194d2ed lib/galaxy/model/mapping.py --- a/lib/galaxy/model/mapping.py +++ b/lib/galaxy/model/mapping.py @@ -2008,12 +2008,14 @@ # Let this go, it could possibly work with db's we don't support log.error( "database_connection contains an unknown SQLAlchemy database dialect: %s" % dialect ) -def init( file_path, url, engine_options={}, create_tables=False, database_query_profiling_proxy=False, object_store=None, trace_logger=None ): +def init( file_path, url, engine_options={}, create_tables=False, database_query_profiling_proxy=False, object_store=None, trace_logger=None, use_pbkdf2=True ): """Connect mappings to the database""" # Connect dataset to the file path model.Dataset.file_path = file_path # Connect dataset to object store model.Dataset.object_store = object_store + # Use PBKDF2 password hashing? + model.User.use_pbkdf2 = use_pbkdf2 # Load the appropriate db module load_egg_for_url( url ) # Should we use the logging proxy? Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 29 May '13

29 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/886a722b0aef/ Changeset: 886a722b0aef Branch: next-stable User: Dave Bouvier Date: 2013-05-29 19:26:38 Summary: Fix for set_environment appearing in multiple action sections. Fix for interpolating multiple environment variables in set_environment actions. Affected #: 2 files diff -r 8c69611137c19dd59d9e6f7cc339484213a88b18 -r 886a722b0aef6560320c1ddf7382e6207731f49f lib/tool_shed/galaxy_install/tool_dependencies/fabric_util.py --- a/lib/tool_shed/galaxy_install/tool_dependencies/fabric_util.py +++ b/lib/tool_shed/galaxy_install/tool_dependencies/fabric_util.py @@ -22,7 +22,6 @@ log = logging.getLogger( __name__ ) -CMD_SEPARATOR = '__CMD_SEP__' INSTALLATION_LOG = 'INSTALLATION.log' VIRTUALENV_URL = 'https://pypi.python.org/packages/source/v/virtualenv/virtualenv-1.9.1.tar.gz' @@ -51,21 +50,90 @@ return output.return_code def handle_environment_variables( app, tool_dependency, install_dir, env_var_dict, set_prior_environment_commands ): + """ + This method works with with a combination of three tool dependency definition tag sets, which are defined in the tool_dependencies.xml file in the + order discussed here. The example for this discussion is the tool_dependencies.xml file contained in the osra repository, which is available at: + + http://testtoolshed.g2.bx.psu.edu/view/bgruening/osra + + The first tag set defines a complex repository dependency like this. This tag set ensures that changeset revision XXX of the repository named + package_graphicsmagick_1_3 owned by YYY in the tool shed ZZZ has been previously installed. + + <tool_dependency> + <package name="graphicsmagick" version="1.3.18"> + <repository changeset_revision="XXX" name="package_graphicsmagick_1_3" owner="YYY" prior_installation_required="True" toolshed="ZZZ" /> + </package> + ... + + * By the way, there is an env.sh file associated with version 1.3.18 of the graphicsmagick package which looks something like this (we'll reference + this file later in this discussion. + ---- + GRAPHICSMAGICK_ROOT_DIR=/<my configured tool dependency path>/graphicsmagick/1.3.18/YYY/package_graphicsmagick_1_3/XXX/gmagick; + export GRAPHICSMAGICK_ROOT_DIR + ---- + + The second tag set defines a specific package dependency that has been previously installed (guaranteed by the tag set discussed above) and compiled, + where the compiled dependency is needed by the tool dependency currently being installed (osra version 2.0.0 in this case) and complied in order for + it's installation and compilation to succeed. This tag set is contained within the <package name="osra" version="2.0.0"> tag set, which implies that + version 2.0.0 of the osra package requires version 1.3.18 of the graphicsmagick package in order to successfully compile. When this tag set is handled, + one of the effects is that the env.sh file associated with graphicsmagick version 1.3.18 is "sourced", which undoubtedly sets or alters certain environment + variables (e.g. PATH, PYTHONPATH, etc). + +  + <action type="set_environment_for_install"> + <repository changeset_revision="XXX" name="package_graphicsmagick_1_3" owner="YYY" toolshed="ZZZ"> + <package name="graphicsmagick" version="1.3.18" /> + </repository> + </action> + + The third tag set enables discovery of the same required package dependency discussed above for correctly compiling the osra version 2.0.0 package, but + in this case the package can be discovered at tool execution time. Using the $ENV[] option as shown in this example, the value of the environment + variable named GRAPHICSMAGICK_ROOT_DIR (which was set in the environment using the second tag set described above) will be used to automatically alter + the env.sh file associated with the osra version 2.0.0 tool dependency when it is installed into Galaxy. * Refer to where we discussed the env.sh file + for version 1.3.18 of the graphicsmagick package above. + + <action type="set_environment"> + <environment_variable action="prepend_to" name="LD_LIBRARY_PATH">$ENV[GRAPHICSMAGICK_ROOT_DIR]/lib/</environment_variable> + <environment_variable action="prepend_to" name="LD_LIBRARY_PATH">$INSTALL_DIR/potrace/build/lib/</environment_variable> + <environment_variable action="prepend_to" name="PATH">$INSTALL_DIR/bin</environment_variable> +  + <environment_variable action="set_to" name="OSRA_DATA_FILES">$INSTALL_DIR/share</environment_variable> + </action> + + The above tag will produce an env.sh file for version 2.0.0 of the osra package when it it installed into Galaxy that looks something like this. Notice + that the path to the gmagick binary is included here since it expands the defined $ENV[GRAPHICSMAGICK_ROOT_DIR] value in the above tag set. + + ---- + LD_LIBRARY_PATH=/<my configured tool dependency path>/graphicsmagick/1.3.18/YYY/package_graphicsmagick_1_3/XXX/gmagick/lib/:$LD_LIBRARY_PATH; + export LD_LIBRARY_PATH + LD_LIBRARY_PATH=/<my configured tool dependency path>/osra/1.4.0/YYY/depends_on/XXX/potrace/build/lib/:$LD_LIBRARY_PATH; + export LD_LIBRARY_PATH + PATH=/<my configured tool dependency path>/osra/1.4.0/YYY/depends_on/XXX/bin:$PATH; + export PATH + OSRA_DATA_FILES=/<my configured tool dependency path>/osra/1.4.0/YYY/depends_on/XXX/share; + export OSRA_DATA_FILES + ---- + """ env_var_value = env_var_dict[ 'value' ] + # env_var_value is the text of an environment variable tag like this: <environment_variable action="prepend_to" name="LD_LIBRARY_PATH"> + # Here is an example of what env_var_value could look like: $ENV[GRAPHICSMAGICK_ROOT_DIR]/lib/ if '$ENV[' in env_var_value and ']' in env_var_value: # Pull out the name of the environment variable to populate. inherited_env_var_name = env_var_value.split( '[' )[1].split( ']' )[0] to_replace = '$ENV[%s]' % inherited_env_var_name - # Build a command line that outputs CMD_SEPARATOR + environment variable value + CMD_SEPARATOR. - set_prior_environment_commands.extend( [ "echo '%s'" % CMD_SEPARATOR, 'echo $%s' % inherited_env_var_name, "echo '%s'" % CMD_SEPARATOR ] ) + # Build a command line that outputs VARIABLE_NAME: <the value of the variable>. + set_prior_environment_commands.append( 'echo "%s: $%s"' % ( inherited_env_var_name, inherited_env_var_name ) ) command = ' ; '.join( set_prior_environment_commands ) # Run the command and capture the output. command_return = handle_command( app, tool_dependency, install_dir, command, return_output=True ) - # And extract anything between the two instances of CMD_SEPARATOR. - environment_variable_value = command_return.split( CMD_SEPARATOR )[1].split( CMD_SEPARATOR )[0].strip( '\n' ) - if environment_variable_value: - log.info( 'Replacing %s with %s in env.sh for this repository.', to_replace, environment_variable_value ) - env_var_value = env_var_value.replace( to_replace, environment_variable_value ) + # And extract anything labeled with the name of the environment variable we're populating here. + if '%s: ' % inherited_env_var_name in command_return: + environment_variable_value = command_return.split( '\n' ) + for line in environment_variable_value: + if line.startswith( inherited_env_var_name ): + inherited_env_var_value = line.replace( '%s: ' % inherited_env_var_name, '' ) + log.info( 'Replacing %s with %s in env.sh for this repository.', to_replace, inherited_env_var_value ) + env_var_value = env_var_value.replace( to_replace, inherited_env_var_value ) else: # If the return is empty, replace the original $ENV[] with nothing, to avoid any shell misparsings later on. log.error( 'Environment variable %s not found, removing from set_environment.', inherited_env_var_name ) diff -r 8c69611137c19dd59d9e6f7cc339484213a88b18 -r 886a722b0aef6560320c1ddf7382e6207731f49f lib/tool_shed/galaxy_install/tool_dependencies/install_util.py --- a/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py +++ b/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py @@ -339,6 +339,7 @@ actions_dict[ 'package_name' ] = package_name actions = [] all_env_shell_file_paths = [] + env_var_dicts = [] # Make sure to skip all comments, since they are now included in the XML tree. for action_elem in actions_elem.findall( 'action' ): action_dict = {} @@ -389,7 +390,6 @@ # <environment_variable name="PYTHONPATH" action="append_to">$INSTALL_DIR/lib/python</environment_variable> # <environment_variable name="PATH" action="prepend_to">$INSTALL_DIR/bin</environment_variable> # </action> - env_var_dicts = [] for env_elem in action_elem: if env_elem.tag == 'environment_variable': env_var_dict = common_util.create_env_var_dict( env_elem, tool_dependency_install_dir=install_dir ) https://bitbucket.org/galaxy/galaxy-central/commits/35ac562e7e81/ Changeset: 35ac562e7e81 User: Dave Bouvier Date: 2013-05-29 19:27:48 Summary: Merge with next-stable. Affected #: 2 files diff -r 964071e63ee3173e35f245d193e3cb8252fabc7c -r 35ac562e7e81f232b99c2c377d415b6aeab538cb lib/tool_shed/galaxy_install/tool_dependencies/fabric_util.py --- a/lib/tool_shed/galaxy_install/tool_dependencies/fabric_util.py +++ b/lib/tool_shed/galaxy_install/tool_dependencies/fabric_util.py @@ -22,7 +22,6 @@ log = logging.getLogger( __name__ ) -CMD_SEPARATOR = '__CMD_SEP__' INSTALLATION_LOG = 'INSTALLATION.log' VIRTUALENV_URL = 'https://pypi.python.org/packages/source/v/virtualenv/virtualenv-1.9.1.tar.gz' @@ -94,7 +93,7 @@ for version 1.3.18 of the graphicsmagick package above. <action type="set_environment"> - <environment_variable action="prepend_to" name="LD_LIBRARY_PATH">$ENV[$GRAPHICSMAGICK_ROOT_DIR]/lib/</environment_variable> + <environment_variable action="prepend_to" name="LD_LIBRARY_PATH">$ENV[GRAPHICSMAGICK_ROOT_DIR]/lib/</environment_variable><environment_variable action="prepend_to" name="LD_LIBRARY_PATH">$INSTALL_DIR/potrace/build/lib/</environment_variable><environment_variable action="prepend_to" name="PATH">$INSTALL_DIR/bin</environment_variable> @@ -102,7 +101,7 @@ </action> The above tag will produce an env.sh file for version 2.0.0 of the osra package when it it installed into Galaxy that looks something like this. Notice - that the path to the gmagick binary is included here since it expands the defined $ENV[$GRAPHICSMAGICK_ROOT_DIR] value in the above tag set. + that the path to the gmagick binary is included here since it expands the defined $ENV[GRAPHICSMAGICK_ROOT_DIR] value in the above tag set. ---- LD_LIBRARY_PATH=/<my configured tool dependency path>/graphicsmagick/1.3.18/YYY/package_graphicsmagick_1_3/XXX/gmagick/lib/:$LD_LIBRARY_PATH; @@ -116,20 +115,25 @@ ---- """ env_var_value = env_var_dict[ 'value' ] + # env_var_value is the text of an environment variable tag like this: <environment_variable action="prepend_to" name="LD_LIBRARY_PATH"> + # Here is an example of what env_var_value could look like: $ENV[GRAPHICSMAGICK_ROOT_DIR]/lib/ if '$ENV[' in env_var_value and ']' in env_var_value: # Pull out the name of the environment variable to populate. inherited_env_var_name = env_var_value.split( '[' )[1].split( ']' )[0] to_replace = '$ENV[%s]' % inherited_env_var_name - # Build a command line that outputs CMD_SEPARATOR + environment variable value + CMD_SEPARATOR. - set_prior_environment_commands.extend( [ "echo '%s'" % CMD_SEPARATOR, 'echo $%s' % inherited_env_var_name, "echo '%s'" % CMD_SEPARATOR ] ) + # Build a command line that outputs VARIABLE_NAME: <the value of the variable>. + set_prior_environment_commands.append( 'echo "%s: $%s"' % ( inherited_env_var_name, inherited_env_var_name ) ) command = ' ; '.join( set_prior_environment_commands ) # Run the command and capture the output. command_return = handle_command( app, tool_dependency, install_dir, command, return_output=True ) - # And extract anything between the two instances of CMD_SEPARATOR. - environment_variable_value = command_return.split( CMD_SEPARATOR )[1].split( CMD_SEPARATOR )[0].strip( '\n' ) - if environment_variable_value: - log.info( 'Replacing %s with %s in env.sh for this repository.', to_replace, environment_variable_value ) - env_var_value = env_var_value.replace( to_replace, environment_variable_value ) + # And extract anything labeled with the name of the environment variable we're populating here. + if '%s: ' % inherited_env_var_name in command_return: + environment_variable_value = command_return.split( '\n' ) + for line in environment_variable_value: + if line.startswith( inherited_env_var_name ): + inherited_env_var_value = line.replace( '%s: ' % inherited_env_var_name, '' ) + log.info( 'Replacing %s with %s in env.sh for this repository.', to_replace, inherited_env_var_value ) + env_var_value = env_var_value.replace( to_replace, inherited_env_var_value ) else: # If the return is empty, replace the original $ENV[] with nothing, to avoid any shell misparsings later on. log.error( 'Environment variable %s not found, removing from set_environment.', inherited_env_var_name ) diff -r 964071e63ee3173e35f245d193e3cb8252fabc7c -r 35ac562e7e81f232b99c2c377d415b6aeab538cb lib/tool_shed/galaxy_install/tool_dependencies/install_util.py --- a/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py +++ b/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py @@ -339,6 +339,7 @@ actions_dict[ 'package_name' ] = package_name actions = [] all_env_shell_file_paths = [] + env_var_dicts = [] # Make sure to skip all comments, since they are now included in the XML tree. for action_elem in actions_elem.findall( 'action' ): action_dict = {} @@ -389,7 +390,6 @@ # <environment_variable name="PYTHONPATH" action="append_to">$INSTALL_DIR/lib/python</environment_variable> # <environment_variable name="PATH" action="prepend_to">$INSTALL_DIR/bin</environment_variable> # </action> - env_var_dicts = [] for env_elem in action_elem: if env_elem.tag == 'environment_variable': env_var_dict = common_util.create_env_var_dict( env_elem, tool_dependency_install_dir=install_dir ) Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: dannon: Merge with next-stable
by commits-noreply＠bitbucket.org 28 May '13

28 May '13

1 new commit in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/964071e63ee3/ Changeset: 964071e63ee3 User: dannon Date: 2013-05-28 23:25:11 Summary: Merge with next-stable Affected #: 0 files Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 28 May '13

28 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/1a9a6e51304c/ Changeset: 1a9a6e51304c User: dannon Date: 2013-05-28 23:23:17 Summary: Bugfix for standard dataset upload Affected #: 1 file diff -r 79889b1c7da331afc80e2fa64e9d568a9c559cbe -r 1a9a6e51304c95d79322d17c9583afbaf305b25b lib/galaxy/tools/actions/upload_common.py --- a/lib/galaxy/tools/actions/upload_common.py +++ b/lib/galaxy/tools/actions/upload_common.py @@ -209,13 +209,12 @@ trans.sa_session.add( dp ) trans.sa_session.flush() return ldda + def new_upload( trans, cntrller, uploaded_dataset, library_bunch=None, history=None, state=None ): if library_bunch: return __new_library_upload( trans, cntrller, uploaded_dataset, library_bunch, state ) - elif history: + else: return __new_history_upload( trans, uploaded_dataset, history=history, state=state ) - else: - raise Exception("new_upload must be called with empty values for library_bunch and history") def get_uploaded_datasets( trans, cntrller, params, precreated_datasets, dataset_upload_inputs, library_bunch=None, history=None ): uploaded_datasets = [] https://bitbucket.org/galaxy/galaxy-central/commits/8c69611137c1/ Changeset: 8c69611137c1 Branch: next-stable User: dannon Date: 2013-05-28 23:23:17 Summary: Bugfix for standard dataset upload Affected #: 1 file diff -r 6b4b91d27c3316960ae0e65bfb7c87371831be41 -r 8c69611137c19dd59d9e6f7cc339484213a88b18 lib/galaxy/tools/actions/upload_common.py --- a/lib/galaxy/tools/actions/upload_common.py +++ b/lib/galaxy/tools/actions/upload_common.py @@ -209,13 +209,12 @@ trans.sa_session.add( dp ) trans.sa_session.flush() return ldda + def new_upload( trans, cntrller, uploaded_dataset, library_bunch=None, history=None, state=None ): if library_bunch: return __new_library_upload( trans, cntrller, uploaded_dataset, library_bunch, state ) - elif history: + else: return __new_history_upload( trans, uploaded_dataset, history=history, state=state ) - else: - raise Exception("new_upload must be called with empty values for library_bunch and history") def get_uploaded_datasets( trans, cntrller, params, precreated_datasets, dataset_upload_inputs, library_bunch=None, history=None ): uploaded_datasets = [] Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 28 May '13

28 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/7395e351ec85/ Changeset: 7395e351ec85 User: jgoecks Date: 2013-05-26 21:05:31 Summary: Trackster: whitespace additions between functions. Affected #: 1 file diff -r 31714646a7b441f34a43748065278a1b08940c3c -r 7395e351ec8564c8c38c38ffcef79a57d02ec836 static/scripts/viz/trackster/tracks.js --- a/static/scripts/viz/trackster/tracks.js +++ b/static/scripts/viz/trackster/tracks.js @@ -1137,9 +1137,11 @@ // FIXME: need to use this approach to enable inheritance of DrawableCollection functions. extend( TracksterView.prototype, DrawableCollection.prototype, { + changed: function() { this.has_changes = true; }, + /** Add or remove intro div depending on view state. */ update_intro_div: function() { if (this.drawables.length === 0) { @@ -1149,6 +1151,7 @@ this.intro_div.hide(); } }, + /** * Triggers navigate events as needed. If there is a delay, * then event is triggered only after navigation has stopped. @@ -1171,6 +1174,7 @@ view.trigger("navigate", new_chrom + ":" + new_low + "-" + new_high); } }, + update_location: function(low, high) { this.location_span.text( commatize(low) + ' - ' + commatize(high) ); this.nav_input.val( this.chrom + ':' + commatize(low) + '-' + commatize(high) ); @@ -1182,6 +1186,7 @@ this.trigger_navigate(chrom, view.low, view.high, true); } }, + /** * Load chrom data for the view. Returns a jQuery Deferred. */ @@ -1230,6 +1235,7 @@ return chrom_data; }, + change_chrom: function(chrom, low, high) { var view = this; // If chrom data is still loading, wait for it. @@ -1365,6 +1371,7 @@ var chrom = view.chrom_select.val(); this.trigger_navigate(chrom, view.low, view.high, true); }, + /** * Add a drawable to the view. */ @@ -1374,11 +1381,13 @@ this.changed(); this.update_intro_div(); }, + add_label_track: function (label_track) { label_track.view = this; label_track.init(); this.label_tracks.push(label_track); }, + /** * Remove drawable from the view. */ @@ -1392,11 +1401,13 @@ }); } }, + reset: function() { this.low = this.max_low; this.high = this.max_high; this.viewport_container.find(".yaxislabel").remove(); }, + /** * Request that view redraw some or all tracks. If a track is not specificied, redraw all tracks. */ @@ -1439,6 +1450,7 @@ this.requested_redraw = true; } }, + /** * Redraws view and tracks. * NOTE: this method should never be called directly; request_redraw() should be used so @@ -1503,6 +1515,7 @@ this.label_tracks[i]._draw(); } }, + zoom_in: function (point, container) { if (this.max_high === 0 || this.high - this.low <= this.min_separation) { return; @@ -1518,6 +1531,7 @@ this.changed(); this.request_redraw(); }, + zoom_out: function () { if (this.max_high === 0) { return; @@ -1530,10 +1544,12 @@ this.changed(); this.request_redraw(); }, + resize_window: function() { this.viewport_container.height( this.container.height() - this.top_container.height() - this.bottom_container.height() ); this.request_redraw(); }, + /** Show a Drawable in the overview. */ set_overview: function(drawable) { if (this.overview_drawable) { @@ -1561,6 +1577,7 @@ view.overview_drawable.request_draw(); this.changed(); }, + view /** Close and reset overview. */ reset_overview: function() { // Update UI. https://bitbucket.org/galaxy/galaxy-central/commits/79889b1c7da3/ Changeset: 79889b1c7da3 User: jgoecks Date: 2013-05-28 21:22:58 Summary: Automated merge Affected #: 1 file diff -r c3199d306781ce61ce121a2c34c846310dac74c7 -r 79889b1c7da331afc80e2fa64e9d568a9c559cbe static/scripts/viz/trackster/tracks.js --- a/static/scripts/viz/trackster/tracks.js +++ b/static/scripts/viz/trackster/tracks.js @@ -1137,9 +1137,11 @@ // FIXME: need to use this approach to enable inheritance of DrawableCollection functions. extend( TracksterView.prototype, DrawableCollection.prototype, { + changed: function() { this.has_changes = true; }, + /** Add or remove intro div depending on view state. */ update_intro_div: function() { if (this.drawables.length === 0) { @@ -1149,6 +1151,7 @@ this.intro_div.hide(); } }, + /** * Triggers navigate events as needed. If there is a delay, * then event is triggered only after navigation has stopped. @@ -1171,6 +1174,7 @@ view.trigger("navigate", new_chrom + ":" + new_low + "-" + new_high); } }, + update_location: function(low, high) { this.location_span.text( commatize(low) + ' - ' + commatize(high) ); this.nav_input.val( this.chrom + ':' + commatize(low) + '-' + commatize(high) ); @@ -1182,6 +1186,7 @@ this.trigger_navigate(chrom, view.low, view.high, true); } }, + /** * Load chrom data for the view. Returns a jQuery Deferred. */ @@ -1230,6 +1235,7 @@ return chrom_data; }, + change_chrom: function(chrom, low, high) { var view = this; // If chrom data is still loading, wait for it. @@ -1365,6 +1371,7 @@ var chrom = view.chrom_select.val(); this.trigger_navigate(chrom, view.low, view.high, true); }, + /** * Add a drawable to the view. */ @@ -1374,11 +1381,13 @@ this.changed(); this.update_intro_div(); }, + add_label_track: function (label_track) { label_track.view = this; label_track.init(); this.label_tracks.push(label_track); }, + /** * Remove drawable from the view. */ @@ -1392,11 +1401,13 @@ }); } }, + reset: function() { this.low = this.max_low; this.high = this.max_high; this.viewport_container.find(".yaxislabel").remove(); }, + /** * Request that view redraw some or all tracks. If a track is not specificied, redraw all tracks. */ @@ -1439,6 +1450,7 @@ this.requested_redraw = true; } }, + /** * Redraws view and tracks. * NOTE: this method should never be called directly; request_redraw() should be used so @@ -1503,6 +1515,7 @@ this.label_tracks[i]._draw(); } }, + zoom_in: function (point, container) { if (this.max_high === 0 || this.high - this.low <= this.min_separation) { return; @@ -1518,6 +1531,7 @@ this.changed(); this.request_redraw(); }, + zoom_out: function () { if (this.max_high === 0) { return; @@ -1530,10 +1544,12 @@ this.changed(); this.request_redraw(); }, + resize_window: function() { this.viewport_container.height( this.container.height() - this.top_container.height() - this.bottom_container.height() ); this.request_redraw(); }, + /** Show a Drawable in the overview. */ set_overview: function(drawable) { if (this.overview_drawable) { @@ -1561,6 +1577,7 @@ view.overview_drawable.request_draw(); this.changed(); }, + view /** Close and reset overview. */ reset_overview: function() { // Update UI. Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0

commit/galaxy-central: 2 new changesets
by commits-noreply＠bitbucket.org 28 May '13

28 May '13

2 new commits in galaxy-central: https://bitbucket.org/galaxy/galaxy-central/commits/6b4b91d27c33/ Changeset: 6b4b91d27c33 Branch: next-stable User: Dave Bouvier Date: 2013-05-28 20:49:19 Summary: Explicitly filter by tag name actions when building the actions dict for tool dependency installation. Affected #: 1 file diff -r 412b9edb65ec78b029cae0969956b194c16ac053 -r 6b4b91d27c3316960ae0e65bfb7c87371831be41 lib/tool_shed/galaxy_install/tool_dependencies/install_util.py --- a/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py +++ b/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py @@ -339,7 +339,8 @@ actions_dict[ 'package_name' ] = package_name actions = [] all_env_shell_file_paths = [] - for action_elem in actions_elem: + # Make sure to skip all comments, since they are now included in the XML tree. + for action_elem in actions_elem.findall( 'action' ): action_dict = {} action_type = action_elem.get( 'type', 'shell_command' ) if action_type == 'shell_command': https://bitbucket.org/galaxy/galaxy-central/commits/c3199d306781/ Changeset: c3199d306781 User: Dave Bouvier Date: 2013-05-28 20:51:18 Summary: Merge with next-stable. Affected #: 1 file diff -r 79d036fbc175dc9e8eeb3c3a8e57352ded1fbbbf -r c3199d306781ce61ce121a2c34c846310dac74c7 lib/tool_shed/galaxy_install/tool_dependencies/install_util.py --- a/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py +++ b/lib/tool_shed/galaxy_install/tool_dependencies/install_util.py @@ -339,7 +339,8 @@ actions_dict[ 'package_name' ] = package_name actions = [] all_env_shell_file_paths = [] - for action_elem in actions_elem: + # Make sure to skip all comments, since they are now included in the XML tree. + for action_elem in actions_elem.findall( 'action' ): action_dict = {} action_type = action_elem.get( 'type', 'shell_command' ) if action_type == 'shell_command': Repository URL: https://bitbucket.org/galaxy/galaxy-central/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.

1 0