[Qgis-developer] RE: postgres provider refactoring

[Jeremy Palmer]

Hi Juergen,

I just tested d5431b3 and I think there is a regression in functionality. 

I used to be able to load a PostGIS layer that contained both polygon and multi-polygon features. Now the geometry type is set to polygon only, and I don't see the multi-polygon geometries. Unless I use the API I have no way to see these multipolgons at all now.

Cheers,
Jeremy


> Message: 3
> Date: Mon, 9 Jan 2012 14:08:02 +0100
> From: J?rgen E. Fischer <jef at norbit.de>
> Subject: [Qgis-developer] 
> To: qgis-developer at lists.osgeo.org
> Message-ID: <20120109130802.GA9675 at norbit.de>
> Content-Type: text/plain; charset=iso-8859-1
> 
> Hi there,
> 
> I just commited a larger change to the postgres provider.  Mostly it's
> refactoring, but it also contains some features.
> 
> You should now be able to use primary keys that are not numeric and even
> primary keys with more that one column should work (not sure how well the
> latter performs).
> 
> The handling of the GEOMETRY type was also changed - the geometry type is now
> specified in the layer uri and therefore the automatically generated subset
> string is not necessary anymore.  For empty tables you can select with geometry
> type to use.
> 
> Views still need numeric primary keys and those need to be selected when
> adding.
> 
> Please test and report.
> 
> 
> Author: Juergen E. Fischer <jef at norbit.de>
> Date:   Thu Dec 29 00:02:47 2011 +0100
> 
>     [FEATURE][API] postgres provider refactoring:
>     - use m prefix for member variables and s prefix for class variables
>     - refactor QgsPostgresProvider::Conn and QgsPostgresConnection to
>       QgsPostgresConn
>     - put QgsColumnTypeThread into separate file
>     - use QgsPostgresConn in QgsPgSourceSelect
>     - cleaner abort of column type thread (e.g. remove unfinished columns from
>       selection list)
>     - [API] move QgsDbTableModel as QgsPgTableModel to provider
>     - [FEATURE] support for arbitrary key (including non-numeric and multi
>       column)
>     - [FEATURE][API] support for requesting a certain geometry type and/or srid
>       in QgsDataSourceURI
>      * no more explicit filter strings
>      * support for empty tables without gemetry_columns entry or GEOMETRY type.
 
#####################################################################################

This message contains information, which is confidential and may be subject to legal privilege. 
If you are not the intended recipient, you must not peruse, use, disseminate, distribute or copy this message.
If you have received this message in error, please notify us immediately (Phone 0800 665 463 or info at linz.govt.nz) and destroy the original message.
LINZ accepts no responsibility for changes to this email, or for any attachments, after its transmission from LINZ.

Thank You.

#####################################################################################