From 621369608f70a8b60b5cec2029fa8a0e83ea4972 Mon Sep 17 00:00:00 2001
From: Dave Foster <dave@axiomdatascience.com>
Date: Wed, 17 May 2017 12:17:12 -0400
Subject: [PATCH] Fixup branching logic in cell_measures, fixes #495

---
 compliance_checker/cf/cf.py | 58 ++++++++++++++++++-------------------
 1 file changed, 29 insertions(+), 29 deletions(-)

diff --git a/compliance_checker/cf/cf.py b/compliance_checker/cf/cf.py
index 051a52e5..a0892f72 100644
--- a/compliance_checker/cf/cf.py
+++ b/compliance_checker/cf/cf.py
@@ -2595,40 +2595,40 @@ def check_cell_measures(self, ds):
                                  "'volume: cell_var' where cell_var is the "
                                  "variable describing the cell measures".format(
                                      var.name))
-                continue
-            valid = True
-            cell_meas_var_name = search_res.groups()[0]
-            # TODO: cache previous results
-            if cell_meas_var_name not in ds.variables:
-                valid = False
-                reasoning.append(
-                    "Cell measure variable {} referred to by "
-                    "{} is not present in dataset variables".format(
-                        var.name, cell_meas_var_name)
-                )
             else:
-                cell_meas_var = ds.variables[cell_meas_var_name]
-                if not hasattr(cell_meas_var, 'units'):
-                    valid = False
-                    reasoning.append(
-                        "Cell measure variable {} is required "
-                        "to have units attribute defined.".format(
-                            cell_meas_var_name)
-                    )
-                if not set(cell_meas_var.dimensions).issubset(var.dimensions):
+                valid = True
+                cell_meas_var_name = search_res.groups()[0]
+                # TODO: cache previous results
+                if cell_meas_var_name not in ds.variables:
                     valid = False
                     reasoning.append(
-                        "Cell measure variable {} must have "
-                        "dimensions which are a subset of "
-                        "those defined in variable {}.".format(
-                            cell_meas_var_name, var.name)
+                        "Cell measure variable {} referred to by "
+                        "{} is not present in dataset variables".format(
+                            var.name, cell_meas_var_name)
                     )
+                else:
+                    cell_meas_var = ds.variables[cell_meas_var_name]
+                    if not hasattr(cell_meas_var, 'units'):
+                        valid = False
+                        reasoning.append(
+                            "Cell measure variable {} is required "
+                            "to have units attribute defined.".format(
+                                cell_meas_var_name)
+                        )
+                    if not set(cell_meas_var.dimensions).issubset(var.dimensions):
+                        valid = False
+                        reasoning.append(
+                            "Cell measure variable {} must have "
+                            "dimensions which are a subset of "
+                            "those defined in variable {}.".format(
+                                cell_meas_var_name, var.name)
+                        )
 
-        result = Result(BaseCheck.MEDIUM,
-                        valid,
-                        ('§7.2 Cell measures', var.name, 'cell_measures'),
-                        reasoning)
-        ret_val.append(result)
+            result = Result(BaseCheck.MEDIUM,
+                            valid,
+                            ('§7.2 Cell measures', var.name, 'cell_measures'),
+                            reasoning)
+            ret_val.append(result)
 
         return ret_val