----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3078/ -----------------------------------------------------------
(Updated 2011-12-08 21:10:12.720302) Review request for rave. Changes ------- Handled the encoding Added DISABLED text to the gadget title to indicate the gadget is disabled Upated the property file Added tests 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-portal-resources/src/main/webapp/WEB-INF/jsp/views/admin/widgetdetail.jsp 1211897 https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/resources/messages.properties 1211897 https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/tag/RegionWidgetTag.java 1211897 https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-core/src/main/java/org/apache/rave/portal/model/Widget.java 1211932 https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/tag/AbstractContextAwareSingletonBeanDependentTag.java 1211897 https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/main/webapp/script/rave.js 1211897 https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-portal-resources/src/test/javascript/raveSpec.js 1211897 Diff: https://reviews.apache.org/r/3078/diff Testing ------- Thanks, Venkat
