diff options
author | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2012-12-23 11:58:51 +0100 |
---|---|---|
committer | Sebastian Spaeth <Sebastian@SSpaeth.de> | 2012-12-23 11:58:51 +0100 |
commit | cfa922295e5ddfaab336a3c2c0403422f77758b6 (patch) | |
tree | f24dcebc6214e2804646c04ff801886b8a11c4b5 /mediagoblin/admin/views.py | |
parent | 785b287fcb42ac9130b222006097e3f68cec3543 (diff) | |
download | mediagoblin-cfa922295e5ddfaab336a3c2c0403422f77758b6.tar.lz mediagoblin-cfa922295e5ddfaab336a3c2c0403422f77758b6.tar.xz mediagoblin-cfa922295e5ddfaab336a3c2c0403422f77758b6.zip |
Convert return HttpException to raise HttpException
controllers (view function) raise HttpException's and do not return them.
Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
Diffstat (limited to 'mediagoblin/admin/views.py')
-rw-r--r-- | mediagoblin/admin/views.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/mediagoblin/admin/views.py b/mediagoblin/admin/views.py index 9c14c55c..d0665151 100644 --- a/mediagoblin/admin/views.py +++ b/mediagoblin/admin/views.py @@ -14,10 +14,11 @@ # You should have received a copy of the GNU Affero General Public License # along with this program. If not, see <http://www.gnu.org/licenses/>. +from werkzeug.exceptions import Forbidden + from mediagoblin.db.util import DESCENDING from mediagoblin.decorators import require_active_login -from mediagoblin.tools.response import (render_to_response, render_403, - render_404) +from mediagoblin.tools.response import render_to_response @require_active_login def admin_processing_panel(request): @@ -26,7 +27,7 @@ def admin_processing_panel(request): ''' # TODO: Why not a "require_admin_login" decorator throwing a 403 exception? if not request.user.is_admin: - return render_403(request) + raise Forbidden() processing_entries = request.db.MediaEntry.find( {'state': u'processing'}).sort('created', DESCENDING) |