eric/pmomusic
1
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases 3 Wiki Activity

♻️ refactor(useRenderers): delegate to centralized getters

Browse Source 
Replace direct accessors on snapshotState with dedicated getter functions (e.g., getRendererById, getStateByid) to eliminate duplication and improve maintainability. This change centralizes renderer/snapshot access logic, ensuring consistency across composables.
This commit is contained in:
Eric Coissac
2026-04-03 22:43:23 +02:00
parent 57995e0bdf
commit 5d081654ef
1 changed files with 7 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
pmoapp/webapp/src/composables/useRenderers.ts
View File

@@ -545,17 +545,14 @@ export function useRenderers() {
export function useRenderer(rendererId: Ref<string>) { export function useRenderer(rendererId: Ref<string>) {
ensureSSEInitialized(); ensureSSEInitialized();
const renderer = computed(() => renderersCache.value.get(rendererId.value)); // Delegates to useRenderers functions - no duplication
const snapshot = computed( const renderer = computed(() => getRendererById(rendererId.value));
() => snapshotState.snapshots.get(rendererId.value) ?? null, const snapshot = computed(() => getSnapshotById(rendererId.value));
); const state = computed(() => getStateById(rendererId.value));
const state = computed(() => snapshot.value?.state ?? null); const queue = computed(() => getQueueById(rendererId.value));
const queue = computed(() => snapshot.value?.queue ?? null); const binding = computed(() => getBindingById(rendererId.value));
const binding = computed(() => snapshot.value?.binding ?? null);
const isStream = computed(() => snapshot.value?.is_stream ?? false); const isStream = computed(() => snapshot.value?.is_stream ?? false);
const queueRefreshing = computed(() => const queueRefreshing = computed(() => isQueueRefreshing(rendererId.value));
snapshotState.queueRefreshingIds.has(rendererId.value),
);
async function refresh(force = true) { async function refresh(force = true) {
await Promise.all([ await Promise.all([