Fixed|World|Sector: Re-evaluate sector cluster linking following cont…

…ainment testing

In this situation there may still be another map hack which contains
the cluster (which, contains the nested map hack). To resolve this
we'll repeat the visual plane remapping process.

Todo for later: Re-implement this algorithm to avoid recursion. This
should wait until after "missing texture" map hack support, however.

doomsday/client/src/world/sector.cpp

@@ -49,6 +49,10 @@ static QRectF qrectFromAABox(AABoxd const &aaBox)
     return QRectF(QPointF(aaBox.minX, aaBox.maxY), QPointF(aaBox.maxX, aaBox.minY));
 }
 
+/**
+ * @todo Redesign the implementation to avoid recursion (note visPlane() calls
+ * this also).
+ */
 void Sector::Cluster::remapVisPlanes()
 {
     // By default both planes are mapped to the parent sector.
@@ -98,7 +102,9 @@ void Sector::Cluster::remapVisPlanes()
             QRectF boundingRect = qrectFromAABox(aaBox());
             if(boundingRect.contains(qrectFromAABox(exteriorCluster->aaBox())))
             {
-                // The contained cluster will link to this.
+                // The contained cluster will link to this. However we may still
+                // need to link this one to another, so re-evaluate.
+                remapVisPlanes();
                 return;
             }
             else