diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-04-16 17:01:50 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2010-04-16 17:01:50 -0400 |
| commit | 3bd09e291b8c9920b043f41438c636bf5bbaeef3 (patch) | |
| tree | f75b00c5f9ffbb6183a6e74aa996638048c970f1 /lib/sqlalchemy | |
| parent | b437bb213221e98fd5909a9082df8b4b6a25fc3f (diff) | |
| download | sqlalchemy-3bd09e291b8c9920b043f41438c636bf5bbaeef3.tar.gz | |
when raising about columns not present, include the name of the property it's trying to configure,
to eliminate confusion over attributes not properly placed inside of a relationship().
Diffstat (limited to 'lib/sqlalchemy')
| -rw-r--r-- | lib/sqlalchemy/orm/mapper.py | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/lib/sqlalchemy/orm/mapper.py b/lib/sqlalchemy/orm/mapper.py index 673e16b29..7a4230099 100644 --- a/lib/sqlalchemy/orm/mapper.py +++ b/lib/sqlalchemy/orm/mapper.py @@ -560,18 +560,21 @@ class Mapper(object): self.mapped_table._reset_exported() mc = self.mapped_table.corresponding_column(c) if mc is None: - raise sa_exc.ArgumentError("Column '%s' is not represented in mapper's table. " + raise sa_exc.ArgumentError("When configuring property '%s' on %s, " + "column '%s' is not represented in the mapper's table. " "Use the `column_property()` function to force this column " - "to be mapped as a read-only attribute." % c) + "to be mapped as a read-only attribute." % (key, self, c)) mapped_column.append(mc) prop = ColumnProperty(*mapped_column) else: - raise sa_exc.ArgumentError("WARNING: column '%s' conflicts with property '%r'. " + raise sa_exc.ArgumentError("WARNING: when configuring property '%s' on %s, column '%s' " + "conflicts with property '%r'. " "To resolve this, map the column to the class under a different " "name in the 'properties' dictionary. Or, to remove all awareness " "of the column entirely (including its availability as a foreign key), " "use the 'include_properties' or 'exclude_properties' mapper arguments " - "to control specifically which table columns get mapped." % (column.key, prop)) + "to control specifically which table columns get mapped." % + (key, self, column.key, prop)) if isinstance(prop, ColumnProperty): col = self.mapped_table.corresponding_column(prop.columns[0]) |