I just noticed, that there is a section called "messages" in core which contains one single function: http://api.drupal.org/api/HEAD/function/message_na

I really think that this one is pretty unnecessary. I attached a patch where all occurences of this function have been replaced with t('n/a').

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kkaefer’s picture

FileSize
3.71 KB

I still don't see any reason why this function should stay there. Updated the patch.

kkaefer’s picture

Status: Active » Needs review
fgm’s picture

It is currently in use in various modules, some of them important. Removing it would mean patching them too:

  • locale.inc
  • blogapi.module
  • filter.module
  • forum.module
  • cvs.module
  • evaluation.module
  • filtercache.module
  • naggregator.module
  • subversion.module
  • theme: civicspace
  • berkes' node_aggregator
  • factoryjoe's groundswell theme
  • jsend's drupal4blog
  • killes' "speed drupal" filter.module
Dries’s picture

Status: Needs review » Needs work

Committed to CVS HEAD. Marking this 'code needs work'. Set to 'fixed' when this change is documented in the upgrade instructions. Thanks timcn.

kkaefer’s picture

FileSize
23.08 KB

Made a patch for contrib CVS to replace message_na() with t('n/a'). This should not break anything, even with non-HEAD versions.

kkaefer’s picture

Status: Needs work » Needs review

Thanks to killes, the change is no documented on the corresponding handbook page.

Dries’s picture

Status: Needs review » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)