diff options
author | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-04-27 14:52:08 +0200 |
---|---|---|
committer | Elrond <elrond+mediagoblin.org@samba-tng.org> | 2013-04-27 15:08:07 +0200 |
commit | 2041ceae1ffd29630988c47fc766a1b9dcf9bbf7 (patch) | |
tree | e9eaf88e658e91ac9164e21be8494fbca4374602 /mediagoblin/submit/views.py | |
parent | 90e7fc673878d4eb68db41ae49c47fb543a35a87 (diff) | |
download | mediagoblin-2041ceae1ffd29630988c47fc766a1b9dcf9bbf7.tar.lz mediagoblin-2041ceae1ffd29630988c47fc766a1b9dcf9bbf7.tar.xz mediagoblin-2041ceae1ffd29630988c47fc766a1b9dcf9bbf7.zip |
Fix translations for collections and drop useless try.
Don't do: _("With some value: %s" % value)
Please do: _("WIth some value: %s") % value
Fixed for collection messages.
Also removed a
try:
some_code.
except Exception as e:
raise
No point in doing that.
Fixing the indentation of some_code comes in an extra
commit, because changing indentation is annoying enough
alone, so don't mix it with other changes.
Diffstat (limited to 'mediagoblin/submit/views.py')
-rw-r--r-- | mediagoblin/submit/views.py | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/mediagoblin/submit/views.py b/mediagoblin/submit/views.py index 9d31c844..c9735fd3 100644 --- a/mediagoblin/submit/views.py +++ b/mediagoblin/submit/views.py @@ -114,6 +114,7 @@ def submit_start(request): {'submit_form': submit_form, 'app_config': mg_globals.app_config}) + @require_active_login def add_collection(request, media=None): """ @@ -122,7 +123,6 @@ def add_collection(request, media=None): submit_form = submit_forms.AddCollectionForm(request.form) if request.method == 'POST' and submit_form.validate(): - try: collection = request.db.Collection() collection.title = unicode(submit_form.title.data) @@ -136,19 +136,18 @@ def add_collection(request, media=None): 'title':collection.title}) if existing_collection: - messages.add_message( - request, messages.ERROR, _('You already have a collection called "%s"!' % collection.title)) + add_message(request, messages.ERROR, + _('You already have a collection called "%s"!') \ + % collection.title) else: collection.save() - add_message(request, SUCCESS, _('Collection "%s" added!' % collection.title)) + add_message(request, SUCCESS, + _('Collection "%s" added!') % collection.title) return redirect(request, "mediagoblin.user_pages.user_home", user=request.user.username) - except Exception as e: - raise - return render_to_response( request, 'mediagoblin/submit/collection.html', |