Thread: [Sqlalchemy-tickets] [sqlalchemy] #2168: clean out column anonymous labeling rules
Brought to you by:
zzzeek
From: sqlalchemy <mi...@zz...> - 2011-05-18 15:52:57
|
#2168: clean out column anonymous labeling rules --------------------+------------------------------------------------------- Reporter: zzzeek | Owner: zzzeek Type: defect | Status: new Priority: medium | Milestone: 0.8.0 Component: sql | Severity: major - 1-3 hours Keywords: | Status_field: awaiting triage --------------------+------------------------------------------------------- I.e. we want to remove: {{{ #!python def label(self, name): if name is None: return self else: return super(ColumnClause, self).label(name) }}} from ColumnClause, so that: {{{ #!python c1 = column('x') assert c1.label(None) is not c1 }}} works as expected, instead of returning self, which doesn't make any sense. The logic within `_Label` should be enhanced to be smarter about when it chooses `name` and when it doesn't. Ideally the same result should always occur for `label(c, None)` versus `c.label(None)`. -- Ticket URL: <http://www.sqlalchemy.org/trac/ticket/2168> sqlalchemy <http://www.sqlalchemy.org/> The Database Toolkit for Python |
From: sqlalchemy <mi...@zz...> - 2012-02-27 00:16:24
|
#2168: clean out column anonymous labeling rules -------------------------------+------------------------------- Reporter: zzzeek | Owner: zzzeek Type: defect | Status: new Priority: medium | Milestone: 0.8.0 Component: sql | Severity: major - 1-3 hours Resolution: | Keywords: Progress State: needs review | -------------------------------+------------------------------- Changes (by zzzeek): * status_field: awaiting triage => needs review -- Ticket URL: <http://www.sqlalchemy.org/trac/ticket/2168#comment:1> sqlalchemy <http://www.sqlalchemy.org/> The Database Toolkit for Python |
From: sqlalchemy <mi...@zz...> - 2012-04-24 17:05:07
|
#2168: clean out column anonymous labeling rules -------------------------------+------------------------------- Reporter: zzzeek | Owner: zzzeek Type: defect | Status: new Priority: medium | Milestone: 0.8.0 Component: sql | Severity: major - 1-3 hours Resolution: | Keywords: Progress State: needs review | -------------------------------+------------------------------- Comment (by zzzeek): r43fe92273d3b in the 0.8 branch -- Ticket URL: <http://www.sqlalchemy.org/trac/ticket/2168#comment:2> sqlalchemy <http://www.sqlalchemy.org/> The Database Toolkit for Python |
From: sqlalchemy <mi...@zz...> - 2012-04-24 23:43:06
|
#2168: clean out column anonymous labeling rules -----------------------------------+------------------------------- Reporter: zzzeek | Owner: zzzeek Type: defect | Status: closed Priority: medium | Milestone: 0.8.0 Component: sql | Severity: major - 1-3 hours Resolution: fixed | Keywords: Progress State: completed/closed | -----------------------------------+------------------------------- Changes (by zzzeek): * status: new => closed * resolution: => fixed * status_field: needs review => completed/closed -- Ticket URL: <http://www.sqlalchemy.org/trac/ticket/2168#comment:3> sqlalchemy <http://www.sqlalchemy.org/> The Database Toolkit for Python |