Skip to content

Commit 932ac9a

Browse files
committed
Remove unnecessary JSON.stringify
1 parent 6983440 commit 932ac9a

File tree

4 files changed

+6
-11
lines changed

4 files changed

+6
-11
lines changed

photon-client/src/stores/settings/CameraSettingsStore.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,7 @@ export const useCameraSettingsStore = defineStore("cameraSettings", {
2727
}),
2828
getters: {
2929
needsCameraConfiguration(): boolean {
30-
return (
31-
JSON.stringify(useCameraSettingsStore().cameras[PlaceholderCameraSettings.uniqueName]) ===
32-
JSON.stringify(PlaceholderCameraSettings)
33-
);
30+
return useCameraSettingsStore().cameras["Placeholder Name"] === PlaceholderCameraSettings;
3431
},
3532
// TODO update types to update this value being undefined. This would be a decently large change.
3633
currentCameraSettings(): UiCameraConfiguration {

photon-client/src/types/SettingTypes.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ export interface CameraSettingsChangeRequest {
274274
quirksToChange: Record<ValidQuirks, boolean>;
275275
}
276276

277-
export const PlaceholderCameraSettings: UiCameraConfiguration = {
277+
export const PlaceholderCameraSettings: UiCameraConfiguration = Object.freeze({
278278
cameraPath: "/dev/null",
279279

280280
nickname: "Placeholder Camera",
@@ -391,7 +391,7 @@ export const PlaceholderCameraSettings: UiCameraConfiguration = {
391391
isConnected: true,
392392
hasConnected: true,
393393
mismatch: false
394-
};
394+
});
395395

396396
export enum CalibrationBoardTypes {
397397
Chessboard = 0,

photon-client/src/views/CameraMatchingView.vue

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ const unmatchedCameras = computed(() => {
9292
const activeVisionModules = computed(() =>
9393
Object.values(useCameraSettingsStore().cameras)
9494
// Ignore placeholder camera
95-
.filter((camera) => JSON.stringify(camera) !== JSON.stringify(PlaceholderCameraSettings))
95+
.filter((camera) => camera !== PlaceholderCameraSettings)
9696
// Display connected cameras first
9797
.sort(
9898
(first, second) =>

photon-client/src/views/DashboardView.vue

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,10 +59,8 @@ const cameraMismatchWarningShown = computed<boolean>(() => {
5959
return (
6060
Object.values(useCameraSettingsStore().cameras)
6161
// Ignore placeholder camera
62-
.filter((camera) => JSON.stringify(camera) !== JSON.stringify(PlaceholderCameraSettings))
63-
.some((camera) => {
64-
return camera.mismatch;
65-
})
62+
.filter((camera) => camera !== PlaceholderCameraSettings)
63+
.some((camera) => camera.mismatch)
6664
);
6765
});
6866

0 commit comments

Comments
 (0)