diff options
| author | mike bayer <mike_mp@zzzcomputing.com> | 2021-04-27 02:03:21 +0000 |
|---|---|---|
| committer | Gerrit Code Review <gerrit@ci3.zzzcomputing.com> | 2021-04-27 02:03:21 +0000 |
| commit | 13d01a03c55890e19f5721405f6e5fb45d37cac4 (patch) | |
| tree | aa1f8aedfd6bd9da1d150d3e9efde3b359a7ce82 /lib/sqlalchemy | |
| parent | 3edb963fc185d4a17a7a98268a7c251c5454a18c (diff) | |
| parent | 802923062720589c0aea0d56a8672407be2cb79c (diff) | |
| download | sqlalchemy-13d01a03c55890e19f5721405f6e5fb45d37cac4.tar.gz | |
Merge "Pass all datetime values to pyodbc for timezone-naive column"
Diffstat (limited to 'lib/sqlalchemy')
| -rw-r--r-- | lib/sqlalchemy/dialects/mssql/pyodbc.py | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/lib/sqlalchemy/dialects/mssql/pyodbc.py b/lib/sqlalchemy/dialects/mssql/pyodbc.py index 424adadfc..95b2bb48e 100644 --- a/lib/sqlalchemy/dialects/mssql/pyodbc.py +++ b/lib/sqlalchemy/dialects/mssql/pyodbc.py @@ -353,6 +353,8 @@ class _ms_binary_pyodbc(object): class _ODBCDateTime(sqltypes.DateTime): """Add bind processors to handle datetimeoffset behaviors""" + has_tz = False + def bind_processor(self, dialect): def process(value): if value is None: @@ -360,7 +362,7 @@ class _ODBCDateTime(sqltypes.DateTime): elif isinstance(value, util.string_types): # if a string was passed directly, allow it through return value - elif isinstance(value, datetime.datetime) and value.tzinfo is None: + elif not value.tzinfo or (not self.timezone and not self.has_tz): # for DateTime(timezone=False) return value else: @@ -379,6 +381,10 @@ class _ODBCDateTime(sqltypes.DateTime): return process +class _ODBCDATETIMEOFFSET(_ODBCDateTime): + has_tz = True + + class _VARBINARY_pyodbc(_ms_binary_pyodbc, VARBINARY): pass @@ -453,7 +459,7 @@ class MSDialect_pyodbc(PyODBCConnector, MSDialect): BINARY: _BINARY_pyodbc, # support DateTime(timezone=True) sqltypes.DateTime: _ODBCDateTime, - DATETIMEOFFSET: _ODBCDateTime, + DATETIMEOFFSET: _ODBCDATETIMEOFFSET, # SQL Server dialect has a VARBINARY that is just to support # "deprecate_large_types" w/ VARBINARY(max), but also we must # handle the usual SQL standard VARBINARY |
