Commit 0aa2b6eb authored by Nitish Rathi's avatar Nitish Rathi

warn on multi-column properties - fixes #575

Show warning instead of raising on multiple-column properties
parent 7bca6d9e
import logging import logging
import warnings
from sqlalchemy.orm.attributes import InstrumentedAttribute from sqlalchemy.orm.attributes import InstrumentedAttribute
from sqlalchemy.orm import joinedload from sqlalchemy.orm import joinedload
...@@ -391,8 +392,7 @@ class ModelView(BaseModelView): ...@@ -391,8 +392,7 @@ class ModelView(BaseModelView):
filtered = tools.filter_foreign_columns(self.model.__table__, p.columns) filtered = tools.filter_foreign_columns(self.model.__table__, p.columns)
if len(filtered) > 1: if len(filtered) > 1:
# TODO: Skip column and issue a warning warnings.warn('Can not convert multiple-column properties (%s.%s)' % (self.model, p.key))
raise TypeError('Can not convert multiple-column properties (%s.%s)' % (self.model, p.key))
column = filtered[0] column = filtered[0]
else: else:
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment