From 808ecaa1beff09c802873d0cc8e50d8b42f1f1e5 Mon Sep 17 00:00:00 2001 From: zhanghongyuan Date: Fri, 23 Oct 2026 03:47:10 +0800 Subject: [PATCH] fix: correct off-by-one in PageOverview row count MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 1. Root cause: PageOverview::updateInfo passed row-1 to setColumnAndRow, but DetailTreeView reserves the last row for expand/collapse button, requiring N+1 rows for N items 2. Fix: change setColumnAndRow(row-1) to setColumnAndRow(row+1) to match PageSingleInfo and PageBoardInfo behavior 3. Impact: single-item maps now display correctly, multi-item maps no longer lose the last row to silent out-of-bounds Log: fix overview table showing empty or missing last item Influence: 1. Test overview page with single item in map 2. Test overview page with multiple items in map 3. Verify all items display correctly in overview table fix: 修复概览页表格行数差一错误 1. 根因:PageOverview::updateInfo 传入 row-1 给 setColumnAndRow, 但 DetailTreeView 保留最后一行用于展开/收起按钮,N 条数据需要 N+1 行 2. 方案:将 setColumnAndRow(row-1) 改为 setColumnAndRow(row+1), 与 PageSingleInfo 和 PageBoardInfo 保持一致 3. 影响:单条目 map 不再显示空表格,多条目 map 不再丢失末行数据 Log: 修复概览页表格显示为空或缺少最后一项的问题 Influence: 1. 测试概览页 map 仅含单条目时的显示 2. 测试概览页 map 含多条目时的显示 3. 验证概览页表格所有条目均正确显示 PMS: BUG-3 --- deepin-devicemanager/src/Page/PageOverview.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/deepin-devicemanager/src/Page/PageOverview.cpp b/deepin-devicemanager/src/Page/PageOverview.cpp index ece07918f..d76459c13 100644 --- a/deepin-devicemanager/src/Page/PageOverview.cpp +++ b/deepin-devicemanager/src/Page/PageOverview.cpp @@ -85,7 +85,7 @@ void PageOverview::updateInfo(const QMap &map) } // 设置表格行数 - mp_Overview->setColumnAndRow(row - 1); + mp_Overview->setColumnAndRow(row + 1); int i = 0;