fix(models): rename sqlalchemy.text import to avoid shadowing by InvariantModel text column
CI / push-validation (pull_request) Successful in 40s
CI / helm (pull_request) Successful in 48s
CI / lint (pull_request) Failing after 1m39s
CI / build (pull_request) Successful in 1m38s
CI / typecheck (pull_request) Failing after 2m8s
CI / quality (pull_request) Successful in 2m38s
CI / security (pull_request) Successful in 3m18s
CI / integration_tests (pull_request) Failing after 7m43s
CI / unit_tests (pull_request) Failing after 10m31s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 2s
CI / push-validation (pull_request) Successful in 40s
CI / helm (pull_request) Successful in 48s
CI / lint (pull_request) Failing after 1m39s
CI / build (pull_request) Successful in 1m38s
CI / typecheck (pull_request) Failing after 2m8s
CI / quality (pull_request) Successful in 2m38s
CI / security (pull_request) Successful in 3m18s
CI / integration_tests (pull_request) Failing after 7m43s
CI / unit_tests (pull_request) Failing after 10m31s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 2s
The InvariantModel class defined a Column named "text" which shadowed the sqlalchemy.text() function used in server_default, postgresql_where, and sqlite_where expressions. Renamed the import from "text" to "sa_text" and updated all 4 usages. ISSUES CLOSED: #8573
This commit is contained in:
@@ -61,7 +61,7 @@ from sqlalchemy import (
|
|||||||
Text,
|
Text,
|
||||||
UniqueConstraint,
|
UniqueConstraint,
|
||||||
create_engine,
|
create_engine,
|
||||||
text,
|
text as sa_text,
|
||||||
)
|
)
|
||||||
from sqlalchemy.orm import (
|
from sqlalchemy.orm import (
|
||||||
Mapped,
|
Mapped,
|
||||||
@@ -1774,7 +1774,7 @@ class ResourceLinkModel(Base): # type: ignore[misc]
|
|||||||
Text,
|
Text,
|
||||||
nullable=False,
|
nullable=False,
|
||||||
default="contains",
|
default="contains",
|
||||||
server_default=text("'contains'"),
|
server_default=sa_text("'contains'"),
|
||||||
)
|
)
|
||||||
|
|
||||||
# Timestamp (ISO-8601 string)
|
# Timestamp (ISO-8601 string)
|
||||||
@@ -2843,8 +2843,8 @@ class DecisionModel(Base): # type: ignore[misc]
|
|||||||
Index(
|
Index(
|
||||||
"idx_decisions_superseded",
|
"idx_decisions_superseded",
|
||||||
"superseded_by",
|
"superseded_by",
|
||||||
postgresql_where=text("superseded_by IS NOT NULL"),
|
postgresql_where=sa_text("superseded_by IS NOT NULL"),
|
||||||
sqlite_where=text("superseded_by IS NOT NULL"),
|
sqlite_where=sa_text("superseded_by IS NOT NULL"),
|
||||||
),
|
),
|
||||||
)
|
)
|
||||||
|
|
||||||
@@ -3270,7 +3270,7 @@ class CorrectionAttemptModel(Base): # type: ignore[misc]
|
|||||||
created_at = Column(
|
created_at = Column(
|
||||||
String(30),
|
String(30),
|
||||||
nullable=False,
|
nullable=False,
|
||||||
server_default=text("(strftime('%Y-%m-%dT%H:%M:%f', 'now'))"),
|
server_default=sa_text("(strftime('%Y-%m-%dT%H:%M:%f', 'now'))"),
|
||||||
)
|
)
|
||||||
completed_at = Column(String(30), nullable=True)
|
completed_at = Column(String(30), nullable=True)
|
||||||
|
|
||||||
@@ -3428,11 +3428,11 @@ class InvariantModel(Base): # type: ignore[misc]
|
|||||||
source_name = Column(String(255), nullable=False)
|
source_name = Column(String(255), nullable=False)
|
||||||
|
|
||||||
# Soft-delete flag (active vs archived)
|
# Soft-delete flag (active vs archived)
|
||||||
active = Column(Boolean, nullable=False, default=True, server_default=text("1"))
|
active = Column(Boolean, nullable=False, default=True, server_default=sa_text("1"))
|
||||||
|
|
||||||
# Whether this invariant cannot be overridden by project-level invariants
|
# Whether this invariant cannot be overridden by project-level invariants
|
||||||
non_overridable = Column(
|
non_overridable = Column(
|
||||||
Boolean, nullable=False, default=False, server_default=text("0"),
|
Boolean, nullable=False, default=False, server_default=sa_text("0"),
|
||||||
)
|
)
|
||||||
|
|
||||||
# Timestamp (ISO-8601 string)
|
# Timestamp (ISO-8601 string)
|
||||||
|
|||||||
Reference in New Issue
Block a user