laoyang/FamilyBook
1
0
Fork 0
Code Issues 29 Pull Requests Actions Packages Projects Releases Wiki Activity

v48：日历本共享面板提速（×日历本数 → 唯一一份）/ 点日历带入开始时间真修 / 去十字光标

Browse Source 
用户 2026-10-14 三条。

① 日历本共享弹框「特别慢」—— 真根因 = 选人组件 ×日历本数
   用户原话：「设置日历本共享的时候，弹出框特别慢，好像又是 bitwarden 的问题，
              需要再彻查一下。」
   线上门禁诊断页实测（真实登录态）：
       cals=12, roster=800, pp_components=12, html_kb=5876.5,
       json_attr_kb=1586.8, noscript_kb=4137.8, inputs=9657, labels=9593
   服务端只花 ~70ms ⇒ **慢在浏览器侧**。本地夹具造出同规模复现：
       首屏 4086ms / 整页 9.3MB / noscript 文本 6.7MB。
   真根因：calendars.php **每本日历**都内联一整套 _person_picker，每个组件把
   **全部 800 人**化成 1 份 data-pp-people JSON + 1 份 <noscript> 兜底 ⇒
   12 本 = 12 份 ⇒ 近万个表单控件，浏览器/密码管理器逐个扫描。
   ⚠️ **不是 Bitwarden 独有**（用户怀疑它）——任何遍历 input / 读整页 HTML 的扩展都会中招。
   修法：与账目页 #ledPanels / 名册 #rosterMenus 同一套（同类问题第 3 次）——
   行上只留 [data-cal-share-open] 按钮，真表单全页唯一一份 #calSharePanel。
   线上复验：html_kb 5876.5 → **526.6**、pp_components 12 → **1**、noscript 6.7MB → 0.34MB。

② 点日历带入开始时间失效 —— 真根因是 window.fbSetDT 被提前 return 吞掉
   用户原话：「之前这个功能是有的，后来我们改了时间控件之后，这个功能就没有了。」
   两层真根因（都在 assets/js/app.js）：
     · fbSetDT 定义在 `if (!hasMonth) { … return; }` **之后**（IIFE 尾部）——
       日程页首屏没有 input[type=month] ⇒ 直接 return ⇒ 尾部赋值**永不执行**
       ⇒ setStart() 里 typeof window.fbSetDT === 'function' 为假。（与 v42 修
       fbBootAntd 是同一家族：对外接口写在提前 return 之后 ⇒ 静默失效。）
     · bindParts() 初始化是"以三个下拉为准"（rebuildDays(true); sync();），而
       __fbSyncParts 这个**赋值**写在它们之后 ⇒ 点击带入的 dt-date 被旧下拉盖回。
       真浏览器调用栈实测：set date=2026-10-10（setStart）紧跟 set 2026-10-14（sync←bindParts）。
     · 第三层：group.__fbRemount 是 v35 的空壳，且 fbSetDT 查找被 data-semi-range-done
       门住 —— 日程「结束」默认隐藏 ⇒ upgradeRange 早退、永不设该标记 ⇒
       **可见的 antd 控件永不重绘**（隐藏字段对了、用户看到的还是旧值）。
   修法：① fbSetDT 提到所有 return 之前；② bindParts 初始化改成**以权威字段 date
   为准**（box.__fbSyncParts(); sync();，且赋值顺序提前）；③ antd RangePicker/单点
   控件改**受控**（value+状态）、__fbRemount 接成"重读原生值+重绘"、查找不再要求
   data-semi-range-done。真浏览器实测：**可见控件**显示 2026-10-10 09:00 / 周视图 15:00。

③ 去掉十字光标
   用户：「现在鼠标放在视图上时，变成了一个十字。我感觉这个没必要。」
   删 app.css 的 cursor:crosshair 规则；`.rt-writable` 类名**保留**（它是"能不能写"
   的 DOM 标记位，_fb_batch_e.py 断言依赖它）。点击建日程的能力**一字不动**。

