This documentation block is at the top of the 6.x version of includes/ajax.inc:

/**
 * @file ajax.inc
 *
 * Handles the server side AJAX interactions of Views.
 *
 * @defgroup ajax Views ajax library
 * @{
 */

This is not right. The @file should be one docblock, and the @defgroup should be a different doc block.

It's currently causing our d7 api.d.o test site to crash in parsing (which is a separate issue:
#1857406: Problem parsing malformed file

But it would be nice to get this fixed... maybe there was a separate issue to fix it already that wasn't backported to d6 because the d7 version does not have this problem (copy the header from the d7 ajax.inc and you'll be fine).

CommentFileSizeAuthor
#2 ajax_doc_block-1857410-2.patch417 bytesIvan Zugec
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

Here's what the header should look like:

/**
 * @file
 * Handles the server side AJAX interactions of Views.
 */

/**
 * @defgroup ajax Views AJAX library
 * @{
 * Handles the server side AJAX interactions of Views.
 */
Ivan Zugec’s picture

Status: Active » Needs review
FileSize
417 bytes

Fixed the doc block within "includes/ajax.inc" as requested.

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! It would be nice to get this committed.

psynaptic’s picture

I need this too. Pretty please? :)

dawehner’s picture

Status: Reviewed & tested by the community » Fixed

Sure, there we go. Thanks for the patch.

Committed and pushed.
Sorry for not seeing this patch :(

psynaptic’s picture

Great, thanks!

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