A better fix for that regression; don't let 'continue' complete a spanner!

Author: gregchapman-dev

music21/_version.py

@@ -50,7 +50,7 @@
 '''
 from __future__ import annotations
 
-__version__ = '9.6.0b13'
+__version__ = '9.6.0b15'
 
 def get_version_tuple(vv):
     v = vv.split('.')

music21/base.py

@@ -27,7 +27,7 @@
 <class 'music21.base.Music21Object'>
 
 >>> music21.VERSION_STR
-'9.6.0b13'
+'9.6.0b15'
 
 Alternatively, after doing a complete import, these classes are available
 under the module "base":

music21/musicxml/xmlToM21.py

@@ -4517,20 +4517,24 @@ def xmlOneSpanner(self, mxObj, target, spannerClass, *, allowDuplicateIds=False)
                 su.placement = placement
             self.spannerBundle.append(su)
 
+        if target is None:
+            return su
+
         # add a reference of this note to this spanner
-        priorLength = len(su)
-        if target is not None:
-            typeAttr = mxObj.get('type')
+        typeAttr = mxObj.get('type')
+        if typeAttr in ('start', 'stop'):
+            priorLength = len(su)
             if typeAttr == 'start':
                 su.insertFirstSpannedElement(target)
                 synchronizeIds(mxObj, su)
             elif typeAttr == 'stop':
                 su.addSpannedElements(target)
+            if priorLength == 1:
+                su.completeStatus = True
+                # only add after complete
+
         # environLocal.printDebug(['adding n', target, id(target), 'su.getSpannedElements',
         #     su.getSpannedElements(), su.getSpannedElementIds()])
-        if priorLength == 1:
-            su.completeStatus = True
-            # only add after complete
 
         return su