Hi,

On 18/01/12 22:18, Christian Boos wrote:
On 1/18/2012 11:13 PM, Olemis Lang wrote:
Hi all !

My comment posted to #9580 @ t.h.o [1]_ makes me wonder whether it is
convenient to reopen #9418 @ t.e.o
I'll try to be more precise and add further details later , but I just
wanted to know if this is better discussed here in the ML or in the
ticket itself (i.e. #9418 @ t.e.o.)

Thnx in advance !

.. [1] Recursion after installing in Trac 0.13 (dev)<= ThemeEnginePlugin
         (http://trac-hacks.org/ticket/9580#comment:1)



I don't see the recursion... care to post a traceback (here)?

-- Christian


That is interesting. In tracking down the Trac code changes associated with the recursion error, I did not consider the possibility that there might be justification for the component containing itself through that method. I believe that the patch I suggested has a bug in the case of the selected themes being disabled.

Anyway, as Olemis has not replied yet and I have just spotted this thread, I thought I would try to replicate the appropriate traceback for the recursion on installation of the unpatched plugin. Sorry I neglected to put this in the trac-hacks ticket that Olemis referenced (traceback replicated on r10876 from trunk):

Traceback (most recent call last):
  File "/home/trac/projects/trac/trac-trunk/trac/web/api.py", line 498, in 
send_error
    data, 'text/html')
  File "/home/trac/projects/trac/trac-trunk/trac/web/chrome.py", line 982, in 
render_template
    template = self.load_template(filename, method=method)
  File "/home/trac/projects/trac/trac-trunk/trac/web/chrome.py", line 942, in 
load_template
    self.get_all_templates_dirs(), auto_reload=self.auto_reload,
  File "/home/trac/projects/trac/trac-trunk/trac/web/chrome.py", line 653, in 
get_all_templates_dirs
    for provider in self.template_providers:
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 78, in 
extensions
    components = [component.compmgr[cls] for cls in classes]
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 199, in 
__getitem__
    component = cls(self)
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 138, in __call__
    self.__init__()
  File "build/bdist.linux-x86_64/egg/themeengine/admin.py", line 21, in __init__
    self.system = ThemeEngineSystem(self.env)
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 138, in __call__
    self.__init__()
  File "build/bdist.linux-x86_64/egg/themeengine/api.py", line 65, in __init__
    for provider in self.providers:

[snip]

  File "build/bdist.linux-x86_64/egg/themeengine/api.py", line 65, in __init__
    for provider in self.providers:
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 78, in 
extensions
    components = [component.compmgr[cls] for cls in classes]
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 199, in 
__getitem__
    component = cls(self)
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 138, in __call__
    self.__init__()
  File "build/bdist.linux-x86_64/egg/themeengine/api.py", line 65, in __init__
    for provider in self.providers:
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 78, in 
extensions
    components = [component.compmgr[cls] for cls in classes]
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 199, in 
__getitem__
    component = cls(self)
  File "/home/trac/projects/trac/trac-trunk/trac/core.py", line 138, in __call__
    self.__init__()
  File "build/bdist.linux-x86_64/egg/themeengine/api.py", line 65, in __init__
    for provider in self.providers:
RuntimeError: maximum recursion depth exceeded while calling a Python object


Thanks for taking an interest :)

Cheers,
    Gary

--
Best wishes,

Gary Martin
Lead Developer
WANdisco, Inc.

http://www.wandisco.com

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

Reply via email to