Use a constant for defaults

jellyfin · Oct 28, 2024 · acde068 · acde068
1 parent 37be617
commit acde068
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/src/apps/stable/features/playback/utils/mediaSegmentSettings.ts b/src/apps/stable/features/playback/utils/mediaSegmentSettings.ts
@@ -5,14 +5,16 @@ import { UserSettings } from 'scripts/settings/userSettings';
 import { MediaSegmentAction } from '../constants/mediaSegmentAction';
 
 const PREFIX = 'segmentTypeAction';
+const DEFAULT_ACTIONS: Partial<Record<MediaSegmentType, MediaSegmentAction>> = {
+    [MediaSegmentType.Intro]: MediaSegmentAction.AskToSkip,
+    [MediaSegmentType.Outro]: MediaSegmentAction.AskToSkip
+};
 
 export const getId = (type: MediaSegmentType) => `${PREFIX}__${type}`;
 
 export function getMediaSegmentAction(userSettings: UserSettings, type: MediaSegmentType): MediaSegmentAction {
     const action = userSettings.get(getId(type), false);
-    let defaultAction = MediaSegmentAction.None;
-    if (type === MediaSegmentType.Intro || type === MediaSegmentType.Outro) {
-        defaultAction = MediaSegmentAction.AskToSkip;
-    }
+    const defaultAction = DEFAULT_ACTIONS[type] || MediaSegmentAction.None;
+
     return action ? action as MediaSegmentAction : defaultAction;
 }