From 7ad60c4471e03526652d24b4fe7776158e3eb495 Mon Sep 17 00:00:00 2001
From: James Turner <zakalawe@mac.com>
Date: Mon, 27 Nov 2017 17:53:46 +0000
Subject: [PATCH] Catalogs: skip set files with missing description

This occurs for at least the A320 copilot, for example. We also log a
warning in the catalog creation script
---
 catalog/catalog.py                              |  7 ++++++-
 catalog/testData/Aircraft/f16/f16broken-set.xml | 11 +++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)
 create mode 100644 catalog/testData/Aircraft/f16/f16broken-set.xml

diff --git a/catalog/catalog.py b/catalog/catalog.py
index c78fad4..ca54010 100644
--- a/catalog/catalog.py
+++ b/catalog/catalog.py
@@ -36,7 +36,12 @@ def scan_set_file(aircraft_dir, set_file, includes):
         return None
 
     variant = {}
-    variant['name'] = sim_node.getValue("description", None)
+    name = sim_node.getValue("description", None)
+    if (name == None or len(name) == 0):
+        print "Set file " + set_file + " is missing a <description>, skipping"
+        return None
+
+    variant['name'] =  name
     variant['status'] = sim_node.getValue("status", None)
 
     if sim_node.hasChild('author'):
diff --git a/catalog/testData/Aircraft/f16/f16broken-set.xml b/catalog/testData/Aircraft/f16/f16broken-set.xml
new file mode 100644
index 0000000..9cb6ab0
--- /dev/null
+++ b/catalog/testData/Aircraft/f16/f16broken-set.xml
@@ -0,0 +1,11 @@
+<?xml version='1.0' encoding='UTF-8'?>
+<PropertyList include="f16-common.xml">
+  <sim>
+    <name>f16broken</name>
+    <!-- <description></description> -->
+    <description></description>
+    <long-description>Blah blah blah</long-description>
+    <variant-of>f16a</variant-of>
+  </sim>
+
+</PropertyList>