Merge pull request #8189 from DonLakeFlyer/TakePhoto

Fix MAV_CMD_IMAGE_START_CAPTURE usage

src/MissionManager/CameraSection.cc

@@ -200,12 +200,13 @@ void CameraSection::appendSectionItems(QList<MissionItem*>& items, QObject* miss
             item = new MissionItem(nextSequenceNumber++,
                                    MAV_CMD_IMAGE_START_CAPTURE,
                                    MAV_FRAME_MISSION,
-                                   0,                                   // Reserved (Set to 0)
-                                   0,                                   // Interval (none)
-                                   1,                                   // Take 1 photo
-                                   qQNaN(), qQNaN(), qQNaN(), qQNaN(),  // reserved
-                                   true,                                // autoContinue
-                                   false,                               // isCurrentItem
+                                   0,                           // Reserved (Set to 0)
+                                   0,                           // Interval (none)
+                                   1,                           // Take 1 photo
+                                   0,                           // No sequence number specified
+                                   qQNaN(), qQNaN(), qQNaN(),   // reserved
+                                   true,                        // autoContinue
+                                   false,                       // isCurrentItem
                                    missionItemParent);
             break;
         }

src/MissionManager/CameraSectionTest.cc

@@ -113,7 +113,8 @@ void CameraSectionTest::init(void)
                                                             0,                              // Reserved (Set to 0)
                                                             0,                              // Interval (none)
                                                             1,                              // Take 1 photo
-                                                            NAN, NAN, NAN, NAN,             // param 4-7 reserved
+                                                            0,                              // Sequence id not used
+                                                            NAN, NAN, NAN,                  // param 5-7 reserved
                                                             true,                           // autoContinue
                                                             false),                         // isCurrentItem
                                                 this);
@@ -975,7 +976,7 @@ void CameraSectionTest::_testScanForTakePhotoSection(void)
     Mission Param #1	Reserved (Set to 0)
     Mission Param #2	Duration between two consecutive pictures (in seconds)
     Mission Param #3	Number of images to capture total - 0 for unlimited capture
-    Mission Param #4	Reserved
+    Mission Param #4	0 Unused sequence id
     */
 
     SimpleMissionItem* newValidTakePhotoItem = new SimpleMissionItem(_offlineVehicle, false /* flyView */, this);