ogr.Geometry __init__ method not functionnal with some attributes

Proposed patch

Some info about the patch :

Forget about my suggestions in my first post. I was wrong. 

The patch seems the right way to do it. Calling ogr.CreateGeometryFromWkt directly from the Geometry constructor gives a Segmentation fault. That's why I had to use the _gdal.OGR_G_CreateGeometryFromWkt function.

Didrik,

I'm a little foggy on the description, but is the problem that the Create*from* methods are returning values?  Have you tried this with the next generation bindings?  Do they still cause the problem?

I'll take on this bug for now.  If you can recall any more details about it, I would greatly appreciate it.

Howard

Hi Howard,

Concerning the ng bindings, I haven't tested them but will in a near future.

About this specific bug problem, here are some clearer details : 

Python __init__ methods can not return a value. So if ogr wants to create a new geometry in the __init__ method, it can do it but without returning the created object. Python will automatically return a reference to the newly created Python object.

The patch I proposed is the best way to do it without changing the Python interface.

For example, the wkt case : 

-            if srs:
-                return CreateGeometryFromWkt(wkt, srs)
+            if srs is None:
+                srs = ''
             else:
-                return CreateGeometryFromWkt(wkt)
+                srs = srs._o
+            _obj  = _gdal.OGR_G_CreateFromWkt(wkt, srs)
+            if _obj is not None and _obj != 'NULL':
+                self._o = _obj
+                self.thisown = 1

The return CreateGeometryFromWkt(wkt, srs) line is not valid in Python. I've added some cleaner testing about the None value for srs, then created the _obj using _gdal.OGR_G_CreateFromWkt and finally updated the self._o object. This is all it has to do in the __init__ method.

Is it clear for you ?

Didrik

Hi Howard,

I've just tested the new generation bindings and everything works well. This is from the official version 1.4.0

BUT 

I guess that it's not possible to create a geometry with ogr.Geometry from wkt and with an srs (from ogr.py):

 __init__(self, OGRwkbGeometryType type=wkbUnknown, char wkt=0, int wkb=0, 

            char wkb_buf=0, char gml=0) -> Geometry



You can do it from ogr.CreateGeometryFromWkt but not anymore from ogr.Geometry.

Because it is possible to create a geometry from Wkt using ogr.Geometry(wkt=...), I guess that the API should allow the user to create a geometry with an srs like this ogr.Geometry(wkt=..., reference=mysrs).

I'm available if you need anymore information.

Didrik

To summarize the problem,

with version 1.3.x : Geometry.__init__ is broken when using srs, wkt, wkb or gml attribute. This can be easily corrected using my proposed patch

with version 1.4.x : the code runs fine.

Didrik

Didrik, 

So does this mean there is no action since the problem is already corrected?
If so, please close this bug with status FIXED and an appropriate note. 

Frank,

I forgot one precision in my summary.

It works fine with version 1.4.x and the ng bindings. 

The 1.4.x with the old bindings has exactly the same problem has the one described for version 1.3.x and must be patched (pymod/ogr.py).

Didrik

committed in r11099.