fix: Deprecate AcquisitionDuration usage for BOLD volume timing

Author: effigies

src/schema/rules/checks/func.yaml

@@ -134,4 +134,22 @@ VolumeTimingMissingFrameAcquisitionDuration:
     - intersects([suffix], ["asl", "bold"])
     - type(sidecar.VolumeTiming) != "null"
   checks:
-    - '"SliceTiming" in sidecar || "FrameAcquisitionDuration" in sidecar'
+    - |
+      "SliceTiming" in sidecar ||
+      "FrameAcquisitionDuration" in sidecar ||
+      "AcquisitionDuration" in sidecar
+
+DeprecatedAcquisitionDuration:
+  issue:
+    code: DEPRECATED_ACQUISITION_DURATION
+    message: |
+      The 'AcquisitionDuration' field has been replaced with 'FrameAcquisitionDuration'
+      to indicate the duration of volume acquisition in sparse acquisition sequences.
+    level: warning
+  selectors:
+    - type(nifti_header) != "null"
+    - intersects([suffix], ["asl", "bold"])
+    - type(sidecar.VolumeTiming) != "null"
+    - type(sidecar.FrameAcquisitionDuration) == "null"
+  checks:
+    - '!("AcquisitionDuration" in sidecar)'

src/schema/rules/sidecars/func.yaml

@@ -63,6 +63,11 @@ MRIFuncTimingParameters:
         field and that do not have the `SliceTiming` field set to allow for
         accurate calculation of "acquisition time"
     DelayAfterTrigger: recommended
+    AcquisitionDuration:
+      level: deprecated
+      description_addendum: |
+        This field was previously used to indicate volume acquisition duration
+        for BOLD data. The `FrameAcquisitionDuration` field should be used instead.
 
 # fMRI task information
 MRIFuncTaskInformation: