Merge pull request #1288 from cuthbertLab/v8-makeRests-return

v8: makeRests(inPlace=True) returns None for scores
cuthbertLab · May 5, 2022 · c838856 · c838856
2 parents 4c5cd6c + c081f61
commit c838856
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/music21/stream/makeNotation.py b/music21/stream/makeNotation.py
@@ -830,6 +830,8 @@ def makeRests(
       - `inPlace` defaults False
       - Recurses into parts, measures, voices
       - Gave priority to `timeRangeFromBarDuration` over `refStreamOrTimeRange`
+
+    Changed in v8: scores (or other streams having parts) edited `inPlace` return `None`.
     '''
     from music21 import stream
 
@@ -847,7 +849,10 @@ def makeRests(
                 refStreamOrTimeRange=refStreamOrTimeRange,
                 timeRangeFromBarDuration=timeRangeFromBarDuration,
             )
-        return returnObj
+        if inPlace:
+            return
+        else:
+            return returnObj
 
     def oHighTargetForMeasure(
         m: Optional[stream.Measure] = None,