You've never heard of module_iterate()? Not surprising, since this function is never used anywhere, in core or in contrib. And not only now - apparently it has been lingering since at least Drupal 4.6, and as unused then as it is now. It is an embarrassment to Drupal, and should be removed. Attached patch does that.

CommentFileSizeAuthor
module_iterate_01.patch703 bytescwgordon7
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cburschka’s picture

Status: Needs review » Reviewed & tested by the community

Amazing. Are there any more easter eggs hidden in the code? :)

No functional changes, as expected.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Nuts. Removed from CVS HEAD. Thanks.

cburschka’s picture

Version: 7.x-dev » 6.x-dev
Status: Fixed » Needs review

Should it be removed from 6.x, or would it be an API change? We cannot be entirely certain that nothing in contrib uses it...

cwgordon7’s picture

Status: Needs review » Closed (won't fix)

Right, this should not be changed in 6.x.

catch’s picture

Status: Closed (won't fix) » Closed (fixed)

This was one of the oldest functions in core, via chx:

git show 8d5b4e7b8615b636eaef8f50eaac45d27dc52b33