Adds KiCad 7.99 compatibility (#109)

`HasProperty()` no longer exists on footprints
and is replaced by `HasFieldByName()`.

Fixes #84, #97

Check kicad api version check

Co-authored-by: Wolfgang Jung <w.jung@polyas.de>

plugins/process.py

@@ -11,6 +11,7 @@ import re
 
 # Interaction with KiCad.
 import pcbnew  # type: ignore
+from .utils import footprint_has_field, footprint_get_field
 
 # Application definitions.
 from .config import *
@@ -174,7 +175,7 @@ class ProcessManager:
             #     2: 'smt'
             # }.get(footprint.GetAttributes())
 
-            skip_footprint = exclude_dnp and (footprint.HasProperty('dnp') or footprint.GetValue().upper() == 'DNP')
+            skip_footprint = exclude_dnp and (footprint_has_field(footprint, 'dnp') or footprint.GetValue().upper() == 'DNP')
 
             if not (footprint.GetAttributes() & pcbnew.FP_EXCLUDE_FROM_POS_FILES) and not skip_footprint:
                 # append unique ID if duplicate footprint designator
@@ -287,12 +288,12 @@ class ProcessManager:
         keys = ['LCSC Part #', 'JLCPCB Part #']
         fallback_keys = ['LCSC Part', 'JLC Part', 'LCSC', 'JLC', 'MPN', 'Mpn', 'mpn']
 
-        if footprint.HasProperty('dnp'):
+        if footprint_has_field(footprint, 'dnp'):
             return 'DNP'
 
         for key in keys + fallback_keys:
-            if footprint.HasProperty(key):
-                return footprint.GetProperty(key)
+            if footprint_has_field(footprint, key):
+                return footprint_get_field(footprint, key)
 
     def _get_top_or_bottom_side_override_from_footprint(self, footprint):
         keys = ['JLCPCB Layer Override']
@@ -304,8 +305,8 @@ class ProcessManager:
         }.get(footprint.GetLayer())
 
         for key in keys + fallback_keys:
-            if footprint.HasProperty(key):
-                temp_layer = footprint.GetProperty(key)
+            if footprint_has_field(footprint, key):
+                temp_layer = footprint_get_field(footprint, key)
                 if (temp_layer[0] == 'b' or temp_layer[0] == 'B'):
                     layer = "bottom"
                     break
@@ -323,8 +324,8 @@ class ProcessManager:
         offset = None
 
         for key in keys + fallback_keys:
-            if footprint.HasProperty(key):
-                offset = footprint.GetProperty(key)
+            if footprint_has_field(footprint, key):
+                offset = footprint_get_field(footprint, key)
                 break
 
         if offset is None or offset == "":
@@ -342,8 +343,8 @@ class ProcessManager:
         offset = None
 
         for key in keys + fallback_keys:
-            if footprint.HasProperty(key):
-                offset = footprint.GetProperty(key)
+            if footprint_has_field(footprint, key):
+                offset = footprint_get_field(footprint, key)
                 break
 
         if offset is None or offset == "":

plugins/utils.py

@@ -0,0 +1,28 @@
+import pcbnew  # type: ignore
+
+def get_version():
+    return float('.'.join(pcbnew.GetBuildVersion().split(".")[0:2]))  #e.g GetBuildVersion(): e.g. '7.99.0-3969-gc5ac2337e4'
+
+def is_v8():
+    version = get_version()
+    return version >= 7.99 and version < 8.99
+
+def is_v7():
+    version = get_version()
+    return version >= 6.99 and version < 7.99
+
+def is_v6():
+    version = get_version()
+    return version >= 5.99 and version < 6.99
+                 
+def footprint_has_field(footprint, field_name):
+    if is_v8():
+        return footprint.HasFieldByName(field_name)
+    else:
+        return footprint.HasProperty(field_name)
+
+def footprint_get_field(footprint, field_name):
+    if is_v8():
+        return footprint.GetFieldByName(field_name).GetText()
+    else:
+        return footprint.GetProperty(field_name)