Remove legacy Package.uploaders PackageVersion.{uploader,readme,changelog} properties

app/handlers/api/package_uploaders.py

Index: app/handlers/api/package_uploaders.py
diff --git a/app/handlers/api/package_uploaders.py b/app/handlers/api/package_uploaders.py
index 1dd9dc0de44a375c3958b9f4b217dd7d1fc5dd18..2f0ea02b3eb0676a0244e07dfeb292c68dbe3e60 100644
--- a/app/handlers/api/package_uploaders.py
+++ b/app/handlers/api/package_uploaders.py
@@ -20,14 +20,12 @@ class PackageUploaders(object):
         Only other uploaders may add new uploaders."""
 
         package = handlers.request().package
-        user_to_add = users.User(email)
-        if package.has_uploader(user_to_add):
+        if package.has_uploader_email(email):
             handlers.http_error(
                 400, "User '%s' is already an uploader for package '%s'." %
                          (email, package.name))
 
-        package.uploaders.append(user_to_add)
-        package.temp_synchronize_uploaders_to_uploaderemails()
+        package.uploaderEmails.append(email)
         package.put()
         package.invalidate_cache()
         return handlers.json_success(
@@ -44,26 +42,27 @@ class PackageUploaders(object):
         uploader may not be deleted until a new one is added.
         """
 
+        # TODO: WHAT IS THIS `format` THING ?
         if format: id = id + '.' + format
 
         package = handlers.request().package
-        user_to_delete = users.User(id)
-        if not package.has_uploader(user_to_delete):
+        email = id
+        if not package.has_uploader_email(email):
             handlers.http_error(
                 400, "'%s' isn't an uploader for package '%s'." %
-                         (user_to_delete.nickname(), package.name))
+                         (email, package.name))
 
-        if len(package.uploaders) == 1:
+        if len(package.uploaderEmails) == 1:
             handlers.http_error(
                 400, ("Package '%s' only has one uploader, so that uploader " +
-                          "can't be removed.") % package.name)
+                      "can't be removed.") % package.name)
 
-        email_to_delete = user_to_delete.email().lower()
-        package.uploaders = [uploader for uploader in package.uploaders
-                             if uploader.email().lower() != email_to_delete]
-        package.temp_synchronize_uploaders_to_uploaderemails()
+        email_to_delete = email.lower()
+        package.uploaderEmails = [email for email in package.uploaderEmails
+                                  if email.lower() != email_to_delete]
         package.put()
         package.invalidate_cache()
         return handlers.json_success(
             "'%s' is no longer an uploader for package '%s'." %
                 (id, package.name))
+