On Saturday 27 June 2009 15:03:05 Kenneth Gonsalves wrote:
> On Saturday 27 June 2009 13:49:28 Brian May wrote:
> > On Sat, Jun 27, 2009 at 01:17:42PM +0530, Kenneth Gonsalves wrote:
> > > decorator---------------------------
> > >
> > > class menu_enabled(object):
> > >     def __init__(self,menuname):
> > >         self.menuname = menuname
> > >
> > >     def __call__(self,f):
> > >         def wrapper(*args,**kwargs):
> > >             try:
> > >                 mn = Menuitem.objects.get(name__iexact = self.menuname)
> > >                 f(*args,**kwargs)
> > >             except:
> > >                 msg = _("You are already registered")
> > >                 return HttpResponseRedirect("/2009/message/%s/" % msg)
> > >
> > >         return wrapper
> > > ------end of decorator
> > >
> > > obviously this f(*args,**kwargs) is not the correct way to call the
> > > view. Any clues?
> >
> > I have never written a decorator, so I may be completely off here,
> > however shouldn't that be "return f(*args,**kwargs)"? ie. don't you need
> > to return the value?
>
> Well this is the first time I am writing a decorator for use! and no - this
> works in python, but in django there is some specific syntax for calling a
> view function

spoke too soon - you are right and it works now - thanks.
-- 
regards
kg
http://lawgon.livejournal.com

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Django users" group.
To post to this group, send email to django-users@googlegroups.com
To unsubscribe from this group, send email to 
django-users+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/django-users?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to