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).
Comment | File | Size | Author |
---|---|---|---|
#2 | ajax_doc_block-1857410-2.patch | 417 bytes | Ivan Zugec |
Comments
Comment #1
jhodgdonHere's what the header should look like:
Comment #2
Ivan Zugec CreditAttribution: Ivan Zugec commentedFixed the doc block within "includes/ajax.inc" as requested.
Comment #3
jhodgdonThanks! It would be nice to get this committed.
Comment #4
psynaptic CreditAttribution: psynaptic commentedI need this too. Pretty please? :)
Comment #5
dawehnerSure, there we go. Thanks for the patch.
Committed and pushed.
Sorry for not seeing this patch :(
Comment #6
psynaptic CreditAttribution: psynaptic commentedGreat, thanks!