From 00380cf3c304ea7f1c7397f17b828166364c36ac Mon Sep 17 00:00:00 2001 From: Mike Bayer Date: Wed, 18 May 2011 12:07:40 -0400 Subject: - Fixed bug whereby nesting a label of a select() with another label in it would produce incorrect exported columns. Among other things this would break an ORM column_property() mapping against another column_property(). [ticket:2167]. Also in 0.6.8 - _Label() is always against a column or selectable. remove uncovered case of label against something else. - start taking notes to clean up some of this labeling stuff, which will be [ticket:2168] --- lib/sqlalchemy/sql/expression.py | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) (limited to 'lib/sqlalchemy/sql/expression.py') diff --git a/lib/sqlalchemy/sql/expression.py b/lib/sqlalchemy/sql/expression.py index 1013f9397..e06eb61be 100644 --- a/lib/sqlalchemy/sql/expression.py +++ b/lib/sqlalchemy/sql/expression.py @@ -3814,11 +3814,7 @@ class _Label(ColumnElement): return self.element._from_objects def _make_proxy(self, selectable, name = None): - if isinstance(self.element, (Selectable, ColumnElement)): - e = self.element._make_proxy(selectable, name=self.name) - else: - e = column(self.name)._make_proxy(selectable=selectable) - + e = self.element._make_proxy(selectable, name=name or self.name) e.proxies.append(self) return e @@ -3934,6 +3930,12 @@ class ColumnClause(_Immutable, ColumnElement): return self.name def label(self, name): + # currently, anonymous labels don't occur for + # ColumnClause. The use at the moment + # is that they do not generate nicely for + # is_literal clauses. We would like to change + # this so that label(None) acts as would be expected. + # See [ticket:2168]. if name is None: return self else: -- cgit v1.2.1