merge from qgis #12
Merged
merge from qgis #12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tests: replace wrong uses of unittest.skip() by unittest.skipIf()
Fixes manual mode in merge features tool
GetFeatureInfo and no data values
Check if a point is 3D before returning its z value
Instead of hijacking 'layerName' which has a different semantics. This modifies commit 84eed01, which returns the database name in the layerName URI component, which is not the right semantic, since layerName should be a OGR layer name. With this fix encodeUri() can roundtrip with decodeUri()
OGR provider: make decodeUri() set a 'databaseName' component
…quoting PG fix relation discovery with table/schema that require quotes
…ore passing geometry to OGR (fixes #39081)
… as such. No functional change
Currently when defining a new OGR Database connection, one must fill the database, host and port numbers. Some/most of them are optional on the OGR side (for example the MYSQL driver only requires the database name, but the PG driver could for example work without any depending on the PG config), so do not require them at all. The 'Test connection' button is already there for users to check they've properly filled the form.
… 'v' might be accessed at non-zero index' false-positive by making code clearer
…ctable as it has pointers
Test for writing weak relation attributes to XML (followup #39090)
… it has pointers (but cannot be enabled because of SIP issue)
…cur, and make code slightly more efficient
…positive from cppcheck
…ent PROJ versions
Background: https://www.qt.io/blog/2011/06/13/string-concatenation-with-qstringbuilder When enabling -DQT_USE_QSTRINGBUILDER, QString() concatenation with operator + are done in a more efficient way, especially with repeated + operators. Qt and creator themselves are built with that enabled.
Reusing the same color resulted in too subtle indentation guides
and move responsibility for showing warning messages to QgsCodeEditor base class, so that the same code can be used by other dialog script editors
It's not free to calculate, and is only used when iterating over a memory layer with a subset string set Results in a big speed up to scripts which fire off many individual feature requests to a memory provider layer
Unreported issue: discovered in the QGIS-Documentation test CI issue https://travis-ci.org/github/qgis/QGIS-Documentation/builds/734917369#L679
…sh-on-save Don't crash on transaction save
…rType-case OGR decodeUri: be tolerant on layerName case
Fix unreported crash
QgsApplication::getThemePixmap
…e restored from XML
…rrectly use this to set the QgsCoordinateReferenceSystem description
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.