From 88ca587eae19afb8e069d896b95580aaed8b0e24 Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Sat, 12 Nov 2016 12:34:01 -0500 Subject: Count columns using PrimaryKeyConstraint.__len__ directly PrimaryKeyConstraint is present on Table however on table() and others it's a ColumnSet. The warning here only needs len() and PrimaryKeyConstraint supports that directly in the same way as ColumnSet. Change-Id: I19c11a39110bfef48cdea49a471e7ab80b537538 Fixes: #3842 --- lib/sqlalchemy/sql/crud.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib/sqlalchemy/sql') diff --git a/lib/sqlalchemy/sql/crud.py b/lib/sqlalchemy/sql/crud.py index 452fe5d9a..9d10fbefc 100644 --- a/lib/sqlalchemy/sql/crud.py +++ b/lib/sqlalchemy/sql/crud.py @@ -681,7 +681,7 @@ def _warn_pk_with_no_anticipated_value(c): "Primary key columns typically may not store NULL." % (c.table.fullname, c.name, c.table.fullname)) - if len(c.table.primary_key.columns) > 1: + if len(c.table.primary_key) > 1: msg += ( " Note that as of SQLAlchemy 1.1, 'autoincrement=True' must be " "indicated explicitly for composite (e.g. multicolumn) primary " -- cgit v1.2.1