From 4fbac180e16c8e6b22df453746d62ae946c4f461 Mon Sep 17 00:00:00 2001 From: laoyang Date: Thu, 15 Oct 2026 10:37:07 +0800 Subject: [PATCH] =?UTF-8?q?v51=EF=BC=9A=E6=97=85=E7=A8=8B=E7=BC=96?= =?UTF-8?q?=E8=BE=91=E4=BF=9D=E5=AD=98=E5=90=8E=E5=88=B7=E6=96=B0=20/=20?= =?UTF-8?q?=E5=90=8D=E5=86=8C=E7=BC=96=E8=BE=91=E9=A1=B5=E5=8F=AF=E6=94=B9?= =?UTF-8?q?=E6=A0=87=E7=AD=BE=20/=20=E6=89=8B=E6=9C=BA=E5=8F=B7=E5=BD=92?= =?UTF-8?q?=E4=B8=80=20/=20=E5=8F=8D=E5=A4=8D=E7=82=B9=E5=87=BB=E5=B8=A6?= =?UTF-8?q?=E5=85=A5=E6=97=B6=E9=97=B4+=E9=BB=98=E8=AE=A41h+=E5=88=86?= =?UTF-8?q?=E6=A0=8F=20/=20=E6=97=A5=E7=A8=8B=E8=AE=BE=E7=BD=AE=E6=8B=86?= =?UTF-8?q?=E5=88=86?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 用户 2026-10-15 六条。 ① 旅程详情「编辑」保存后页面不刷新（改了题图/标题，页头纹丝不动） **两个 bug 串在一起**： A. 只换 `#nodeListHost`（行程列表），而标题/题图/时间城市都在那容器**外面**（服务端直出）； B. 🔴 `form.id === 'tripForm'` **永远为假** —— HTML 表单把**同名具名控件**挂成自己的属性， `tripForm` 里有 `` ⇒ `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 个一开始没变红，都是注入打偏、不是断言弱。 --- app/Person.php | 62 ++++++++++- app/Schema.php | 52 +++++++++ assets/css/app.css | 9 ++ assets/js/app.js | 42 ++++++- index.php | 69 +++++++++++- templates/calendar/_settings_menu.php | 121 ++++++-------------- templates/calendar/calendars.php | 4 + templates/calendar/display.php | 99 ++++++++++++++++ templates/calendar/index.php | 155 ++++++++++++++++++++++++-- templates/people/form.php | 71 ++++++++++++ templates/trips/show.php | 30 ++++- 11 files changed, 610 insertions(+), 104 deletions(-) create mode 100644 templates/calendar/display.php diff --git a/app/Person.php b/app/Person.php index 38457ce..e25057e 100644 --- a/app/Person.php +++ b/app/Person.php @@ -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'] ?? '')); diff --git a/app/Schema.php b/app/Schema.php index d855115..ec063ae 100644 --- a/app/Schema.php +++ b/app/Schema.php @@ -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) { diff --git a/assets/css/app.css b/assets/css/app.css index 7765e7e..5bf9980 100644 --- a/assets/css/app.css +++ b/assets/css/app.css @@ -179,6 +179,15 @@ a:hover { text-decoration: underline; } .cal-set-view input { accent-color: var(--brand); } /* 本子清单：最多占 42vh，超了内部滚动 —— 菜单本身不滚，否则保存按钮会被推出可视区 */ .cal-set-books { max-height: 42vh; overflow-y: auto; } +/* 🔴 v51（用户 2026-10-15 第 6 条）：显示设置**独立成页之后**， + `.cal-set-books` 那条 `max-height:42vh; overflow-y:auto` 就不该再套用了 —— + 它是"塞在下拉里"时的妥协（下拉不能太高）。整页里再套一个 42vh 的滚动区， + 等于"页面里再开一个小窗"，日历本多的时候要**滚动两次**才能看完。 + ⚠️ 用 `.cal-set-page` 作用域覆盖，**不动**上面那条基础规则 —— + 它现在没有别的使用者，但留着万一将来又要在别处（弹框里）用同一套列表。 */ +.cal-set-page .cal-set-books { max-height: none; overflow-y: visible; } +/* 页面上把每一本排成一行（与下拉里同一套行样式，只是不滚动） */ +.cal-set-page .cal-set-book { font-size: 14px; } .cal-set-book { display: flex; align-items: center; gap: 7px; padding: 5px 4px; font-size: 13px; cursor: pointer; border-radius: 6px; white-space: normal; diff --git a/assets/js/app.js b/assets/js/app.js index e61ac76..a519392 100644 --- a/assets/js/app.js +++ b/assets/js/app.js @@ -1585,11 +1585,49 @@ `mountAntd` **后**跑并把 `group.__fbRemount` **覆盖**成"重读原生值 + 重绘 antd"。 所以： · 有 antd 的页面 → 最终是 mountAntd 那版（下面的空壳被覆盖）； - · 没有 antd 的页面 → 就是这一版，把两个端点的下拉对齐到原生值。 */ - group.__fbRemount = function () { + · 没有 antd 的页面 → 就是这一版，把两个端点的下拉对齐到原生值。 + + 🔴🔴 v51（用户 2026-10-15）——**这里原来是"降级覆盖"，害了日程页的带入时间**。 + 用户原话：「在周视图和日视图的某一个地方点击，它会把当前的时间带进来， + 但是**只有第一次**带进来的时间是对的……当我关闭了它，再次点击的时候， + 无论我点什么地方，它带进来的还是 8 点钟。」 + + **真根因（真机插桩实测，不是猜的）**： + 日程页首屏那对起止字段是**藏在弹框里**的（`#calAddCard` 默认 `hidden`） + ⇒ boot 时 `upgradeRange()` 因为"两端不可见"**直接 return**（见上面的守卫） + ⇒ 那一刻 **`mountAntd` 是唯一给 `group.__fbRemount` 赋值的人** + （它那一版会"重读原生值 + `renderPicker()` 重绘 antd"）。 + 而**一打开弹框**（元素第一次可见），页内 `openNew()` 会调 + `window.fbUpgradeRange()` 补升级 ⇒ `upgradeRange()` 这次跑到底， + 在**它的尾部**把 `group.__fbRemount` **重新赋成了下面这一版** —— + 只会对齐两个原生下拉、**根本不碰 antd**。 + ⇒ 此后每次 `fbSetDT`（点日历带入时间）都走到这一版： + 隐藏字段与原生下拉都更新了，**可见的 antd 控件永远停在第一次的值**。 + 这正是"第一次对、后面都不动"。 + + ⇒ 修法：**别覆盖已经更强的重绘实现**。 + · 已存在 `__fbRemount`（= mountAntd 那版）时：把"对齐原生下拉"**串在它前面**， + 而不是把它顶掉； + · 不存在时：才用下面这版（纯原生页面的兜底）。 + ⚠️ 顺序：先对齐下拉再重绘 antd —— mountAntd 那版会从原生值重读， + 先让原生侧一致，重绘出来的才对。 + ⚠️ 幂等：`upgradeRange` 若因任何原因再跑一次，也不会把自己套进自己 + （用 `__fbRangeSync` 标记，套过的就不重复套）。 */ + var syncPartsOnce = function () { if (a.__fbSyncParts) { a.__fbSyncParts(); } if (b.__fbSyncParts) { b.__fbSyncParts(); } }; + if (typeof group.__fbRemount === 'function' && !group.__fbRangeSync) { + /* 已有更强的实现（mountAntd）→ 只把"对齐下拉"挂到它前面 */ + var stronger = group.__fbRemount; + group.__fbRangeSync = true; + group.__fbRemount = function () { + syncPartsOnce(); + return stronger(); + }; + } else if (!group.__fbRemount) { + group.__fbRemount = syncPartsOnce; + } guardRequired(a); guardRequired(b); group.setAttribute('data-semi-range-done', '1'); diff --git a/index.php b/index.php index 2ce82ac..a96b01c 100644 --- a/index.php +++ b/index.php @@ -248,8 +248,17 @@ function validate_contacts(array $contacts): string if ($kind === 'email' && !filter_var($v, FILTER_VALIDATE_EMAIL)) { return '第 ' . $no . ' 个补充邮箱的格式看起来不对：' . $v; } - if ($kind === 'phone' && !preg_match('/^\+?[0-9\- ]{6,25}$/', $v)) { - return '第 ' . $no . ' 个补充电话的格式看起来不对：' . $v; + if ($kind === 'phone') { + /* 🔴 v51（用户第 5 条）：与主手机号**同一套口径** —— 先去分隔符再校验。 + 原来这里收 `[0-9\- ]`（含半角空格）， + 但**全角空格 U+3000** 同样会被判错（PCRE 的 `\s` 不加 /u 不认它）。 + 两处口径不同 = 同一个号码填主字段报错、填补充字段却通过 ⇒ 用户懵。 + ⇒ 统一成"先归一（抹掉空格/短横/括号/全角空格）、再判纯数字"。 */ + $n = (string)preg_replace('/[\s\-()]+/u', '', str_replace( + ["\xE3\x80\x80", "\xC2\xA0"], ' ', $v)); + if (!preg_match('/^\+?[0-9]{5,25}$/', $n)) { + return '第 ' . $no . ' 个补充电话的格式看起来不对：' . $v; + } } } } @@ -1981,6 +1990,18 @@ switch ($r) { 'jobRows' => PersonJob::list($id), // 我对他的私备注（第 5 条）：**只有我自己**那一份 'myNote' => $isSelf ? '' : PersonNote::get($id, (int)Auth::personId()), + /* 🔴 v51（用户 2026-10-15 第 4 条）：「名册这个功能里面，当我编辑资料的时候， + 在这个页面竟然**没有办法选择它的标签**。按道理讲，名册编辑成员这个地方 + 应该能编辑这一个人的**所有信息**才对。」 + ⇒ 这一页补上「标签」编辑（与名册列表上那颗标签按钮同一份数据源）。 + ⚠️ 标签是**每人一份**的（owner = 我），所以只有"我"名册里的标签有意义： + · `myTags` 这个人当前在我名下的标签 → 回填输入框； + · `tagChoices` 我名下**用过**的全部标签 → 给可点选的候选（与列表页同一个 + `PersonTag::cloud()`，两处口径一致）。 + ⚠️ 改自己（$isSelf）不给标签：标签是"我给别人打的"，给自己打没有意义 + （名册页那颗标签按钮也只在 `$canMove` 时出现）。 */ + 'myTags' => $isSelf ? [] : PersonTag::of((int)Auth::personId(), $id), + 'tagChoices' => $isSelf ? [] : array_column(PersonTag::cloud((int)Auth::familyId(), (int)Auth::personId()), 'tag'), ], 'people'); break; @@ -2052,6 +2073,12 @@ switch ($r) { 'phoneValue' => $d['phone'], 'idCardValue' => $d['id_card'], 'contacts' => $contacts, + /* 🔴 v51：校验失败**必须把标签也带回去**，否则用户刚打的一串标签 + 随着这一屏重画就没了（"改了半天的东西突然消失"）。 + ⚠️ 取自 `$_POST['tags']`（用户刚提交的那份），**不是**库里的旧值 —— + 这才叫"保留用户填的东西"，`clear_old`/`old()` 那一套同理。 */ + 'myTags' => $isSelf ? [] : PersonTag::clean((string)($_POST['tags'] ?? '')), + 'tagChoices' => $isSelf ? [] : array_column(PersonTag::cloud($familyId, (int)Auth::personId()), 'tag'), ], 'people'); exit; } @@ -2081,12 +2108,27 @@ switch ($r) { 'phoneValue' => $d['phone'], 'idCardValue' => $d['id_card'], 'contacts' => $contacts, + // v51：同上面那条 —— 重画时保留用户刚提交的标签 + 'myTags' => $isSelf ? [] : PersonTag::clean((string)($_POST['tags'] ?? '')), + 'tagChoices' => $isSelf ? [] : array_column(PersonTag::cloud($familyId, (int)Auth::personId()), 'tag'), ], 'people'); exit; } Person::replaceContacts($id, $contacts); // 工作经历：整组替换（第 4 条）。空的那组会把这段经历清空 —— 与联系人同一口径。 PersonJob::replace($id, (array)($d['jobs'] ?? [])); + /* 🔴 v51（用户 2026-10-15 第 4 条）：编辑页也能改「标签」了。 + ⚠️ **owner 永远是"我"**（`PersonTag::setFor(家庭, 我, 他, ...)`）—— + 标签是"我给别人打的"，改它不能影响全家的标签（与列表页那颗标签按钮同一口径）。 + ⚠️ 守卫与"私备注"同一条思路，但不同： + · 改自己时表单里**没有** tags 字段 → 不能读（读到空串会把我的标签清空）； + · 所以先用 `array_key_exists` 判"这次提交里有没有这个字段"， + 没有就**不碰**（而不是"当成清空"）。 + ⚠️ 这正是 v49 那次踩过的形状（"字段不在就当成空"会把数据弄丢）—— + 凡"条件出现的字段"都必须按"存在与否"分流，不能按"值是否为空"。 */ + if ((int)$id !== (int)Auth::personId() && array_key_exists('tags', $_POST)) { + PersonTag::setFor($familyId, (int)Auth::personId(), $id, (string)$_POST['tags']); + } // 「我对他的私人备注」（第 5 条）：落在**写的人自己**名下（person_notes）。 // ⚠️ 两条守卫： // ① 编辑自己时表单里没有这个字段（不需要"我对我的备注"）→ 别把已有的清掉； @@ -4693,6 +4735,29 @@ $want = array_keys($want); redirect('settings/me', [], 'api'); break; + /** + * 日程「显示设置」独立页（v51 用户 2026-10-15 第 6 条）。 + * + * 原来「默认视图 + 显示哪些日历本」是**铺在设置下拉里**的一个表单； + * 日历本一多，下拉就被撑到屏幕外面（用户原话："下面的设置就看不到了， + * 要拉下来才能看到"）⇒ 挪成独立页面，下拉里只留一行「显示设置」。 + * + * ⚠️ 保存仍走既有的 `calendar/prefs`（一条没改）—— 它只是从"下拉里的表单" + * 变成"这一页上的表单"，提交契约、字段名、服务端逻辑完全不动。 + */ + case 'calendar/display': + $familyId = (int)Auth::familyId(); + $me = (int)Auth::personId(); + $cals = Calendar::mine($familyId, $me); + View::render('calendar/display', [ + 'title' => '显示设置', + 'cals' => $cals, + 'hiddenCals' => array_keys(Calendar::hiddenCalendars($me)), + 'view' => (string)($_GET['view'] ?? Calendar::defaultView($me)), + 'anchor' => (string)($_GET['anchor'] ?? date('Y-m-d')), + ], 'calendar'); + break; + case 'calendar/calendars': $familyId = (int)Auth::familyId(); $me = (int)Auth::personId(); diff --git a/templates/calendar/_settings_menu.php b/templates/calendar/_settings_menu.php index 6e5f376..b7852c9 100644 --- a/templates/calendar/_settings_menu.php +++ b/templates/calendar/_settings_menu.php @@ -1,26 +1,36 @@ '', 'books' => '', - 'import' => '', + 'sub' => '', 'dav' => '', ]; $d = $paths[$name] ?? ''; @@ -41,78 +51,19 @@ $setIcon = static function (string $name): string { . ' fill="none" stroke="currentColor" stroke-width="1.5" stroke-linecap="round"' . ' stroke-linejoin="round">' . $d . ''; }; - -$myCals = (array)($cals ?? []); -$hiddenIds = array_map('intval', (array)($hiddenCals ?? [])); -$curView = (string)($view ?? 'month'); -$backView = $curView; ?>
- - -
- - - -
-
默认打开时看
-
- $vw): ?> - - -
-
- -
-
显示哪些日历本
- - - -

