RADEMAKERS Tanguy wrote: > you have two calls to function urlencodedformat: > > #urlencodedformat(main_jpeg)# > #urlencodedformat(tempPopTitle)# > > could that have something to do with it?
Well, I wouldn't do that, nor would I get rid of the javascript: bit in the link (as such). Try this: <cfset popupLink = "href="http://XXXXXX/Show_Movies_Image.cfm?Image="; & URLEncodedFormat(main_jpeg) & "&type=DVD&title=" & URLEncodedFormat(tempPopTitle)> <cfsavecontent var="file"> <cfoutput> <a href="http://XXXXXX/DVD/#URL#"; style="color:navy;font-size:15px">#Trim(tempTitle)# DVD <a onclick="popupBoxArtDVD(this.href, 236, 335);return false" onmouseover="window.status = 'See Box Art'" onmouseout="window.status = ''" href="#popupLink#"><img src="http://XXXXXX/gif/excal2/Files_Icon.gif"; alt="Box Art" hspace="4" vspace="0" border="0" align="bottom"></a> </cfoutput> </savecontent> <cffile action="append" file="#complete_filename#" addnewline="No" output="#file#"> Also, look for any variables that you're repeatedly reprocessing within the loop, but that don't actually change in value and move them out of it. Doing something just once is always faster than doing it over and over again. Also, I removed a few trims that looked redundant from the above. K. ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Logware (www.logware.us): a new and convenient web-based time tracking application. Start tracking and documenting hours spent on a project or with a client with Logware today. Try it for free with a 15 day trial account. http://www.houseoffusion.com/banners/view.cfm?bannerid=67 Message: http://www.houseoffusion.com/lists.cfm/link=i:4:204947 Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/4 Subscription: http://www.houseoffusion.com/lists.cfm/link=s:4 Unsubscribe: http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.4 Donations & Support: http://www.houseoffusion.com/tiny.cfm/54
![http://XXXXXX/gif/excal2/Files_Icon.gif"](http://XXXXXX/gif/excal2/Files_Icon.gif"){kind=link}