Sorry for the lack of diff but I just found that the $imagepath was referencing relative to the node.

updated line 477 as follows:

$imagepath = base_path().drupal_get_path('module','filefield') .'/ico/'. $ext .'.png';

catting the basepath() to the top of the string.

Though getting this out of the monules directory might be a good idea, but that's another issue all togather.

Comments

jpetso’s picture

Status: Active » Fixed

Thanks, I committed this change to HEAD. The issue didn't normally occur, but it seems you used a theme where the HTML "base" property was set (like in Drupal 4.7 by default, but not in Drupal 5). However, your suggestion seems worthwhile, if only as guard against non-standard themes that use "base".

Anonymous’s picture

Status: Fixed » Closed (fixed)