Merge PR#19 from 'sieverssj/RAVE-1275'
diff --git a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateRegion.java b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateRegion.java
index 0a4f5ad..9ea89a8 100644
--- a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateRegion.java
+++ b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateRegion.java
@@ -20,6 +20,7 @@
 
 import javax.xml.bind.annotation.XmlTransient;
 import java.util.List;
+import java.util.Map;
 
 /**
  */
@@ -44,4 +45,19 @@
 
     void setLocked(boolean locked);
 
+    /**
+     * Generic property bag for extension of the region object.
+     *
+     * Rave makes no attempt to understand the shape of properties in the bag.
+     *
+     * @return a valid Map of String to JSON Serializable Object.
+     */
+    Map<String, Object> getProperties();
+
+    /**
+     * Overrides the current properties with a new set.
+     *
+     * @param properties a non-null map of string to JSON serializable objects
+     */
+    void setProperties(Map<String, Object> properties);
 }
diff --git a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateWidget.java b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateWidget.java
index 23e7bfb..0b64dfb 100644
--- a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateWidget.java
+++ b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/PageTemplateWidget.java
@@ -19,6 +19,7 @@
 package org.apache.rave.model;
 
 import javax.xml.bind.annotation.XmlTransient;
+import java.util.Map;
 
 /**
  */
@@ -46,4 +47,20 @@
     boolean isHideChrome();
 
     void setHideChrome(boolean hideChrome);
+
+    /**
+     * Generic property bag for extension of the page template widget object.
+     *
+     * Rave makes no attempt to understand the shape of properties in the bag.
+     *
+     * @return a valid Map of String to JSON Serializable Object.
+     */
+    Map<String, Object> getProperties();
+
+    /**
+     * Overrides the current properties with a new set.
+     *
+     * @param properties a non-null map of string to JSON serializable objects
+     */
+    void setProperties(Map<String, Object> properties);
 }
diff --git a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/Region.java b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/Region.java
index 93b0a1f..37fe534 100644
--- a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/Region.java
+++ b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/Region.java
@@ -20,6 +20,7 @@
 
 import javax.xml.bind.annotation.XmlTransient;
 import java.util.List;
+import java.util.Map;
 
 @XmlTransient
 public interface Region {
@@ -40,4 +41,20 @@
     boolean isLocked();
 
     void setLocked(boolean locked);
+
+    /**
+     * Generic property bag for extension of the region object.
+     *
+     * Rave makes no attempt to understand the shape of properties in the bag.
+     *
+     * @return a valid Map of String to JSON Serializable Object.
+     */
+    Map<String, Object> getProperties();
+
+    /**
+     * Overrides the current properties with a new set.
+     *
+     * @param properties a non-null map of string to JSON serializable objects
+     */
+    void setProperties(Map<String, Object> properties);
 }
diff --git a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/RegionWidget.java b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/RegionWidget.java
index 9fe3e7f..36cb82d 100644
--- a/rave-components/rave-core-api/src/main/java/org/apache/rave/model/RegionWidget.java
+++ b/rave-components/rave-core-api/src/main/java/org/apache/rave/model/RegionWidget.java
@@ -23,6 +23,7 @@
 
 import javax.xml.bind.annotation.XmlTransient;
 import java.util.List;
+import java.util.Map;
 
 @XmlTransient
 public interface RegionWidget {
@@ -60,4 +61,20 @@
     boolean isHideChrome();
 
     void setHideChrome(boolean hideChrome);
+
+    /**
+     * Generic property bag for extension of the region widget object.
+     *
+     * Rave makes no attempt to understand the shape of properties in the bag.
+     *
+     * @return a valid Map of String to JSON Serializable Object.
+     */
+    Map<String, Object> getProperties();
+
+    /**
+     * Overrides the current properties with a new set.
+     *
+     * @param properties a non-null map of string to JSON serializable objects
+     */
+    void setProperties(Map<String, Object> properties);
 }
diff --git a/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/Region.java b/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/Region.java
index 0d347bc..b207d60 100644
--- a/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/Region.java
+++ b/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/Region.java
@@ -22,6 +22,8 @@
 import javax.xml.bind.annotation.*;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Map;
