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.


Dave Reid’s picture

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