You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shindig.apache.org by rb...@apache.org on 2012/07/20 08:17:55 UTC

svn commit: r1363663 - /shindig/trunk/features/src/main/javascript/features/actions/actions_container.js

Author: rbaxter85
Date: Fri Jul 20 06:17:55 2012
New Revision: 1363663

URL: http://svn.apache.org/viewvc?rev=1363663&view=rev
Log:
SHINDIG-1819
Committed for Marshall Shi
With declarative actions, if two gadgets claim the same 'id' there is no warning that only one gadget will function.

Modified:
    shindig/trunk/features/src/main/javascript/features/actions/actions_container.js

Modified: shindig/trunk/features/src/main/javascript/features/actions/actions_container.js
URL: http://svn.apache.org/viewvc/shindig/trunk/features/src/main/javascript/features/actions/actions_container.js?rev=1363663&r1=1363662&r2=1363663&view=diff
==============================================================================
--- shindig/trunk/features/src/main/javascript/features/actions/actions_container.js (original)
+++ shindig/trunk/features/src/main/javascript/features/actions/actions_container.js Fri Jul 20 06:17:55 2012
@@ -541,6 +541,8 @@
         // check if action already exists
         if (!registry.getItemById(actionClone.id)) {
           addAction(actionClone, url);
+        } else {
+          gadgets.warn(['Duplicated gadget action [', actionClone.id, '] detected, make sure the gadget actions have unique ids.'].join(''));
         }
       }
     }