Bug: SQLAlchemy models with default=func.now() error with Unexpected default type about litestar HOT 5 CLOSED

@sion908 I am introducing litestar to a project which already had sqlalchemy in it. I would like to keep new litestar code separated from older sqlalchemy stuff. Meaning I don't want to import AuditColumns from litestar. In case I wanted to remove litestar from the project in the future, this would create problems.

But I will try using lambda: datetime.now(), thanks.

from litestar.

Closing this as it's not really a bug. Thanks @sion908 for providing the explanation and alternative solution :)

from litestar.

@MitPitt
Hi!!
I also encountered the same error.
And this seems to be an expected error.
As a concept like a responsibility branching point, it seems better not to let the db generate values ​​as much as possible, so use lambda: datetime.now(timezone.utc) instead of func.nou(). However, in this case, I think it can be said that it is good.

like this
And it's in hereadvanced-alchemy

class AuditColumns:
    """Created/Updated At Fields Mixin."""

    created_at: Mapped[datetime] = mapped_column(
        DateTimeUTC(timezone=True),
        default=lambda: datetime.now(timezone.utc),
    )
    """Date/time of instance creation."""
    updated_at: Mapped[datetime] = mapped_column(
        DateTimeUTC(timezone=True),
        default=lambda: datetime.now(timezone.utc),
    )
    """Date/time of instance last update."""

you can import from here

from litestar.plugins.sqlalchemy import AuditColumns

from litestar.

I hope it was helpful

from litestar.

@sion908 I am introducing litestar to a project which already had sqlalchemy in it. I would like to keep new litestar code separated from older sqlalchemy stuff. Meaning I don't want to import AuditColumns from litestar. In case I wanted to remove litestar from the project in the future, this would create problems.

You can just import it from advanced-alchemy directly:

from advanced_alchemy.base import AuditColumn

from litestar.