[SEDONA-118] ST_Within uses wrong JTS Function

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.2.1
• Fix Version/s: 1.2.1
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Dimitris Bilidas
• Created: 22/Apr/22 16:20
• Updated: 04/Sep/22 08:18
• Resolved: 04/Sep/22 08:18

Description

The eval method of ST_Within (in file Predicates.scala) calls the function coveredBy of JTS. Instead, it should call the within function. 

As a result, now, the expression 

ST_Within(ST_GeomFromWKT('POINT(0 0)'),ST_GeomFromWKT('POLYGON ((0.0 0.0, 10.0 0.0, 10.0 10.0, 0.0 10.0, 0.0 0.0))')) 

returns true, whereas the correct behaviour according to the simple features specification for sql should be false.

Also, the toString method of the ST_Within is wrong.

Issue Links

• links to: GitHub Pull Request #623