Let topics have topic from other module as parent

yched - September 13, 2008 - 00:08
Project:Advanced help
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

(Not sure this is doable easily, or even that it's a good idea to begin with)

At some point, CCK will need a 'field types' page. All 'core' field types will provide a child page describing the specifics of this field type (what it stores, info on the available settings, etc...)
It would be cool if contrib field modules like date or filefield could provide their own 'field description' topic as children of CCK's 'field types' topic.

#1

merlinofchaos - September 15, 2008 - 18:17
Status:active» fixed

They can. Just use modulename%topicname -- and this is documented.

Note that I didn't extensively test this feature, so it may break down somewhere but I did do some basic testing of it.

#2

yched - September 17, 2008 - 16:47

Hm. Always embarrassing :-/

#3

Anonymous (not verified) - October 1, 2008 - 16:52
Status:fixed» closed

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

#4

KarenS - October 22, 2008 - 13:20
Status:closed» active

We're trying this out and I ran into a bug in the code that keeps it from working. Attached is a patch to fix it.

AttachmentSize
advanced_help.module.patch 969 bytes

#5

KarenS - October 22, 2008 - 13:21
Category:feature request» bug report
Status:active» needs review

Changing status.

#6

KarenS - October 22, 2008 - 13:23

Oops, I see another issue #323867: parent module%topic format does not work, but that isn't a patch so I'll mark it a duplicate.

#7

Pasqualle - October 22, 2008 - 18:36
Status:needs review» reviewed & tested by the community

same patch from me, and tested..

#8

KarenS - October 26, 2008 - 10:58

Just a note, CCK Advanced help is broken until this is fixed.

#9

Roi Danton - October 27, 2008 - 17:55

Patch works for my own module + submodules (not at drupal.org for now), too. Without patch module%topic didn't work.

#10

merlinofchaos - October 27, 2008 - 20:59
Status:reviewed & tested by the community» fixed

Thanks Karen! Sorry about taking a bit to get to this; I only do advanced help passes every now and then, and this came in right after I released 1.1.

#11

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

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

 
 

Drupal is a registered trademark of Dries Buytaert.