attachments module improvements

Hi

I have so far used some of the old search_attachments module to enhance the normal search so it also includes attachments. But this is looking very good and I'm thinking of changing over to it. Two things, though, I've found from looking at the code and which might be worth adding to it are:

1. caching of content:
   

         $cache_text = cache_get("file_text:".$fid);
         if ($cache_text && is_object($cache_text) && $cache_text->data) {
           $file_text .= " " . $cache_text->data;
   ...
             cache_set("file_text:".$file->fid, $new_text);
   

2. the file system path doesn't look correct - shouldn't it be:

         $full_path = $_SERVER['DOCUMENT_ROOT'] . '/' . file_directory_path() . '/' . $file->filepath;
   

   My apologies if this has already been dealt with.

   All the best