Opened 13 years ago

Closed 13 years ago

#794 closed task (fixed)

Rename function ST_ModEdgesSplit to ST_ModEdgeSplit

Reported by: robe Owned by: strk
Priority: medium Milestone: PostGIS 2.0.0
Component: topology Version: master
Keywords: Cc:

Description

This we discussed in: http://www.postgis.org/pipermail/postgis-devel/2011-January/011541.html

That in current and older versions this function was transcribed wrong and the name is not SQL-MM compliant. But strk felt we should keep it. I personally feel we should just scrap ST_ModEdgesSplit to minimize on confusion (with new users accidentally using the old function name) with a change note in the docs and readme that this function name has changed and can be aliased back to old name if really needed. Perhaps put in a legacy script like we are building for the core postgis stuff I'm dissecting out.

I consider this a new beginning for topology and just want to shed old confusing baggage so we can add really useful baggage.

Change History (2)

comment:1 by strk, 13 years ago

Summary: Deprecate (or delete) function ST_ModEdgesSplit and add ST_ModEdgeSplitRename function ST_ModEdgesSplit to ST_ModEdgeSplit

You convinced me !

comment:2 by strk, 13 years ago

Resolution: fixed
Status: newclosed

Fixed with r6948

Note: See TracTickets for help on using tickets.