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

scott.mclewin’s picture

Assigned: Unassigned » scott.mclewin

Dom,

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

scott.mclewin’s picture

Status: Active » Closed (fixed)

This has been addressed in cvs HEAD and DRUPAL-4-7. Thanks for reporting it.