Opened 8 months ago

Closed 7 months ago

Last modified 4 weeks ago

#28841 closed Cleanup/optimization (fixed)

rename GIS function ForceRHR to ForcePolygonCW

Reported by: Sergey Fedoseev Owned by: Sergey Fedoseev
Component: GIS Version: master
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 (8)

comment:1 Changed 8 months ago by Sergey Fedoseev

Owner: changed from nobody to Sergey Fedoseev
Status: newassigned

comment:2 Changed 8 months ago by Sergey Fedoseev

Summary: rename GIS function ST_ForceRHR to ST_ForcePolygonCWrename GIS function ForceRHR to ForcePolygonCW

comment:3 Changed 8 months ago by Sergey Fedoseev

comment:4 Changed 8 months ago by Sergey Fedoseev

Has patch: set

comment:5 Changed 8 months ago by Tim Graham

Triage Stage: UnreviewedAccepted

comment:6 Changed 7 months ago by Tim Graham <timograham@…>

Resolution: fixed
Status: assignedclosed

In aefe624:

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

comment:7 Changed 4 weeks ago by Tim Graham <timograham@…>

In 12018cef:

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

comment:8 Changed 4 weeks ago by Tim Graham <timograham@…>

In afcf5aad:

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

Backport of 12018cef83b2396131e8c32d997baa0f86b4bf00 from master

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