fixed previous commit so that SingleXMLFileBacked plugins use xinclude, and GroupPatterns uses SingleXMLFileBacked correctly

1 files changed, 27 insertions, 18 deletions

diff --git a/src/lib/Server/Plugin.py b/src/lib/Server/Plugin.py
index 0038632e4..414713793 100644
--- a/src/lib/Server/Plugin.py
+++ b/src/lib/Server/Plugin.py
@@ -551,12 +551,34 @@ class XMLFileBacked(FileBacked):
     def __init__(self, filename):
         self.label = "dummy"
         self.entries = []
-        self.extras = []
         FileBacked.__init__(self, filename)
 
     def Index(self):
         """Build local data structures."""
         try:
+            self.xdata = XML(self.data)
+        except XMLSyntaxError:
+            logger.error("Failed to parse %s" % (self.name))
+            return
+        self.entries = self.xdata.getchildren()
+        if self.__identifier__ is not None:
+            self.label = self.xdata.attrib[self.__identifier__]
+
+    def __iter__(self):
+        return iter(self.entries)
+
+
+class SingleXMLFileBacked(XMLFileBacked):
+    """This object is a coherent cache for an independent XML file."""
+    def __init__(self, filename, fam):
+        XMLFileBacked.__init__(self, filename)
+        self.extras = []
+        self.fam = fam
+        self.fam.AddMonitor(filename, self)
+
+    def Index(self):
+        """Build local data structures."""
+        try:
             self.xdata = lxml.etree.XML(self.data, base_url=self.name)
         except lxml.etree.XMLSyntaxError:
             err = sys.exc_info()[1]
@@ -568,7 +590,10 @@ class XMLFileBacked(FileBacked):
         if included:
             for name in included:
                 if name not in self.extras:
-                    self.add_monitor(name)
+                    self.fam.AddMonitor(os.path.join(os.path.dirname(self.name),
+                                                     name),
+                                        self)
+                    self.extras.append(name)
             try:
                 self.xdata.getroottree().xinclude()
             except lxml.etree.XIncludeError:
@@ -579,22 +604,6 @@ class XMLFileBacked(FileBacked):
         if self.__identifier__ is not None:
             self.label = self.xdata.attrib[self.__identifier__]
 
-    def add_monitor(self, fname):
-        """Add a fam monitor for an included file"""
-        self.fam.AddMonitor(os.path.join(os.path.dirname(self.name), fname),
-                            self)
-        self.extras.append(fname)
-
-    def __iter__(self):
-        return iter(self.entries)
-
-
-class SingleXMLFileBacked(XMLFileBacked):
-    """This object is a coherent cache for an independent XML file."""
-    def __init__(self, filename, fam):
-        XMLFileBacked.__init__(self, filename)
-        fam.AddMonitor(filename, self)
-
 
 class StructFile(XMLFileBacked):
     """This file contains a set of structure file formatting logic."""