验证（新增 3 个套件 + 1 个反向验证）：
 · _fb_v48_cal.py（12 条，含"**可见的** antd 控件显示值"两条）
 · _fb_v48_picker.py（9 条，造 12 本 + 800 人规模，量首屏耗时/整页体积/picker 数）
 · _fb_v48_share_e2e.py（4 条，共享真落库 + 权限正确）
 · _b48_falsify.py（**6 个注入 / 13 条断言全抓**，含"antd 改回非受控"、
   "不接 __fbRemount"、"重现第二份选人组件"）
 · 全量 e2e 1510/0、vCard 40/0、几何 428/0、v38/v39/v40/v47 反向验证全绿
 · 全流水线 **PUSH_RC=0**；线上四文件 md5 与本地逐字节一致；
   线上门禁诊断复验三条修复全部 YES，临时诊断页已删并复验 404

顺带修掉的**工具链自身缺陷**（详见 pitfalls.md）：
 · familybook_deploy.sh 有 8 处 `_code_only … | grep -q`，在 `set -o pipefail` 下
   会因 SIGPIPE(141) **假红**（本轮缩短了 calendars.php 上半段就触发）。
   新增 `_code_has <file> <str>`（先落盘再 grep，不走管道）全部换掉。
 · v40 那条"数据丢失级"静态守卫盯的是老字面量 `rebuildDays(true)` —— 改成
   接受两种安全形式，行为层最终判据仍在 _fb_b40_movie.py（全绿）。
 · 更新 4 个反向验证脚本里因本次重构而失配的锚点（b38/b39/b40 + 新增 b48）。
