Return minimal QgsRectangle as boundingBox if geometry not available, since QgsRectangle(0,0,0,0) may be a valid bounding box for some geometries

nyalldawson · nyalldawson · commit f6d2d8a5765b · 2013-08-04T22:21:38.000+10:00
diff --git a/src/core/qgsgeometry.cpp b/src/core/qgsgeometry.cpp
@@ -3607,7 +3607,10 @@ QgsRectangle QgsGeometry::boundingBox()
   if ( !mGeometry )
   {
     QgsDebugMsg( "WKB geometry not available!" );
-    return QgsRectangle( 0, 0, 0, 0 );
+    // Return minimal QgsRectangle
+    QgsRectangle invalidRect;
+    invalidRect.setMinimal();
+    return invalidRect;
   }
 
   // consider endian when fetching feature type

Original file line number	Diff line number	Diff line change
`@@ -3607,7 +3607,10 @@ QgsRectangle QgsGeometry::boundingBox()`
`3607`	`3607`	`if ( !mGeometry )`
`3608`	`3608`	`{`
`3609`	`3609`	`QgsDebugMsg( "WKB geometry not available!" );`
`3610`		`- return QgsRectangle( 0, 0, 0, 0 );`
	`3610`	`+ // Return minimal QgsRectangle`
	`3611`	`+ QgsRectangle invalidRect;`
	`3612`	`+ invalidRect.setMinimal();`
	`3613`	`+ return invalidRect;`
`3611`	`3614`	`}`
`3612`	`3615`
`3613`	`3616`	`// consider endian when fetching feature type`