Kevin Dangoor <[EMAIL PROTECTED]> writes:

> Hmm... I'm not 100% sure that this will fix 503, but it might.
>
> I just committed this patch (manually applied). Please check it to
> make sure my application of it works.

It did here.  At least I'm still getting the error I reported on 498, with the
following traceback:

Traceback (most recent call last):
  File 
"/usr/lib/python2.4/site-packages/CherryPy-2.2.0beta-py2.4.egg/cherrypy/_cphttptools.py",
 line 99, in _run
    self.main()
  File 
"/usr/lib/python2.4/site-packages/CherryPy-2.2.0beta-py2.4.egg/cherrypy/_cphttptools.py",
 line 247, in main
    body = page_handler(*virtual_path, **self.params)
  File "<string>", line 3, in index
  File 
"/home/godoy/desenvolvimento/python/TurboGears/trunk/turbogears/controllers.py",
 line 215, in expose
    func, tg_format, html, fragment, **kw)
  File 
"/home/godoy/desenvolvimento/python/TurboGears/trunk/turbogears/database.py", 
line 193, in run_with_transaction
    retval = func(*args, **kw)
  File 
"/home/godoy/desenvolvimento/python/TurboGears/trunk/turbogears/controllers.py",
 line 243, in _execute_func
    output = dispatch_error(func, self, **kw)
  File "<string>", line 5, in dispatch_error
  File 
"/home/godoy/desenvolvimento/python/TurboGears/trunk/turbogears/controllers.py",
 line 295, in _default_error_handler
    return getattr(self, error_source.__name__ )(**kw)
  File "<string>", line 3, in index
  File 
"/home/godoy/desenvolvimento/python/TurboGears/trunk/turbogears/controllers.py",
 line 211, in expose
    output = _execute_func(self, func, tg_format, html, fragment, **kw)
  File 
"/home/godoy/desenvolvimento/python/TurboGears/trunk/turbogears/controllers.py",
 line 240, in _execute_func
    raise error
TypeError: unsupported operand type(s) for +: 'lazystring' and 'str'

> By the way, I just opened ticket 504 to get unit tests for identity.
> It would be nice to have some tests that exercise at least some of
> Identity's features to catch this kind of thing.
>
> http://trac.turbogears.org/turbogears/ticket/504#preview

Indeed, including testing it integrated with different decorators and in
different orders of decorators, to be sure that 503 doesn't happen.

-- 
Jorge Godoy      <[EMAIL PROTECTED]>

Reply via email to