Write delayexecution only for a property with a value

pipeline/fiftyone_pipeline_core/src/fiftyone_pipeline_core/jsonbundler.py

@@ -175,13 +175,6 @@ def process_internal(self, flowdata):
                 null_reason = "Unknown"
                 no_value = False
 
-                # Check if property has delayed execution and set in JSON if yes
-
-
-                if propertykey in property_cache["delayExecutionList"]:
-                    output[flow_element.datakey.lower()][propertykey.lower() + "delayexecution"] = True
-
-
                 # Check if property has any delayed execution evidence properties and set in JSON if yes
 
                 if propertykey in property_cache["evidenceProperties"]:
@@ -247,6 +240,12 @@ def process_internal(self, flowdata):
                     if value is None:
                         output[flow_element.datakey.lower()][propertykey.lower() + "nullreason"] = null_reason
 
+                # A Property with delayed execution is marked only when it
+                # has a value.
+                if (value is not None and
+                        propertykey in property_cache["delayExecutionList"]):
+                    output[flow_element.datakey.lower()][propertykey.lower() + "delayexecution"] = True
+
                 if sequence < 10:
 
                     if "type" in propertymeta and propertymeta["type"].lower() == "javascript":

pipeline/fiftyone_pipeline_core/tests/test_jsonbundler_delayexecution.py

@@ -0,0 +1,69 @@
+# *********************************************************************
+# This Original Work is copyright of 51 Degrees Mobile Experts Limited.
+# Copyright 2026 51 Degrees Mobile Experts Limited, Davidson House,
+# Forbury Square, Reading, Berkshire, United Kingdom RG1 3EU.
+#
+# This Original Work is licensed under the European Union Public Licence
+# (EUPL) v.1.2 and is subject to its terms as set out below.
+#
+# If a copy of the EUPL was not distributed with this file, You can obtain
+# one at https://opensource.org/licenses/EUPL-1.2.
+#
+# The 'Compatible Licences' set out in the Appendix to the EUPL (as may be
+# amended by the European Commission) shall be deemed incompatible for
+# the purposes of the Work and the provisions of the compatibility
+# clause in Article 5 of the EUPL shall not apply.
+#
+# If using the Work as, or as part of, a network application, by
+# including the attribution notice(s) required under Article 5 of the EUPL
+# in the end user terms of the application under an appropriate heading,
+# such notice(s) shall fulfill the requirements of that article.
+# *********************************************************************
+
+"""The JSON Builder writes delayexecution only for a Property that has a
+value, as the JSON Builder specification's meta-Property table says."""
+
+import unittest
+
+from fiftyone_pipeline_core.aspectproperty_value import AspectPropertyValue
+from fiftyone_pipeline_core.elementdata_dictionary import ElementDataDictionary
+from fiftyone_pipeline_core.flowelement import FlowElement
+from fiftyone_pipeline_core.pipelinebuilder import PipelineBuilder
+
+
+def delayed(name):
+    return {"name": name, "type": "javascript", "delayexecution": True}
+
+
+class DelayedElement(FlowElement):
+
+    def __init__(self):
+        super(DelayedElement, self).__init__()
+        self.datakey = "delayed"
+        self.properties = {
+            "withvalue": delayed("WithValue"),
+            "novalue": delayed("NoValue"),
+            "plainnull": delayed("PlainNull")}
+
+    def process_internal(self, flowdata):
+        flowdata.set_element_data(ElementDataDictionary(self, {
+            "withvalue": AspectPropertyValue(value="script"),
+            "novalue": AspectPropertyValue(no_value_message="No script"),
+            "plainnull": None}))
+
+
+class JsonBuilderDelayExecutionTests(unittest.TestCase):
+
+    def test_delayexecution_is_written_only_for_a_property_with_a_value(self):
+        pipeline = PipelineBuilder({}).add(DelayedElement()).build()
+        self.addCleanup(pipeline.close)
+        flowdata = pipeline.create_flowdata()
+        flowdata.process()
+        section = flowdata.jsonbundler.json["delayed"]
+        self.assertTrue(section["withvaluedelayexecution"])
+        self.assertNotIn("novaluedelayexecution", section)
+        self.assertNotIn("plainnulldelayexecution", section)
+
+
+if __name__ == "__main__":
+    unittest.main()