Small CL to change the way format is used for package versions.

app/handlers/api/package_versions.py

 # Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 from cStringIO import StringIO
 from contextlib import closing
 from uuid import uuid4
 import json
 import logging
 import time
@@ skipping 146 matching lines @@
         with files.open(write_path, 'a') as f:
             f.write(file.file.read())
         files.finalize(write_path)
 
         if success_action_redirect:
             raise cherrypy.HTTPRedirect(success_action_redirect)
         cherrypy.response.status = 204
         return ""
 
     @handlers.api(2)
-    def show(self, package_id, id, format):
+    def show(self, package_id, id, format=None):

[Bob Nystrom, 2013/12/18 19:24:01]

What does it mean for format to be omitted? Default to JSON? Document this.

         """Retrieve the document describing a package version."""
         # The mapper thinks the final version digit is the format.
-        id = id + '.' + format
+        if format: id = id + '.' + format
         return json.dumps(
             handlers.request().package_version(id).as_dict(full=True))
 
     @handlers.api(1)
     @handlers.requires_private_key
     @handlers.requires_uploader
     def new_dartdoc(self, package_id, id):
         """Retrieve the form for uploading dartdoc for this package version."""
         version = handlers.request().package_version(id)
         upload = cloud_storage.Upload(version.dartdoc_storage_path,
                                       acl='public-read',
                                       size_range=(0, Package.MAX_SIZE))
 
         return upload.to_json()