listofsettings allways an array, test for length of array

E3SM-Project · Aug 5, 2016 · fa1cb49 · fa1cb49
1 parent a6a3d33
commit fa1cb49
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/scripts/Tools/xmlchange b/scripts/Tools/xmlchange
@@ -88,7 +88,7 @@ formatter_class=argparse.ArgumentDefaultsHelpFormatter
 def xmlchange(caseroot, listofsettings=None, xmlfile=None, xmlid=None, xmlval=None, subgroup=None,
               append=None, noecho=False, warn=None, force=False):
     with Case(caseroot, read_only=False) as case:
-        if listofsettings:
+        if len(listofsettings):
             logger.warning("List of attributes to change: %s" , listofsettings)
 
             # Error handling, exit with grace if xmlval or equal sign is missing. Check first before changing any attributes