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

(Updated 2011-12-09 19:38:25.650053)


Review request for rave.


Changes
-------

Removed '-DISABLED' from title as suggested
Added Dutch language to messages_nl.properties file (using Google translate)
Created new JIRA ticket to address the broader Javascript security issues: 
https://issues.apache.org/jira/browse/RAVE-385


Summary
-------

RAVE-210  Administrators should be able to disable widgets

To disable a gadget, login as Admin and go to "Admin Interface" and to the 
Widgets tab and click on a widget to get its metadata. Here you will find two 
new columns, "Disable Gadget" and "Disable Gadget Message". One can disable the 
gadget from rendering if you check he disable gadget checkbox and whatever you 
put in the message will be displayed on the gadget when the gadget is rendered. 
If you want to enable it, uncheck the checkbox. The gadget will be displayed as 
usual (even if the disable message is still there in the metadata, since the 
checkbox overrides it).


Diffs (updated)
-----

  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/Widget.java
 1212554 
  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/tag/AbstractContextAwareSingletonBeanDependentTag.java
 1212554 
  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/tag/RegionWidgetTag.java
 1212554 
  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/resources/messages.properties
 1212554 
  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/resources/messages_nl.properties
 1212554 
  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/webapp/WEB-INF/jsp/views/admin/widgetdetail.jsp
 1212554 
  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/webapp/script/rave.js
 1212554 
  
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/test/javascript/raveSpec.js
 1212554 

Diff: https://reviews.apache.org/r/3078/diff


Testing
-------


Thanks,

Venkat

Reply via email to