+
 @XmlAccessorType(XmlAccessType.FIELD)
 @XmlType(name = "Region", propOrder = {
         "locked", "regionWidgets"
@@ -35,6 +37,8 @@
     @XmlElementWrapper(name = "regionWidgets")
     @XmlElement(name="RegionWidget")
     private List<RegionWidget> regionWidgets;
+    @XmlElement(name = "properties")
+    private Map<String, Object> properties;
 
     public Region() { }
 
@@ -42,6 +46,7 @@
         this.locked = source.isLocked();
         this.regionWidgets = createRegionWidgets(source);
         this.id = source.getId();
+        this.properties = source.getProperties();
     }
 
     public boolean isLocked() {
@@ -68,6 +73,14 @@
         this.regionWidgets = regionWidgets;
     }
 
+    public Map<String, Object> getProperties() {
+        return this.properties;
+    }
+
+    public void setProperties(Map<String, Object> properties) {
+        this.properties = properties;
+    }
+
     private List<RegionWidget> createRegionWidgets(org.apache.rave.model.Region source) {
         List<RegionWidget> created = null;
         List<org.apache.rave.model.RegionWidget> widgets = source.getRegionWidgets();
diff --git a/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/RegionWidget.java b/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/RegionWidget.java
index cef2499..28a70e2 100644
--- a/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/RegionWidget.java
+++ b/rave-components/rave-core-api/src/main/java/org/apache/rave/rest/model/RegionWidget.java
@@ -24,6 +24,7 @@
 import javax.xml.bind.annotation.*;
 import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 
 @XmlAccessorType(XmlAccessType.FIELD)
 @XmlType(name = "RegionWidget", propOrder = {
@@ -52,6 +53,8 @@
     protected String ownerId;
     @XmlElement(name = "userPrefs")
     protected HashMap<String, String> userPrefs;
+    @XmlElement(name = "properties")
+    protected Map<String,Object> properties;
 
     public RegionWidget() {  }
 
@@ -64,6 +67,7 @@
         this.ownerId = widget.getRegion().getPage().getOwnerId();
         this.hideChrome = widget.isHideChrome();
         this.userPrefs = createPrefs(widget);
+        this.properties = widget.getProperties();
     }
 
     public String getId() {
@@ -146,6 +150,14 @@
         this.ownerId = ownerId;
     }
 
+    public Map<String, Object> getProperties() {
+        return this.properties;
+    }
+
+    public void setProperties(Map<String, Object> properties) {
+        this.properties = properties;
+    }
+
     protected HashMap<String, String> createPrefs(org.apache.rave.model.RegionWidget widget) {
         HashMap<String, String> created = null;
         List<RegionWidgetPreference> preferences = widget.getPreferences();
diff --git a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateRegionImpl.java b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateRegionImpl.java
index 312bd98..df733e8 100644
--- a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateRegionImpl.java
+++ b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateRegionImpl.java
@@ -23,6 +23,7 @@
 import org.apache.rave.model.PageTemplateWidget;
 
 import java.util.List;
+import java.util.Map;
 
 public class PageTemplateRegionImpl implements PageTemplateRegion {
     private String id;
@@ -30,6 +31,7 @@
     private PageTemplate pageTemplate;
     private List<PageTemplateWidget> pageTemplateWidgets;
     private boolean locked;
+    private Map<String, Object> properties;
 
     public PageTemplateRegionImpl() {
 
@@ -80,6 +82,16 @@
     }
 
     @Override
+    public Map<String, Object> getProperties() {
+        return this.properties;
+    }
+
+    @Override
+    public void setProperties(Map<String, Object> properties) {
+        this.properties = properties;
+    }
+
+    @Override
     public boolean equals(Object o) {
         if (this == o) return true;
         if (!(o instanceof PageTemplateRegionImpl)) return false;
diff --git a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateWidgetImpl.java b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateWidgetImpl.java
index e7d610a..11898e3 100644
--- a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateWidgetImpl.java
+++ b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/PageTemplateWidgetImpl.java
@@ -21,6 +21,8 @@
 import org.apache.rave.model.PageTemplateRegion;
 import org.apache.rave.model.PageTemplateWidget;
 
+import java.util.Map;
+
 public class PageTemplateWidgetImpl implements PageTemplateWidget {
     private String id;
     private PageTemplateRegion pageTemplateRegion;
@@ -28,6 +30,7 @@
     private String widgetId;
     private boolean locked;
     private boolean hideChrome;
+    private Map<String, Object> properties;
 
     public PageTemplateWidgetImpl() {
 
@@ -86,6 +89,16 @@
     }
 
     @Override
+    public Map<String, Object> getProperties() {
+        return this.properties;
+    }
+
+    @Override
+    public void setProperties(Map<String, Object> properties) {
+        this.properties = properties;
+    }
+
+    @Override
     public boolean equals(Object o) {
         if (this == o) return true;
         if (!(o instanceof PageTemplateWidgetImpl)) return false;
diff --git a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionImpl.java b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionImpl.java
index 31f0c13..962fb80 100644
--- a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionImpl.java
+++ b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionImpl.java
@@ -24,6 +24,7 @@
 import com.fasterxml.jackson.annotation.JsonBackReference;
 
 import java.util.List;
+import java.util.Map;
 
 public class RegionImpl implements Region {
     private String id;
@@ -31,6 +32,7 @@
     private Boolean locked = false;
     private Integer renderOrder = 0;
     private List<RegionWidget> regionWidgets;
+    private Map<String, Object> properties;
 
     public RegionImpl() {
 
@@ -97,6 +99,16 @@
     }
 
     @Override
+    public Map<String, Object> getProperties() {
+        return this.properties;
+    }
+
+    @Override
+    public void setProperties(Map<String, Object> properties) {
+        this.properties = properties;
+    }
+
+    @Override
     public boolean equals(Object o) {
         if (this == o) return true;
         if (!(o instanceof RegionImpl)) return false;
diff --git a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionWidgetImpl.java b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionWidgetImpl.java
index 4d6ac6e..4104b9b 100644
--- a/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionWidgetImpl.java
+++ b/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/impl/RegionWidgetImpl.java
@@ -24,6 +24,7 @@
 import org.apache.rave.model.RegionWidgetPreference;
 
 import java.util.List;
+import java.util.Map;
 
 public class RegionWidgetImpl implements RegionWidget {
     private String id;
@@ -34,6 +35,7 @@
     private List<RegionWidgetPreference> preferences;
     private Boolean locked = false;
     private Boolean hideChrome = false;
+    private Map<String,Object> properties;
 
     @JsonBackReference
     private Region region;
@@ -149,6 +151,16 @@
     }
 
     @Override
+    public Map<String, Object> getProperties() {
+        return this.properties;
+    }
+
+    @Override
+    public void setProperties(Map<String, Object> properties) {
+        this.properties = properties;
+    }
+
+    @Override
     public boolean equals(Object o) {
         if (this == o) return true;
         if (!(o instanceof RegionWidgetImpl)) return false;
diff --git a/rave-components/rave-core/src/main/java/org/apache/rave/portal/util/PageUtil.java b/rave-components/rave-core/src/main/java/org/apache/rave/portal/util/PageUtil.java
index ee8037d..c2021f9 100644
--- a/rave-components/rave-core/src/main/java/org/apache/rave/portal/util/PageUtil.java
+++ b/rave-components/rave-core/src/main/java/org/apache/rave/portal/util/PageUtil.java
@@ -82,6 +82,7 @@
             region.setPage(page);
             region.setLocked(ptr.isLocked());
             region.setRegionWidgets(convertWidgets(ptr.getPageTemplateWidgets(), region, createIds));
+            region.setProperties(ptr.getProperties());
             regions.add(region);
         }
         return regions;
@@ -110,6 +111,7 @@
             regionWidget.setHideChrome(ptw.isHideChrome());
             regionWidget.setRenderOrder((int) ptw.getRenderSeq());
             regionWidget.setWidgetId(ptw.getWidgetId());
+            regionWidget.setProperties(ptw.getProperties());
             widgets.add(regionWidget);
         }
         return widgets;
diff --git a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateRegion.java b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateRegion.java
index c6048b4..349320b 100644
--- a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateRegion.java
+++ b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateRegion.java
@@ -19,6 +19,7 @@
 

 package org.apache.rave.portal.model;

 

+import com.google.common.collect.Maps;

 import org.apache.rave.model.PageTemplate;

 import org.apache.rave.model.PageTemplateRegion;

 import org.apache.rave.model.PageTemplateWidget;

@@ -29,6 +30,7 @@
 import java.io.Serializable;

 import java.util.ArrayList;

 import java.util.List;

+import java.util.Map;

 

 @Entity

 @Table(name = "page_template_region")

@@ -127,4 +129,14 @@
     public void setLocked(boolean locked) {

         this.locked = locked;

     }

+

+    @Override

+    public Map<String, Object> getProperties() {

+        return Maps.newHashMap();

+    }

+

+    @Override

+    public void setProperties(Map<String, Object> properties) {

+        // TODO: Implement this

+    }

 }

diff --git a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateWidget.java b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateWidget.java
index fbbd6b1..dba1307 100644
--- a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateWidget.java
+++ b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaPageTemplateWidget.java
@@ -18,12 +18,14 @@
  */

 package org.apache.rave.portal.model;

 

+import com.google.common.collect.Maps;

 import org.apache.rave.model.PageTemplateRegion;

 import org.apache.rave.model.PageTemplateWidget;

 import org.apache.rave.portal.model.conversion.JpaConverter;

 

 import javax.persistence.*;

 import java.io.Serializable;

+import java.util.Map;

 

 @Entity

 @Table(name= "page_template_widget")

@@ -127,4 +129,14 @@
     public void setHideChrome(boolean hideChrome) {

         this.hideChrome = hideChrome;

     }

+

+    @Override

+    public Map<String, Object> getProperties() {

+        return Maps.newHashMap();

+    }

+

+    @Override

+    public void setProperties(Map<String, Object> properties) {

+        // TODO: Implement this

+    }

 }

diff --git a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegion.java b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegion.java
index 9ad1887..823b68b 100644
--- a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegion.java
+++ b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegion.java
@@ -19,6 +19,7 @@
 package org.apache.rave.portal.model;
 
 import com.fasterxml.jackson.annotation.JsonManagedReference;
+import com.google.common.collect.Maps;
 import org.apache.rave.model.Page;
 import org.apache.rave.model.Region;
 import org.apache.rave.model.RegionWidget;
@@ -33,6 +34,7 @@
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Map;
 
 /**
  * A region of a page, which can contain widget instances {@link RegionWidget}
@@ -193,6 +195,16 @@
     }
 
     @Override
+    public Map<String, Object> getProperties() {
+        return Maps.newHashMap();
+    }
+
+    @Override
+    public void setProperties(Map<String, Object> properties) {
+        // TODO: Implement this
+    }
+
+    @Override
     public boolean equals(Object obj) {
         if (obj == null) {
             return false;
diff --git a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegionWidget.java b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegionWidget.java
index 2ee328d..62f73ff 100644
--- a/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegionWidget.java
+++ b/rave-components/rave-jpa/src/main/java/org/apache/rave/portal/model/JpaRegionWidget.java
@@ -18,6 +18,7 @@
  */
 package org.apache.rave.portal.model;
 
+import com.google.common.collect.Maps;
 import org.apache.rave.model.Region;
 import org.apache.rave.model.RegionWidget;
 import org.apache.rave.model.RegionWidgetPreference;
@@ -29,6 +30,7 @@
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.List;
+import java.util.Map;
 
 /**
  * A widget within a region
@@ -262,6 +264,16 @@
     }
 
     @Override
+    public Map<String, Object> getProperties() {
+        return Maps.newHashMap();
+    }
+
+    @Override
+    public void setProperties(Map<String, Object> properties) {
+        // TODO: Implement this
+    }
+
+    @Override
     public boolean equals(Object obj) {
         if (obj == null) {
             return false;
diff --git a/rave-components/rave-mongodb/src/main/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageConverter.java b/rave-components/rave-mongodb/src/main/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageConverter.java
index 04ffaef..63b94fa 100644
--- a/rave-components/rave-mongodb/src/main/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageConverter.java
+++ b/rave-components/rave-mongodb/src/main/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageConverter.java
@@ -169,6 +169,7 @@
         String regionId = region.getId() == null ? generateId() : region.getId();
         Region converted = new RegionImpl(regionId, null, region.getRenderOrder());
         converted.setLocked(region.isLocked());
+        converted.setProperties(region.getProperties());
         if (region.getRegionWidgets() != null) {
             List<RegionWidget> convertedWidgets = Lists.newArrayList();
             for (RegionWidget widget : region.getRegionWidgets()) {
diff --git a/rave-components/rave-mongodb/src/test/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageTemplateConverterTest.java b/rave-components/rave-mongodb/src/test/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageTemplateConverterTest.java
index ae19bb4..cd5c0d1 100644
--- a/rave-components/rave-mongodb/src/test/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageTemplateConverterTest.java
+++ b/rave-components/rave-mongodb/src/test/java/org/apache/rave/portal/model/conversion/impl/MongoDbPageTemplateConverterTest.java
@@ -30,6 +30,7 @@
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
+import java.util.Map;
 
 import static org.easymock.EasyMock.*;
 import static org.hamcrest.CoreMatchers.*;
@@ -200,7 +201,17 @@
                 public void setLocked(boolean locked) {
                     //To change body of implemented methods use File | Settings | File Templates.
                 }
-            };
+
+                @Override
+                public Map<String, Object> getProperties() {
+                    return null;
+                }
+
+                @Override
+                public void setProperties(Map<String, Object> properties) {
+
+                }
+        };
     }
 
     @Test
diff --git a/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/model/OpenSocialRegionWidget.java b/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/model/OpenSocialRegionWidget.java
index 6aaebda..5cdfcb3 100644
--- a/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/model/OpenSocialRegionWidget.java
+++ b/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/model/OpenSocialRegionWidget.java
@@ -54,6 +54,7 @@
         this.hideChrome = base.isHideChrome();
         this.ownerId = base.getOwnerId();
         this.userPrefs = base.getUserPrefs();
+        this.properties = base.getProperties();
     }
 
     public String getSecurityToken() {
diff --git a/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetWrapperRenderer.java b/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetWrapperRenderer.java
index 04aa618..aaf6203 100644
--- a/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetWrapperRenderer.java
+++ b/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetWrapperRenderer.java
@@ -76,7 +76,9 @@
             " locked: %10$s," +
             " hideChrome: %11$s," +
             " subPage: {id: %12$s, name: '%13$s', isDefault: %14$s}," +
-            " properties: %15$s" +
+            " properties: %15$s," +
+            " regionProperties: %16$s," +
+            " regionWidgetProperties: %17$s" +
             "})});</script>";
     private static final String MARKUP = "<!-- RegionWidget '%1$s' placeholder -->";
 
@@ -157,7 +159,9 @@
                 pageId,
                 pageName,
                 isDefault,
-                JsonUtils.stringify(widget.getProperties())
+                JsonUtils.stringify(widget.getProperties()),
+                JsonUtils.stringify(item.getRegion().getProperties()),
+                JsonUtils.stringify(item.getProperties())
                 );
     }
 
diff --git a/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/test/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetRendererTest.java b/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/test/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetRendererTest.java
index b81e108..30f73e2 100644
--- a/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/test/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetRendererTest.java
+++ b/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/test/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetRendererTest.java
@@ -19,6 +19,7 @@
 
 package org.apache.rave.provider.opensocial.web.renderer;
 
+import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Maps;
 import org.apache.rave.exception.NotSupportedException;
 import org.apache.rave.model.*;
@@ -103,6 +104,7 @@
         props.put("foo","bar");
 
         Region region = new RegionImpl(REGION_ID);
+        region.setProperties(ImmutableMap.<String, Object>of("regionFoo", "regionBar"));
         region.setPage(subPage);
         RegionWidget rw = new RegionWidgetImpl(REGION_WIDGET_ID);
         rw.setCollapsed(VALID_COLLAPSED);
@@ -113,6 +115,7 @@
         rw.setPreferences(Arrays.asList((RegionWidgetPreference) new RegionWidgetPreferenceImpl("1", "color", "blue"),
                 new RegionWidgetPreferenceImpl("1", "speed", "fast"),
                 new RegionWidgetPreferenceImpl("1", null, null)));
+        rw.setProperties(ImmutableMap.<String, Object>of("rwFoo", "rwBar"));
 
         final String markup =
                 "<script>require(['rave'], function(rave){" +
@@ -127,7 +130,9 @@
                         " locked: " + VALID_LOCKED + "," +
                         " hideChrome: " + VALID_HIDE_CHROME + "," +
                         " subPage: {id: '" + VALID_SUBPAGE_ID + "', name: '" + VALID_SUBPAGE_NAME + "', isDefault: " + VALID_IS_DEFAULT_SUBPAGE + "}," +
-                        " properties: {\"foo\":\"bar\"}" +
+                        " properties: {\"foo\":\"bar\"}," +
+                        " regionProperties: {\"regionFoo\":\"regionBar\"}," +
+                        " regionWidgetProperties: {\"rwFoo\":\"rwBar\"}" +
                         "})" +
                         "});</script>";
 
@@ -178,7 +183,9 @@
                         " locked: false," +
                         " hideChrome: false," +
                         " subPage: {id: null, name: '', isDefault: false}," +
-                        " properties: null" +
+                        " properties: null," +
+                        " regionProperties: null," +
+                        " regionWidgetProperties: null" +
                         "})" +
                         "});</script>";