还没有日历本。

- - - - - -

- 观影、旅行、有截止时间的任务会自动出现在这里，不用另建本子。 -

-
- -
- -
-
- - -
- >日历本 · 颜色 / 共享 / 导入 - CalDAV 订阅（在手机日历里看） - - DAV 口令（日历 + 通讯录） -
+ + >显示设置 + >日历本管理 + >订阅管理 + + >DAV 口令
共 本
返回日程 + +
diff --git a/templates/calendar/display.php b/templates/calendar/display.php new file mode 100644 index 0000000..0d45c56 --- /dev/null +++ b/templates/calendar/display.php @@ -0,0 +1,99 @@ + +
+

显示设置

+ 默认看哪个视图、日历上显示哪几本 +
+ 返回日程 + +
+
+ +
+ + + + + +
+

默认打开时看 + 打开日程页默认落在哪个视图

+
+ $vw): ?> + + +
+
+ +
+

显示哪些日历本 + 关掉的本子只是不在日历上画出来，数据都还在

+ +

还没有日历本。

+ +
+ + + +
+ +

+ 观影、旅行、有截止时间的任务会自动出现在这里，不用另建本子。 +

+
+ + 取消 +
+
+
+ \ No newline at end of file diff --git a/templates/calendar/index.php b/templates/calendar/index.php index 0a2187b..15790fe 100644 --- a/templates/calendar/index.php +++ b/templates/calendar/index.php @@ -421,7 +421,21 @@ $headText = $view === 'month' ? date('Y 年 n 月', $ts) // trip 的条目指向"某个旅程里的某段行程"，编辑器在旅程详情页里 —— 这里不给「编辑」 elseif ($evKind === 'anniv' && $evRef > 0) { $evEdit = url('days/edit', ['id' => $evRef, 'embed' => 1]); } ?> + data-fb-entry='' style="background: ; top: %; height: %; @@ -526,19 +540,57 @@ $headText = $view === 'month' ? date('Y 年 n 月', $ts) var startSec = startField ? startField.querySelector('select.dt-second') : null; function setStart(dateStr, hh, mm) { - // ⚠️ 优先走全局接口：它会把原生字段与 Semi 控件**一起**更新。 - // 直接写 startDate.value 的话，Semi 是非受控组件、显示不会跟着变 —— + var H = (hh === null || hh === undefined || hh === '') ? '09' : String(hh); + var M = (mm === null || mm === undefined || mm === '') ? '00' : String(mm); + // ⚠️ 优先走全局接口：它会把原生字段与 Semi/antd 控件**一起**更新。 + // 直接写 startDate.value 的话，非受控控件显示不会跟着变 —— // 用户会看到"我点了 15:00，控件里还写着 09:00"（提交的却是 15:00）。 - if (typeof window.fbSetDT === 'function' - && window.fbSetDT('starts_at', dateStr + ' ' + (hh || '09') + ':' + (mm || '00') + ':00')) { - return; + var okStart = (typeof window.fbSetDT === 'function' + && window.fbSetDT('starts_at', dateStr + ' ' + H + ':' + M + ':00')); + if (!okStart) { + // 兜底：Semi/antd 还没加载完（或加载失败）→ 原生控件照旧能用 + if (!startDate) return; + startDate.value = dateStr; + if (hh !== null && hh !== undefined && startHour) { startHour.value = hh; } + if (mm !== null && mm !== undefined && startMin) { startMin.value = mm; } + if (startSec) { startSec.value = '00'; } + } + /* 🔴🔴 v51（用户 2026-10-15 第 2 条）：「我们的时间范围**默认给它设 1 个小时**， + 就是说你点了之后带进来的是 8 点钟，那么它结束时间应该是当天的 9 点钟， + 就按照 1 个小时把结束时间也给它放上去。」 + ⇒ 带入开始时间时**同时**把结束设为「开始 + 1 小时」。 + ⚠️ 跨午夜要顺延到次日（23:30 点下去 → 次日 00:30），不能让结束时间 + 在日期上跑回"比开始还早的今天"（那种提交上去是负时长）。 + ⚠️ 结束时间的设置**只在这里**做（用户"点了某一处"这个动作）—— + 不要捆进 fbSetDT：那个接口也被"编辑已有日程"复用， + 把它的结束时间也强改成 +1h 会把用户原本填的时长弄丢。 */ + var st = new Date(dateStr + 'T' + H + ':' + M + ':00'); + if (!isNaN(st.getTime())) { + st.setHours(st.getHours() + 1); + var pad2 = function (n) { return (n < 10 ? '0' : '') + n; }; + var eDate = st.getFullYear() + '-' + pad2(st.getMonth() + 1) + '-' + pad2(st.getDate()); + var eStr = eDate + ' ' + pad2(st.getHours()) + ':' + pad2(st.getMinutes()) + ':00'; + if (typeof window.fbSetDT !== 'function' || !window.fbSetDT('ends_at', eStr)) { + /* 兜底：按 data-dt 里的 JSON `name` 找结束端点（**不能**用 `[data-dt*="ends_at"]` + —— 那个属性是一整段 JSON，里面含 `name` 键，朴素子串匹配能中但语义不对， + 将来加个字段就误命中）。 */ + var endField = null; + Array.prototype.forEach.call(document.querySelectorAll('.dt-input[data-dt]'), function (bx) { + if (endField) { return; } + var cfg = {}; + try { cfg = JSON.parse(bx.getAttribute('data-dt') || '{}'); } catch (e) {} + if (cfg.name === 'ends_at') { endField = bx; } + }); + if (endField) { + var ed = endField.querySelector('input.dt-date'); + var eh = endField.querySelector('select.dt-hour'); + var em = endField.querySelector('select.dt-minute'); + if (ed) { ed.value = eDate; } + if (eh) { eh.value = pad2(st.getHours()); } + if (em) { em.value = pad2(st.getMinutes()); } + } + } } - // 兜底：Semi 还没加载完（或加载失败）→ 原生控件照旧能用 - if (!startDate) return; - startDate.value = dateStr; - if (hh !== null && startHour) { startHour.value = hh; } - if (mm !== null && startMin) { startMin.value = mm; } - if (startSec) { startSec.value = '00'; } } /** @@ -785,6 +837,80 @@ $headText = $view === 'month' ? date('Y 年 n 月', $ts) return true; } + /* 🔴🔴 v51（用户 2026-10-15 第 3 条）：「添加一个日程之后，只是把这个日程放在了 + 周视图或者是日视图上面。如果它这个时间段已经有日程了，它会**直接盖在上面**， + 却并没有按照**分栏**的方式左右显示。这个需要再调整一下。」 + + 真根因：`putTimed()` 只把自己那条色块按 top/height **整宽铺上去**， + 完全不知道同一列里已有的日程 —— 于是两条重叠时后加的直接压在先有的上面 + （服务端渲染时是有分栏的：`App\Calendar::conflictLanes`，但那是**整页重绘**时才算的）。 + + ⇒ 修法：**在浏览器里重算整列的分栏**（与服务端同一套算法），插入后重排该列。 + 为什么"重算整列"而不是"给新块腾个位置"： + 新日程可能同时与**多条**已有日程冲突（比如插进一个 3 段重叠的时段）， + 只挪新块会让已有多条的宽度/位置也不对（它们现在是 2 列、重算后该是 3 列） + —— 必须让整列一起重排才是自洽的。 + ⚠️ 算法**照抄** app/Calendar.php 的 conflictLanes（贪心分列 + 分组复列）： + 两边算错一个边界，刷新一次位置就会跳。 + ⚠️ 高度仍按各自"声明的起止"算（没结束的用 +30′）—— 与服务端那套一致。 + （服务端在"没填结束"时还会拿同列下一条的起点当视觉结束；这里简化成 +30′， + 差别只在"未填结束"这一种情况，且插入后用户一刷新就归位。 */ + function minsAttr(el, attr) { + var v = el.getAttribute(attr); + if (v === null || v === '') { return null; } + var n = parseInt(v, 10); + return isNaN(n) ? null : n; + } + + /** 重算某一列里**所有** .tg-block 的分栏（left/width）与 top/height */ + function relayoutColumn(col) { + var blocks = Array.prototype.slice.call(col.querySelectorAll('.tg-block')); + if (!blocks.length) { return; } + // ① 归一成 [{el, s, e}]（缺 data-e → +30′；缺 data-s → 从 top 反推兜底） + var rows = []; + blocks.forEach(function (el) { + var s = minsAttr(el, 'data-s'); + if (s === null) { + var tp = parseFloat(el.style.top) || 0; + s = Math.round(tp / 100 * 1440); + } + var e = minsAttr(el, 'data-e'); + if (e === null || e <= s) { e = s + 30; } + rows.push({ el: el, s: s, e: Math.min(e, 1440) }); + }); + // ② 按开始时间升序（同开始则长的在前）—— 与服务端 usort 同序 + rows.sort(function (a, b) { return a.s - b.s || b.e - a.e; }); + // ③ 划分冲突组（与组内最晚结束仍重叠就并组） + var groups = [], cur = [], endMax = -1; + rows.forEach(function (r) { + if (cur.length && r.s >= endMax) { groups.push(cur); cur = []; endMax = -1; } + cur.push(r); + endMax = Math.max(endMax, r.e); + }); + if (cur.length) { groups.push(cur); } + // ④ 组内贪心分列（每列记最后结束分钟，放进编号最小的可用列） + groups.forEach(function (g) { + var laneEnd = [], laneOf = []; + g.forEach(function (r, k) { + var lane = -1; + for (var li = 0; li < laneEnd.length; li++) { + if (laneEnd[li] <= r.s) { lane = li; break; } + } + if (lane < 0) { lane = laneEnd.length; laneEnd.push(r.e); } + else { laneEnd[lane] = r.e; } + laneOf[k] = lane; + }); + var cols = Math.max(1, laneEnd.length); + g.forEach(function (r, k) { + var left = laneOf[k] / cols * 100, width = 100 / cols; + r.el.style.left = 'calc(2px + ' + left.toFixed(4) + '%)'; + r.el.style.width = 'calc(' + width.toFixed(4) + '% - 4px)'; + r.el.style.top = (r.s / 1440 * 100).toFixed(4) + '%'; + r.el.style.height = ((Math.max(15, r.e - r.s)) / 1440 * 100).toFixed(4) + '%'; + }); + }); + } + /** 周/日视图：按"列高 = 24 小时"算 top/height 插时间块 */ function putTimed(ev) { if (ev.all_day) { @@ -832,6 +958,11 @@ $headText = $view === 'month' ? date('Y 年 n 月', $ts) var blk = document.createElement('a'); blk.className = 'tg-block'; blk.setAttribute('data-cal', String(ev.cal || '')); + /* 🔴 v51：带上"声明的起止分钟"，重算分栏要用（服务端渲染的同名属性）。 + ⚠️ 与服务端口径一致：结束缺失/倒挂 → 不带 data-e（按 +30′ 处理）。 */ + blk.setAttribute('data-s', String(st)); + var declE = minsOf(ev.ends_at); + if (declE > st) { blk.setAttribute('data-e', String(declE)); } blk.style.background = ev.color || ''; blk.style.top = top.toFixed(4) + '%'; blk.style.height = hgt.toFixed(4) + '%'; @@ -839,6 +970,8 @@ $headText = $view === 'month' ? date('Y 年 n 月', $ts) blk.setAttribute('title', ev.title || ''); blk.textContent = ev.title || ''; target.appendChild(blk); + /* 🔴 v51（第 3 条）：插完**重排整列** —— 否则这条会整宽盖在已有日程上（用户报的现象）。 */ + relayoutColumn(target); return true; } diff --git a/templates/people/form.php b/templates/people/form.php index 3b6188e..a087229 100644 --- a/templates/people/form.php +++ b/templates/people/form.php @@ -347,6 +347,47 @@ $ctHints = [ + + +
+ + +
+ + + +
+ + + + + 一个人可以有多个标签 —— 他既是小学同学、又是初中同学，两个都打上就行。
+ 标签是你自己名册里的：你给他打「球友」不会让全家的名册都多出这个词。 + +
+ +
@@ -490,5 +531,35 @@ $ctHints = [ }); } } + + /* 🔴 v51（用户第 4 条）：标签的「勾选框 ↔ 文本框」双向同步。 + ⚠️ 文本框（name="tags"）是**唯一提交源**，与服务端只读一个字段的约定一致； + 勾选框只是快捷方式。判据与名册列表那个浮层**逐字一致** + （split(/[\s,，]+/) + 去掉前导 #）—— 两处口径不同的话， + "在列表页打的标签"与"在编辑页打的标签"会归一成不同的结果。 + ⚠️ 手打的新词没有勾选框 ⇒ 同步时要把它们**保留**（否则勾一下就把新词冲掉了）。 */ + (function () { + var block = document.getElementById('ptBlock'); + if (!block) { return; } + var inp = document.getElementById('pfTags'); + var picks = block.querySelectorAll('.pt-pick input'); + if (!inp || !picks.length) { return; } + + function syncFromPicks() { + var on = []; + Array.prototype.forEach.call(picks, function (c) { if (c.checked) { on.push(c.value); } }); + var known = {}; + Array.prototype.forEach.call(picks, function (c) { known[c.value] = true; }); + var keep = []; + String(inp.value || '').split(/[\s,，]+/).forEach(function (w) { + w = w.replace(/^#/, ''); + if (w !== '' && !known[w]) { keep.push(w); } + }); + inp.value = on.concat(keep).join(' '); + } + block.addEventListener('change', function (e) { + if (e.target && e.target.closest && e.target.closest('.pt-pick')) { syncFromPicks(); } + }); + })(); })(); diff --git a/templates/trips/show.php b/templates/trips/show.php index 5877156..80ce84e 100644 --- a/templates/trips/show.php +++ b/templates/trips/show.php @@ -707,8 +707,36 @@ foreach ($bysStage as $lst) { $shown += count($lst); } } }); } - /* 成功：服务端 302 回来 → 只换列表容器（保留滚动位置、不闪白） */ + /* 成功：服务端 302 回来 → 换列表容器（保留滚动位置、不闪白）。 + 🔴🔴 v51（用户 2026-10-15 第 1 条）：「在旅程的详情页面编辑的时候，当我编辑完了之后， + 点击确定之后，页面并没有刷新，就是原来的详情页面并没有刷新。比方说我修改了一张图片， + 当我确定完了之后，原来的页面应该刷新一下，把最新的这个旅程的信息给更新出来才对呀。」 + + 真根因：这里只换 `#nodeListHost`（**行程列表**那一段），而旅程自己的 + **标题（页头 h1）/ 题图 / 时间与城市（.trip-hero）** 都在那个容器**外面** —— + 它们由服务端直出、不在被替换的那块里 ⇒ 改了标题/题图/简介，页头与题图纹丝不动。 + 用户看到的就是"确定点了、框关了，但上面还是旧的"。 + + ⇒ 修法：**按表单来源分流**—— + · `tripForm`（编辑旅程本体：标题/题图/时间…）→ **整页刷新**。 + 理由：它改的正是"页面骨架"（页头 + 题图 + 城市），这些散落在容器外、 + 无法靠换一小块 DOM 覆盖；而"保留滚动位置"在这里没有价值 + （用户改完就是要看新的题图与标题，回到顶部反而正合意）。 + · `nodeForm`（行程增删改）→ 仍然只换行程列表（用户当年明确要的"不整页跳"， + 而且它改的东西**确实**只在那块容器里）。 + ⚠️ 判据用表单 id，不要用"有没有题图字段"去猜 —— id 是稳定的契约。 + ⚠️⚠️ **必须 `getAttribute('id')`，不能写 `form.id`** —— + HTML 表单元素会把**同名的具名控件**挂成自己的属性： + `tripForm` 里有 `` ⇒ `form.id` 返回的是 + **那个 input 元素**（字符串化就是 `[object HTMLInputElement]`）， + 永远不等于 `'tripForm'` ⇒ 这个分支**从来不会命中**（本轮实测： + 改完标题点确定，页面纹丝不动；插桩打印出来才看到 + `form.id === '[object HTMLInputElement]'`）。 + 同类还有 form.name / form.action / form.method 都可能被同名控件遮蔽。 + ⇒ 一律用 `getAttribute('id')` 取"写在标签上的那个 id"。 */ + var isTripForm = (form.getAttribute('id') === 'tripForm'); return r.text().then(function (html) { + if (isTripForm) { window.location.reload(); return; } var doc = new DOMParser().parseFromString(html, 'text/html'); var fresh = doc.getElementById(RELOAD) || doc.getElementById('tripListHost'); var cur = document.getElementById(RELOAD) || document.getElementById('tripListHost');