ardupilot: Don't early return on NAMED_VALUE_FLOAT messages

src/libs/vehicle/ardupilot/ardupilot.ts

@@ -324,28 +324,28 @@ export abstract class ArduPilotVehicle<Modes> extends Vehicle.AbstractVehicle<Mo
 
     const messageType = mavlink_message.message.type
 
-    // Special handling for NAMED_VALUE_FLOAT messages
+    // Inject variables from the MAVLink messages into the DataLake
     if (messageType === 'NAMED_VALUE_FLOAT') {
+      // Special handling for NAMED_VALUE_FLOAT messages
       const name = (mavlink_message.message.name as string[]).join('').replace(/\0/g, '')
       const path = `${messageType}/${name}`
       if (getDataLakeVariableInfo(path) === undefined) {
         createDataLakeVariable(new DataLakeVariable(path, path, 'number'))
       }
       setDataLakeVariableData(path, mavlink_message.message.value)
-      return
+    } else {
+      // For all other messages, use the flattener
+      const flattened = flattenData(mavlink_message.message)
+      flattened.forEach(({ path, value }) => {
+        if (value === null) return
+        if (typeof value !== 'string' && typeof value !== 'number') return
+        if (getDataLakeVariableInfo(path) === undefined) {
+          createDataLakeVariable(new DataLakeVariable(path, path, typeof value === 'string' ? 'string' : 'number'))
+        }
+        setDataLakeVariableData(path, value)
+      })
     }
 
-    // For all other messages, use the flattener
-    const flattened = flattenData(mavlink_message.message)
-    flattened.forEach(({ path, value }) => {
-      if (value === null) return
-      if (typeof value !== 'string' && typeof value !== 'number') return
-      if (getDataLakeVariableInfo(path) === undefined) {
-        createDataLakeVariable(new DataLakeVariable(path, path, typeof value === 'string' ? 'string' : 'number'))
-      }
-      setDataLakeVariableData(path, value)
-    })
-
     // Update our internal messages
     this._messages.set(mavlink_message.message.type, { ...mavlink_message.message, epoch: new Date().getTime() })