v24 第 17 批:城市聚合 / 高德地址 / 表单精简(用户 10 条)
用户 10 条原话要点与落地:
- ①「统计出来的不是目的地,也不是车站,而是城市…城市要聚合,有多段行程,
都有青岛作为出发地或者目的地,那么就只记一次」
→ trip_nodes 补 from_city/to_city(Schema v22),Trip::cities() 改成
「出发地 ∪ 到达地」去重、按最早时间 + 出发地优先排。
详情页信息区显示的就是这个聚合结果(第 4 条:「图片右侧显示的是聚合的城市,
而不是终点地址」)。
- ②「类型不够丰富,你看看 triplog 里面都有哪些,都拿过来」
→ NodeType::TRIPLOG_TYPES(休息/突发/购物/拜访亲友),在 all() 里读时自愈补齐。
- ③「出行方式,只有选择自驾,才让选车辆;车辆下拉框不需要出现什么财务里登记的车,
只把车辆列出来就行了」→ DRIVE_MODES 收窄成 ['drive'];
Vehicle::pickable() 只回车辆库;车辆页删掉「财务里已登记的车」那块。
- ③「飞机、铁路,只记录车次、航班号就可以…座位什么的不记录了」
→ seat_info / gate 从表单与列表**彻底移除**(列保留,老数据不丢;
saveNode 的 UPDATE 也不再写这两列,否则会抹掉迁来的数据)。
- ③「出发地和到达地,都用高德地图的接口,记录详细地址」+「参与人弹框选择」
+「给参与人标注角色这个功能去掉」
→ 新增 templates/trips/_loc_field.php(高德联想 + 详细地址 + 坐标 + 城市 +
导航链接,没配 key 也能手输);参与人改用全局弹框选人组件;
角色输入框删除,saveNodeMembers 改成差集增删(否则老 role_tag 被静默抹掉)。
- ④「triplog 中的行程花销数据,没有同步过来,需要迁移过来」
→ 新增 _tooling/_fb_b17_backfill.php:迁地址(20 条)、花销(4 笔 ¥8584)、
车辆(3 台)。线上已执行并核对(笔数与金额一分不差)。
- ④「行程中,时间明明具体到了时分秒,怎么不显示?」
→ Trip::nodeSpan():行程时间精确到时分(同一天只写一次日期、跨天写月日、
两头都没时分就不写 00:00)。
其他:
- App\Geo 扩展 cityFromPoint / cityFromText / navUrl / jsConfig。
中文地址提城市修掉四种错法(白名单→黑名单、取最后→取第一个、直辖市最先认、
离线表只能兜底),并加了「五大连池 ≠ 大连」的位置守卫。
- config.example.php 加 amap 段(key / security_js / nav_src)。
验证:
- e2e 920/0(+29 条本批断言)· 升级 149/0(+15 条 v22 断言)·
城市解析 32/32(新套件,样例全是 TripLog 线上真实地址)·
桌面几何 428/0 · 移动端 688/0 · 交互 92/0 · 窄屏溢出 24/0。
- 反向验证 5/5:城市退回读 place / 只算到达地 / 时间退回只到日期 /
打车也要车 / 类型不补 —— 五种坏法全部被断言抓住。
- 部署:服务器自检 19/19;线上 schema v22;补迁后花销 4 笔 ¥8584 与源库对齐。
This commit is contained in:
+21
-8
@@ -48,9 +48,13 @@ final class Vehicle
|
||||
/**
|
||||
* 财务里已登记的车子(**只读引用**,不复制进 vehicles)。
|
||||
*
|
||||
* 取的是所有账本里 kind='car' 的账目 —— 用户说"直接读取就可以"。
|
||||
* 返回结构与自有车辆**同形**(id 用负数区分:`-账目id`,`from` 标来源),
|
||||
* 这样模板里画下拉、画配置列表都能用同一套代码。
|
||||
* 🔴 **自 v24 第 17 批(用户第 3 条)起,界面上已不再使用** ——
|
||||
* 用户原话「车辆下拉框,不需要出现什么财务里登记的车,只把车辆列出来就行了」。
|
||||
* 方法本身保留的两个理由:
|
||||
* ① `vehicles.ledger_id` 是**软关联**(某台出行车在财务里对应哪条资产账目),
|
||||
* 将来"按车统计油费/保养"要用它把两边的数字对起来;
|
||||
* ② 删掉方法会让那层对应关系在代码里失去唯一的说明处(注释也跟着没了)。
|
||||
* **不要再把它接回选车下拉** —— 那正是这一批要改掉的东西。
|
||||
*/
|
||||
public static function fromLedger(int $familyId): array
|
||||
{
|
||||
@@ -86,16 +90,25 @@ final class Vehicle
|
||||
}
|
||||
|
||||
/**
|
||||
* 选车下拉用的合并列表:财务的车子在前(用户已经录好了),自家车辆在后。
|
||||
* 选车下拉用的列表 —— **只有 vehicles 表里的车**。
|
||||
*
|
||||
* @return array 每项都带 `from` = 'ledger' | 'local'
|
||||
* ⚠️ 用户 2026-10-05 第 17 批 第 3 条把上一批的做法**推翻**了:
|
||||
* 「车辆下拉框,不需要出现什么财务里登记的车,只把车辆列出来就行了。」
|
||||
* 第 16 批第 8 条当时说的是"财务里的车直接读取就可以",这一批明确不要了。
|
||||
* 理由(用户的思路):**出行的车**与**财务里的车子账目**是两回事 ——
|
||||
* 后者记的是"这台车值多少钱"(资产),没有车牌车型,混进选车列表里
|
||||
* 只会让人在"大众朗逸"和"家里的老车"之间发懵。
|
||||
* 所以现在选车只认 vehicles 表(自家 / 租的 / 借的都录在这里)。
|
||||
*
|
||||
* ⚠️ `fromLedger()` 方法**暂时保留但不再被任何界面调用**(见下方注释):
|
||||
* 财务里那些车是真实存在的资产,将来若要"按车统计油费/保养"还会用到;
|
||||
* 删掉它反而会丢掉那层对应关系(`vehicles.ledger_id` 软关联还指着它)。
|
||||
*
|
||||
* @return array 每项都带 `from` = 'local'
|
||||
*/
|
||||
public static function pickable(int $familyId): array
|
||||
{
|
||||
$out = [];
|
||||
foreach (self::fromLedger($familyId) as $v) {
|
||||
$out[] = $v;
|
||||
}
|
||||
foreach (self::all($familyId) as $v) {
|
||||
$v['from'] = 'local';
|
||||
$out[] = $v;
|
||||
|
||||
Reference in New Issue
Block a user