[postgis-tickets] r17978 - Fix PLPGSQL functions missing the schema qualification

Raul raul at rmr.ninja
Fri Oct 18 09:32:35 PDT 2019


Author: algunenano
Date: 2019-10-18 09:32:35 -0700 (Fri, 18 Oct 2019)
New Revision: 17978

Modified:
   branches/2.5/NEWS
   branches/2.5/postgis/geography.sql.in
   branches/2.5/postgis/postgis.sql.in
   branches/2.5/raster/rt_pg/rtpostgis.sql.in
Log:
Fix PLPGSQL functions missing the schema qualification

References #4546


Modified: branches/2.5/NEWS
===================================================================
--- branches/2.5/NEWS	2019-10-18 16:31:41 UTC (rev 17977)
+++ branches/2.5/NEWS	2019-10-18 16:32:35 UTC (rev 17978)
@@ -20,6 +20,7 @@
   - #4537, Fix leak in WKT collection parser (Raúl Marín)
   - #4547, Fix AddRasterConstraints handling of empty tables (Sandro Santilli)
   - #4549, Fix schema qualification of internal types (Raúl Marín)
+  - #4546, Fix PLPGSQL functions missing the schema qualification (Raúl Marín)
 
 
 PostGIS 2.5.3

Modified: branches/2.5/postgis/geography.sql.in
===================================================================
--- branches/2.5/postgis/geography.sql.in	2019-10-18 16:31:41 UTC (rev 17977)
+++ branches/2.5/postgis/geography.sql.in	2019-10-18 16:32:35 UTC (rev 17978)
@@ -652,7 +652,7 @@
 -- Availability: 1.5.0 - this is just a hack to prevent unknown from causing ambiguous name because of geography
 CREATE OR REPLACE FUNCTION ST_Length(text)
 	RETURNS float8 AS
-	$$ SELECT ST_Length($1::@extschema at .geometry);  $$
+	$$ SELECT @extschema at .ST_Length($1::@extschema at .geometry);  $$
 	LANGUAGE 'sql' IMMUTABLE STRICT _PARALLEL;
 
 -- Availability: 2.0.0

Modified: branches/2.5/postgis/postgis.sql.in
===================================================================
--- branches/2.5/postgis/postgis.sql.in	2019-10-18 16:31:41 UTC (rev 17977)
+++ branches/2.5/postgis/postgis.sql.in	2019-10-18 16:32:35 UTC (rev 17978)
@@ -2078,7 +2078,7 @@
         IF upper(gc_old.type) = 'GEOMETRY' THEN
         -- This is an unconstrained geometry we need to do something
         -- We need to figure out what to set the type by inspecting the data
-            EXECUTE 'SELECT @extschema at .ST_srid(' || quote_ident(gcs.attname) || ') As srid, GeometryType(' || quote_ident(gcs.attname) || ') As type, @extschema at .ST_NDims(' || quote_ident(gcs.attname) || ') As dims ' ||
+            EXECUTE 'SELECT @extschema at .ST_srid(' || quote_ident(gcs.attname) || ') As srid, @extschema at .GeometryType(' || quote_ident(gcs.attname) || ') As type, @extschema at .ST_NDims(' || quote_ident(gcs.attname) || ') As dims ' ||
                      ' FROM ONLY ' || quote_ident(gcs.nspname) || '.' || quote_ident(gcs.relname) ||
                      ' WHERE ' || quote_ident(gcs.attname) || ' IS NOT NULL LIMIT 1;'
                 INTO gc;
@@ -5012,7 +5012,7 @@
 	RETURNS geometry
 	AS '
 	SELECT CASE
-	WHEN geometrytype(@extschema at .ST_GeomFromText($1, $2)) = ''GEOMETRYCOLLECTION''
+	WHEN @extschema at .geometrytype(@extschema at .ST_GeomFromText($1, $2)) = ''GEOMETRYCOLLECTION''
 	THEN @extschema at .ST_GeomFromText($1,$2)
 	ELSE NULL END
 	'
@@ -5358,8 +5358,8 @@
 DECLARE
 	geomtext alias for $1;
 	srid alias for $2;
-	mline geometry;
-	geom geometry;
+	mline @extschema at .geometry;
+	geom @extschema at .geometry;
 BEGIN
 	mline := @extschema at .ST_MultiLineStringFromText(geomtext, srid);
 
