@name is not supported by API module doxygen [#898040]

There are several places in Drupal core that have @name in docblocks.

The API module does not support name in our version of doxygen. They should have been @defgroups in most/all cases.

I made a patch here http://drupal.org/files/issues/895600.patch on this issue #895600: @ingroup @{ @} with classes should only include the class, not its members that included a fix for this. We need to take the @name part out of that patch and attach it here as a patch and get it committed, while we debate what to do about that other issue (which is really unrelated).

Comment	File	Size	Author
#4	898040.patch	3.95 KB	jhodgdon
#2	name.patch	4.09 KB	jhodgdon

Comments

Comment #1

jhodgdon

she/her

English

commented 31 August 2010 at 02:08

Title:

@name is not supported

» @name is not supported by API module doxygen

Comment #2

jhodgdon

she/her

English

commented 31 August 2010 at 14:45

Status:

Active

» Needs review

Status	File	Size
new	name.patch	4.09 KB

Here's the patch.

Comment #3

31 August 2010 at 15:00

Status:

Needs review

» Needs work

The last submitted patch, name.patch, failed testing.

Comment #4

jhodgdon

she/her

English

commented 31 August 2010 at 16:27

Status:

Needs work

» Needs review

Status	File	Size
new	898040.patch	3.95 KB

Here's a patch that should work better.

Comment #5

markabur commented 8 October 2010 at 00:05

Status:

Needs review

» Reviewed & tested by the community

Looks good; patch still applies and all of the "@name Title" are gone and replaced with "@defgroup group_id Title". I'm assuming "@defgroup content_flags Content markers" isn't a mistake -- the rest of the group ids match the human-friendly title more closely.