You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@shindig.apache.org by dd...@apache.org on 2011/12/07 16:24:25 UTC

svn commit: r1211472 - /shindig/trunk/features/src/main/javascript/features/container/container.js

Author: ddumont
Date: Wed Dec  7 15:24:24 2011
New Revision: 1211472

URL: http://svn.apache.org/viewvc?rev=1211472&view=rev
Log:
SHINDIG-1670 Fix issue with osapi.container.Container.prototype.getNavigatedGadgetUrls_ and URL sites

Modified:
    shindig/trunk/features/src/main/javascript/features/container/container.js

Modified: shindig/trunk/features/src/main/javascript/features/container/container.js
URL: http://svn.apache.org/viewvc/shindig/trunk/features/src/main/javascript/features/container/container.js?rev=1211472&r1=1211471&r2=1211472&view=diff
==============================================================================
--- shindig/trunk/features/src/main/javascript/features/container/container.js (original)
+++ shindig/trunk/features/src/main/javascript/features/container/container.js Wed Dec  7 15:24:24 2011
@@ -841,10 +841,12 @@ osapi.container.Container.prototype.getA
 osapi.container.Container.prototype.getNavigatedGadgetUrls_ = function() {
   var result = {};
   for (var siteId in this.sites_) {
-    var site = this.sites_[siteId],
-        holder = (site.getActiveGadgetHolder || site.getActiveUrlHolder).call(site);
-    if (holder) {
-      result[holder.getUrl()] = null;
+    var site = this.sites_[siteId];
+    if (site instanceof osapi.container.GadgetSite) {
+      var holder = (site.getActiveGadgetHolder || site.getActiveUrlHolder).call(site);
+      if(holder) {
+        result[holder.getUrl()] = null;
+      }
     }
   }
   return result;