@@ -5370,7 +5370,7 @@
 
 	geom := @extschema at .ST_BuildArea(mline);
 
-	IF GeometryType(geom) != 'POLYGON'
+	IF @extschema at .GeometryType(geom) != 'POLYGON'
 	THEN
 		RAISE EXCEPTION 'Input returns more then a single polygon, try using BdMPolyFromText instead';
 	END IF;
@@ -5400,8 +5400,8 @@
 DECLARE
 	geomtext alias for $1;
 	srid alias for $2;
-	mline geometry;
-	geom geometry;
+	mline @extschema at .geometry;
+	geom @extschema at .geometry;
 BEGIN
 	mline := @extschema at .ST_MultiLineStringFromText(geomtext, srid);
 
@@ -5858,14 +5858,14 @@
   RETURNS geometry AS
 $$
 	DECLARE
-	vexhull GEOMETRY;
-	var_resultgeom geometry;
-	var_inputgeom geometry;
-	vexring GEOMETRY;
-	cavering GEOMETRY;
-	cavept geometry[];
+	vexhull @extschema at .geometry;
+	var_resultgeom @extschema at .geometry;
+	var_inputgeom @extschema at .geometry;
+	vexring @extschema at .geometry;
+	cavering @extschema at .geometry;
+	cavept @extschema at .geometry[];
 	seglength double precision;
-	var_tempgeom geometry;
+	var_tempgeom @extschema at .geometry;
 	scale_factor float := 1;
 	i integer;
 	BEGIN
@@ -5922,7 +5922,7 @@
 		-- make sure result covers original (#3638)
 		-- Using ST_UnaryUnion since SFCGAL doesn't replace with its own implementation
 		-- and SFCGAL one chokes for some reason
-		var_resultgeom := @extschema at .ST_UnaryUnion(ST_Collect(param_inputgeom, var_resultgeom) );
+		var_resultgeom := @extschema at .ST_UnaryUnion(@extschema at .ST_Collect(param_inputgeom, var_resultgeom) );
 		RETURN var_resultgeom;
 
 	END;
@@ -5934,18 +5934,18 @@
 CREATE OR REPLACE FUNCTION ST_ConcaveHull(param_geom geometry, param_pctconvex float, param_allow_holes boolean DEFAULT false) RETURNS geometry AS
 $$
 	DECLARE
-		var_convhull geometry := @extschema at .ST_ForceSFS(@extschema at .ST_ConvexHull(param_geom));
-		var_param_geom geometry := @extschema at .ST_ForceSFS(param_geom);
+		var_convhull @extschema at .geometry := @extschema at .ST_ForceSFS(@extschema at .ST_ConvexHull(param_geom));
+		var_param_geom @extschema at .geometry := @extschema at .ST_ForceSFS(param_geom);
 		var_initarea float := @extschema at .ST_Area(var_convhull);
 		var_newarea float := var_initarea;
 		var_div integer := 6; /** this is the 1/var_div is the percent increase we will allow per geometry to keep speed decent **/
-		var_tempgeom geometry;
-		var_tempgeom2 geometry;
-		var_cent geometry;
-		var_geoms geometry[4]; /** We will cut the current geometry into 4 triangular quadrants along the centroid/extent **/
-		var_enline geometry;
-		var_resultgeom geometry;
-		var_atempgeoms geometry[];
+		var_tempgeom @extschema at .geometry;
+		var_tempgeom2 @extschema at .geometry;
+		var_cent @extschema at .geometry;
+		var_geoms @extschema at .geometry[4]; /** We will cut the current geometry into 4 triangular quadrants along the centroid/extent **/
+		var_enline @extschema at .geometry;
+		var_resultgeom @extschema at .geometry;
+		var_atempgeoms @extschema at .geometry[];
 		var_buf float := 1; /** tolerance so that geometries that are right on the extent don't get accidentally clipped off **/
 	BEGIN
 		-- We start with convex hull as our base

Modified: branches/2.5/raster/rt_pg/rtpostgis.sql.in
===================================================================
--- branches/2.5/raster/rt_pg/rtpostgis.sql.in	2019-10-18 16:31:41 UTC (rev 17977)
+++ branches/2.5/raster/rt_pg/rtpostgis.sql.in	2019-10-18 16:32:35 UTC (rev 17978)
@@ -2093,7 +2093,7 @@
 	RETURNS bytea
 	AS $$
 	DECLARE
-		rast2 raster;
+		rast2 @extschema at .raster;
 		num_bands int;
 		i int;
 	BEGIN
@@ -2171,7 +2171,7 @@
 	RETURNS bytea
 	AS $$
 	DECLARE
-		rast2 raster;
+		rast2 @extschema at .raster;
 		num_bands int;
 		i int;
 		pt text;
@@ -2385,7 +2385,7 @@
 	RETURNS raster
 	AS $$
 	DECLARE
-		g geometry;
+		g @extschema at .geometry;
 		g_srid integer;
 
 		ul_x double precision;
@@ -3900,7 +3900,7 @@
 	RETURNS raster
 	AS $$
 	DECLARE
-		_rast raster;
+		_rast @extschema at .raster;
 		_nband integer;
 		_pixtype text;
 		_pixwidth double precision;
@@ -3907,7 +3907,7 @@
 		_pixheight double precision;
 		_width integer;
 		_height integer;
-		_customextent raster;
+		_customextent @extschema at .raster;
 		_extenttype text;
 	BEGIN
 		_customextent := customextent;
@@ -3957,7 +3957,7 @@
 	scale double precision DEFAULT 1.0,	interpolate_nodata boolean DEFAULT FALSE
 )
 	RETURNS raster
