-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Fix PG trust metadata layer loading #38833
Fix PG trust metadata layer loading #38833
Conversation
Perfect - I confirm that it fixes my issue!
Thanks a lot,
Andreas
On 2020-09-17 13:04, Alessandro Pasotti wrote:
Fixes #38809 [1]
Followup 760347c [2]
-------------------------
YOU CAN VIEW, COMMENT ON, OR MERGE THIS PULL REQUEST ONLINE AT:
#38833
COMMIT SUMMARY
* Fix PG trust metadata layer loading
FILE CHANGES
* M src/core/qgsvectorlayer.cpp [3] (2)
* M src/providers/postgres/qgspostgresprovider.cpp [4] (4)
* M tests/src/python/test_provider_postgres.py [5] (37)
PATCH LINKS:
* https://github.com/qgis/QGIS/pull/38833.patch
* https://github.com/qgis/QGIS/pull/38833.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub [6], or unsubscribe [7].
Links:
------
[1] #38809
[2]
760347c
[3]
https://github.com/qgis/QGIS/pull/38833/files#diff-106be592650b55ccae79c83eecd15ceb
[4]
https://github.com/qgis/QGIS/pull/38833/files#diff-374a3d3ebe5430cec252d7f2c7d5f457
[5]
https://github.com/qgis/QGIS/pull/38833/files#diff-99b101819133a316786604243f3abf6e
[6] #38833
[7]
https://github.com/notifications/unsubscribe-auth/AAGX57EMZBGWRL3HVTXW5HTSGHUKHANCNFSM4RQIFFOQ
|
Fixes qgis#38809 Followup 760347c
c6dcc64
to
9d436cd
Compare
@@ -1544,7 +1544,7 @@ bool QgsVectorLayer::readXml( const QDomNode &layer_node, QgsReadWriteContext &c | |||
} | |||
|
|||
QgsDataProvider::ProviderOptions options { context.transformContext() }; | |||
QgsDataProvider::ReadFlags flags = QgsDataProvider::ReadFlags(); | |||
QgsDataProvider::ReadFlags flags; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The generated code is totally equivalent, I just find redundant call the default ctor explicitly. It's more a style matter, nothing important.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rldhont np.
On another topic I did not check if the same quotedIdentifier
/ quotedValue
mistake was done elsewhere with your original PR or any followup. Can you please check that it only affected PG?
Fixes #38809
Followup 760347c