Add namespace option

Author: i-like-robots

jquery.dfp.js

@@ -75,7 +75,8 @@
             refreshExisting: true,
             disablePublisherConsole: false,
             disableInitialLoad: false,
-            noFetch: false
+            noFetch: false,
+            namespace: undefined
         };
 
         if (typeof options.setUrlTargeting === 'undefined' || options.setUrlTargeting)
@@ -349,7 +350,7 @@
      */
     getName = function ($adUnit) {
 
-        return $adUnit.data('adunit') || $adUnit.attr('id');
+        return $adUnit.data('adunit') || dfpOptions.namespace || $adUnit.attr('id');
 
     },
 

jquery.dfp.min.js

@@ -50,4 +50,26 @@ describe('Ad units', function () {
 
     });
 
+    it("Google ad unit object get attached to the ad unit container (with namespace)", function () {
+        var namespace = 'my-long-namespace';
+
+        runs(function () {
+            $('body').append('<div class="adunit"></div>');
+            $.dfp({dfpID: 'xxxxxxx', namespace: namespace});
+        }, "Kick off loader");
+
+        waitsFor(function () {
+            if (typeof window.googletag.getVersion === 'function') {
+                return true;
+            } else {
+                return false;
+            }
+        }, "getVersion function to exist", 5000);
+
+        runs(function () {
+            expect($('.adunit').data('googleAdUnit').getName()).toEqual('/xxxxxxx/' + namespace);
+        });
+
+    });
+
 });