Changeset 1050

Timestamp:

12/07/11 18:34:31 (18 months ago)

Author:

tal

Message:

Validate kerning during reloadKerning. This partially addresses ticket #16.

File:

• packages/defcon/branches/ufo3/Lib/defcon/objects/font.py (modified) (2 diffs)

packages/defcon/branches/ufo3/Lib/defcon/objects/font.py

@@ -1087 +1087 @@
         Reload the data in the :class:`Kerning` object from the
         kerning.plist file in the UFO.
+
+        This validates the kerning against the groups loaded into the
+        font. If groups are being reloaded in the same pass, the groups
+        should always be reloaded before reloading the kerning.
         """
         if self._kerning is None:
@@ -1092 +1096 @@
         else:
             reader = UFOReader(self._path)
-            d = reader.readKerning()
+            kerning = reader.readKerning()
+            if self._groups is not None:
+                if not kerningValidator(kerning, self._groups):
+                    raise DefconError("The kerning data is not valid.")
             self._kerning.clear()
-            self._kerning.update(d)
+            self._kerning.update(kerning)
             self._stampKerningDataState(reader)