SL display HTML instead of actual buttons and links

nodegate - May 24, 2008 - 14:38
Project:Amadou
Version:6.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:active
Description

Hi,

Could you have a look at my website:
http://www.secury.net/news/microsoft-patch-tuesday-six-vulnerabilities-f...

Service links display as source html instead of showing text and images. How to workaround this? Input type is set to PHP.

#1

apsivam - May 25, 2008 - 14:53
Status:active» active (needs more info)

The above page not found (404).
I opened another page but there is no Service Links installed

#2

kylehase - August 28, 2008 - 22:45

I can confirm this on my site as well. Interestingly I'm using the same theme so it could be an incompatability with the Amadou theme. Instead of the actual graphics I see unparsed HTML img src tags.

I'll leave it broken for now so you can see it.
http://kylehasegawa.com/content/my-2-cents-wifi-myths-busted-part-1-ssid

* <img src="/sites/all/modules/service_links/images/delicious.png" alt="Delicious" /> Delicious
* <img src="/sites/all/modules/service_links/images/digg.png" alt="Digg" /> Digg
* <img src="/sites/all/modules/service_links/images/stumbleit.png" alt="StumbleUpon" /> StumbleUpon
* <img src="/sites/all/modules/service_links/images/reddit.png" alt="Reddit" /> Reddit
* <img src="/sites/all/modules/service_links/images/facebook.png" alt="Facebook" /> Facebook
* <img src="/sites/all/modules/service_links/images/technorati.png" alt="Technorati" /> Technorati

#3

kylehase - August 29, 2008 - 03:27
Status:active (needs more info)» active

Amadou for Drupal 6 is using the Drupal 5 link function arguments on line 87 of template.php.

Here's a fix. amadou/template.php line 87.

      if (isset($link['href'])) {
        $output .= l($link['title'], $link['href'], $link['attributes'], $link['query'], $link['fragment'], FALSE, $html);
      }
}

to

      if (isset($link['href'])) {
        $link_options = array('attributes'  => $link['attributes'],
                              'query'       => $link['query'],
                              'fragment'    => $link['fragment'],
                              'absolute'    => FALSE,
                              'html'        => $html);
        $output .= l($link['title'], $link['href'], $link_options);
      }

#4

kylehase - August 29, 2008 - 03:35
Project:Service links» Amadou
Version:6.x-1.0» 6.x-1.1

Moving bug to Amadou.

Actually, I see there's a dupe bug in Amadou but the resolution is different.
http://drupal.org/node/267615

 
 

Drupal is a registered trademark of Dries Buytaert.