Author: [email protected]
Date: Tue Dec 20 12:45:12 2011
New Revision: 1864
Log:
[AMDATUOPENSOCIAL-151] Added support for absolute URLs in whiteboard style
registered gadgets. In fact the only thing that prevented this from working was
the URL validation.
Modified:
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/service/InternalGadgetStoreImpl.java
Modified:
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/service/InternalGadgetStoreImpl.java
==============================================================================
---
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/service/InternalGadgetStoreImpl.java
(original)
+++
trunk/amdatu-opensocial/opensocial-gadgetmanagement/src/main/java/org/amdatu/opensocial/gadgetmanagement/service/InternalGadgetStoreImpl.java
Tue Dec 20 12:45:12 2011
@@ -97,22 +97,6 @@
}
private String isValid(GadgetDefinitionProvider provider) {
- for (GadgetDefinition gadgetDef : provider.getDefinitions()) {
- if (!gadgetDef.isRelativeUrl()) {
- String errorMsg = "The gadget definition URL '" +
gadgetDef.getUrl() + "' ";
- errorMsg += "' registered by provider '" + provider.getClass()
+ "' ";
- errorMsg += "is absolute. Providers can only register relative
URLs.";
- return errorMsg;
- }
-
- if (!gadgetDef.getUrl().startsWith("/")) {
- String errorMsg = "The gadget definition URL '" +
gadgetDef.getUrl() + "' ";
- errorMsg += "' registered by provider '" + provider.getClass()
+ "'";
- errorMsg += "should start with a /.";
- return errorMsg;
- }
- }
-
// Check that there are not URL conflicts: the gadget URL must be
unique
for (GadgetDefinition gadget : m_gadgets) {
for (GadgetDefinition providedGadgetDef :
provider.getDefinitions()) {
_______________________________________________
Amdatu-commits mailing list
[email protected]
http://lists.amdatu.org/mailman/listinfo/amdatu-commits
