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

Delete member fails when IASP is involved. #629

Closed
m-tyler opened this issue Apr 15, 2022 · 2 comments · Fixed by #649
Closed

Delete member fails when IASP is involved. #629

m-tyler opened this issue Apr 15, 2022 · 2 comments · Fixed by #649
Assignees
Labels
enhancement New feature or request

Comments

@m-tyler
Copy link

m-tyler commented Apr 15, 2022

Describe the bug
Delete Member right-click action on a memeber filter attempts to run the RMVM command but fails with a command that is filled out like this system "RMVM FILE(WIASP/WFIINTG) MBR()". "WIASP" is our IASP name, "WFIINTG" is the source file library.

To Reproduce

  1. You will need a system with an IASP defined.
  2. You need to have source in the IASP.
  3. Create a filter to that source.
  4. Add a member that you are willing to delete
  5. Run the right-click action Delete Member.

Expected behavior
Parse the member string name properly. Do you always need to store the IASP value in all member references? could it be prepended to the member path when needed for those actions that need it, like IFS source editing? RMVM does don't need it for standard QSYS functions.

Environment (please complete the following information):

  • Extension version 1.2.6
  • IBM i OS version 7.3

This is a Med PTY for me.

@worksofliam
Copy link
Contributor

@m-tyler Need some more info from you:

  1. Are you specifying an ASP name in the connection settings, or are you leaving it blank?
  2. Please share the Code for IBM i output so I can see what command it is running.

I know the issue, but need to recreate it before I move on. I also do not have access to a system with an ASP right now.

@worksofliam worksofliam added the info-needed More information is requied from the issuer label Apr 21, 2022
@m-tyler
Copy link
Author

m-tyler commented Apr 21, 2022

ASP setting.
image

Attempt to delete member
image
image
image

Looking at the code I think the parsing of the item string does not expect an IASP as part of the string.
image

@worksofliam worksofliam self-assigned this Apr 21, 2022
@worksofliam worksofliam added enhancement New feature or request and removed info-needed More information is requied from the issuer labels Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants