Getting Image/Img_Assist and FileManager to Work Together

OK, here are the relevant snippets of code I added to image.module to make this work. By the way, I am using category.module in place of the core taxonomy module, so this code will probably not work as-is if you are using core taxonomy.

Also, keep in mind that this is one of my first Drupal hacks, and as such, I am quite certain that this code can be optimized and improved upon. I'm almost embarrassed to post it, just b/c I dashed this off very quickly and, once it worked, didn't spend any time looking for a better way. I would request that anyone who sees a better way to do this would please post suggestions for improvements to the code.

That said, it does work, and for now, that's good enough for me.

1. First, add the $node parameter to the _image_filename function and adjust all references to this function in image.module to pass in $node accordingly. If you are running Img_Assist, there is a call to this function there which must be adjusted as well.

2. Add an else condition to the if ($temp) statement in _image_filename:

if ($temp) {
	  $path .= 'temp/';
  }else{
	  $path .= getCategoryDirectory($node, $path);
}

3. Add the full getCategoryDirectory function. It works, but are there better ways of doing this? (For example, I'm sure Drupal has its own "create directory" function, but I didn't have time to look for it, so I just used mkdir.) Also, keep in mind that if the node has multiple categories, this will only grab one of them.

function getCategoryDirectory($node, $path){	
    //try to find the category cid
    if (is_array($node->category)) {
      foreach ($node->category as $key => $cat) {
        if ($key != 'tags' && $key != 'legacy') {
          if (is_array($cat)) {
            foreach ($cat as $cid) {
              if (!empty($cid)) {
                $catcid = $cid;
              }
            }
          }
          else if (isset($cat->cid)) {
            $catcid = $cat->cid;
          }
          else if ($cat) {
            $catcid = $cat;
          }
        }
      }
    }
    
    if (isset($catcid)) {
	$result = db_query(db_rewrite_sql('SELECT n.title FROM {node} n WHERE n.nid=' . $catcid)); 
      while ($cattitle = db_fetch_object($result)) {
	      $img_clean_str = imagecat_cleanstring($cattitle->title) . '/';
	      $path .= $img_clean_str;
	      
	      //create the directory if it doesn't exist
	      if (!file_exists($_SERVER["DOCUMENT_ROOT"] . '/files/' . $path)) {
		      mkdir($_SERVER["DOCUMENT_ROOT"] . '/files/' . $path);
	      }
	      return $img_clean_str;
        break;
      }
    }
    return '';
}

4. Add the imagecat_cleanstring function. I borrowed most of this code from pathauto module. This will clean the category name of spaces and other bad characters.

