Apply suggestions from code review

pidarped · samadDotDev · web-flow · commit f08a7d810894 · 2025-11-06T10:54:57.000-08:00
Co-authored-by: Abdul Samad &lt;abdul.samad@smartthings.com&gt;
diff --git a/src/app/clusters/camera-av-stream-management-server/camera-av-stream-management-server.cpp b/src/app/clusters/camera-av-stream-management-server/camera-av-stream-management-server.cpp
@@ -1015,8 +1015,8 @@ void CameraAVStreamMgmtServer::ModifyVideoStream(const uint16_t streamID, const
             if (wasModified)
             {
                 PersistAndNotify<Attributes::AllocatedVideoStreams::Id>();
+                ChipLogProgress(Camera, "Modified video stream with ID: %d", streamID);
             }
-            ChipLogProgress(Camera, "Modified video stream with ID: %d", streamID);
             return;
         }
     }
@@ -1043,8 +1043,8 @@ void CameraAVStreamMgmtServer::ModifySnapshotStream(const uint16_t streamID, con
             if (wasModified)
             {
                 PersistAndNotify<Attributes::AllocatedSnapshotStreams::Id>();
+                ChipLogProgress(Camera, "Modified snapshot stream with ID: %d", streamID);
             }
-            ChipLogProgress(Camera, "Modified snapshot stream with ID: %d", streamID);
             return;
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -1015,8 +1015,8 @@ void CameraAVStreamMgmtServer::ModifyVideoStream(const uint16_t streamID, const`
`1015`	`1015`	`if (wasModified)`
`1016`	`1016`	`{`
`1017`	`1017`	`PersistAndNotify<Attributes::AllocatedVideoStreams::Id>();`
	`1018`	`+ ChipLogProgress(Camera, "Modified video stream with ID: %d", streamID);`
`1018`	`1019`	`}`
`1019`		`- ChipLogProgress(Camera, "Modified video stream with ID: %d", streamID);`
`1020`	`1020`	`return;`
`1021`	`1021`	`}`
`1022`	`1022`	`}`
`@@ -1043,8 +1043,8 @@ void CameraAVStreamMgmtServer::ModifySnapshotStream(const uint16_t streamID, con`
`1043`	`1043`	`if (wasModified)`
`1044`	`1044`	`{`
`1045`	`1045`	`PersistAndNotify<Attributes::AllocatedSnapshotStreams::Id>();`
	`1046`	`+ ChipLogProgress(Camera, "Modified snapshot stream with ID: %d", streamID);`
`1046`	`1047`	`}`
`1047`		`- ChipLogProgress(Camera, "Modified snapshot stream with ID: %d", streamID);`
`1048`	`1048`	`return;`
`1049`	`1049`	`}`
`1050`	`1050`	`}`