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

INTERLIS Export - Logs are not always there where they should be #82

Closed
sjib opened this issue Nov 26, 2022 · 3 comments
Closed

INTERLIS Export - Logs are not always there where they should be #82

sjib opened this issue Nov 26, 2022 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@sjib
Copy link
Contributor

sjib commented Nov 26, 2022

Describe the bug
Logs or only a part of the logs are not always there where they are expected to be.

To Reproduce

  1. Click on INTERLIS Export
  2. Select or deselect "Save logs next to"
    20221126_qgep-export_LK_labels_NOT_next_to_logs
  3. Choose if you want to export labels or not
  4. Click OK
  5. Logs are not always where they are expected

Expected behavior
All Logs are in the folder where they should be depending on selection

Screenshots / data
20221126_qgep-export_NO_labels_next_to_logs_fileoverview
WRONG: qgep-export_NO_labels_next_to_logs_fileoverview - only export log in next to folder

20221126_qgep-export_NO_labels_NOT_next_to_logs_fileoverview
WRONG: qgep-export_NO_labels_NOT_next_to_logs_fileoverview - error log in next to folder instead of temp folder

20221126_qgep-export_LK_labels_NOT_next_to_logs_fileoverview
WRONG: qgep-export_LK_labels_NOT_next_to_logs_fileoverview - all logs in wrong place

20221126_qgep-export_LK_labels_next_to_logs_fileoverview
OK: qgep-export_LK_labels_next_to_logs_fileoverview

Desktop (please complete the following information):

  • QGEP plugin version 1.5.6
  • QGEP datamodel version 1.5.5
  • QGEP project file version
  • QGIS Version 3.26.3
  • OS Windows 10

Additional context
#77 not implemented as expected

@sjib sjib added the bug Something isn't working label Nov 26, 2022
@olivierdalang
Copy link
Collaborator

Ok my bad, the logic for the checkbox was flawed. Fix incoming: #86

@olivierdalang
Copy link
Collaborator

Can you test again (from master qgep plugin) and see if it works ?

This is what I get here, once with the checkbox checked and once with the checkbox unchecked. (note that the main qgepqwat2ili log is expected as it holds important information about the export itself)

image

@sjib
Copy link
Contributor Author

sjib commented Nov 30, 2022

@olivierdalang Looks much better now and as expected. Thanks for the fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants