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

v16.1 with some bugs fixed #9

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Sicitronic
Copy link

@Sicitronic Sicitronic commented Mar 12, 2020

v16.1 with some bugs fixed

[x] This contribution adheres to CONTRIBUTING.md.

TODO: Check the above box with an 'x' indicating you've read and followed CONTRIBUTING.md.

What does this Pull Request accomplish?

TODO: Include high-level description of the changes in this pull request.

Why should this Pull Request be merged?

TODO: Justify why this contribution should be part of the project.

What testing has been done?

TODO: Detail what testing has been done to ensure this submission meets requirements.

Into Replication and Deployment Utility.vi -> rad_Deploy Application Image.vi -> rad_Extract Full Image.vi. sometimes the task that copies .lvappimg image into lvtemporary_xxxxxx.tmp\lvappimage.contents is executed before deleting the content into lvtemporary_xxxxxx.tmp\lvappimage.contents, so the image is deleted. The program flow must guarantee to delete the content into lvtemporary_xxxxxx.tmp\lvappimage.contents before copy the image into it.
@Sicitronic
Copy link
Author

Bug solution to following problem:

Into Replication and Deployment Utility.vi -> rad_Deploy Application Image.vi -> rad_Extract Full Image.vi. sometimes the task that copies .lvappimg image into lvtemporary_xxxxxx.tmp\lvappimage.contents is executed before deleting the content into lvtemporary_xxxxxx.tmp\lvappimage.contents, so the image is deleted. The program flow must guarantee to delete the content into lvtemporary_xxxxxx.tmp\lvappimage.contents before copy the image into it.

Fix the following bug:

When retrieving a new image, if you press "back" on the "Choose Image Directory" step and then press "OK" again, the application image path shows as incorrect. Every time you press back and OK, a new level is added to the image path.
Rad App Version = 16.1
Updated README.md with the fixed bugs
@Sicitronic
Copy link
Author

Fixed Bugs:

  • Into rad_Extract Full Image.vi. sometimes the task that copies .lvappimg image into lvtemporary_xxxxxx.tmp\lvappimage.contents is executed before deleting the content into lvtemporary_xxxxxx.tmp\lvappimage.contents, so the image is deleted. The program flow must guarantee to delete the content into lvtemporary_xxxxxx.tmp\lvappimage.contents before copy the image into it.

  • Into rad_Get Image Save Location.vi when retrieving a new image, if you press "back" on the "Choose Image Directory" step and then press "OK" again, the application image path shows as incorrect. Every time you press back and OK, a new level is added to the image path.

Updated:

  • rad_Globals.vi updated with the following changes Rad App Version = 16.1

  • README.md

@Sicitronic Sicitronic changed the title Bug solution in "rad_Extract Full Image.vi" v16.1 with some bugs fixed Mar 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant