I added piclens support to the node_view of my particular node type. When view /node I get multiple instances of the piclens javascript in the head.

<script type="text/javascript" src="http://lite.piclens.com/current/piclens_optimized.js"></script>
<link rel="alternate" type="application/rss+xml" title="dev - mediaRSS" href="http://localhost/sliderdev/piclens/slider_piclens/579" />

<script type="text/javascript" src="http://lite.piclens.com/current/piclens_optimized.js"></script>
<link rel="alternate" type="application/rss+xml" title="dev - mediaRSS" href="http://localhost/sliderdev/piclens/slider_piclens/560" />

This is because a check for duplicates is not made nor is drupal_add_js() used and/or does not support external JS files.

Comments

swentel’s picture

swentel commented
Version: 5.x-1.5 » 5.x-1.x-dev

I'll use a static var to fix the inclusion of the piclens javascript.
The multiple rss feed links are ok to stay, right ?

(changed version to dev also, since I'm going to work on that - it has way more features and bugfixes allready and this should be included in next release - at least the D5 version)

swentel’s picture

swentel commented
Status: Active » Fixed

commited to drupal 5 and 6 branch, thanks for the report!

dldege’s picture

dldege commented

Yes, each feed should be listed. I will test this new version. Thanks.

dldege’s picture

dldege commented

This is fixed in my testing. Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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