admin/momentry_studio
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Compare commits

base: admin:f727b21e99c8adcf82ec5433458b4939d594d0fe
Branches Tags
admin:main
...
compare: admin:5db1b68fa060d8bb365cf9b72d016eda4add1248
Branches Tags
admin:main

2 Commits
f727b21e99 ... 5db1b68fa0

Author SHA1 Message Date
Momentry Studio
5db1b68fa0 debug: add allGroupsForMove computed for Move Faces Modal 
- Explicitly define data source for Move Faces Modal
- Use clusterAsPending directly
- Add console.log for debugging

Purpose: Diagnose why only one group shows in Move Faces Modal
 2026-07-25 15:40:48 +08:00
Momentry Studio
ec8d8d9d1a fix: add fallback for missing group names in Move Faces Modal 
Problem:
- Some groups may not have name field
- Display would show blank or undefined

Solution:
- Add fallback: c.name || 'Unnamed Group'
- Add fallback: c.trace_count || 0
- Ensures always shows meaningful text

Impact:
- Move Faces Modal now always shows group names
- No blank entries even with incomplete data
 2026-07-25 15:40:01 +08:00
1 changed files with 3 additions and 3 deletions
Expand all files Collapse all files

6
src/views/PeopleView.vue
View File

@@ -313,7 +313,7 @@
<hr class="ms-merge-divider">
<div class="ms-merge-grid">
<div v-for="c in clusterAsPending" :key="c.identity_uuid" class="ms-merge-face-card" @click="moveToGroup(c)">
<div v-for="c in allGroupsForMove" :key="c.identity_uuid" class="ms-merge-face-card" @click="moveToGroup(c)">
<div class="ms-merge-face-img">
<img v-if="c.rep_trace && faceThumbsCache[thumbTraceKey(c.rep_trace)]" :src="faceThumbsCache[thumbTraceKey(c.rep_trace)]" alt="">
<svg v-else viewBox="0 0 120 120" fill="none" style="width:100%;height:100%;">
@@ -321,8 +321,8 @@
<ellipse cx="60" cy="105" rx="40" ry="25" fill="var(--border-color)"/>
</svg>
</div>
<span class="ms-merge-face-name">{{ c.name }}</span>
<span class="ms-merge-face-count">{{ c.trace_count }} faces</span>
<span class="ms-merge-face-name">{{ c.name || 'Unnamed Group' }}</span>
<span class="ms-merge-face-count">{{ c.trace_count || 0 }} faces</span>
</div>
</div>
</div>