Error if all module's topics are childs of other modules

Roi Danton - October 27, 2008 - 18:14
Project:Advanced help
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Bug report

If a linodef_bueditor.help.ini file is like this:

; $Id$
[advanced help settings]
line break = TRUE

[usage]
title = "How to use Linodef - BUEditor"
parent = linodef%editors
weight = 0

[insert-button]
title = "Insert buttons that supports Linodef filter in BUEditor"
parent = usage
weight = 0

the modules name is listed at the Advanced Help overview nevertheless. By clicking on it the following error is display because no topics are defined
    * warning: uasort() [function.uasort]: The argument should be an array in modulepath/advanced_help/advanced_help.module on line 241.
    * warning: Invalid argument supplied for foreach() in modulepath/advanced_help/advanced_help.module on line 243.

Feature request

To prevent this error, a module only with child topics of other modules shouldn't be listed on the Advanced help page.

#1

merlinofchaos - October 27, 2008 - 21:21
Status:active» fixed

It's actually a bit difficult to auto check to see if there are no top level topics (or, rather, it's a bit intensive, as you have to scan every topic). Instead, I added a 'hide' directive to the global structure to remove your module from the list. This would be recommended in cases like this.

#2

Roi Danton - October 27, 2008 - 21:46

Thx! Using HEAD this works excellent.

#3

Anonymous (not verified) - November 10, 2008 - 21:53
Status:fixed» closed

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

#4

markus_petrux - November 27, 2008 - 23:56

Hi,

I was about to report this same issue, but I noticed it's already fixed in HEAD. Any change to get a new release (6.x-1.2) with latest bug fixes included?

 
 

Drupal is a registered trademark of Dries Buytaert.