function imagecat_cleanstring($string) {
  static $translations = array(
    'À'=>'A','�'=>'A','Â'=>'A','Ã'=>'A','Ä'=>'A','Å'=>'A','Ā'=>'A','Ą'=>'A','Ă'=>'A',
    'à '=>'a','á'=>'a','â'=>'a','ã'=>'a','ä'=>'a','Ã¥'=>'a','Ä?'=>'a','Ä…'=>'a','ă'=>'a',
    'Æ'=>'Ae',
    'æ'=>'ae',
    'Ç'=>'C','Ć'=>'C','Č'=>'C','Ĉ'=>'C','Ċ'=>'C',
    'ç'=>'c','ć'=>'c','Ä?'=>'c','ĉ'=>'c','Ä‹'=>'c',
    'ÄŽ'=>'D','Ä?'=>'D','Ã?'=>'D',
    'Ä?'=>'d','Ä‘'=>'d','ð'=>'d',
    'È'=>'E','É'=>'E','Ê'=>'E','Ë'=>'E','Ē'=>'E','Ę'=>'E','Ě'=>'E','Ĕ'=>'E','Ė'=>'E',
    'è'=>'e','é'=>'e','ê'=>'e','ë'=>'e','ē'=>'e','ę'=>'e','ě'=>'e','ĕ'=>'e','ė'=>'e',
    'Æ’'=>'f',
    'Äœ'=>'G','Äž'=>'G','Ä '=>'G','Ä¢'=>'G',
    'Ä?'=>'g','ÄŸ'=>'g','Ä¡'=>'g','Ä£'=>'g',
    'Ĥ'=>'H','Ħ'=>'H',
    'ĥ'=>'h','ħ'=>'h',
    'Ì'=>'I','�'=>'I','Î'=>'I','�'=>'I','Ī'=>'I','Ĩ'=>'I','Ĭ'=>'I','Į'=>'I','İ'=>'I',
    'ì'=>'i','í'=>'i','î'=>'i','ï'=>'i','ī'=>'i','ĩ'=>'i','ĭ'=>'i','į'=>'i','ı'=>'i',
    'IJ'=>'Ij',
    'ij'=>'ij',
    'Ä´'=>'J',
    'ĵ'=>'j',
    'Ķ'=>'K',
    'ķ'=>'k','ĸ'=>'k',
    'Å?'=>'L','Ľ'=>'L','Ĺ'=>'L','Ä»'=>'L','Ä¿'=>'L',
    'ł'=>'l','ľ'=>'l','ĺ'=>'l','ļ'=>'l','ŀ'=>'l',
    'Ñ'=>'N','Ń'=>'N','Ň'=>'N','Ņ'=>'N','Ŋ'=>'N',
    'ñ'=>'n','ń'=>'n','ň'=>'n','ņ'=>'n','ʼn'=>'n','ŋ'=>'n',
    'Ã’'=>'O','Ó'=>'O','Ô'=>'O','Õ'=>'O','Ö'=>'O','Ø'=>'O','ÅŒ'=>'O','Å?'=>'O','ÅŽ'=>'O',
    'ò'=>'o','ó'=>'o','ô'=>'o','õ'=>'o','ö'=>'o','ø'=>'o','Å?'=>'o','Å‘'=>'o','Å?'=>'o',
    'Å’'=>'Oe',
    'Å“'=>'oe',
    'Ŕ'=>'R','Ř'=>'R','Ŗ'=>'R',
    'Å•'=>'r','Å™'=>'r','Å—'=>'r',
    'Ś'=>'S','Š'=>'S','Ş'=>'S','Ŝ'=>'S','Ș'=>'S',
    'Ť'=>'T','Ţ'=>'T','Ŧ'=>'T','Ț'=>'T','Þ'=>'T',
    'þ'=>'t',
    'Ù'=>'U','Ú'=>'U','Û'=>'U','Ü'=>'U','Ū'=>'U','Ů'=>'U','Ű'=>'U','Ŭ'=>'U','Ũ'=>'U','Ų'=>'U',
    'ú'=>'u','û'=>'u','ü'=>'u','ū'=>'u','ů'=>'u','ű'=>'u','ŭ'=>'u','ũ'=>'u','ų'=>'u',
    'Å´'=>'W',
    'ŵ'=>'w',
    '�'=>'Y','Ŷ'=>'Y','Ÿ'=>'Y','Y'=>'Y',
    'ý'=>'y','ÿ'=>'y','ŷ'=>'y',
    'Ź'=>'Z','Ž'=>'Z','Ż'=>'Z',
    'ž'=>'z','ż'=>'z','ź'=>'z',
    'ß'=>'ss','ſ'=>'ss');

  // Drop apostrophes
  $output = str_replace("'", "", $string);
  // Convert accented characters to their ASCII counterparts...
/*  $output = strtr(utf8_decode($output),
       "\xA1\xAA\xBA\xBF".
       "\xC0\xC1\xC2\xC3\xC5\xC7\xC8\xC9\xCA\xCB\xCC\xCD\xCE\xCF".
       "\xD0\xD1\xD2\xD3\xD4\xD5\xD8\xD9\xDA\xDB\xDD".
       "\xE0\xE1\xE2\xE3\xE5\xE7\xE8\xE9\xEA\xEB\xEC\xED\xEE\xEF".
       "\xF0\xF1\xF2\xF3\xF4\xF5\xF8\xF9\xFA\xFB\xFD\xFF",
       "!ao?AAAAACEEEEIIIIDNOOOOOUUUYaaaaaceeeeiiiidnooooouuuyy"); 
  // ...and ligatures too
  $output = utf8_encode(strtr($output, array("\xC4"=>"Ae", "\xC6"=>"AE", "\xD6"=>"Oe", 
    "\xDC"=>"Ue", "\xDE"=>"TH", "\xDF"=>"ss", "\xE4"=>"ae", "\xE6"=>"ae", 
    "\xF6"=>"oe", "\xFC"=>"ue", "\xFE"=>"th")));*/
  $output = strtr($output, $translations);

  // Preserve alphanumerics, everything else becomes a separator
  $pattern = '/[^a-zA-Z0-9]+/ ';
  $separator = variable_get('pathauto_separator', '_');
  $output = preg_replace($pattern, $separator, $output);

  // Trim any leading or trailing separators (note the need to
  // escape the separator if and only if it is not alphanumeric)
  if ($separator) {
    if (ctype_alnum($separator)) {
      $seppattern = $separator;
    } else {
      $seppattern = '\\'.$separator;
    }
    $output = preg_replace("/^$seppattern+|$seppattern+$/", "", $output);
  }
  
  return $output;
}

Well, I guess that's it. I hope this helps someone else...and I also hope to get some feedback on improvements to this code.