ValidateTopology: don't skip the Universal Face

[aperi2007]

Hi, testing the ValidateTopology I see it don't skip the UniverseFace. The bug is in this code:

	-- Now create a temporary table to construct all face geometries
	-- for checking their consistency
	EXECUTE 'CREATE TEMP TABLE face_check ON COMMIT DROP AS '
	  || 'SELECT face_id, topology.ST_GetFaceGeometry('
	  || quote_literal(toponame) || ', face_id) as geom, mbr FROM '
	  || quote_ident(toponame) || '.face';

It don't skip the UniverseFace (face_id=0) in the face table. The UniverseFace has geom=NULL so the GetFaceGeometry give an ISO sql/mm error .

I guess the code can be patched with this :

	-- Now create a temporary table to construct all face geometries
	-- for checking their consistency
	EXECUTE 'CREATE TEMP TABLE face_check ON COMMIT DROP AS '
	  || 'SELECT face_id, topology.ST_GetFaceGeometry('
	  || quote_literal(toponame) || ', face_id) as geom, mbr FROM '
	  || quote_ident(toponame) || '.face'
        || ' WHERE face_id<>0;';

A attach the patch to this ticket.

Also I change the Validate Regress Test filling the MBR geom field of the Face table that I notice was empty.

[aperi2007]

Patch for ValidateTopology function and LoadTopology Regress Test

[strk]

Fixed in r7233