fix up version numbers

sourcing/model.py

@@ -162,7 +162,7 @@ class Item(TimeStampedModel):
         return url_for('.view_item',
                        username=self.user.username,
                        hashid=self.hashid,
-                       v=version.transaction_id)
+                       v=version)
 
     @property
     def history_url(self):

sourcing/templates/history.html

@@ -28,7 +28,7 @@
 
 <ul>
   {% for v in doc.versions %}
-    <li><a href="{{ doc.version_url(v) }}">{{ v.modified.strftime('%H:%M:%S, %d %B %Y') }}</a></li>
+    <li><a href="{{ doc.version_url(loop.index) }}">{{ v.modified.strftime('%H:%M:%S, %d %B %Y') }}</a></li>
   {% endfor %}
 </ul>
 

sourcing/view.py

@@ -213,11 +213,14 @@ def view_item(username, hashid, raw=False):
     if raw:
         return Response(item.text, mimetype='text/plain')
 
-    if 'v' in request.args and request.args['v'].isdigit():
+    v = request.args.get('v')
-        ItemVersion = version_class(Item)
+    if v:
-        version = (session.query(ItemVersion)
+        if not v.isdigit():
-                          .filter_by(transaction_id=int(request.args['v']))
+            abort(404)
-                          .first())
+        try:
+            version = item.versions[int(v) - 1]
+        except IndexError:
+            abort(404)
         text = version.text
     else:
         version = None