Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions core/frontend/src/components/video-manager/VideoManager.vue
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,11 @@ export default Vue.extend({
if (device.name === 'Fake source') {
return has_active_stream(device) || settings.is_pirate_mode
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

issue (bug_risk): The condition and return value seem to contradict the comment about not showing RadCam's secondary stream.

This returns has_active_stream(device) || settings.is_pirate_mode, matching the "Fake source" behavior and still allowing the secondary stream to be shown when active or in pirate mode. If the goal is to always hide this secondary stream, this should return false (or similar) instead of delegating to has_active_stream. Otherwise, please adjust the comment to describe the intended nuanced behavior so it doesn’t mislead future readers.

}

// Do not show RadCam's secondary stream
if (device.name === 'UnderwaterCam - IPCamera (UnderwaterCam)' && device.source.endsWith('/stream_1')) {
return has_active_stream(device) || settings.is_pirate_mode
}
return true
}

Expand Down
Loading