Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2494)

Unified Diff: app/handlers/__init__.py

Issue 815973002: Remove legacy Package.uploaders PackageVersion.{uploader,readme,changelog} properties (Closed) Base URL: https://github.com/dart-lang/pub-dartlang.git@master
Patch Set: Small fix in changelog/readme handling for new uploads Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | app/handlers/api/package_uploaders.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: app/handlers/__init__.py
diff --git a/app/handlers/__init__.py b/app/handlers/__init__.py
index 1101c0a680d1f5fc02a68cc9ab57a52626a78b4b..0b6a27bf63b4a2155f6999d7008586faa0b8e56e 100644
--- a/app/handlers/__init__.py
+++ b/app/handlers/__init__.py
@@ -238,7 +238,7 @@ def requires_uploader(fn, *args, **kwargs):
if is_current_user_admin(): return fn(*args, **kwargs)
package = request().maybe_package
- if package and not package.has_uploader(get_current_user()):
+ if package and not package.has_uploader_email(get_current_user().email()):
message = "You aren't an uploader for package '%s'." % package.name
if request().is_json:
http_error(403, message)
« no previous file with comments | « no previous file | app/handlers/api/package_uploaders.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698