Opened 11 years ago
Last modified 7 years ago
#2630 new defect
Typo in UpdateGeometrySRID parameter name
Reported by: | strk | Owned by: | robe |
---|---|---|---|
Priority: | medium | Milestone: | PostGIS Fund Me |
Component: | postgis | Version: | 2.1.x |
Keywords: | Cc: |
Description
The signature is:
UpdateGeometrySRID( catalogn_name varchar, schema_name varchar, table_name varchar, column_name varchar, new_srid_in integer )
Changing the name of the first argument would require a DROP function…
Assigning to Regina as she feels it's ok to DROP
Change History (5)
comment:1 by , 11 years ago
comment:2 by , 11 years ago
Milestone: | PostGIS 2.1.2 → PostGIS 2.2.0 |
---|
I feel it's okay to drop in a minor release. I'm out to lunch in micro — it's borderline. In most cases I wouldn't unless it was causing problems like ambiguous name conflict.
no I will not name ST_, cause then I'd have to keep the old name anyway and also do the same with all the other management functions — http://postgis.net/docs/manual-dev/reference.html#Management_Functions
Massive renamings should only be done in major releases in my opinion where people can expect some non-backward compatibility.
comment:3 by , 11 years ago
oh and we never do renaming of args in micros — that's a big no no since some people used named args
comment:4 by , 9 years ago
Milestone: | PostGIS 2.2.0 → PostGIS Future |
---|
BTW, should it be ST_UpdateGeometrySRID ?