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.It looks like system_list() is using both $list['module_enable'] and $list['module_enabled'] for the same thing (note the enable/enabled change) -- someone might want to check this out? Probably the caching isn't working in this function due to this typo?
e.g.
elseif (!isset($lists['module_enable'])) {
$lists = array(
'module_enabled' => array(),
'theme' => array(),
'filepaths' => array(),
);
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 648262-typo-module-inc.patch | 2.01 KB | brianV |
Comments
Comment #1
brianV CreditAttribution: brianV commentedNice catch. Patch attached, also with a few whitespace fixes.
Comment #2
jhodgdonIf you're going to fix the whitespace in the doc block, you might remove the extra blank line. There should just be one blank line between paragraphs, or between paragraph/@param. And there should be a blank line between the last @param and the @return if there isn't already, according to current standards.
Other than that, I'll let the testing bot see if the patch is OK.
Comment #3
Dries CreditAttribution: Dries commentedCommitted to CVS HEAD. Leaving status as is so we can follow-up with new line patches, if we want to.
Comment #4
jhodgdonLet's not worry about that now. It's a very minor doc thing that doesn't change how doc is displayed on api.drupal.org, and not relevant to this issue really.