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.

Members fund testing for the Drupal project. Drupal Association Learn more


Dave Reid’s picture

Status: Active » Needs review
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
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’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.