tortoise/tortoise-orm

on_delete should be required

Open

#1.801 aberto em 5 de dez. de 2024

Ver no GitHub
 (3 comments) (4 reactions) (0 assignees)Python (333 forks)batch import
enhancementgood first issue

Métricas do repositório

Stars
 (3.863 stars)
Métricas de merge de PR
 (Mesclagem média 15d 7h) (9 fundiu PRs em 30d)

Description

Is your feature request related to a problem? Please describe. I encountered that when declaring ForeignKeyField on_delete = CASCADE, I believe that this is dangerous and in some scenarios may lead to undesirable consequences.

def ForeignKeyField(
    model_name: str,
    related_name: Union[Optional[str], Literal[False]] = None,
    on_delete: OnDelete = CASCADE,
    db_constraint: bool = True,
    null: bool = False,
    **kwargs: Any,
) -> "ForeignKeyRelation[MODEL] | ForeignKeyNullableRelation[MODEL]":

Describe the solution you'd like The best solution would be to make on_delete a mandatory parameter, I believe that actions with data should be explicit.

def ForeignKeyField(
    model_name: str,
    on_delete: OnDelete,
    related_name: Union[Optional[str], Literal[False]] = None,
    db_constraint: bool = True,
    null: bool = False,
    **kwargs: Any,
) -> "ForeignKeyRelation[MODEL] | ForeignKeyNullableRelation[MODEL]":

Guia do colaborador