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

Multi-grid file mask #96

Closed
ianneilmacleod opened this issue Apr 25, 2018 · 4 comments
Closed

Multi-grid file mask #96

ianneilmacleod opened this issue Apr 25, 2018 · 4 comments
Assignees
Labels
Projects

Comments

@ianneilmacleod
Copy link
Contributor

See line 54 in ...\gxpy\examples\om-extensions\test_project.py

When the dialog is presented, pressing [...] brings this up:

2018-04-25_15-28-01

@jacquesbeaurain
Copy link
Contributor

@ianneilmacleod I have a fix for this that would require a rebuild of 9.4 Desktop Application. I am not sure how this was not a pre-existing issue in 9.3.

Using *.grd in a file edit in a GRC would cause our DAT file browse to be displayed. In the case of multi file browsing the code involved in determining if it is an open vs. save dialog had a bug that caused it to call the save dialog code. Having multi file and save flags combined are considered an "improper argument" by the Windows APIs we use for displaying file dialogs, hence the error.

@ianneilmacleod ianneilmacleod removed this from GXAPI in 9.4 Apr 30, 2018
@ianneilmacleod
Copy link
Contributor Author

@jbgeosoft I'll move this to 9.4.1

@ianneilmacleod ianneilmacleod added this to gxpy in 9.5 Apr 30, 2018
@ianneilmacleod ianneilmacleod moved this from gxpy to gxapi in 9.5 Apr 30, 2018
@ianneilmacleod ianneilmacleod added this to GXAPI related in Future Backlog Apr 30, 2018
@ianneilmacleod ianneilmacleod removed this from GXAPI related in Future Backlog Apr 30, 2018
@jacquesbeaurain
Copy link
Contributor

jacquesbeaurain commented Apr 30, 2018

Fixed in 9.5 testing (desk repo commit 7ba2f5f4b7c3a860b40461f4468547b2204bbbcd).

Fix available in testing build Build#: 13352 or later.

Needs to be merged in 9.4.1 once those branches exists.

ianneilmacleod added a commit that referenced this issue May 10, 2018
@ianneilmacleod
Copy link
Contributor Author

Verified fixed for 9.4.2.

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

No branches or pull requests

2 participants