v51：旅程编辑保存后刷新 / 名册编辑页可改标签 / 手机号归一 / 反复点击带入时间+默认1h+分栏 / 日程设置拆分
用户 2026-10-15 六条。 ① 旅程详情「编辑」保存后页面不刷新（改了题图/标题，页头纹丝不动） **两个 bug 串在一起**： A. 只换 `#nodeListHost`（行程列表），而标题/题图/时间城市都在那容器**外面**（服务端直出）； B. 🔴 `form.id === 'tripForm'` **永远为假** —— HTML 表单把**同名具名控件**挂成自己的属性， `tripForm` 里有 `<input name="id">` ⇒ `form.id` 返回的是**那个 input** （`String(form.id)` = `[object HTMLInputElement]`）。同类还有 form.name/action/method。 ⇒ 修法：`form.getAttribute('id')` 判型；tripForm（编辑旅程本体）→ **整页刷新**， nodeForm（行程增删改）→ 仍只换行程列表（那是 v41 用户明确要的"不整页跳"）。 ② 名册编辑页不能选标签（"应该能编辑这个人的所有信息才对"） `templates/people/form.php` 里压根没有标签 UI ⇒ 新增 `#ptBlock`： 候选勾选（PersonTag::cloud）+ 自由输入（`name="tags"`，唯一提交源，勾选框只是快捷方式）。 ⚠️ `PersonTag::setFor(家庭, **我**, 他, …)` —— owner 永远是"我"，不影响全家的标签。 ⚠️ 守卫按**字段存在与否**分流（array_key_exists），不是"值是否为空"——否则改自己时 表单没这个字段，按"空值=清空"会把人家的标签抹掉。 ⚠️ 校验失败的两条重渲染路径也要带上 `myTags`/`tagChoices`（取 `$_POST['tags']`）。 ③ 导入的手机号带空格 → 编辑时被判"格式不对"（"统一处理一下"） 真根因：主手机号正则 `^\+?[0-9\-]{6,20}$` **空格一律判错**，而补充电话那条收 `[0-9\- ]` ⇒ 同一个号码填主字段报错、填补充字段通过。 ⇒ ① `Person::normPhone()` 去空格/短横/括号/**全角空格 U+3000**； ② 校验改成「先归一再判纯数字」——⚠️ 只把正则放宽到含 `\s` **不够**： PCRE 的 `\s`（无 /u/UCP）**不认全角空格**，通讯录导出里真有； ③ create/update/replaceContacts **入库前归一**（原来 hash 内部归一但**密文存原样**）； ④ `Schema::ensure` 加**一次性迁移**（哨兵 `phone_normalized_v51`）把已入库的 带分隔符号码解密→归一→写回（手机号是加密存的，SQL 里没法 LIKE/REPLACE）。 ④ 周/日视图"只有第一次带入的时间是对的" 真根因（真机给 `__fbRemount` 装 getter/setter 才抓到）：起止字段藏在弹框里 ⇒ boot 时 `upgradeRange()` 因"两端不可见"提前 return ⇒ 那一刻**只有 mountAntd** 给它赋值 （会"重读原生值+重绘 antd"）；而**一打开弹框** `fbUpgradeRange()` 补跑 ⇒ 在尾部把它**重新赋成"只对齐原生下拉、不碰 antd"**的一版 ⇒ 此后 antd 控件永远停在第一次的值。 ⇒ `upgradeRange()` 不再降级覆盖更强的实现，改成"把对齐下拉串在它前面"（`__fbRangeSync` 防自套）。 ⑤ 默认结束 = 开始 + 1 小时；新增日程要按分栏错开（不再叠盖） · `setStart()` 里顺带设 ends_at = +1h（⚠️ 跨午夜顺延到次日；⚠️ 不捆进 fbSetDT， 那个接口被"编辑已有日程"复用，强改会弄丢用户填的时长）。 · 真根因：`putTimed()` 只把自己那条**整宽**铺上去，不知道同列已有日程 （服务端整页渲染有分栏，但那是整页重绘时才算的）⇒ 前端 `relayoutColumn(col)` **照抄服务端 `conflictLanes`** 重排整列；模板加 `data-s`/`data-e`（**声明的起止分钟**， 不能从 top/height 反推 —— 那是"实际显示结束"，不是真实结束分钟）。 ⑥ 日程设置下拉太长（日历本一多就把下面的设置挤出屏幕）→ 拆成"四行列表 + 显示设置独立页" 新建 `templates/calendar/display.php` + 路由 `calendar/display`（默认视图 + 显示哪些本子）； `_settings_menu.php` 从"内联表单"改成**纯列表四行**（显示设置/日历本管理/订阅管理/DAV 口令）； ⚠️ `.cal-set-books` 原本 `max-height:42vh`（"塞在下拉里"的妥协）⇒ 用 `.cal-set-page` 作用域覆盖，否则整页里再套一层内滚 = 要滚两次。 保存仍走既有 `calendar/prefs`（一个字段没改）。 验证：`_fb_v51_ui.py`（25 条真机）+ `_b51_falsify.py`（**12 注入 / 25 断言全抓**）； e2e 1511/0；`_fb_batch_e.py` 与 `familybook_deploy.sh` 里"设置菜单含 show_present"等 旧断言随 #6 搬到 display 页（意图一字不改）；全流水线 PUSH_RC=0、部署 0 FAIL（含"第51批形态"）； 线上 10 文件 md5 逐字节一致。 ⚠️ 排查纪律（已写进 pitfalls.md）： · `form.id` 被同名控件遮蔽 —— 判型一律用 `getAttribute('id')`； · 反向验证的注入可能"打偏"（改的那处不是探针实际走的分支）—— 本轮 12 个注入有 2 个一开始没变红，都是注入打偏、不是断言弱。
This commit is contained in:
+59
-3
@@ -160,7 +160,14 @@ final class Person
|
||||
*/
|
||||
public static function create(int $familyId, array $d): int
|
||||
{
|
||||
$phone = (string)($d['phone'] ?? '');
|
||||
/* 🔴 v51（用户第 5 条）：主手机号**入库前归一**成纯数字。
|
||||
`Crypto::hash()` 内部本来就归一（所以"138 0000 0000"与"13800000000"
|
||||
一直是**同一个人**，登录不受影响），但**密文里存的是原样**
|
||||
⇒ 名册列表/详情页显示出来就是"138 0000 0000"这种带空格的形态。
|
||||
用户这次要求"统一处理一下" ⇒ 这里也归一，两处产物（hash / 密文）口径一致。
|
||||
⚠️ 只用 `normalizePhone` 的"去分隔符"这一半，**不**去掉国别前缀 86
|
||||
（那是登录查找用的宽松口径；显示与存储要保留用户填的完整号码）。 */
|
||||
$phone = self::normPhone($d['phone'] ?? '');
|
||||
$bd = Birthday::normalize($d);
|
||||
$sql = 'INSERT INTO persons
|
||||
(family_id, kind, name, gender, birth_rank, birthday,
|
||||
@@ -201,7 +208,8 @@ final class Person
|
||||
|
||||
public static function update(int $id, array $d): void
|
||||
{
|
||||
$phone = (string)($d['phone'] ?? '');
|
||||
// v51：与 create 同一口径（主手机号入库前归一成纯数字，见上面的说明）
|
||||
$phone = self::normPhone($d['phone'] ?? '');
|
||||
$bd = Birthday::normalize($d);
|
||||
$sql = 'UPDATE persons SET
|
||||
name = ?, gender = ?, birth_rank = ?, birthday = ?,
|
||||
@@ -251,6 +259,33 @@ final class Person
|
||||
return $v === '' ? null : mb_substr($v, 0, 60, 'UTF-8');
|
||||
}
|
||||
|
||||
/**
|
||||
* 主手机号入库前的归一（v51 用户第 5 条）。
|
||||
*
|
||||
* 用户原话：「名册的导进来的手机号，有些它是一共 11 个数字，前 3 个数字和后 4 个数字，
|
||||
* 中间有两个空格。这个看看能不能把它给处理一下，**统一处理一下**。……
|
||||
* 都提示我数据不对，当我编辑它的时候，提示我数据的格式不对。」
|
||||
*
|
||||
* 做什么：去掉号码里的**空格 / 短横 / 圆括号**（号码里最常见的人类排版），
|
||||
* 并把全角空格一并处理掉 —— 通讯录导出里真的有全角空格。
|
||||
*
|
||||
* ⚠️ 与 `Crypto::normalizePhone()` 的差别（别把两者合并）：
|
||||
* · `normalizePhone` 是**登录查找**口径，还会把前缀 `+86`/`86` 去掉
|
||||
* （"13800000000" 与 "+8613800000000" 必须是同一个人）；
|
||||
* · 本方法是**存储与显示**口径，**不去国别前缀** ——
|
||||
* 用户填的是什么号就存什么号（只是把分隔符抹平），显示出来才是他认得的号码。
|
||||
* 两者都从同一套分隔符集合出发，所以 hash 与密文永远指向同一个号码。
|
||||
* ⚠️ 保留 `+`：`+8613800000000` 这种要原样留着（去掉会变成另一个语义的号码）。
|
||||
*/
|
||||
private static function normPhone($raw): string
|
||||
{
|
||||
$s = (string)$raw;
|
||||
// 全角空格 → 半角（通讯录导出的常见形态），再去掉所有分隔符
|
||||
$s = str_replace(["\xE3\x80\x80", "\xC2\xA0"], ' ', $s); // U+3000 / U+00A0
|
||||
$s = (string)preg_replace('/[\s\-()]/u', '', trim($s));
|
||||
return $s;
|
||||
}
|
||||
|
||||
/**
|
||||
* 自助改自己的资料。
|
||||
*
|
||||
@@ -338,6 +373,13 @@ final class Person
|
||||
}
|
||||
$label = mb_substr(trim((string)($row['label'] ?? '')), 0, 20, 'UTF-8');
|
||||
if ($kind === 'phone') {
|
||||
/* 🔴 v51（用户第 5 条）：补充电话与主手机号**同一口径** —— 入库前去掉
|
||||
分隔符（空格 / 短横 / 括号 / 全角空格），避免同一个人在"主号 / 补充号"
|
||||
两处存出两种排版（名册列表显示时一个带空格一个不带，看着像两种数据）。 */
|
||||
$value = self::normPhone($value);
|
||||
if ($value === '') {
|
||||
continue;
|
||||
}
|
||||
$value = Crypto::encrypt(mb_substr($value, 0, 60, 'UTF-8'));
|
||||
} else {
|
||||
$value = mb_substr($value, 0, 500, 'UTF-8');
|
||||
@@ -444,7 +486,21 @@ final class Person
|
||||
$err['name'] = '姓名不能为空';
|
||||
}
|
||||
$phone = trim((string)($d['phone'] ?? ''));
|
||||
if ($phone !== '' && !preg_match('/^\+?[0-9\-]{6,20}$/', $phone)) {
|
||||
/* 🔴🔴 v51（用户 2026-10-15 第 5 条）：「名册的导进来的手机号，有些它是一共 11 个数字，
|
||||
前 3 个数字和后 4 个数字，中间有两个空格……看起来还有不少这样的数据呢，
|
||||
都提示我数据不对，当我编辑它的时候，提示我数据的格式不对。」
|
||||
⇒ 根因就是这个正则：原来只收 `[0-9-]`，**空格一律判错** ⇒
|
||||
通讯录导入进来的 "138 0000 0000" 这种（vCard/CSV 里很常见的人类排版）
|
||||
一进编辑页就红着，保存不了。
|
||||
⇒ 做法：**先按存储口径归一、再校验归一后的串**。
|
||||
这样校验的与真正入库的是**同一个值**，两边不会互相打脸
|
||||
（分隔符、全角空格都已被 `normPhone()` 抹平）。
|
||||
⚠️ 曾经试过"把正则的字符集放宽到含 `\s`"——**不够**：
|
||||
PCRE 的 `\s`（不加 /u / UCP）只认 ASCII 空白，**不认全角空格 U+3000**，
|
||||
而通讯录导出里真的会出现全角空格 ⇒ 那种号码照样被判错。
|
||||
"先归一再校验"从根上避免了这类"字符集列举不全"的问题。 */
|
||||
$phone = self::normPhone($phone);
|
||||
if ($phone !== '' && !preg_match('/^\+?[0-9]{5,25}$/', $phone)) {
|
||||
$err['phone'] = '手机号格式看起来不对';
|
||||
}
|
||||
$email = trim((string)($d['email'] ?? ''));
|
||||
|
||||
@@ -2183,6 +2183,58 @@ SQL);
|
||||
}
|
||||
});
|
||||
|
||||
/* 🔴 v51（用户 2026-10-15 第 5 条）：把**已入库**的带分隔符手机号归一成纯数字。
|
||||
用户原话：「名册的导进来的手机号，有些它是一共 11 个数字，前 3 个数字和后 4 个数字，
|
||||
中间有两个空格……看起来还有不少这样的数据呢，都提示我数据不对。」
|
||||
⇒ 光放宽校验（`Person::validate`）只能让**以后**编辑不再报错；
|
||||
库里**已经存进去**的那些仍带着空格，名单上显示出来还是"138 0000 0000"。
|
||||
用户明确要"统一处理一下" ⇒ 这里做一次性数据归一。
|
||||
|
||||
⚠️ 手机号是**加密存**的（`phone_encrypted`），SQL 里没法直接 LIKE/REPLACE ——
|
||||
只能把整表读出来、解密、归一、再写回（见下面）。
|
||||
⚠️ 用**哨兵键**保证"只做一次"（与 `person_tags_migrated_v28` 同一套路）：
|
||||
这是**一次性数据转换**，不是持续同步；否则用户哪天存回一个带空格的号，
|
||||
下次 ensure 又给他抹掉，会出现"我改的号码自己变了"。
|
||||
超长/超大库也不怕：条件只挑"解密后含分隔符"的行，
|
||||
绝大多数行一条 SQL 的代价都没有（先全量解密是可接受的量级：
|
||||
名册千人级，一次 ensure 跑完就再也不跑）。
|
||||
⚠️ 幂等：写完 `phone_hash` 与原来**同一个值**（`Crypto::hash` 本来就归一），
|
||||
所以唯一约束不会因为这次改写而冲突。 */
|
||||
self::tryStep('手机号归一（去掉导入数据里的空格/短横）', static function () use ($db) {
|
||||
$flag = 'phone_normalized_v51';
|
||||
$has = $db->query("SELECT COUNT(*) FROM `settings` WHERE `k` = '" . $flag . "'")->fetchColumn();
|
||||
if ((int)$has > 0) {
|
||||
return; // 已经归过一次，永不再做（见上面的说明）
|
||||
}
|
||||
if (!self::hasColumn($db, 'persons', 'phone_encrypted')) {
|
||||
return;
|
||||
}
|
||||
$rows = $db->query(
|
||||
"SELECT id, phone_encrypted FROM persons
|
||||
WHERE phone_encrypted IS NOT NULL AND phone_encrypted <> ''"
|
||||
)->fetchAll();
|
||||
$upd = $db->prepare('UPDATE persons SET phone_encrypted = ? WHERE id = ?');
|
||||
foreach ($rows as $r) {
|
||||
$plain = \App\Crypto::decrypt((string)$r['phone_encrypted']);
|
||||
if ($plain === '') {
|
||||
continue;
|
||||
}
|
||||
// 只有"含有分隔符"的才需要改写（其余原样，避免无谓的写库）
|
||||
$norm = (string)preg_replace('/[\s\-()]+/u', '', str_replace(
|
||||
["\xE3\x80\x80", "\xC2\xA0"], ' ', $plain));
|
||||
if ($norm === $plain || $norm === '') {
|
||||
continue;
|
||||
}
|
||||
$upd->execute([\App\Crypto::encrypt($norm), (int)$r['id']]);
|
||||
}
|
||||
// 记哨兵（settings 是家庭级的，用最小 family_id 归属，与其它迁移一致）
|
||||
$fid = (int)$db->query('SELECT MIN(id) FROM families')->fetchColumn();
|
||||
if ($fid > 0) {
|
||||
$db->prepare("INSERT IGNORE INTO `settings` (family_id, `k`, `v`) VALUES (?,?,?)")
|
||||
->execute([$fid, $flag, date('Y-m-d H:i:s')]);
|
||||
}
|
||||
});
|
||||
|
||||
// 被容错跳过的数据步骤记进设置，系统设置页会把它显出来 ——
|
||||
// 不然「升级没跑完」这件事只活在日志里，跟没发生一样。
|
||||
if (self::$stepErrors) {
|
||||
|
||||
Reference in New Issue
Block a user