diff options
| author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-12-24 00:08:28 +0100 |
|---|---|---|
| committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2011-12-27 20:41:31 +0100 |
| commit | 05751758469a03835975dd2998aa727fa29c9a16 (patch) | |
| tree | f101ef46aa0d6441232335f678415ad1555b7f45 /mediagoblin/db/mongo | |
| parent | 0eb649ff7ac3f1eb71eb1d2cb66019a860b2c5c7 (diff) | |
| download | mediagoblin-05751758469a03835975dd2998aa727fa29c9a16.tar.lz mediagoblin-05751758469a03835975dd2998aa727fa29c9a16.tar.xz mediagoblin-05751758469a03835975dd2998aa727fa29c9a16.zip | |
Turn media.get_uploader into a property
sqlalchemy gives autoloading (hopefully caching) link to
other objects as properties. So turn get_uploader on the
current mongo based stuff into a property to ease
transition.
Diffstat (limited to 'mediagoblin/db/mongo')
| -rw-r--r-- | mediagoblin/db/mongo/models.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/mediagoblin/db/mongo/models.py b/mediagoblin/db/mongo/models.py index 8cd0da1b..5de59c12 100644 --- a/mediagoblin/db/mongo/models.py +++ b/mediagoblin/db/mongo/models.py @@ -261,7 +261,7 @@ class MediaEntry(Document): Use a slug if we have one, else use our '_id'. """ - uploader = self.get_uploader() + uploader = self.get_uploader if self.get('slug'): return urlgen( @@ -297,6 +297,7 @@ class MediaEntry(Document): for media in cursor: return media.url_for_self(urlgen) + @property def get_uploader(self): return self.db.User.find_one({'_id': self.uploader}) |