The HTML HEAD link to wishlist's css file is incorrect for sites where drupal is installed in a sub-directory of the site.
Wishlist link comes up as 'modules/wishlist/wishlist.css' when it ought to come up as 'correctpath/modules/wishlist/wishlist.css'
I've no idea how CVS and Patches etc work, (so sorry for not posting a patch etc) but I believe you can correct the fault by amending the code as shown below.
Line 355 of wishlist.module reads :
drupal_set_html_head('@import url('. drupal_get_path('module','wishlist') .'/wishlist.css);');
Wishlist module uses drupal_set_html_head when it ought (?) to be using:
theme_add_style(drupal_get_path('module', 'wishlist') .'/wishlist.css');
If you replace the code as shown the CSS reference is correct.
Kind Regards
Dom
Comments
Comment #1
scott.mclewin commentedDom,
No patch needed - what you submitted is all I need. I intend to cycle up a set of wishlist module changes this week and will include this change. I missed this change (the move to theme_add_style()) when I did my upgrade from 4.6 to 4.7, thanks for catching it.
Scott
Comment #2
scott.mclewin commentedThis has been addressed in cvs HEAD and DRUPAL-4-7. Thanks for reporting it.