-	AS $$ SELECT @extschema at .ST_slope($1, $2, NULL::raster, $3, $4, $5, $6) $$
+	AS $$ SELECT @extschema at .ST_slope($1, $2, NULL::@extschema at .raster, $3, $4, $5, $6) $$
 	LANGUAGE 'sql' IMMUTABLE _PARALLEL;
 
 -----------------------------------------------------------------------
@@ -4083,12 +4083,12 @@
 	RETURNS raster
 	AS $$
 	DECLARE
-		_rast raster;
+		_rast @extschema at .raster;
 		_nband integer;
 		_pixtype text;
 		_width integer;
 		_height integer;
-		_customextent raster;
+		_customextent @extschema at .raster;
 		_extenttype text;
 	BEGIN
 		_customextent := customextent;
@@ -4135,7 +4135,7 @@
 	interpolate_nodata boolean DEFAULT FALSE
 )
 	RETURNS raster
-	AS $$ SELECT @extschema at .ST_aspect($1, $2, NULL::raster, $3, $4, $5) $$
+	AS $$ SELECT @extschema at .ST_aspect($1, $2, NULL::@extschema at .raster, $3, $4, $5) $$
 	LANGUAGE 'sql' IMMUTABLE _PARALLEL;
 
 -----------------------------------------------------------------------
@@ -4289,7 +4289,7 @@
 	RETURNS RASTER
 	AS $$
 	DECLARE
-		_rast raster;
+		_rast @extschema at .raster;
 		_nband integer;
 		_pixtype text;
 		_pixwidth double precision;
@@ -4296,7 +4296,7 @@
 		_pixheight double precision;
 		_width integer;
 		_height integer;
-		_customextent raster;
+		_customextent @extschema at .raster;
 		_extenttype text;
 	BEGIN
 		_customextent := customextent;
@@ -4349,7 +4349,7 @@
 	interpolate_nodata boolean DEFAULT FALSE
 )
 	RETURNS RASTER
-	AS $$ SELECT @extschema at .ST_hillshade($1, $2, NULL::raster, $3, $4, $5, $6, $7, $8) $$
+	AS $$ SELECT @extschema at .ST_hillshade($1, $2, NULL::@extschema at .raster, $3, $4, $5, $6, $7, $8) $$
 	LANGUAGE 'sql' IMMUTABLE _PARALLEL;
 
 -----------------------------------------------------------------------
@@ -4449,7 +4449,7 @@
 	RETURNS raster
 	AS $$
 	DECLARE
-		_rast raster;
+		_rast @extschema at .raster;
 		_nband integer;
 		_pixtype text;
 		_pixwidth double precision;
@@ -4456,7 +4456,7 @@
 		_pixheight double precision;
 		_width integer;
 		_height integer;
