Comments on: group_required decorator

By: User links about "decorator" on iLinkShare

User links about "decorator" on iLinkShare — Sat, 04 Apr 2009 15:18:06 +0000

[...] ago2 votesDecorator Pattern – Decorating among Decorators>> saved by uberwald 16 days ago3 votesgroup_required decorator>> saved by fatalscorpion 32 days ago6 votesDesign Patterns – Decorator Pattern>> saved by ferlachat [...]

By: Recent URLs tagged Decorator - Urlrecorder

Recent URLs tagged Decorator - Urlrecorder — Thu, 30 Oct 2008 05:01:30 +0000

[...] Recent public urls tagged “decorator” → group_required decorator [...]

By: mattd

mattd — Fri, 20 Jun 2008 23:27:58 +0000

Makes sense to me good call.

By: michiel_1981

michiel_1981 — Fri, 20 Jun 2008 19:54:41 +0000

Hi,

To make it less error prone move the return view line in a else statement.

try:
getting group
except:
return redirect
else:
return view

This way if a Group.DoesNotExist is raised in your view function it won’t be handled by the decorator.
Which most likely shouldn’t redirect.

- Michiel