fix(warehouse): preserve cached transfer list context on detail navigation

This commit is contained in:
2026-10-16 16:28:49 +08:00
parent d89830fe41
commit ded625f2a1
2 changed files with 51 additions and 1 deletions
@@ -44,7 +44,8 @@ const props = withDefaults(defineProps<{ service: ServiceType; createPage?: bool
const auth = useAuthStore()
const router = useRouter()
const route = useRoute()
const detailNumber = computed(() => props.detailNo || String(route.params.detailNo || ''))
// 缓存列表不订阅其他详情页的路由参数,避免离开时清空筛选并重复请求。
const detailNumber = computed(() => props.detailPage ? props.detailNo || String(route.params.detailNo || '') : '')
const canCreate = computed(() => auth.hasPermission(({
labeling: 'merchant:overseas-warehouse:relabel:create',
transfer: 'merchant:overseas-warehouse:transfer:create',
@@ -0,0 +1,49 @@
import test from 'node:test'
import assert from 'node:assert/strict'
import { readFileSync } from 'node:fs'
import { computed, effectScope, nextTick, reactive, ref, watch } from 'vue'
import ts from 'typescript'
const source = readFileSync(new URL('./WarehouseServiceView.vue', import.meta.url), 'utf8')
const detailNumber = source.match(/^const detailNumber = .*$/m)[0]
const routeWatcher = source.slice(source.indexOf('watch(() => [props.service,'), source.indexOf('watch(mode,'))
const activation = source.slice(source.indexOf('let activatedOnce = false'), source.indexOf('onBeforeUnmount(() =>'))
const code = ts.transpileModule([detailNumber, routeWatcher, activation].join('\n'), { compilerOptions: { target: ts.ScriptTarget.ES2022 } }).outputText
function harness(detailPage = false) {
const scope = effectScope(), props = reactive({ service: 'transfer', createPage: false, detailPage, detailNo: '' })
const route = reactive({ params: { detailNo: detailPage ? 'TRANSFER-A' : undefined } })
const mode = ref(detailPage ? 'detail' : 'list'), page = ref(3), filters = reactive({ number: 'TRANSFER-FILTER', warehouse: 'warehouse-1', status: 'RECEIVED' })
const calls = [], activationCallbacks = []
const resetFilters = () => { calls.push('resetFilters'); Object.assign(filters, { number: '', warehouse: '', status: '' }) }
const loadPage = () => { calls.push({ kind: 'list', page: page.value, ...filters }) }
scope.run(() => new Function('computed', 'watch', 'props', 'route', 'mode', 'page', 'resetFilters', 'resetForm', 'openCreate', 'loadDetailPage', 'loadPage', 'onActivated', code)(computed, watch, props, route, mode, page, resetFilters, () => calls.push('resetForm'), () => calls.push('create'), () => calls.push('detail'), loadPage, callback => activationCallbacks.push(callback)))
return { scope, props, route, mode, page, filters, calls, activate: () => activationCallbacks.forEach(callback => callback()) }
}
test('cached transfer list ignores detail navigation and refreshes once on return with its original filters and page', async () => {
const state = harness()
try {
state.activate()
state.route.params.detailNo = 'TRANSFER-A'
await nextTick()
assert.deepEqual(state.calls, [])
assert.equal(state.page.value, 3)
assert.equal(state.filters.number, 'TRANSFER-FILTER')
state.route.params.detailNo = 'TRANSFER-B'
await nextTick()
assert.deepEqual(state.calls, [])
state.route.params.detailNo = undefined
await nextTick()
state.activate()
assert.deepEqual(state.calls, [{ kind: 'list', page: 3, number: 'TRANSFER-FILTER', warehouse: 'warehouse-1', status: 'RECEIVED' }])
} finally { state.scope.stop() }
})
test('real detail pages still react to another detail number without loading a background list', async () => {
const state = harness(true)
try {
state.route.params.detailNo = 'TRANSFER-B'
await nextTick()
assert.deepEqual(state.calls, ['resetFilters', 'resetForm', 'detail'])
} finally { state.scope.stop() }
})