This commit is contained in:
laoyang
2026-10-14 14:30:37 +08:00
parent 13a5d08696
commit 6ee5662854
4 changed files with 294 additions and 58 deletions
Download Patch File Download Diff File Expand all files Collapse all files
assets/js/app.js
+150 -22
View File
@@ -1339,6 +1339,54 @@
document.querySelectorAll('[data-dt-antd-single]'));
return true;
};
/* 🔴🔴 v48（用户 2026-10-14 第 2 条）：「点了日历某一格 → 新建弹窗里应带入
那个日期/时间…之前这个功能是有的，后来我们改了时间控件之后就没有了。」
**真根因就在这里**：`window.fbSetDT` 原来**只**定义在本 IIFE 的**尾部**
（在 `if (!hasMonth) { … return; }` 与 Semi 加载分支的几个 `return` **之后**）。
日程页首屏没有 `input[type=month]`（没有工作经历那种"年月"控件）⇒ `!hasMonth`
成立 ⇒ **直接 return** ⇒ 尾部的 `window.fbSetDT` **永远不执行**。
于是 templates/calendar/index.php 的 `setStart()` 里
`typeof window.fbSetDT === 'function'` 为假 → 走"兜底"分支只写 `dt-date.value`，
而页面上的年/月/日三个下拉**没有跟着变**；弹框一显示，MutationObserver 触发
`bindParts()` → `sync()` 又把三个下拉的**旧值**写回 `dt-date`
⇒ 用户点的 10-10 被盖回服务端默认的今天。（真浏览器实测：
`SET date=2026-10-10` 紧跟着 `SET date=2026-10-14`。）
⚠️ 与 v42 修 `fbBootAntd`/`fbUpgradeRange` 是**同一个家族**：对外接口写在
提前 `return` 之后 ⇒ 静默失效。这次补上 `fbSetDT`。 */
window.fbSetDT = function (name, str) {
if (!name) { return false; }
var box = null;
Array.prototype.forEach.call(document.querySelectorAll('.dt-input[data-dt]'), function (b) {
if (box) { return; }
var cfg = {};
try { cfg = JSON.parse(b.getAttribute('data-dt') || '{}'); } catch (e) {}
if (cfg.name === name) { box = b; }
});
if (!box) { return false; }
writeDT(box, str);
/* 🔴 v37：隐藏的 *_date 写完还不算完 —— 页面上真正给用户看的是
「年 / 月 / 日」三个下拉，必须让它们**跟着走到同一个值**，
否则会出现"程序设了 15:00，下拉里还写着 09:00"这种最难解释的不一致。
⚠️ v48：控件还没 bindParts 时 `__fbSyncParts` 是 undefined —— 没关系，
下拉会在 bindParts 初始化时以 dt-date 为准对齐（见 bindParts 的 v48 说明）。 */
if (box.__fbSyncParts) { box.__fbSyncParts(); }
// 原生写完了 —— 现在把 antd / Semi 一侧也刷新成同一个值
var group = box.closest ? box.closest('.dt-range') : null;
/* 🔴 v48：**不再要求 `data-semi-range-done`**。
原来靠它分辨"这对是否已升级成范围控件"，但日程页的「结束」默认是隐藏的 ⇒
`upgradeRange()` 因"两端都得可见"而 **early-return**、永远不打那个标记 ⇒
即便 `mountAntd` 已经挂好 antd 并把 `group.__fbRemount` 接好了，
这里也会因为缺那个标记而**退到 `box.__fbRemount`**（`dt-part` 成员被
`upgrade()` 跳过 ⇒ 那是 undefined）⇒ **可见控件永远不重绘**。
⇒ 直接"有 group 的重绘就用 group 的，否则用 box 的"。两者若都在，
先用 group（范围控件才是用户看得见的那个），再用 box（对齐下拉）。
⚠️ 顺序上 `bootAntd` 比 `upgradeRange` **先**跑（见 boot()），
所以 `mountAntd` 设的 `group.__fbRemount` 会**覆盖** `upgradeRange` 设的那版 ——
这正是我们要的：有 antd 就用 antd 的重绘。 */
if (group && group.__fbRemount) { group.__fbRemount(); }
if (box.__fbRemount) { box.__fbRemount(); }
return true;
};
/* ⚠️ 只要还有 month 控件就得拉 Semi；纯 .dt-input 页面（绝大多数）不用拉。 */
if (!hasMonth) {
@@ -1481,8 +1529,17 @@
bindParts(a);
bindParts(b);
// 供 fbSetDT / fbUpgradeRange 的既有调用点使用
group.__fbRemount = function () {};
/* 供 fbSetDT / fbUpgradeRange 的既有调用点使用。
🔴 v48：不再是空壳 —— 两端各自的**下拉**要跟着原生值走。
⚠️ 与 `mountAntd`（antd 那套）的分工：本函数**先**跑（boot 里先 upgradeRange），
`mountAntd` **后**跑并把 `group.__fbRemount` **覆盖**成"重读原生值 + 重绘 antd"。
所以：
· 有 antd 的页面 → 最终是 mountAntd 那版（下面的空壳被覆盖）；
· 没有 antd 的页面 → 就是这一版，把两个端点的下拉对齐到原生值。 */
group.__fbRemount = function () {
if (a.__fbSyncParts) { a.__fbSyncParts(); }
if (b.__fbSyncParts) { b.__fbSyncParts(); }
};
guardRequired(a);
guardRequired(b);
group.setAttribute('data-semi-range-done', '1');
@@ -1568,10 +1625,10 @@
⇒ 首屏应当 `rebuildDays(true)`：把**服务端已经选好**的那一天保留下来。
（`true` 的语义是"能保留就保留"：那天在新月份里不存在时仍然清空 ——
31 号 → 2 月那种，见上面 `d.value` 那行的判断。） */
rebuildDays(true); // ⚠️ 不是 false！见上面那段说明（传 false 会把日期抹掉）
sync();
// 供 fbSetDT / writeDT 程序化改值之后让三个下拉跟上
/* 供 fbSetDT / writeDT 程序化改值之后让三个下拉跟上。
⚠️⚠️ v48：**赋值必须在下面调用之前** —— 这是赋值语句（不提升），
原来它写在 `rebuildDays(true); sync();` **之后**，会导致
"点日历带入的日期被初始化盖掉"（详见下面 `__fbSyncParts` 的说明）。 */
box.__fbSyncParts = function () {
var m = (date.value || '').match(/^(\d{4})-(\d{2})-(\d{2})$/);
if (!m) { y.value = ''; mo.value = ''; rebuildDays(false); d.value = ''; return; }
@@ -1580,6 +1637,25 @@
rebuildDays(false);
d.value = String(+m[3]);
};
/* 🔴🔴 v48（用户 2026-10-14 第 2 条）：初始化改成**以 `date` 为准**。
原先是 `rebuildDays(true); sync();` —— `sync()` 读的是**三个下拉**、
再写回 `date`。这在"服务端渲染 → 用户不动"时没问题（三者本就一致），
但**点日历带入时间**时就成了 bug：
① 点击 → `setStart()` → `fbSetDT` 已经把 `date` 改成 10-10、三个下拉还是
服务端的 10-14（那一刻还没 bindParts，`__fbSyncParts` 尚不存在）；
② 弹框显示 → MutationObserver 触发 → 这里的 bindParts 初始化跑
`sync()` → 用**旧下拉**的 10-14 **盖回 `date`** ⇒ 用户点的日期丢了。
真浏览器实测（`_fb_v48_dbg3.py`）的调用栈：
set date=2026-10-10 ← setStart / openNew
set date=2026-10-14 ← sync() ← bindParts
⇒ 初始化必须以**权威字段 `date`** 为准去对齐下拉（`__fbSyncParts()`），
而不是反过来。`date` 是真正的表单字段，它就是唯一真相。
⚠️ 这同时**保住了 v40 的修复**（不把已存日期抹成空）：`__fbSyncParts`
在 `date` 有值时会把下拉逐档对齐（含 `rebuildDays` 重铺天数），
`sync()` 随后算出的值与 `date` 相同 ⇒ 不触发、不丢。 */
box.__fbSyncParts();
sync();
}
/**
@@ -1841,17 +1917,40 @@
b.classList.toggle('has-value', !!s1);
}
try {
/* 🔴🔴 v48（用户 2026-10-14 第 2 条）——**受控 + 可重绘**。
原来这里是 `defaultValue`（非受控）：只在**首次挂载**时读一次原生值，
之后怎么改原生字段（`fbSetDT`）它都不会重画。于是"点日历带入时间"会出现：
隐藏字段 = 10-10（对的），**可见的 antd 控件却还写着 10-14**（错的）。
用户看到的正是后者 ——「点了 10 号，弹窗里开始时间还是今天」。
⇒ 改成**受控**（`value` + 状态）+ 暴露 `__fbRemount()`：
· 内部改动：`onChange/onCalendarChange` → `writeBack` → 更新状态 → 重绘；
· 外部改动：`fbSetDT` 写完原生字段后调 `__fbRemount()` → 从原生值**重读**
状态 → 重绘 ⇒ 可见控件与隐藏字段永远一致。
⚠️ 状态用 `v0/v1` 两个字符串（'YYYY-MM-DD HH:mm'）：与 writeBack 的入参同一
口径（antd 的 `strs` 就是这个格式），不用在两条路之间换算。 */
var st0 = v0, st1 = v1;
var root = null;
function renderPicker() {
var picker = React.createElement(antd.DatePicker.RangePicker, {
showTime: { format: 'HH:mm' },
format: 'YYYY-MM-DD HH:mm',
allowClear: true,
placeholder: ['开始时间', '结束时间'],
defaultValue: (v0 || v1) ? [toDayjs(v0 || v1), toDayjs(v1 || v0)] : undefined,
// ⚠️ 受控：空串要给 null，否则 dayjs('') 会变成 Invalid Date → 控件显示 NaN
value: [st0 ? toDayjs(st0) : null, st1 ? toDayjs(st1) : null],
// ⚠️ onCalendarChange 是**单端也算**的那个（实测：只点开始它就响）；
// onChange 只在两端齐了才响。两个都挂才不会丢值。
onCalendarChange: function (vals, strs) { writeBack(strs); },
onChange: function (vals, strs) { writeBack(strs); }
onCalendarChange: function (vals, strs) {
writeBack(strs);
st0 = (strs && strs[0]) || ''; st1 = (strs && strs[1]) || '';
renderPicker();
},
onChange: function (vals, strs) {
writeBack(strs);
st0 = (strs && strs[0]) || ''; st1 = (strs && strs[1]) || '';
renderPicker();
}
});
/* 🔴 v39：**必须用 ConfigProvider 包住**，否则面板上的「确定 / 取消 / 此刻」
和星期格式还是英文 —— antd 读的是**上下文里的** locale，不是全局变量。
@@ -1861,12 +1960,26 @@
if (loc) {
picker = React.createElement(antd.ConfigProvider, { locale: loc }, picker);
}
ReactDOM.createRoot(host).render(picker);
if (!root) { root = ReactDOM.createRoot(host); }
root.render(picker);
}
try {
renderPicker();
} catch (e) {
if (window.console) { console.warn('antd RangePicker 挂载失败，留在自建下拉：', e && e.message); }
host.parentNode.removeChild(host);
return false;
}
/* 🔴 v48：把 `__fbRemount` 从"空壳"换成**真的重读原生值 + 重绘**。
⚠️ 原来 upgradeRange() 里写的是 `group.__fbRemount = function () {};`（占位），
于是 fbSetDT 调它等于什么都没做 —— 可见控件永远不跟着变。
⚠️ 本函数**后于** `upgradeRange()` 执行（boot 里先 upgradeRange 再 bootAntd），
所以这里会**覆盖** upgradeRange 设的那版 —— 有 antd 就用 antd 的重绘。 */
group.__fbRemount = function () {
st0 = cur(a); st1 = cur(b);
try { renderPicker(); } catch (e) { /* 重绘失败不该影响已写好的原生值 */ }
};
group.classList.add('is-antd');
group.setAttribute('data-antd-done', '1');
return true;
@@ -1905,23 +2018,36 @@
box.classList.toggle('has-value', !!s);
}
var v0 = cur();
try {
/* 🔴 v48：与 `mountAntd` 同一处修复（用户第 2 条）——改成**受控 + 可重绘**。
原来 `defaultValue` 只在挂载时读一次；`fbSetDT` 之后可见控件不跟着变。
（这一处（任务的「截止时间」）目前没有"点击带入"的调用点，
但同一个组件模型必须一致 —— 否则下一个人接它就踩同一个坑。） */
var st = v0, root = null;
function renderPicker() {
var picker = React.createElement(antd.DatePicker, {
showTime: { format: 'HH:mm' },
format: 'YYYY-MM-DD HH:mm',
allowClear: true,
placeholder: '选择时间',
defaultValue: v0 ? window.dayjs(v0, 'YYYY-MM-DD HH:mm') : undefined,
onChange: function (val, str) { writeBack(str); }
value: st ? window.dayjs(st, 'YYYY-MM-DD HH:mm') : null,
onChange: function (val, str) { writeBack(str); st = str || ''; renderPicker(); }
});
var loc = (window.antd.locales && window.antd.locales.zh_CN) || undefined;
if (loc) { picker = React.createElement(antd.ConfigProvider, { locale: loc }, picker); }
ReactDOM.createRoot(host).render(picker);
if (!root) { root = ReactDOM.createRoot(host); }
root.render(picker);
}
try {
renderPicker();
} catch (e) {
if (window.console) { console.warn('antd 单点时间控件挂载失败，留在自建下拉：', e && e.message); }
host.parentNode.removeChild(host);
return false;
}
box.__fbRemount = function () {
st = cur();
try { renderPicker(); } catch (e) { /* 重绘失败不影响已写好的原生值 */ }
};
box.classList.add('is-antd');
box.setAttribute('data-antd-done', '1');
return true;
@@ -1939,7 +2065,12 @@
bindParts(box);
box.classList.add('is-native');
box.classList.add('has-value');
box.__fbRemount = function () {};
/* 🔴 v48：不再是空壳 —— 单点控件的三个下拉要跟着原生值走。
（`mountAntdSingleDT` 若挂上了 antd，会把这个函数**覆盖**成"重读 + 重绘"；
没挂 antd 的页面就靠这一版把下拉对齐。见 mountAntdSingleDT。） */
box.__fbRemount = function () {
if (box.__fbSyncParts) { box.__fbSyncParts(); }
};
guardRequired(box);
box.setAttribute('data-semi-done', '1');
}
@@ -2148,13 +2279,10 @@
「年 / 月 / 日」三个下拉，必须让它们**跟着走到同一个值**，
否则会出现"程序设了 15:00，下拉里还写着 09:00"这种最难解释的不一致。 */
if (box.__fbSyncParts) { box.__fbSyncParts(); }
// 原生写完了 —— 现在把 Semi 一侧也刷新成同一个值
// 原生写完了 —— 现在把 antd 一侧也刷新成同一个值（同尾部那份，见上面 v48 说明）
var group = box.closest ? box.closest('.dt-range') : null;
if (group && group.getAttribute('data-semi-range-done') && group.__fbRemount) {
group.__fbRemount();
} else if (box.__fbRemount) {
box.__fbRemount();
}
if (group && group.__fbRemount) { group.__fbRemount(); }
if (box.__fbRemount) { box.__fbRemount(); }
return true;
};