kate522/wordpress-theme-aksrt
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(admin): expand the ticket process form inline without duplicating the row

Browse Source 
Restore the reference-theme direction: processing happens inline in the
filtered list, not on a separate page. The actual defect was the same
ticket being rendered twice (list row + form) while the bare <form><table>
inside <tbody> got foster-parented by the browser, displacing the extra
row info below the submit button. Now the processing ticket renders only
as the form, wrapped in a valid <tr><td colspan=6>, and every other row
renders normally.
This commit is contained in:
kate522
2026-09-20 15:23:08 +08:00
parent 09cff17d72
commit 0a958644b8
2 changed files with 18 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Admin/Native/UsersMenusTickets.php
+13 -23
View File
@@ -186,29 +186,7 @@ trait UsersMenusTickets {
$process_id = isset($_GET['id']) ? (string) wp_unslash($_GET['id']) : '';
if ($process_action === 'process') {
// 独立处理视图：只渲染目标工单的处理表单（参考主题同构）。
// 列表与分页不得出现在“确认提交”下方——此前表单被当作 <tr>
// 追加进列表表格，属于无效 HTML 嵌套，浏览器重排后工单列表
// 全部堆到表单下方。
$processing = null;
foreach (array_merge($tickets, $rows) as $row) {
if ($process_id === (string) $row['kind'] . '-' . (string) $row['id']) {
$processing = $row;
break;
}
}
echo '<a class="button" href="' . esc_url(remove_query_arg(['action', 'id'], $page_url)) . '" style="margin:8px 0;">' . esc_html__('查看全部', 'aksrt') . '</a>';
if ($processing === null) {
echo '<div class="table-box" style="max-width:860px;"><p>' . esc_html__('工单不存在或已被删除', 'aksrt') . '</p></div>';
return;
}
echo '<div class="table-box" style="max-width:860px;">';
$this->render_ticket_process_form($processing, $action_url);
echo '</div>';
return;
}
echo '<div class="table-box" style="overflow-y:auto;width:100%;">';
@@ -242,7 +220,19 @@ trait UsersMenusTickets {
}
foreach ($tickets as $row) {
$this->render_ticket_row($row, $page_url);
$is_processing = $process_action === 'process'
&& $process_id === (string) $row['kind'] . '-' . (string) $row['id'];
if ($is_processing) {
// 被处理工单：行信息由处理表单承载，不再重复渲染列表行——
// 此前 row + 表单把同一工单信息展示两次，且表单裸嵌 <tbody>
// 属无效 HTML，被浏览器 foster-parent 后那次多余的行信息
// 错位到“确认提交”下方。合法包裹为 colspan=6 单元格。
echo '<tr><td colspan="6">';
$this->render_ticket_process_form($row, $action_url);
echo '</td></tr>';
} else {
$this->render_ticket_row($row, $page_url);
}
}
echo '</tbody></table></div>';
tests/widget-system/report-ban-contract.test.cjs
+5 -4
View File
@@ -179,10 +179,11 @@ test('native ban page mirrors the reference theme layout', () => {
assert.match(tickets, /name="action" value="' \. esc_attr\(\$kind === 'report' \? 'aksrt_users_update_report' : 'aksrt_users_process_appeal'\)/)
assert.match(usersMenus, /admin_post_aksrt_users_update_report/)
assert.match(usersMenus, /admin_post_aksrt_users_process_appeal/)
// 独立处理视图：action=process 时只渲染目标工单的处理表单（提前 return），
// 列表循环不得再注入表单——无效 <tr> 嵌套曾把工单列表挤到“确认提交”下方。
assert.match(tickets, /if \(\$process_action === 'process'\) \{[\s\S]*?render_ticket_process_form\(\$processing, \$action_url\)[\s\S]*?return;/)
assert.doesNotMatch(tickets, /foreach \(\$tickets as \$row\) \{\s*\$this->render_ticket_row[\s\S]{0,400}?render_ticket_process_form/)
// 内联处理视图（参考主题方向）：被处理工单以 colspan=6 合法单元格展开
// 表单，且不再渲染其列表行——此前 row + 表单重复展示同一工单信息，
// 表单裸嵌 tbody 被 foster-parent 后多余行信息错位到“确认提交”下方。
assert.match(tickets, /<tr><td colspan="6">';\s*\$this->render_ticket_process_form/)
assert.match(tickets, /if \(\$is_processing\) \{[\s\S]*?render_ticket_process_form\(\$row, \$action_url\)[\s\S]*?\} else \{\s*\$this->render_ticket_row/)
// 分页对齐参考主题 zibpay_admin_pagenavi（每页 20 条）
assert.match(tickets, /TICKETS_PER_PAGE = 20/)