Changes between Initial Version and Version 3 of Ticket #25665


Ignore:
Timestamp:
Nov 4, 2015, 12:15:34 PM (9 years ago)
Author:
Sergey Fedoseev
Comment:

I was motivated to deprecate num_points and crs because number of GEOSGeometry members looks disturbing for me.

In [2]: GEOSGeometry.
In [2]: from django.contrib.gis.geos import GEOSGeometry as Geometry

In [3]: Geometry.
Geometry.append            Geometry.geojson           Geometry.prepared
Geometry.area              Geometry.geom_type         Geometry.ptr
Geometry.boundary          Geometry.geom_typeid       Geometry.ptr_type
Geometry.buffer            Geometry.get_srid          Geometry.relate
Geometry.centroid          Geometry.has_cs            Geometry.relate_pattern
Geometry.clone             Geometry.hasz              Geometry.remove
Geometry.contains          Geometry.hex               Geometry.reverse
Geometry.convex_hull       Geometry.hexewkb           Geometry.ring
Geometry.coord_seq         Geometry.index             Geometry.set_srid
Geometry.count             Geometry.insert            Geometry.simple
Geometry.crosses           Geometry.intersection      Geometry.simplify
Geometry.crs               Geometry.intersects        Geometry.sort
Geometry.difference        Geometry.json              Geometry.srid
Geometry.dims              Geometry.kml               Geometry.srs
Geometry.disjoint          Geometry.length            Geometry.sym_difference
Geometry.distance          Geometry.mro               Geometry.touches
Geometry.empty             Geometry.normalize         Geometry.transform
Geometry.envelope          Geometry.num_coords        Geometry.union
Geometry.equals            Geometry.num_geom          Geometry.valid
Geometry.equals_exact      Geometry.num_points        Geometry.valid_reason
Geometry.ewkb              Geometry.ogr               Geometry.within
Geometry.ewkt              Geometry.overlaps          Geometry.wkb
Geometry.extend            Geometry.point_on_surface  Geometry.wkt
Geometry.extent            Geometry.pop 

I'll change ticket summary because there are also getters/setters for coordinates of Point and probably there are others.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #25665

    • Property Owner changed from nobody to Sergey Fedoseev
    • Property Status newassigned
    • Property Triage Stage UnreviewedAccepted
    • Property Version 1.8master
    • Property Summary alias members of `GEOSGeometry` (`num_points`, `crs`, `get_srid`, `set_srid`) should be deprecated or documenteddeprecate public getters/setters for properties of `GEOSGeometry` and its subclasses
  • Ticket #25665 – Description

    initial v3  
    1 I'd prefer to deprecate them because I see no benefit in having them, I think they just litter class namespace.
Back to Top