'Adroll' and the 'adding acatalog' issue again

[graphicz]

Please can someone kindly check this for me?

Client has been give this code by Adroll http://www.adroll.com/

<script type="text/javascript">
adroll_adv_id = "5RXXXXXXXXXXXXXXXXXXXX";
adroll_pix_id = "2KXXXXXXXXXXXXXXXXXXXX";
(function () {
var oldonload = window.onload;
window.onload = function(){
   __adroll_loaded=true;
   var scr = document.createElement("script");
   var host = (("https:" == document.location.protocol) ? "https://s.adroll.com" : "http://a.adroll.com");
   scr.setAttribute('async', 'true');
   scr.type = "text/javascript";
   scr.src = host + "/j/roundtrip.js";
   ((document.getElementsByTagName('head') || [null])[0] ||
    document.getElementsByTagName('script')[0].parentNode).appendChild(scr);
   if(oldonload){oldonload()}};
}());
</script>

Sellerdeck was adding 'acatalog' on the home page ( scr.src = host + "/j/roundtrip.js"; became scr.src = host + "acatalog//j/roundtrip.js"; )

I have therefore edited the code thus:

<script type="text/javascript">
adroll_adv_id = "5RXXXXXXXXXXXXXXXXXXXX";
adroll_pix_id = "2KXXXXXXXXXXXXXXXXXXXX";
(function () {
var oldonload = window.onload;
window.onload = function(){
   __adroll_loaded=true;
   var scr = document.createElement("script");
   var host = (("https:" == document.location.protocol) ? "https://s.adroll.com" : "http://a.adroll.com");
   scr.setAttribute('async', 'true');
   scr.type = "text/javascript";
   scr.src = host + "/j/roundtrip"+"."+"js";
   ((document.getElementsByTagName('head') || [null])[0] ||
    document.getElementsByTagName('script')[0].parentNode).appendChild(scr);
   if(oldonload){oldonload()}};
}());
</script>

Is that correct or have I made a gaff?

Thank you.

[NormanRouxel]

The line

scr.src = host + "/j/roundtrip"+"."+"js";

Looks fine to me. Try it and see.

[graphicz]

Thanks Norman