comparison view/View.cpp @ 1539:bfacecf7ea7e

Overhaul choice of layer to show scale, in Pane::drawVerticalScale, making it call out to the same logic used by View::getVisibleExtentsForUnit when appropriate. Also refuse to accept scale from dormant top layer.

 bool
 View::getVisibleExtentsForUnit(QString unit,
                                double &min, double &max,
                                bool &log) const
 {
-    bool have = false;
-
-    // Iterate in reverse order, so as to return display extents of
-    // topmost layer that fits the bill
+    Layer *layer = getScaleProvidingLayerForUnit(unit);
+    if (!layer) {
+        return false;
+    }
+
+    //!!! clumsy
+
+    QString layerUnit;
+    double layerMin, layerMax;
+
+    if (layer->getValueExtents(layerMin, layerMax, log, layerUnit)) {
+        if (layer->getDisplayExtents(min, max)) {
+            return true;
+        } else {
+            min = layerMin;
+            max = layerMax;
+            return true;
+        }
+    } else {
+        return false;
+    }
+}
+        
+Layer *
+View::getScaleProvidingLayerForUnit(QString unit) const
+{
+    // Iterate in reverse order, so as to use topmost layer that fits
+    // the bill
     
     for (auto i = m_layerStack.rbegin(); i != m_layerStack.rend(); ++i) { 
 
         Layer *layer = *i;
 

         }
         if (layerUnit.toLower() != unit.toLower()) {
             continue;
         }
 
-        double displayMin = 0.0, displayMax = 0.0;
-        
-        if (layer->getDisplayExtents(displayMin, displayMax)) {
-
-            min = displayMin;
-            max = displayMax;
-            log = layerLog;
-            have = true;
-            break;
-
-        } else {
-
-            if (!have || layerMin < min) min = layerMin;
-            if (!have || layerMax > max) max = layerMax;
-            if (!have && layerLog) log = true;
-            have = true;
-        }
-    }
-
-    return have;
+        return layer;
+    }
+
+    return nullptr;
 }
 
 bool
 View::getVisibleExtentsForAnyUnit(double &min, double &max,
                                   bool &log, QString &unit) const