Use node_get_types() rather than a database query

This:

        $sql = "select name, type from {node_type} order by name";
        $result = db_query($sql);
        while ($data = db_fetch_object($result)) {
                $node = node_load($data->nid);
                $cats[$data->type] = $data->name;
        }

Seems like it could just as easily be a call to http://api.drupal.org/api/function/node_get_types/6 and you could avoid an extra database hit.