[QGIS-Developer] pyqgis - layer.saveDefaultStyle() for PostGIS layers
kimaidou
kimaidou at gmail.com
Wed May 9 11:02:09 PDT 2018
Great news, thanks for the report ;)
2018-05-04 14:30 GMT+02:00 Etienne Trimaille <etienne.trimaille at gmail.com>:
> For your information, with the big hint from Matthias in the last email,
> it's now fixed in 2.18.20. I checked in QGIS 3, it was working fine already.
>
> 2017-09-26 9:07 GMT-04:00 Matthias Kuhn <matthias at opengis.ch>:
>
>> For 2.18 it should be ok to make it /InOut/ (there was no return value
>> before, so it shouldn't break anything).
>> For 3.0 this can be made /Out/ as proposed.
>>
>> Matthias
>>
>> On 09/26/2017 12:06 PM, Luigi Pirelli wrote:
>> > there is a SIP specification error
>> > https://github.com/qgis/QGIS/blob/release-2_18/python/core/q
>> gsvectorlayer.sip#L492
>> > should be QString &msgError /out/ );
>> >
>> > that's the reason that error is not returned in the python side => I
>> > suppose SIP would change C++ api in:
>> > msg = saveStyleToDatabase( name, description, useAsDefault,
>> uiFileContent)
>> > Luigi Pirelli
>> >
>> > ************************************************************
>> **************************************
>> > * Boundless QGIS Support/Development: lpirelli AT boundlessgeo DOT com
>> > * LinkedIn: https://www.linkedin.com/in/luigipirelli
>> > * Stackexchange: http://gis.stackexchange.com/users/19667/luigi-pirelli
>> > * GitHub: https://github.com/luipir
>> > * Mastering QGIS 2nd Edition:
>> > * https://www.packtpub.com/big-data-and-business-intelligence/
>> mastering-qgis-second-edition
>> > ************************************************************
>> **************************************
>> >
>> >
>> > On 25 September 2017 at 16:21, kimaidou <kimaidou at gmail.com> wrote:
>> >> I have tried another method
>> >>
>> >> * first add default style into DB via UI -> it works
>> >>
>> >> * the use console to create a new, not default, style
>> >> msg = ''
>> >> layer.saveStyleToDatabase( "a new style", "some description", False,
>> None,
>> >> msg );
>> >>
>> >> It does not work (not new line created, and msg remains empty. No log
>> in the
>> >> database, which means NO query has been sent (I have change postgresql
>> >> config to log all requests).
>> >>
>> >> But the following method works
>> >> layer.listStylesInDatabase()
>> >>
>> >> And a new line appears in the postgresql log
>> >> 2017-09-25 16:19:18.248 CEST [8984] mdouchin at qgis LOG: 00000:
>> duration:
>> >> 0.217 ms statement: SELECT id,styleName,description FROM layer_styles
>> WHERE
>> >> NOT (f_table_catalog='qgis' AND f_table_schema='sup' AND
>> >> f_table_name='geo_commune' AND f_geometry_column='geom') ORDER BY
>> >> update_time DESC
>> >>
>> >> It seems the method saveStyleToDatabase returns with no error, which
>> can be
>> >> caused by 2 reasons
>> >> https://qgis.org/api/2.18/qgsvectorlayer_8cpp_source.html#l04347
>> >>
>> >> 4369 this->exportNamedStyle( qmlDocument, msgError );
>> >> 4370 if ( !msgError.isNull() )
>> >> 4371 {
>> >> 4372 return;
>> >> 4373 }
>> >> 4374 qmlStyle = qmlDocument.toString();
>> >> 4375
>> >> 4376 this->exportSldStyle( sldDocument, msgError );
>> >> 4377 if ( !msgError.isNull() )
>> >> 4378 {
>> >> 4379 return;
>> >> 4380 }
>> >>
>> >> /me investigates
>> >>
>> >> 2017-09-25 15:50 GMT+02:00 Jorge Gustavo Pereira Bastos Rocha
>> >> <jgr at di.uminho.pt>:
>> >>>
>> >>> Hi,
>> >>>
>> >>> It already happened to me. The first user to save a style became the
>> owner
>> >>> of the table. No other users were able to write on the layer_style
>> table,
>> >>> because there were DEFAULT PRIVILEGES defined.
>> >>>
>> >>> Can you check the postgrsql log to see if there is any problem related
>> >>> with permissions?
>> >>>
>> >>> Regards,
>> >>>
>> >>> Jorge Gustavo
>> >>>
>> >>> ________________________________________
>> >>> De: QGIS-Developer [qgis-developer-bounces at lists.osgeo.org] em nome
>> de
>> >>> kimaidou [kimaidou at gmail.com]
>> >>> Enviado: 25 de Setembro de 2017 14:23
>> >>> Para: Luigi Pirelli
>> >>> Cc: qgis-developer at lists.osgeo.org
>> >>> Assunto: Re: [QGIS-Developer] pyqgis - layer.saveDefaultStyle() for
>> >>> PostGIS layers
>> >>>
>> >>> Hi,
>> >>>
>> >>> @ Tom : I really would like to trust QGIS method instead of managing
>> >>> manually different providers.
>> >>>
>> >>> @ Luigi
>> >>> No error message, even worse : i got a message telling everything went
>> >>> well ;)
>> >>>
>> >>> layer.saveDefaultStyle()
>> >>> (u'The style dbname=\'qgis\' host=localhost port=5432
>> user=\'mdouchin\'
>> >>> sslmode=disable key=\'ogc_fid\' estimatedmetadata=true srid=2154
>> >>> type=MultiPolygon table="sup"."geo_commune" (geom) sql= was updated
>> in the
>> >>> database.', True)
>> >>>
>> >>> But the line is not created in the layer_styles table.
>> >>>
>> >>> Michaël
>> >>>
>> >>>
>> >>> 2017-09-25 14:48 GMT+02:00 Luigi Pirelli
>> >>> <luipir at gmail.com<mailto:luipir at gmail.com>>:
>> >>> from the code:
>> >>>
>> >>> mLayer->saveStyleToDatabase and mLayer->saveDefaultStyle
>> >>>
>> >>> and in
>> >>> QString QgsMapLayer::saveDefaultStyle( bool & theResultFlag )
>> >>> {
>> >>> return saveNamedStyle( styleURI(), theResultFlag );
>> >>> }
>> >>>
>> >>> => seems you are doing correct.
>> >>>
>> >>> any log message?
>> >>>
>> >>> ******************************************
>> >>>
>> >>>
>> >>> void QgsVectorLayerProperties::saveDefaultStyle_clicked()
>> >>> {
>> >>> apply();
>> >>> QString errorMsg;
>> >>> if ( mLayer->dataProvider()->isSaveAndLoadStyleToDBSupported() )
>> >>> {
>> >>> QMessageBox askToUser;
>> >>> askToUser.setText( tr( "Save default style to: " ) );
>> >>> askToUser.setIcon( QMessageBox::Question );
>> >>> askToUser.addButton( tr( "Cancel" ), QMessageBox::RejectRole );
>> >>> askToUser.addButton( tr( "Local database" ), QMessageBox::NoRole
>> );
>> >>> askToUser.addButton( tr( "Datasource database" ),
>> QMessageBox::YesRole
>> >>> );
>> >>>
>> >>> switch ( askToUser.exec() )
>> >>> {
>> >>> case 0:
>> >>> return;
>> >>> case 2:
>> >>> mLayer->saveStyleToDatabase( "", "", true, "", errorMsg );
>> >>> if ( errorMsg.isNull() )
>> >>> {
>> >>> return;
>> >>> }
>> >>> break;
>> >>> default:
>> >>> break;
>> >>> }
>> >>> }
>> >>>
>> >>> bool defaultSavedFlag = false;
>> >>> errorMsg = mLayer->saveDefaultStyle( defaultSavedFlag );
>> >>> if ( !defaultSavedFlag )
>> >>> {
>> >>> QMessageBox::warning( this, tr( "Default Style" ), errorMsg );
>> >>> }
>> >>> }
>> >>> Luigi Pirelli
>> >>>
>> >>>
>> >>> ************************************************************
>> **************************************
>> >>> * Boundless QGIS Support/Development: lpirelli AT boundlessgeo DOT com
>> >>> * LinkedIn: https://www.linkedin.com/in/luigipirelli
>> >>> * Stackexchange: http://gis.stackexchange.com/u
>> sers/19667/luigi-pirelli
>> >>> * GitHub: https://github.com/luipir
>> >>> * Mastering QGIS 2nd Edition:
>> >>> *
>> >>> https://www.packtpub.com/big-data-and-business-intelligence/
>> mastering-qgis-second-edition
>> >>>
>> >>> ************************************************************
>> **************************************
>> >>>
>> >>>
>> >>> On 25 September 2017 at 13:44, Tom Chadwin
>> >>> <tom.chadwin at nnpa.org.uk<mailto:tom.chadwin at nnpa.org.uk>> wrote:
>> >>>> Perhaps you have to do it manually via a PostGIS query matching
>> >>>> f_table_catalog, f_table_schema, and f_table_name, and updating
>> styleqml
>> >>>> and
>> >>>> stylesld?
>> >>>>
>> >>>> Tom
>> >>>>
>> >>>>
>> >>>>
>> >>>> -----
>> >>>> Buy Pie Spy: Adventures in British pastry 2010-11 on Amazon
>> >>>> --
>> >>>> Sent from:
>> >>>> http://osgeo-org.1560.x6.nabble.com/QGIS-Developer-f4099106.html
>> >>>> _______________________________________________
>> >>>> QGIS-Developer mailing list
>> >>>> QGIS-Developer at lists.osgeo.org<mailto:QGIS-Developer at lists.osgeo.org
>> >
>> >>>> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>> >>>> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>> >>>
>> >>
>> > _______________________________________________
>> > QGIS-Developer mailing list
>> > QGIS-Developer at lists.osgeo.org
>> > List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>> > Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>> >
>> _______________________________________________
>> QGIS-Developer mailing list
>> QGIS-Developer at lists.osgeo.org
>> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>>
>
>
> _______________________________________________
> QGIS-Developer mailing list
> QGIS-Developer at lists.osgeo.org
> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer
> Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/qgis-developer/attachments/20180509/f0ccd609/attachment-0001.html>
More information about the QGIS-Developer
mailing list