PostGIS 2.0 ST_AsBinary drops forcing 2d, breaks WKB parser of QGIS

[qgib]

Author Name: Sandro Santilli (@strk)
Original Redmine Issue: 4767
Affected QGIS version: 1.7.3
Redmine category:data_provider/postgis
Assignee: Tim Sutton

---

Up to PostGIS-1.5 ST_AsBinary used to internally force the geometry dimensionality to 2d, due to lack of standard specifications for higher dimensions. As of PostGIS-2.0 the new SQL/MM higher dimension specifications are used. Qgis is unable to parse such WKB and complains as follows:

Debug: /usr/src/qgis/qgis-1.7/src/core/qgsgeometry.cpp: 3701: (boundingBox) Unknown WkbType ENCOUNTERED

Mapserver contains an explicit Force2D call since a long time. Qgis could do the same for a quick fix.

[qgib]

Author Name: Sandro Santilli (@strk)

---

Here's a fix for the 1.7 branch, which also improve the support for geography (fixing opening a geography table with no type advertised in geometry_columns). Note: this was tested against PostGIS 1.4, 1.5 and 2.0svn, opening geometry and geography columns with 4 dimensions.

The following changes since commit 7a9110ab8de73475313f44ebeae76aaa31bf4c43:
  Alexander Bruy (1):
        always save last used dir in Raster terrain analysis plugin (fix #14578)

are available in the git repository at:

  git@github.com:strk/Quantum-GIS.git release-1_7-pgis-2_0

Sandro Santilli (1):
      Simplify code handling geography and support postgis-2.0 WKB (sqlmm)

 src/providers/postgres/qgspostgresprovider.cpp |   34 +++++++++++------------
 1 files changed, 16 insertions(+), 18 deletions(-)

---

• version was changed from master to 1.7.3

[qgib]

Author Name: Sandro Santilli (@strk)

---

I'll postpone port of the fix to master as master is currently undergoing a refactoring

---

• category_id was configured as Data Provider/PostGIS

[qgib]

Author Name: Sandro Santilli (@strk)

---

2b739d9 by jef adapted the patch to master branch.
Does it need to be also backported in a 1.8 branch ?

[qgib]

Author Name: Sandro Santilli (@strk)

---

Fixed in changeset "8d65c9728065d5b34b4e8c0418ad7767ea6f514e".

---

• done_ratio was changed from 0 to 100
• status_id was changed from Open to Closed

[qgib]

Author Name: Sandro Santilli (@strk)

---

The commit references don't make it obvious which branch contains the commit :(
I've no time to file a bug for that against hub...

Anyway, I'm assuming 8d65c97 is in 1.7.
Do we need to port in 1.8 as well ?

[qgib]

Author Name: Sandro Santilli (@strk)

---

Done in 1.7 branch and master (2.0).
Still needs to be taken care of in 1.8 branch.

---

• status_id was changed from Closed to Reopened

[qgib]

Author Name: Jürgen Fischer (@jef-n)

---

Sandro Santilli wrote:

Fixed in changeset "8d65c9728065d5b34b4e8c0418ad7767ea6f514e".

FTR I changed the commit message to contain "fixes #14630".

[qgib]

Author Name: Sandro Santilli (@strk)

---

@jef: great, thanks !

@tim: I've been told you're the one taking care of backports to 1.8 :)

---

• assigned_to_id was configured as Tim Sutton

[qgib]

Author Name: Giuseppe Sucameli (@brushtyler)

---

The fix seems already present in branch 1.8.

---

• resolution was changed from to fixed
• status_id was changed from Reopened to Closed