Download & Extend

Remove include_once('skeleton_common.inc');

Project:Skeleton outlines
Version:6.x-1.x-dev
Component:Code
Category:task
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

Comments

#1

Status:active» needs review

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.

AttachmentSize
skeleton_include.patch 2.95 KB

#2

Status:needs review» fixed

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

#3

Status:fixed» closed (fixed)

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

nobody click here