Opened 7 years ago

Closed 7 years ago

Last modified 6 years ago

#28841 closed Cleanup/optimization (fixed)

rename GIS function ForceRHR to ForcePolygonCW

Reported by: Sergey Fedoseev Owned by: Sergey Fedoseev
Component: GIS Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

ST_ForceRHR name is confusing, PostGIS ​doc recommends to use its alias ST_ForcePolygonCW introduced in PostGIS 2.4.
Moreover there is also SpatiaLite function for doing exactly the same thing but it's called ST_ForceRHR. Here's related thread in ​spatialite-users group.

Change History (9)

comment:1 by Sergey Fedoseev, 7 years ago

Owner: changed from nobody to Sergey Fedoseev
Status: new → assigned

comment:2 by Sergey Fedoseev, 7 years ago

Summary: rename GIS function ST_ForceRHR to ST_ForcePolygonCW → rename GIS function ForceRHR to ForcePolygonCW

comment:3 by Sergey Fedoseev, 7 years ago

​PR

comment:4 by Sergey Fedoseev, 7 years ago

Has patch: set

comment:5 by Tim Graham, 7 years ago

Triage Stage: Unreviewed → Accepted

comment:6 by Tim Graham <timograham@…>, 7 years ago

Resolution: → fixed
Status: assigned → closed

In aefe624:

Fixed #28841 -- Added ForcePolygonCW GIS function and deprecated ForceRHR.

comment:7 by Tim Graham <timograham@…>, 6 years ago

In 12018cef:

Refs #28841 -- Added ForcePolygonCW to GIS database functions table.

comment:8 by Tim Graham <timograham@…>, 6 years ago

In afcf5aad:

[2.1.x] Refs #28841 -- Added ForcePolygonCW to GIS database functions table.

Backport of 12018cef83b2396131e8c32d997baa0f86b4bf00 from master

comment:9 by Tim Graham <timograham@…>, 6 years ago

In 6f23996a:

Refs #28841 -- Removed ForceRHR function per deprecation timeline.

Note: See TracTickets for help on using tickets.
Back to Top