agronholm / agronholm/sqlacodegen

Dataclass Mapping changed in sqalchemy 2.0

Aperta
#248 12 commenti 0 reazioni 1 assegnatario Rivendicata da @sheinbergon Vedi su GitHub
Lingua principale
Python
Stelle
2.4k
Fork
284
Merge medio
2g 1h
PR unite (30g)
1

Descrizione

Refer: https://docs.sqlalchemy.org/en/20/orm/dataclasses.html#orm-declarative-native-dataclasses

Current documented dataclass style is:

```python
from sqlalchemy.orm import DeclarativeBase
from sqlalchemy.orm import Mapped
from sqlalchemy.orm import mapped_column
from sqlalchemy.orm import MappedAsDataclass

class Base(MappedAsDataclass, DeclarativeBase):
"""subclasses will be converted to dataclasses"""

class User(Base):
__tablename__ = "user_account"

id: Mapped[int] = mapped_column(init=False, primary_key=True)
name: Mapped[str]
```

or at least:

```python
from sqlalchemy.orm import Mapped
from sqlalchemy.orm import mapped_column
from sqlalchemy.orm import registry

reg = registry()

@reg.mapped_as_dataclass
class User:
__tablename__ = "user_account"

id: Mapped[int] = mapped_column(init=False, primary_key=True)
name: Mapped[str]
```

Columns type should be `Mapped[int]` not `int` which is different with current sqlacodegen and the new `mapped_column` is much more readable.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.