You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by aw...@gmail.com on 2009/07/18 02:45:14 UTC

Rename config param from opensocial-0.8 to opensocial

Reviewers: plindner_linkedin.com, chabotc,

Message:
This changes the name of the config entry from "opensocial-0.8" (just
wrong, since it applies to all opensocial versions) to "opensocial".  As
a consequence of the Java config loading code, this necessitates
introducing an "opensocial" feature, and making "opensocial-0.9" a
trivial alias.

This includes a .php change, which is totally untested, but seemed the
right thing, so putting Chris on the review as well.



Please review this at http://codereview.appspot.com/95071

Affected files:
   M     config/container.js
   M     features/src/main/javascript/features/features.txt
   A     features/src/main/javascript/features/opensocial-0.9/feature.xml
   M     features/src/main/javascript/features/opensocial-current/feature.xml
   M      
java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/ActivityHandler.java
   M      
java/social-api/src/main/java/org/apache/shindig/social/opensocial/service/PersonHandler.java
   M      
java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/ActivityHandlerTest.java
   M      
java/social-api/src/test/java/org/apache/shindig/social/opensocial/service/PersonHandlerTest.java
   M     php/test/gadgets/ContainerConfigTest.php