labull-framework 0.1.34__tar.gz → 0.1.35__tar.gz

{labull_framework-0.1.34/labull_framework.egg-info → labull_framework-0.1.35}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: labull-framework
-Version: 0.1.34
+Version: 0.1.35
 Summary: Labull Framework: a pluggable Django app with Authing OIDC login and magic blocks
 Author: Labull Framework
 License-Expression: MIT

{labull_framework-0.1.34 → labull_framework-0.1.35}/labull_framework/canvas/data.py

@@ -450,10 +450,32 @@ def _fields_of(model: Any) -> tuple[list[dict[str, Any]], list[dict[str, Any]],
                 )
             )
             continue
-        if getattr(field, 'one_to_many', False) and not getattr(field, 'one_to_one', False):
-            # 🔴 **反向那一侧也要画**：从 `Customer` 看不到"它被多少订单引用"，
-            #    图就是半个 —— 老项目那套连线两边都连。反向是**只读**的（库里没有这一列）。
-            # ⚠️ 反向的一对一**跳过**：它的正向那一边已经画了一条线，两边都画就成了两条重合的线。
+        # 🔴 **反向那一侧也要画**：从 `Customer` 看不到"它被多少订单引用"，
+        #    图就是半个 —— 老项目那套连线两边都连。反向是**只读**的（库里没有这一列）。
+        # 🔴 **反向的一对一必须并进来**（`one_to_one` 那一档）——老代码写的是
+        #    `one_to_many and not one_to_one`，而 Django 的反向一对一是一个
+        #    `OneToOneRel`：它的 `one_to_many` 是 **False**、`one_to_one` 是 **True**
+        #    ⇒ 老判据把它整个漏掉，**一条反向一对一都不画**。
+        #    代价不是"少画一根线"而是**一片假差异**：基线那一侧是 `shape_ast._reverses()`
+        #    算的，它对一对一**照画不误**（`name_back = related or model.name.lower()`）
+        #    ⇒ 每次打开画布都报"删除了 `supplier.bank_account` / `labull_user.extra`"
+        #    （在 `_relation_diff()` 那里成了 `deleted` 一档），而那两张表根本没动过。
+        #    🔴 "正向那边已经画了一条线"这件事**不成立**：反向一对一指向的是**子表**，
+        #    正向那条线指向的是**父表**，两张卡片、两条线，不是重合。
+        #    口径与 `shape_ast._reverses()` 那一段**逐字对齐**（那边的 `elif kind == 'oneToOne'`）。
+        # 🔴 **前向的一对一不能被这一档吃掉**（这一条是我修上面那一档时当场踩出来的，
+        #    而且报文一模一样是"改过"）：前向的 `OneToOneField` 在 `_meta` 上
+        #    `one_to_one` 也是 **True** —— 拿它当选判据，`ReceiveBankAccount.supplier`
+        #    这条**真列**会被当成"反向那一侧"，于是 `kind` 记成 `reverse`、`column` 记成
+        #    `None`、`null`/`unique` 记成 `None` ⇒ 基线里那条 `('supplier','oneToOne','1',True,True)`
+        #    与它一比就是"改过"（实测：`receive_bank_account.supplier` / `user_extra_info.user`
+        #    两张表各报一处假的修改）。
+        #    判据：**反向的那几个都是 `ForeignObjectRel`**（`auto_created` 为真），
+        #    前向字段是 `Field`（`auto_created` 为假）—— 只有前一种才走这一档。
+        reverse_rel = getattr(field, 'one_to_many', False) or (
+            bool(getattr(field, 'one_to_one', False)) and bool(getattr(field, 'auto_created', False))
+        )
+        if reverse_rel:
             target = field.related_model
             # 🔴 **对端是"我们不画的那几张表"时不补这一条**（`django.contrib.*` / 会话 / 迁移）：
             #    它的正向声明住在 **Django 自己的源码**里，而"上一版"那一份只扫项目与装着的
@@ -464,7 +486,13 @@ def _fields_of(model: Any) -> tuple[list[dict[str, Any]], list[dict[str, Any]],
             #    **用同一条判据**就能对齐：不画的表，两边都不挂它的反向关联）。
             if _table_of_model(target) in SKIP_TABLES:
                 continue
-            relations.append(_relation_of(field, 'reverse', '*', target, readonly=True))
+            # 🔴 一对一反过来的**基数也是 `1`**：这一头只对应一行（`OneToOneRel.one_to_one`）。
+            #    报成 `*` 会让前端画出"一对多"那种端口，也与基线那一份（`back_card = '1'`）对不上
+            #    ⇒ `_relation_key()` 一比就是"改过"，又一处假差异。
+            once = bool(getattr(field, 'one_to_one', False))
+            relations.append(
+                _relation_of(field, 'reverse', '1' if once else '*', target, readonly=True)
+            )
             continue
         if not hasattr(field, 'attname'):
             continue
@@ -477,7 +505,46 @@ def _fields_of(model: Any) -> tuple[list[dict[str, Any]], list[dict[str, Any]],
                 _relation_of(field, kind, '1', target, column=str(field.attname))
             )
         fields.append(_shape_field(field, enums))
-    return fields, relations, enums
+    return fields, _dedupe_relation_names(relations), enums
+
+
+def _dedupe_relation_names(relations: list[dict[str, Any]]) -> list[dict[str, Any]]:
+    """按**关联名**去重：真列（前向那条）优先，说不出名字的反向那条让位。
+
+    🔴 为什么必须有这一步（修完"反向一对一也要画"那一档才露出来的）：差异比对
+       （`_relations_of()` 与 `_relation_diff()`）是拿 `row['field']` 当键的，
+       **同名就互相盖**。而反向一对一在**没有 `related_name`** 时，Django 给它的访问器名是
+       **子模型名的小写**（`UserExtraInfo` ⇒ `extra`、`ReceiveBankAccount` ⇒ `bank_account`）——
+       这一档本身没问题；问题是那 13 条前向一对一/外键里，只要有一条的**字段名**与
+       另一条反向关系的名字撞上，两条里必然被盖掉一条，于是 `_relation_diff()` 拿着同名键
+       比出**两条不同的关系**，报出"`receive_bank_account.supplier` 改过"这种假差异
+       （基线里那一条是 `('supplier','oneToOne')`，现在这一份被盖成了
+       `('supplier','reverse')` —— 实测就是这个）。
+    🔴 **前向那条（`readonly` 不为真）一定留下**：它是库里真有的那一列，信息量更大；
+       而名字冲突时 Django 本来也不会给反向那一侧生成访问器属性。
+       ⚠️ 所以判据是"先收前向、再补反向"，**不是**"先来后到"——`_meta.get_fields()`
+       今天恰好是"先列、后反向"，靠顺序等于靠一个没写进契约的实现细节。
+    """
+    forward = {
+        str(row.get('field') or '')
+        for row in relations
+        if not row.get('readonly')
+    }
+    out: list[dict[str, Any]] = []
+    seen: set[str] = set()
+    for row in relations:
+        name = str(row.get('field') or '')
+        if not name:
+            out.append(row)
+            continue
+        if name in seen:
+            continue
+        # 反向那条与某个真列同名 ⇒ 让位（它没有自己的访问器，见 docstring）。
+        if row.get('readonly') and name in forward:
+            continue
+        seen.add(name)
+        out.append(row)
+    return out
 
 
 def _type_of(field: Any) -> str: