Hi,
Just stumbled across this. The following path should be updated in this module's implementation of HOOK_HELP:

case 'admin/settings/upload'
should be...
case 'admin/settings/uploads'

With this amendment the help text appears.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dave Reid’s picture

Status: Active » Needs review
FileSize
2.46 KB

That's pretty funny that we haven't noticed that for so long. Patch for review against 6.x-1.x-dev.

For an additional lol, we were linking to the admin/settings/types page, which was moved to admin/content/types in D6...

Dave Reid’s picture

Version: 6.19 » 6.x-dev
FileSize
2.49 KB

We should also link to the upload section in the permissions page rather than making the user scroll. This does not change any translation strings.

Sivaji_Ganesh_Jojodae’s picture

Status: Needs review » Reviewed & tested by the community

Patch #2 looks good, applies cleanly and really no brainier. Marking RTBC.

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Looks good, thanks, committed. Drupal 7 has no upload.module so not applicable there.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.