r16775 - ST_Intersеcts support GEOMETRYCOLLECTION

[Darafei]

Author: komzpa
Date: 2018-09-13 06:35:33 -0700 (Thu, 13 Sep 2018)
New Revision: 16775

Modified:
   trunk/NEWS
   trunk/doc/reference_measure.xml
   trunk/postgis/lwgeom_geos.c
   trunk/regress/tickets.sql
   trunk/regress/tickets_expected
Log:
ST_Intersеcts support GEOMETRYCOLLECTION

By some overlook ST_Intersects has a guard against GEOMETRYCOLLECTION, while GEOS can handle it.

Closes #4176
Closes https://github.com/postgis/postgis/pull/297


Modified: trunk/NEWS
===================================================================
--- trunk/NEWS	2018-09-13 13:34:33 UTC (rev 16774)
+++ trunk/NEWS	2018-09-13 13:35:33 UTC (rev 16775)
@@ -12,6 +12,7 @@
   - #4161, MVT: Drop geometries smaller than the resolution (Raúl Marín)
   - #4172, Fix memory leak in lwgeom_offsetcurve (Raúl Marín)
   - #4173, Fix undefined behaviour in ptarray_segmentize2d (Raúl Marín)
+  - #4176, ST_Intersects supports GEOMETRYCOLLECTION (Darafei Praliaskouski)
 
 PostGIS 2.5.0rc1
 2018/08/19

Modified: trunk/doc/reference_measure.xml
===================================================================
--- trunk/doc/reference_measure.xml	2018-09-13 13:34:33 UTC (rev 16774)
+++ trunk/doc/reference_measure.xml	2018-09-13 13:35:33 UTC (rev 16775)
@@ -3474,12 +3474,8 @@
 				returns true, then the geometries also spatially intersect.
 				Disjoint implies false for spatial intersection.</para>
 
-	  <important>
-		<para>Do not call with a <varname>GEOMETRYCOLLECTION</varname> as an argument for geometry version.  The geography
-			version supports GEOMETRYCOLLECTION since its a thin wrapper around distance implementation.</para>
-	  </important>
-	  <para>Enhanced: 2.3.0 Enhancement to PIP short-circuit extended to support MultiPoints with few points. Prior versions only supported point in polygon.</para>
-
+			<para>Enhanced: 2.5.0 Supports GEOMETRYCOLLECTION.</para>
+			<para>Enhanced: 2.3.0 Enhancement to PIP short-circuit extended to support MultiPoints with few points. Prior versions only supported point in polygon.</para>
 			<para>Performed by the GEOS module (for geometry), geography is native</para>
 			<para>Availability: 1.5 support for geography was introduced.</para>
 			<note>

Modified: trunk/postgis/lwgeom_geos.c
===================================================================
--- trunk/postgis/lwgeom_geos.c	2018-09-13 13:34:33 UTC (rev 16774)
+++ trunk/postgis/lwgeom_geos.c	2018-09-13 13:35:33 UTC (rev 16775)
@@ -2148,7 +2148,6 @@
 	geom1 = PG_GETARG_GSERIALIZED_P(0);
 	geom2 = PG_GETARG_GSERIALIZED_P(1);
 
-	errorIfGeometryCollection(geom1,geom2);
 	error_if_srid_mismatch(gserialized_get_srid(geom1), gserialized_get_srid(geom2));
 
 	/* A.Intersects(Empty) == FALSE */

Modified: trunk/regress/tickets.sql
===================================================================
--- trunk/regress/tickets.sql	2018-09-13 13:34:33 UTC (rev 16774)
+++ trunk/regress/tickets.sql	2018-09-13 13:35:33 UTC (rev 16775)
@@ -1103,3 +1103,5 @@
 
 -- Clean up
 DELETE FROM spatial_ref_sys;
+
+SELECT '#4176', ST_Intersects('POLYGON((0 0, 10 10, 3 5, 0 0))', 'GEOMETRYCOLLECTION(POINT(10 10), LINESTRING(0 0, 3 3))');

Modified: trunk/regress/tickets_expected
===================================================================
--- trunk/regress/tickets_expected	2018-09-13 13:34:33 UTC (rev 16774)
+++ trunk/regress/tickets_expected	2018-09-13 13:35:33 UTC (rev 16775)
@@ -336,3 +336,4 @@
 ERROR:  lwgeom_pointonsurface: GEOS Error: TopologyException: Input geom 1 is invalid: Self-intersection
 #4081|f|t
 ERROR:  The 'coordinates' in GeoJSON polygon are not sufficiently nested
+#4176|t