Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[15.10] Fix for no longer being able to access data managers. #879

Merged
merged 1 commit into from Oct 8, 2015

Conversation

blankenberg
Copy link
Member

Have datamanager links point to root with tool_id provided while targeting a new window.

tool_form.mako no longer exists and all tools have an empty string for their default_template value.

…r links point to root with tool_id provided while targeting a new window.
@martenson
Copy link
Member

we have 10 more calls to controller='tool_runner' in the codebase (fulltext search), should we be concerned? ping @guerler

@blankenberg
Copy link
Member Author

There is a hard coded check for 'default' against tool.type in the controller method. Any tool that is not a default tool will probably fail to load against tool_runner index/default (rerun should be OK).

@blankenberg
Copy link
Member Author

I'm not super happy with using a new window, but showing only the tool form (within the admin iframe) without panels seemed problematic.

@martenson
Copy link
Member

👍 for this as a hotfix; is there an easy way how to make this work nicer @guerler ?
If not I guess admins can chew through the new window. :)

@dannon
Copy link
Member

dannon commented Oct 8, 2015

Is the Galaxy.app.display functionality available from within the admin context? And, can the data managers be rendered using the js tool form?

If so, it should be pretty straightforward to use the Galaxy.app.display function with some appropriate target (the right hand admin frame), and use the new tool form javascript.

(then again, that seems like a whole lot of enhancement work and this works for now -- +1 and merging)

dannon added a commit that referenced this pull request Oct 8, 2015
[15.10] Fix for no longer being able to access data managers.
@dannon dannon merged commit 38b9ca3 into galaxyproject:release_15.10 Oct 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants