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

Refresh the tree when recalling a data set #3471

Open
JWaters02 opened this issue Feb 20, 2025 · 5 comments · May be fixed by #3479
Open

Refresh the tree when recalling a data set #3471

JWaters02 opened this issue Feb 20, 2025 · 5 comments · May be fixed by #3479
Assignees
Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.

Comments

@JWaters02
Copy link
Member

Is your feature request related to a problem? Please describe.

My colleagues are frustrated that they have to manually refresh the tree after recalling a data set to get it to show up.

Describe the solution you'd like

When you successfully recall a data set, the tree is refreshed automatically.

Describe alternatives you've considered

Additional context

Copy link

Thank you for raising this enhancement request.
The community has 90 days to vote on it.
If the enhancement receives at least 10 upvotes, it is added to our development backlog.
If it receives fewer votes, the issue is closed.

@JWaters02
Copy link
Member Author

JWaters02 commented Feb 20, 2025

Okay, the bot did just show me that the code is already doing a refresh on the node, and to be honest, what it suggested was exactly what I was going to try doing as a quick fix... which is kind of scary - even moreso if the quick fix works.

However, since the refresh already exists, even if the quick fix does work, there is still the question of the underlying problem of why the existing refresh does not work - and perhaps that is a bug?

datasetProvider.refreshElement(node.getParent());

@traeok
Copy link
Member

traeok commented Feb 20, 2025

This definitely seems like a bug with refreshElement or the tree nodes themselves - I would expect that call to have the same effect, as the parent of the data set is the profile node (as you can only recall PDS or a sequential data set, which is a child of a profile node). Maybe something has changed with how data sets are listed (dirty flag, updated processing, etc.)?

@JTonda JTonda added bug Something isn't working Research Needed and removed enhancement New feature or request labels Feb 20, 2025
Copy link

Thank you for creating a bug report.
We will investigate the bug and evaluate its impact on the product.
If you haven't already, please ensure you have provided steps to reproduce the bug and as much context as possible.

@traeok traeok linked a pull request Feb 24, 2025 that will close this issue
15 tasks
@traeok traeok linked a pull request Feb 24, 2025 that will close this issue
15 tasks
@JTonda JTonda added priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex. and removed Research Needed labels Feb 25, 2025
@JTonda JTonda moved this from New Issues to In Progress in Zowe Explorer for VS Code Feb 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.
Projects
Status: In Progress
Development

Successfully merging a pull request may close this issue.

4 participants
@JTonda @JWaters02 @traeok and others