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.

Comments

apsivam’s picture

Status: Active » Postponed (maintainer needs more info)

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

kylehase’s picture

Project: Amadou » Service links
Version: 6.x-1.1 » 6.x-1.0
Assigned: jwolf » Unassigned
Status: Active » Postponed (maintainer needs more info)

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


* <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

kylehase’s picture

Status: Postponed (maintainer 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);
      }
kylehase’s picture

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

jwolf’s picture

Assigned: Unassigned » jwolf

Great! Thank you. Will add this fix to the next release.

jwolf’s picture

Project: Service links » Amadou
Version: 6.x-1.0 » 6.x-1.1
Assigned: Unassigned » jwolf
Status: Postponed (maintainer needs more info) » Patch (to be ported)
sk33lz’s picture

Status: Patch (to be ported) » Fixed

Added to dev release. Seemed like a more comprehensive fix than the previous issue fix.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.