kate522
afd08885fc
AdminListPagination 的输出标记从 WP 原生 paginate_links() 换回「卡密管理」
原版的四按钮式：`« ‹ 第 X 页，共 Y 页 › »` +「每页 X 共Y」计数，没有数字页码
链接。七个页面调用点不变，外观统一为卡密管理那一套。
- 删除 $total_pages <= 1 早退：只要总数 > 0 就渲染分页条，单页时四枚按钮全
disabled —— 与原版 render_pagenavi 行为一致
- 基准 URL 与页码夹取沿用现有实现（显式拼装、夹到 [1, total_pages]），
这两处缺陷修复不动
- 标题属性由 esc_attr_e() 改为 esc_attr__()：输出相同，且同文件与 dev harness
都只用后者
- 新增私有静态 print_assets()，由 render() 调用并以 $assets_printed 保证每页
只输出一次；内容只有 .tablenav-pages a.disabled { pointer-events: none; }
及其注释。这些原生后台页面不加载主题 CSS，故随标记内联
清理 harness 中已无人调用的 number_format_i18n() 与 paginate_links() stub。
验证：tests/admin/ 76 项通过；tests/widget-system/ 458/459（仅剩与本次无关的
lazyload-contract 既有失败）；用 git show HEAD~1 取出原 render_pagenavi() 与
新助手在 8 组输入（含单页、总数为 0、per_page 为 0、页码越界）下比对，规范化
空白后输出逐字节一致；harness 渲染售后面板确认四按钮标记与内联样式就位。
107 lines
5.1 KiB
JavaScript
107 lines
5.1 KiB
JavaScript
'use strict';
|
|
|
|
/**
|
|
* 「卡密管理」分页契约。
|
|
*
|
|
* 背景（两次演进）：
|
|
* 1) 翻页链接此前写作 add_query_arg('paged', N) —— 只有两个参数时 WordPress 会落到
|
|
* $_SERVER['REQUEST_URI']，链接绑在当前请求上，随请求形态漂移，列条件也容易丢。
|
|
* 2) render_pagenavi() 的 URL 推导是一串 if/elseif。页码越界时（例如 per_page 缩小后
|
|
* URL 里还留着 paged=2，而总数只剩 1 页）三个分支一个都不命中，上一页/下一页
|
|
* 同时停在 '#'：按钮既不打 disabled 也点不动，还停在「第 1 页」状态无法自洽。
|
|
* 极端情况下 offset 落到数据之外，表格直接变空态并且连分页条都不输出，只能手改 URL 退出。
|
|
* 3) ice_perpage 只在服务端被读取、默认每页条数偏大。条数小于默认值时永远只有一页，
|
|
* 四个按钮恒为 disabled —— 看上去就是「分页是坏的」。
|
|
*
|
|
* 现状：渲染已迁移到共享助手 AdminListPagination，四按钮标记由助手统一输出
|
|
* （与这里原来那套逐行一致），基准 URL 与页码夹取也都在那里做。
|
|
* 本文件只固化卡密管理仍特有的部分：
|
|
* - ice_perpage 仍作为 URL 参数被尊重，且工具栏不新增「每页显示」控件；
|
|
* - 越界页码回退到最后一页并重查，不留空态死胡同；
|
|
* - 筛选参数白名单化拼装，排序与每页条数随链接带走。
|
|
*/
|
|
|
|
const assert = require('node:assert/strict');
|
|
const fs = require('node:fs');
|
|
const path = require('node:path');
|
|
const test = require('node:test');
|
|
|
|
const root = path.resolve(__dirname, '..', '..');
|
|
const page = fs.readFileSync(path.join(root, 'src/Admin/Native/PayChargeCards.php'), 'utf8');
|
|
|
|
const count = (src, pattern) => (src.match(pattern) || []).length;
|
|
|
|
const slice = (from, to) => {
|
|
const start = page.indexOf(from);
|
|
const end = page.indexOf(to);
|
|
if (start === -1 || end === -1 || end <= start) {
|
|
throw new Error(`切片失败：${from} → ${to}`);
|
|
}
|
|
return page.slice(start, end);
|
|
};
|
|
|
|
const list = slice('private function render_list(', 'private function render_add_form(');
|
|
const args = slice('private function list_query_args(', 'private function print_assets(');
|
|
|
|
test('渲染已迁移到共享助手 AdminListPagination', () => {
|
|
assert.match(
|
|
list,
|
|
/AdminListPagination::render\(self::PAGE_SLUG, \$count_all, \$paged, \$per_page, \$query_args\);/
|
|
);
|
|
|
|
// 自定义标记与专属常量都应清空。
|
|
assert.doesNotMatch(page, /render_pagenavi/);
|
|
assert.doesNotMatch(page, /first-page button|prev-page button|next-page button|last-page button/);
|
|
assert.doesNotMatch(page, /public const DEFAULT_PER_PAGE/);
|
|
assert.doesNotMatch(page, /self::DEFAULT_PER_PAGE/);
|
|
assert.match(page, /AdminListPagination::DEFAULT_PER_PAGE/);
|
|
|
|
// 只服务自定义禁用按钮的 CSS 规则已随标记一起搬进助手。
|
|
assert.doesNotMatch(page, /\.tablenav-pages a\.disabled \{ pointer-events: none; \}/);
|
|
});
|
|
|
|
test('ice_perpage 仍作为 URL 参数被尊重，默认每页 20', () => {
|
|
assert.match(
|
|
list,
|
|
/\(int\) \(\$_REQUEST\['ice_perpage'\] \?\? AdminListPagination::DEFAULT_PER_PAGE\)/
|
|
);
|
|
|
|
// 明确约定：不在列表工具栏增加「每页显示」下拉/按钮。
|
|
assert.doesNotMatch(page, /per_page_selector/);
|
|
assert.doesNotMatch(page, /PER_PAGE_CHOICES/);
|
|
assert.doesNotMatch(page, /aksrt-perpage-form/);
|
|
assert.doesNotMatch(page, /name="ice_perpage"/);
|
|
assert.doesNotMatch(page, /order-header-right/);
|
|
|
|
// 但 ice_perpage 仍随链接带走（无界面入口，供 URL / 程序化调用）。
|
|
assert.match(args, /\$args\['ice_perpage'\] =/);
|
|
|
|
// 工具栏保持「过滤链接 + 搜索表单」两段式。
|
|
assert.match(list, /echo '<div class="order-header"><ul class="subsubsub">' \. \$but_html \. '<\/ul>' \. \$search \. '<\/div>';/);
|
|
});
|
|
|
|
test('越界页码回退到最后一页，不再出现无分页条的死胡同', () => {
|
|
assert.match(list, /\$max_page\s*=\s*max\(1, \(int\) \(\$result\['max_pages'\] \?\? 1\)\)/);
|
|
assert.match(list, /if \(!\$items && \$count_all > 0 && \$paged > \$max_page\)/);
|
|
assert.match(list, /\$paged\s*=\s*\$max_page;/);
|
|
// 回退之后必须重查，否则表格依旧是空的。
|
|
assert.equal(count(list, /\$svc->get_list\(/g), 2);
|
|
// 分页条必须在空态分支之后渲染，否则空态 return 会把分页条一起吞掉。
|
|
assert.ok(
|
|
list.indexOf('AdminListPagination::render(') > list.indexOf('未找到对应内容'),
|
|
'分页渲染必须晚于空态处理'
|
|
);
|
|
});
|
|
|
|
test('筛选参数白名单化拼装，排序与每页条数随链接带走', () => {
|
|
for (const key of ['status', 'type', 'other', 's', 'orderby', 'desc', 'ice_perpage']) {
|
|
assert.match(args, new RegExp(`\\$args\\['${key}'\\] =`), `缺少 ${key}`);
|
|
}
|
|
// 默认值不进 URL，避免链接里塞满无意义参数。
|
|
assert.match(args, /\$orderby !== 'modified_time'/);
|
|
assert.match(args, /\$order === 'ASC'/);
|
|
assert.match(args, /\$per_page !== AdminListPagination::DEFAULT_PER_PAGE/);
|
|
// paged 由 AdminListPagination 单独追加，不能混进基准参数里（否则会被自己覆盖）。
|
|
assert.doesNotMatch(args, /\$args\['paged'\]/);
|
|
});
|