Investigate circular dependencies

[core-ai-bot]

Issue by gruehle
Wednesday Oct 31, 2012 at 00:09 GMT
Originally opened as adobe/brackets#1998

---

Running madge on the Brackets source shows that we have a number of circular dependencies.

Some of these are known, like DocumentManager<-->EditorManager, and would be very difficult to break. Others should be easy fixes. We should investigate each of these and split the easy fixes into separate bugs.

> madge --circular src/
document/DocumentManager -> editor/EditorManager
editor/EditorManager -> editor/Editor -> command/Menus
editor/EditorManager -> editor/Editor -> command/Menus -> widgets/PopUpManager
editor/EditorManager -> editor/Editor -> editor/CodeHintManager
editor/EditorManager -> editor/Editor
preferences/PreferencesManager -> preferences/PreferenceStorage
document/DocumentManager -> editor/EditorManager -> editor/InlineTextEditor
editor/EditorManager -> editor/InlineTextEditor
editor/EditorManager -> editor/InlineTextEditor -> editor/InlineWidget
editor/EditorManager -> widgets/StatusBar
document/DocumentManager -> project/ProjectManager
project/ProjectManager -> preferences/PreferencesDialogs
document/DocumentManager -> project/ProjectManager -> project/FileViewController

[core-ai-bot]

Comment by peterflynn
Wednesday Oct 31, 2012 at 00:23 GMT

---

EditorManager -> Statusbar is covered by #1869

Editor -> Menus is only there to close popups on scroll, which might be changed by #1381.

[core-ai-bot]

Comment by pthiess
Tuesday Nov 06, 2012 at 00:17 GMT

---

Reviewed