rave-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matt Franklin" <mfrank...@apache.org>
Subject Re: Review Request: RAVE-624 dynamic debug preference
Date Sat, 28 Jul 2012 18:23:07 GMT


> On July 18, 2012, 4:29 p.m., Anthony Carlucci wrote:
> > Overall looks good, will make a few minor tweaks with commit:
> > 1) add apache header to PortalPreferenceKeys
> > 2) ignore pom.xml changes (looks like local changes you shouldn't have included)
> > 3) OpenSocialEnvironment - created a constant string for the value of the "debug
on" instead of inlining it

Please close if this has been applied


- Matt


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5844/#review9242
-----------------------------------------------------------


On July 16, 2012, 6:12 p.m., Aaron Gooch wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> 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
> -----
> 
>   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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message