Commit ec6473c5 authored by Julien Muchembled's avatar Julien Muchembled

Add comment about recent workaround for circular import dependency

See commit a93ea5f1.
parent d4cde62e
Pipeline #6655 passed with stage
in 0 seconds
...@@ -603,6 +603,9 @@ def cancelProcessShutdown(): ...@@ -603,6 +603,9 @@ def cancelProcessShutdown():
is_running_lock.release() is_running_lock.release()
has_processed_shutdown = False has_processed_shutdown = False
# Due to a circular import dependency between this module and
# Products.ERP5Type.Core.Folder, both modules must import after the definitions
# of getCurrentNode and Folder (the later is a base class of BaseTool).
from Products.ERP5Type.Tool.BaseTool import BaseTool from Products.ERP5Type.Tool.BaseTool import BaseTool
# Activating a path means we tried to avoid loading useless # Activating a path means we tried to avoid loading useless
# data in cache so there would be no gain to expect. # data in cache so there would be no gain to expect.
......
...@@ -1665,6 +1665,7 @@ class Folder(OFSFolder2, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn): ...@@ -1665,6 +1665,7 @@ class Folder(OFSFolder2, CMFBTreeFolder, CMFHBTreeFolder, Base, FolderMixIn):
pass pass
return '%s/%s' % (url, icon) return '%s/%s' % (url, icon)
# See comment in Products.CMFActivity.ActivityTool about the import of BaseTool.
from Products.CMFActivity.ActivityTool import getCurrentNode from Products.CMFActivity.ActivityTool import getCurrentNode
# We browse all used class from btree and hbtree and set not implemented # We browse all used class from btree and hbtree and set not implemented
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment