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

[22.01] Do not store toolbox on ToolPanelViewSearch #13776

Conversation

cat-bro
Copy link
Contributor

@cat-bro cat-bro commented Apr 21, 2022

For issue #13775

Updated tools have been disappearing from tool search results until the next time galaxy is restarted. The ToolPanelViewSearch has self.toolbox assigned in __init__ and this is not updated when app.toolbox is updated for a new tool. The out of date self.toolbox is used in the build_index function.

This change is passing app.toolbox to ToolBoxSearch.build_index and passing it from there to ToolPanelViewSearch.build_index so that the newly installed tool can be found in the toolbox when the search index is updated. This is the only place where ToolPanelViewSearch.toolbox is used so ToolPanelViewSearch does not need self.toolbox.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these contributions under Galaxy's current license.
  • I agree to allow the Galaxy committers to license these and all my past contributions to the core galaxy codebase under the MIT license. If this condition is an issue, uncheck and just let us know why with an e-mail to [email protected].

@github-actions github-actions bot added this to the 22.05 milestone Apr 21, 2022
@cat-bro cat-bro changed the title Do not store toolbox on ToolPanelSearchView Do not store toolbox on ToolPanelViewSearch Apr 21, 2022
@mvdbeek mvdbeek changed the title Do not store toolbox on ToolPanelViewSearch [22.01] Do not store toolbox on ToolPanelViewSearch Apr 21, 2022
Copy link
Member

@nsoranzo nsoranzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice bugfix, thanks @cat-bro !

@nsoranzo nsoranzo merged commit e612a38 into galaxyproject:release_22.01 Apr 21, 2022
@bgruening
Copy link
Member

Very cool, I think this has bitten us as well. Thanks a lot @cat-bro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants