The include line (line 12) is not necessary, since the common file is loaded by all the subfiles. This include structure predates the D6 menu system and should be removed.

CommentFileSizeAuthor
#1 skeleton_include.patch2.95 KBagentrickard

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Needs review
StatusFileSize
new skeleton_include.patch2.95 KB

To make this work, we need to move the two menu loader functions from the common file to the module file, which is more in keeping with D6.

deviantintegral’s picture

deviantintegral commented
Status: Needs review » Fixed

Thanks - that was something I was meaning to do but hadn't done yet. It makes the code much cleaner.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.