You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wookie.apache.org by sc...@apache.org on 2011/05/06 23:04:40 UTC
svn commit: r1100374 - in /incubator/wookie/trunk: ivy.xml
shindig/dist/shindig-features-2.0.0.jar
shindig/features/core.prefs/feature.xml
src/org/apache/wookie/util/gadgets/GadgetUtils.java
src/org/apache/wookie/util/opensocial/OpenSocialUtils.java
Author: scottbw
Date: Fri May 6 21:04:39 2011
New Revision: 1100374
URL: http://svn.apache.org/viewvc?rev=1100374&view=rev
Log:
Updated Shindig connector to work with Shindig 2.0.0 (see WOOKIE-170). I've also modified the Ivy configuration so that only the very small subset of dependencies from Shindig actually required by Wookie are explicitly named and included in the deployment, as some of the downstream dependencies that we don't use can cause conflicts with the servlet container, and a lot of other jars simply aren't needed. I've tested this with Shinding 2.0.0 on Tomcat 6.0.20.
Added:
incubator/wookie/trunk/shindig/dist/shindig-features-2.0.0.jar (with props)
Modified:
incubator/wookie/trunk/ivy.xml
incubator/wookie/trunk/shindig/features/core.prefs/feature.xml
incubator/wookie/trunk/src/org/apache/wookie/util/gadgets/GadgetUtils.java
incubator/wookie/trunk/src/org/apache/wookie/util/opensocial/OpenSocialUtils.java
Modified: incubator/wookie/trunk/ivy.xml
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/ivy.xml?rev=1100374&r1=1100373&r2=1100374&view=diff
==============================================================================
--- incubator/wookie/trunk/ivy.xml (original)
+++ incubator/wookie/trunk/ivy.xml Fri May 6 21:04:39 2011
@@ -41,7 +41,7 @@
</dependency>
<dependency org="commons-configuration" name="commons-configuration" rev="1.4" conf="deploy->default">
</dependency>
- <dependency org="commons-fileupload" name="commons-fileupload" rev="1.0" conf="deploy->default">
+ <dependency org="commons-fileupload" name="commons-fileupload" rev="1.2.1" conf="deploy->default">
</dependency>
<dependency org="commons-httpclient" name="commons-httpclient" rev="3.0.1" conf="deploy->default">
</dependency>
@@ -59,8 +59,22 @@
</dependency>
<dependency org="org.slf4j" name="slf4j-log4j12" rev="1.5.2" conf="deploy->default">
</dependency>
- <dependency org="org.apache.shindig" name="shindig-common" rev="1.1-BETA5-incubating" conf="deploy->default">
+
+ <!-- Note that we only require the "BlobCrypterToken" part of Shindig in order to create valid tokens, so
+ we don't need to include the whole of Shindig common's dependencies as well, hence transitive is
+ marked "false" and we include the two modules that the token creation depends on.
+
+ While it may be simpler to just include the whole of Shindig, not only does it bloat the application, it
+ also can cause deployment conflicts with servlet containers.
+ -->
+ <dependency org="org.apache.shindig" name="shindig-common" transitive="false" rev="2.0.0" conf="deploy->default">
+ </dependency>
+ <dependency org="com.google.collections" name="google-collections" rev="latest.integration" conf="deploy->default">
+ </dependency>
+ <dependency org="commons-codec" name="commons-codec" rev="latest.integration" conf="deploy->default">
</dependency>
+
+
<dependency org="org.jdom" name="jdom" rev="1.1" conf="deploy->default">
</dependency>
<dependency org="org.json" name="json" rev="20080701" conf="deploy->default">
@@ -111,6 +125,7 @@
</dependency>
<exclude module="juel-api"/>
+ <exclude module="juel-impl"/>
<exclude module="spring"/>
<exclude module="struts"/>
<exclude module="webwork"/>
Added: incubator/wookie/trunk/shindig/dist/shindig-features-2.0.0.jar
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/shindig/dist/shindig-features-2.0.0.jar?rev=1100374&view=auto
==============================================================================
Binary file - no diff available.
Propchange: incubator/wookie/trunk/shindig/dist/shindig-features-2.0.0.jar
------------------------------------------------------------------------------
svn:mime-type = application/octet-stream
Modified: incubator/wookie/trunk/shindig/features/core.prefs/feature.xml
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/shindig/features/core.prefs/feature.xml?rev=1100374&r1=1100373&r2=1100374&view=diff
==============================================================================
--- incubator/wookie/trunk/shindig/features/core.prefs/feature.xml (original)
+++ incubator/wookie/trunk/shindig/features/core.prefs/feature.xml Fri May 6 21:04:39 2011
@@ -18,6 +18,7 @@
-->
<feature>
<name>core.prefs</name>
+ <dependency>globals</dependency>
<dependency>core.util</dependency>
<gadget>
<!--- wookie integration js -->
Modified: incubator/wookie/trunk/src/org/apache/wookie/util/gadgets/GadgetUtils.java
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/util/gadgets/GadgetUtils.java?rev=1100374&r1=1100373&r2=1100374&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/util/gadgets/GadgetUtils.java (original)
+++ incubator/wookie/trunk/src/org/apache/wookie/util/gadgets/GadgetUtils.java Fri May 6 21:04:39 2011
@@ -55,7 +55,7 @@ public class GadgetUtils {
/**
* The default Container ID
*/
- private static final String CONTAINER_ID = "wookie";
+ private static final String CONTAINER_ID = "default";
/**
* Create a new W3CWidget from the request supplied
Modified: incubator/wookie/trunk/src/org/apache/wookie/util/opensocial/OpenSocialUtils.java
URL: http://svn.apache.org/viewvc/incubator/wookie/trunk/src/org/apache/wookie/util/opensocial/OpenSocialUtils.java?rev=1100374&r1=1100373&r2=1100374&view=diff
==============================================================================
--- incubator/wookie/trunk/src/org/apache/wookie/util/opensocial/OpenSocialUtils.java (original)
+++ incubator/wookie/trunk/src/org/apache/wookie/util/opensocial/OpenSocialUtils.java Fri May 6 21:04:39 2011
@@ -35,9 +35,8 @@ public class OpenSocialUtils {
static Logger _logger = Logger.getLogger(OpenSocialUtils.class.getName());
- private static final String DOMAIN_ID = "wookie"; //$NON-NLS-1$
- // TODO once we have an API Key implementation, we can convey the actual container id rather than Wookie
- private static final String CONTAINER_ID = "wookie"; //$NON-NLS-1$
+ private static final String DOMAIN_ID = "default"; //$NON-NLS-1$
+ private static final String CONTAINER_ID = "default"; //$NON-NLS-1$
/**
@@ -45,7 +44,7 @@ public class OpenSocialUtils {
*
* VIEWER = who the current user is (ie. Instance.getUserId)
* OWNER = who the page belongs to (assumed to be Instance.getUserId until we change plugin API)
- * APP_ID = what gadget this is (i.e. Widget.guid)
+ * APP_ID/APP URL = what gadget this is (i.e. Widget URI)
* MOD_ID = which instance of it (i.e. Instance.id/idkey)
*
* Alas, MOD_ID doesn't really map onto instance.IdKey (incompatible types), so we just have to use instance.id