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

Explicitly list each GDAL vector export format #1779

Merged
merged 6 commits into from
Nov 16, 2020
Merged

Explicitly list each GDAL vector export format #1779

merged 6 commits into from
Nov 16, 2020

Conversation

dg0yt
Copy link
Member

@dg0yt dg0yt commented Nov 15, 2020

No description provided.

Don't construct the filter string everytime an extension is added,
but only when it is actually used.
Replace parentheses in the filter name by brackets: parentheses
mark the filename patterns.
When QFileDialog is used with option HideNameFilterDetails, the
string returning the selected filter does not contain the list of
extensions. So the matching by filter must ignore the extensions.
Similar to the OCD format export variants, register a separate export
format for each GDAL driver which supports vector data export. Adjust
the export dialog to list these formats explicitly.
Resolves GH-1778 (List all possible vector formats).
Sometimes we need to add a default extension to a new filename. This
change moves existing code from MainWindow to FileFormat. It makes
use of the new function for geospatial vector data export.

The function is modified to work without manipulating the given
list of extensions. It no longer uses the deprecated QRegExp class.
Mapper is limited to GDAL vector export formats which do have registered
extensions. This must be reflected by this tool which provides input
for editing documentation.
@dg0yt dg0yt merged commit d17d1e4 into master Nov 16, 2020
@dg0yt dg0yt deleted the issue-1778 branch November 16, 2020 08:25
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