Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.In the current d6 branch of the Views project, there is a file ajax.inc that seems to be causing some parsing problems.
The file is not following our docs standards, due to this header at the top:
/**
* @file ajax.inc
*
* Handles the server side AJAX interactions of Views.
*
* @defgroup ajax Views ajax library
* @{
*/
The @file and the @defgroup should be in separate doc blocks (I'll file an issue in Views to get that fixed). But it seems that when the API module encounters this situation, it's getting into some kind of a recursion problem (I think?). Hard to say without more investigation, but the file is getting parsed over and over, the group/topic page is weird, and the file page is missing the file source. In any case, even though this docblock is wrong, it shouldn't cause the parser to crash.
Comments
Comment #1
jhodgdonAlso filed
#1857410: Doc block at top of ajax.inc is wrong
to get the file fixed in the Views project.
Comment #2
jhodgdonI was able to reproduce the problem in a test file, and then fix the problem in parser.inc.
I decided the best solution if there was a @file docblock that contained either @mainpage or @defgroup (both would cause the same problem) was to just remove the offending @mainpage or @defgroup and put in a watchdog message saying that it was ignored. So at least the file entry would be created (it wasn't, which causes all kinds of problems).
I also wrote a test for this. Since parser.inc had to change, I am running all the API module tests before committing this patch, but it should be fixed shortly.
Comment #3
jhodgdonThe fix for this is committed.
Comment #4
jhodgdonwhoops, the commit for this included a fix for a Drush command, see
#1832610: Drush commands to update a file need attention