diff options
author | Mike Bayer <mike_mp@zzzcomputing.com> | 2006-01-02 05:28:05 +0000 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2006-01-02 05:28:05 +0000 |
commit | 0f7484094cfa7077e90865acdb5cad175608f567 (patch) | |
tree | 34a891d67059ca0f0bf42b0895e36525b4b861bc /lib/sqlalchemy/sql.py | |
parent | 6980730e7d3f962768f44a1d5f5a7013d048cf85 (diff) | |
download | sqlalchemy-0f7484094cfa7077e90865acdb5cad175608f567.tar.gz |
order by oid -> order by *default_order_by() when called by mapper layer.
- one layer of abstraction between the "oid" and default ordering
Diffstat (limited to 'lib/sqlalchemy/sql.py')
-rw-r--r-- | lib/sqlalchemy/sql.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/sqlalchemy/sql.py b/lib/sqlalchemy/sql.py index 25b73c776..3012708fe 100644 --- a/lib/sqlalchemy/sql.py +++ b/lib/sqlalchemy/sql.py @@ -484,6 +484,11 @@ class FromClause(Selectable): def _get_from_objects(self): # this could also be [self], at the moment it doesnt matter to the Select object return [] + def default_order_by(self): + if not self.engine.default_ordering: + return None + else: + return [self.oid_column] def hash_key(self): return "FromClause(%s, %s)" % (repr(self.id), repr(self.from_name)) def accept_visitor(self, visitor): @@ -906,9 +911,6 @@ class TableImpl(FromClause): self.id = self.table.name def _oid_col(self): - if not self.table.engine.default_ordering: - return None - if not hasattr(self, '_oid_column'): if self.table.engine.oid_column_name() is not None: self._oid_column = schema.Column(self.table.engine.oid_column_name(), sqltypes.Integer, hidden=True) |