-		_customextent raster;
+		_customextent @extschema at .raster;
 		_extenttype text;
 	BEGIN
 		_customextent := customextent;
@@ -4501,7 +4501,7 @@
 	pixeltype text DEFAULT '32BF', interpolate_nodata boolean DEFAULT FALSE
 )
 	RETURNS RASTER
-	AS $$ SELECT @extschema at .ST_tpi($1, $2, NULL::raster, $3, $4) $$
+	AS $$ SELECT @extschema at .ST_tpi($1, $2, NULL::@extschema at .raster, $3, $4) $$
 	LANGUAGE 'sql' IMMUTABLE _PARALLEL;
 
 -----------------------------------------------------------------------
@@ -4585,7 +4585,7 @@
 	RETURNS raster
 	AS $$
 	DECLARE
-		_rast raster;
+		_rast @extschema at .raster;
 		_nband integer;
 		_pixtype text;
 		_pixwidth double precision;
@@ -4592,7 +4592,7 @@
 		_pixheight double precision;
 		_width integer;
 		_height integer;
-		_customextent raster;
+		_customextent @extschema at .raster;
 		_extenttype text;
 	BEGIN
 		_customextent := customextent;
@@ -4632,7 +4632,7 @@
 	pixeltype text DEFAULT '32BF', interpolate_nodata boolean DEFAULT FALSE
 )
 	RETURNS RASTER
-	AS $$ SELECT @extschema at .ST_roughness($1, $2, NULL::raster, $3, $4) $$
+	AS $$ SELECT @extschema at .ST_roughness($1, $2, NULL::@extschema at .raster, $3, $4) $$
 	LANGUAGE 'sql' IMMUTABLE _PARALLEL;
 
 -----------------------------------------------------------------------
@@ -4731,7 +4731,7 @@
 	RETURNS raster
 	AS $$
 	DECLARE
-		_rast raster;
+		_rast @extschema at .raster;
 		_nband integer;
 		_pixtype text;
 		_pixwidth double precision;
@@ -4738,7 +4738,7 @@
 		_pixheight double precision;
 		_width integer;
 		_height integer;
-		_customextent raster;
+		_customextent @extschema at .raster;
 		_extenttype text;
 	BEGIN
 		_customextent := customextent;
@@ -4783,7 +4783,7 @@
 	pixeltype text DEFAULT '32BF', interpolate_nodata boolean DEFAULT FALSE
 )
 	RETURNS RASTER
-	AS $$ SELECT @extschema at .ST_tri($1, $2, NULL::raster, $3, $4) $$
+	AS $$ SELECT @extschema at .ST_tri($1, $2, NULL::@extschema at .raster, $3, $4) $$
 	LANGUAGE 'sql' IMMUTABLE _PARALLEL;
 
 -----------------------------------------------------------------------
@@ -4843,7 +4843,7 @@
 
 		nrast integer;
 		idx integer;
-		rast raster;
+		rast @extschema at .raster;
 		nband integer;
 
 		stats summarystats;
@@ -5232,7 +5232,7 @@
     $$
     DECLARE
         params text[];
-        rastout raster;
+        rastout @extschema at .raster;
     BEGIN
         IF rast IS NULL THEN
             RAISE WARNING 'Cannot set georeferencing on a null raster in st_setgeoreference.';
@@ -6738,7 +6738,7 @@
 	RETURNS raster
 	AS $$
 	DECLARE
-		rtn raster;
+		rtn @extschema at .raster;
 		_returnband text;
 		newnodata1 float8;
 		newnodata2 float8;
@@ -7691,8 +7691,8 @@
 		_tileheight integer;
 		_alignment boolean;
 
-		_covextent geometry;
-		_covrast raster;
+		_covextent @extschema at .geometry;
+		_covrast @extschema at .raster;
 	BEGIN
 		fqtn := '';
 		IF length($1) > 0 THEN
@@ -8992,7 +8992,7 @@
 		sql := 'UPDATE ' || fqtn ||
 			' SET ' || quote_ident($3) ||
 			' =  @extschema at .ST_SetSRID(' || quote_ident($3) ||
-			'::raster, ' || srid || ')';
+			'::@extschema at .raster, ' || srid || ')';
 		RAISE NOTICE 'sql = %', sql;
 		EXECUTE sql;
 



More information about the postgis-tickets mailing list