Minor link errors in UI - fails with unclean URLS

Just evaluating this module for the first time on a raw site.
At first install, the [settings...][more help...] links were broken, and wrong even when fixed for unclean URLs.
Here's my fix-up: webfm.module:1249

  $output  = '<div class="more-help-link">'. (($user->uid == 1||user_access('administer webfm')) ? t('[<a href="@link">settings...</a>]', array('@link' => url("admin/settings/webfm"))) : '') . t('[<a href="@link">more help...</a>]', array('@link' => url("admin/help/webfm"))) .'</div>';

There may be other similar cases, but I thought I'd send in this small patch rather than try to do it all, I've only had the module open 5 mins.
Possibly more to come.

I'm also encountering "illegal name for ftp root directory" which is probably because I'm using a out-of-webroot files path, but that's a different problem. ...