b45cf671d1c1340547da094f9d28269260a5c6be
3
Commits
| Author | SHA1 | Message | Date | |
|---|---|---|---|---|
 laoyang
|
32e9e9ab84 |
v47：地址联想三处接入 / 读书页慢的真根因（会话锁）/ 笔记两处 / 日程状态下线
用户 2026-10-13 七条。拍板：影院只记地址、不存经纬度（后续用高德反查）；
日程状态只删界面与展示、数据库列保留（「后续会有一个整体清理的工作」）。
①🔴 读书页慢 —— 真根因是**会话锁**，不是封面（我先判断错了方向）
用户原话：「读书现在还是非常慢，就是进入读书这个页面，刚开始还是非常的慢。」
线上实测（真公网、同一会话连打 3 次）：
第 1 次 TTFB 0.08s ← 触发微信读书同步
第 2 次 TTFB **8.73s** ← 用户说的"刚开始非常慢"
第 3 次 TTFB 0.07s
插桩拿到的服务端时间线把因果钉死：
21:18:05.483 QUEUED ← 第 1 个请求排队
21:18:05.501 FINISH 2ms ← 响应确实提前发出去了（这步是好的）
21:18:14.674 SYNC 9.2s ← 但它仍占着这个人的会话文件锁跑完 9.2 秒
21:18:14.682 QUEUED ← 第 2 个请求此刻才真正开始执行
⇒ **`fastcgi_finish_request()` 只断开客户端连接、不释放 PHP 文件会话锁。**
后台同步要 9.2 秒（真打微信读书接口），同一个人的下一个请求一路等到它跑完。
修法（缺一不可）：
· `WeRead::queueBackgroundSync()` 回调里 **先 `session_write_close()`、
再 `fastcgi_finish_request()`**（顺序不能反）；
· `index.php` 两处"渲染前排同步"的入口排完**立刻放锁**
（shutdown 回调不一定执行：客户端中断/服务器超时才不放过）；
· 顺带修 **`Holiday::queueBackgroundFetch()` 的同型问题**（联网 12 秒）。
为什么以前没发现：v29 引入"后台化"时只验证了"响应是不是提前发出去"，
**单次请求测量永远看不到这个 bug** —— 必须同一会话连打两次才现形。
线上复验：修复后连打 4 次 = 0.15 / 0.15 / 0.09 / 0.15 秒（修复前第 2 次 8.73s）；
后台同步仍正常（`weread_sync_at` 已更新）。
② 封面：顺手也修了（但这不是用户说的那个问题）
· `media.php` 读 OSS 前先放会话锁（原来从不放，35 张封面串行）；
· 新增 `Storage::cacheLocally()` 磁盘二级缓存（原来 `read()` 只读不写回）；
· 让 304 不再先拉整张图（原来先取回 body、再判 `If-None-Match`）。
线上实测：冷取 149ms（走 OSS）→ 热取 35ms（本地缓存、字节一致）
→ 带 `If-None-Match` 返回 **304 / 19.8ms / 空体**；35 张封面 103ms → 20ms（5 倍）。
③ 地址联想：抽成全站共用一份（`assets/js/addr_suggest.js`）+ 接入三处
· 影院管理页、观影新增页的「新建影院」弹框、日程地点、行程出发/到达；
· `layout.php` 按需加载（`[data-addr]` 出现才拉 SDK + 我们的脚本），
并暴露 `fbLoadAddr(scope)` 给 AJAX 片段（v42 踩过"内联 script 不执行"）；
· 安全密钥在 SDK 之前设置；回填支持 `data-addr-fill` / `data-addr-for`、
**只补空白字段**（不覆盖用户手写）、直辖市 `province==city` 去重；
· 字段名有两套（影院 `address`/`place`，行程 `from_addr`/`to_addr` 缩写 `_addr`），
`fieldFor()` 带同义词表 + 后缀匹配；
· 影院**只存地址、不存经纬度**（按用户要求）。
④ 笔记两条
· 「点保存没反应」真因：附件区是个 `<form>` **套在保存表单里** ——
HTML 不允许 form 套 form，浏览器丢掉内层标签、保留它的 `</form>`，
于是**保存按钮成了孤儿**（`button.form === null`），点了不提交也不报错。
真浏览器实测：修前 `btnInsideForm:false` / 点后 POST=0；修后 true / POST=1。
改法：附件区改 `<div id="noteAttForm">`，上传按钮用 `formaction`。
· 详情页宽度：`.note-read { max-width: 820px }` → `none`
（内容卡 820px vs 下方共享卡 1032px；修后两者都是 1032px）。
⑤ 日程状态下线
· 删表单里的「状态」下拉、`Calendar::statusWord()`、浮层 JSON 的 `status_word`；
· **保留** `events.status` 列与 `Calendar::statusOf()`（契约不动，等整体清理）。
验证（新增 5 个套件）：
· `_fb_v47_addr.py`（接线 24 条）/ `_fb_v47_addr_sim.py`（真浏览器 + 假高德，16 条）
· `_fb_v47_readperf.py`（缓存与会话锁，23 条）
· `_fb_v47_notes.py`（真浏览器，9 条）
· `_b47_readperf_falsify.py`（反向验证 9 个注入）—— **9/9 全抓住**
· 全量 e2e **1510/0**；vCard 规格+往返 40/0；TB 探针 44/0；
v44 反向验证 17/0；全流水线 **PUSH_RC=0**（含"第47批形态"静态断言）
· **线上终验**：读书页整页 81ms、封面 103→20ms、media 冷 149→热 35→304 19.8ms、
修复前第 2 次请求 8.73s → 修复后 0.15s、地址联想 meta[fb-addr] 带真高德 key
顺带修掉三处**探针自身**的假红/假绿（详见 pitfalls.md）：
· `_b44_falsify.py` 的"整小时"注入在**整点**与正确实现等价 ⇒ 换成恒错值；
· `_fb_b44_ui.py` 3b 的并发计时断言在单进程 `php -S` 下必然假红 ⇒ 改断言确定事实；
· 多处探针的"清理删除"撞沙箱删除门禁 ⇒ 探针**中途死掉**看起来像产品坏了
⇒ 统一改成 `_safe_rm()` / 不删。
|
||
|
laoyang
|
c3d00ad66a |
v24 第 20 批：拖动即时上传 / 观影图片带出 / 行程保存修复 / 日程红线与删除范围与多提醒 / 悬停统一（用户 9 条）
用户 9 条原话要点：
①「观影模块，编辑观影，已经保存的海报和票根、合影，都没带进来。」
②「个人设置页面，"只影响你自己看到的样子"这个提示删除，不要了。我怎么称呼家人，
这个卡片，不要了。我的资料这个卡片，不要了。」
③「旅程模块，编辑行程，点击保存没有反应。」
④「所有上传图片视频和附件的模块，都允许多次拖动添加文件上传，拖动一个进去，
就开始上传，还可以继续拖动，继续上传，最后统一保存即可。」
⑤「添加行程，线路名、承运方，都不要了。车辆，只在出行方式选择自驾时才出现，
其他情况下默认不出现。出发地和到达地，应该在同一行。」
⑥「日程模块，周视图和日视图，应该在视图中间显示一条红线，用来表示当前进行的时间…
用一个不断从上到下的红线，来表示当前时间在视图上的刻度和位置。」
⑦「对于重复的日程，当点击删除的时候，应该提示是删除这一次，还是全部都删除，
还是删除这一次及以后的。」
⑧「日程的提醒，是可以多选的，就是说，一个日程，我可以选择让它 5 分钟提醒，
10 分钟提醒等等。」
⑨「很多按钮的悬停效果非常奇怪，感觉纯粹是换了一个背景色…应该调整所有按钮的悬停效果，
保持统一，但是不要单纯的换一个不协调的背景色。」
🔴 三个真根因（都是"看着不像 bug"的地方）：
1. **③ 点保存没反应 = `<form>` 嵌套**（HTML 非法）：`trips/node_form.php` 的照片缩略图上
套了 `<form action=photo_del>`，而整块在**外层行程表单内部**。浏览器解析到内层
`</form>` 时**把外层表单提前关掉** → 「保存」按钮**掉出表单**（真机实测
`button.form === null`）。源码里完全看不出，**只有真浏览器量 `btn.form` 才现形**；
且**只有这段行程已经有照片时才触发** —— 所以表现为"有时候好、有时候坏"。
→ 按钮留在原位（布局不动），用 HTML5 `form="id"` 关联到**表单外**的独立表单。
2. **⑤ 车辆藏不住（第 17 批就写过、一直没生效）**：`.field` 自己声明 `display:flex`，
作者样式压过浏览器默认的 `[hidden]{display:none}` → `hidden` 形同虚设。
**本项目第 4 次栽同一个坑**（.att-item / .pv / .pp-item 各一次）→ 补 `.field[hidden]`。
3. **④ 拖动即时上传的卡片"看不见"**：模板给 `.stage-list` 加了 `hidden`（没人上传时
不留空边框），JS 建卡片时忘了 `list.hidden = false` → 卡片在 DOM 里但整条清单
`display:none`、尺寸 0×0。**HTTP 层断言全绿**（HTML 里确实有卡片），
**只有真机量尺寸才发现** —— 由新写的真机探针抓到。
本批新增能力：
- **拖动即时上传**（第 4 条）：新增 `upload_stage.php` + `app/StagedUpload.php` +
`templates/_stage_drop.php`。拖进来 → **立刻** POST 到暂存目录 → 回 `stoken` →
放进隐藏字段 `stage[]` → 保存时在 `attach_save_files()` 里**认领**（移文件 + 建附件行）。
`media.php` 加 `?stage=` 预览（拖完立刻看得见缩略图）。
⚠️ 三个必须踩准：**串行队列**（并发会让缩略图顺序乱跳）、**提交守卫**
（还有没传完就 preventDefault，否则"拖了但没保存上"必然发生）、**卡片清单要 un-hide**。
⚠️ 海报/票根走**独立字段名** `stage_poster`/`stage_ticket`（各自单槽位、要按比例重裁）。
⚠️ 落点**就近取**（`input.closest('.field')`）：一个表单里多个上传区不能串到第一个去。
- **观影编辑带出已存图片**（第 1 条）：编辑页显示海报 / 票根 / 合影 + 各自的删除入口；
新增 `movies/media_del`（只删不换也能删，槽位走白名单）；`movies/update` 收编辑页
新选的照片时**合并**而不是替换（否则一编辑就把已有合影冲掉）。
- **行程表单**（第 5 条）：去掉线路名 / 承运方（**库里的值保留**：UPDATE 时不参与，
与 seat_info/gate 同一套）；出发地/到达地包进 `.loc-row` 真同一行；车辆按自驾显隐。
- **当前时间红线**（第 6 条）：位置在服务端算（与 `.tg-block` 同一套"24h = 100%"坐标），
前端每分钟自己挪（对齐到下一分钟整再起定时器），跨午夜 reload；只画在**今天那一列**。
- **重复日程删除三档**（第 7 条）：与「编辑」那三档同名同序（全部 / 这一次及以后 /
只这一次）。「只删这一次」→ `skipOnce`（**绝不删整条系列**）；「这一次及以后」→
老系列截 UNTIL，**必须清掉 COUNT**（两者并存时 COUNT 优先，不清就截不断）。
- **提醒多选**（第 8 条）：同列存逗号串（`5,15,60`）；Schema 把它从 **INT 改 VARCHAR**
（INT 存逗号串会被隐式转换，多档会静默只剩第一档）；ICS 导出吐**多个 VALARM**。
- **悬停统一**（第 9 条）：原来 14 种互不相干的 hover 底色 → 统一成**两套手法**：
有底色 `filter: brightness(.92)` 同色系加深 + 轻抬升；描边白底 `--hov-veil` 中性薄雾
+ 边框微显。放在 CSS **最末**用一条兜底收口（新按钮自动继承），并把十几条旧规则
清成"交给统一段"。`person_picker.js` 同时改为按需加载。
验证（本地全套）：
- e2e **1004/0**（+21 条本批断言）· 升级 149/0 · 桌面几何 428/0 ·
**真机探针 25/0**（连续拖两次 / 红线位置与颜色与几何 / 车辆显隐 / 出发到达同行）。
- **反向验证 5/5**：①删除按钮改回嵌套 form ②卡片清单不 un-hide ③观影编辑不渲染海报
④车辆隐藏失效 ⑤「只删这一次」做成删全部 —— 五种坏法全部被抓住，还原后复绿 1004/0。
过程中还**抓到两条弱断言**（只查 DOM 有无、不查可见性/是否真渲染），已一并加强。
- 部署脚本静态断言段本地空跑 **305 条全过**。
|
||
|
laoyang
|
7e9b4520c0 |
登录居中 · 个人 slogan · 三种生日 · 称呼视角化 · 附件统一走代理
按九点反馈一次性改造： 1. 登录页「图标 + 用手机号登录你的家书」包进 .auth-head 并真正居中 （以前只加了个 class 名，没有样式，看着像做了其实没做）。 2. slogan 改成两段式：系统级 slogan 作兜底，每个人可在自己的资料里写一个， 填了就显示自己的、留空就回落到系统那句。settings 里「家庭名称」字段去掉 （只在安装时归档，不参与展示）。 3. 三个生日：身份证生日（从证件号截取，永不改动）／真实阳历／农历。 同一人填了其一即自动推出另一个（靠 birthday_source 指明填的是哪一侧， 不依赖 JS）。三者落在同一天时合并成一条提醒，否则分别提醒。 农历表不是照抄流传的那份 lunarInfo（它在 5 个年份上是错的）—— 用五个独立实现按 7.3 万天多数表决重新推导，再要求 100% 重现。 4. 称呼改成「视角属性」：nicknames(family_id, viewer_id, target_id, nickname)。 「小明」不是某个人的属性——同一个人被不同人叫法不同，所以是「谁看」的属性。 旧的 persons.nickname 由 Nickname::migrateLegacy() 搬到新表，只搬一次。 5. 点自己的头像/名字直接进「我的资料」，不必再绕 「名册 → 找到自己 → 编辑资料」三步。 6. 附件（头像与其它上传）一律走 media.php 代理，不再直连 uploads/ 或云端地址： 可读性由登录态决定而不是「知道文件名」，Content-Type/缓存/下载头由服务端统一给。 uploads/ 整个目录 deny（含历史遗留备份文件），并有真实探针验证。 7. OSS 设置页加「连通性测试」（本地 / 云端两个按钮），逐步给出结论与耗时。 8. 生日输入改用「原生日期框 + 24 小时制时:分:秒 三个下拉」： datetime-local 的格式由浏览器 UI 语言决定（中文环境渲染成「下午 03:30」， 没有秒、还容易把上下午填反）。点输入框任意一处都能弹出选择器（showPicker）。 9. 结构版本号收敛成 Schema::VERSION 一处，安装时即盖章，避免首次请求白跑升级。 验证（全部为「先注入 bug 证伪、再跑通过」）： php -l 42 个文件；e2e 149 条；老库升级专测 37 条；反向验证 23 个注入全被抓； 桌面几何 108 条（768/1280/1920/2560）；移动端真机几何 212 条（360/375/390/414）； 交互检查 49 条（生日整框可点、登录居中）。 |