-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5844/
-----------------------------------------------------------
(Updated July 16, 2012, 6:12 p.m.)
Review request for rave.
Changes
-------
Fixed patch file.
Description
-------
Allow dynamic debug mode switching. Preferences will now allow toggle between
debug on and off for container.js. An Event listening interface was created to
fire the change and the DefaultScriptManager implements the interface listening
for an event to happen.
This addresses bug RAVE-624.
https://issues.apache.org/jira/browse/RAVE-624
Diffs (updated)
-----
trunk/rave-components/rave-commons/src/main/java/org/apache/rave/persistence/jpa/impl/H2OpenJpaDialect.java
1362126
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/events/PortalPreferenceJavascriptDebugModeEventListener.java
PRE-CREATION
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/events/PortalPreferenceJavascriptDebugModeSaveEvent.java
PRE-CREATION
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/events/RaveEvent.java
PRE-CREATION
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/events/RaveEventListener.java
PRE-CREATION
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/events/RaveEventManager.java
PRE-CREATION
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/PortalPreferenceService.java
1362126
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/service/impl/DefaultPortalPreferenceService.java
1362126
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/web/renderer/ScriptManager.java
1362126
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/web/renderer/impl/DefaultScriptManager.java
1362126
trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/web/util/PortalPreferenceKeys.java
PRE-CREATION
trunk/rave-components/rave-core/src/main/resources/org/apache/rave/core-applicationContext.xml
1362126
trunk/rave-components/rave-core/src/test/java/org/apache/rave/portal/service/impl/DefaultPortalPreferenceServiceTest.java
1362126
trunk/rave-components/rave-core/src/test/java/org/apache/rave/portal/web/renderer/ScriptManagerTest.java
1362126
trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/controller/WidgetStoreController.java
1362126
trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/controller/admin/WidgetController.java
1362126
trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/tag/RegionWidgetTag.java
1362126
trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/tag/ScriptTag.java
1362126
trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/util/PortalPreferenceKeys.java
1362126
trunk/rave-components/rave-web/src/test/java/org/apache/rave/portal/web/controller/admin/PortalPreferenceControllerTest.java
1362126
trunk/rave-components/rave-web/src/test/java/org/apache/rave/portal/web/controller/admin/WidgetControllerTest.java
1362126
trunk/rave-components/rave-web/src/test/java/org/apache/rave/portal/web/tag/RegionWidgetTagTest.java
1362126
trunk/rave-portal-resources/src/main/resources/log4j.xml 1362126
trunk/rave-portal-resources/src/main/webapp/script/rave.js 1345097
trunk/rave-portal/pom.xml 1362126
trunk/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/config/OpenSocialEnvironment.java
1362126
trunk/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/main/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetRenderer.java
1362126
trunk/rave-providers/rave-opensocial-provider/rave-opensocial-client/src/test/java/org/apache/rave/provider/opensocial/web/renderer/OpenSocialWidgetRendererTest.java
1362126
trunk/rave-providers/rave-w3c-provider/src/main/java/org/apache/rave/provider/w3c/web/renderer/W3cWidgetRenderer.java
1362126
Diff: https://reviews.apache.org/r/5844/diff/
Testing
-------
Thanks,
Aaron Gooch
