Skip to main content

Source code revision

svn

summary: http://java.net/jira/browse/JAVASERVERFACES-3005 Re-enable Groovy support
revision: 12449
author: edburns
date: 2013-08-26 12:55:34 UTC (3 years)
message: http://java.net/jira/browse/JAVASERVERFACES-3005 Re-enable Groovy support

Backport from 2.2.x. 558 tests run successfully.

Ever since the 2012-08-12 commit for JAVASERVERFACES-2491 Groovy support
has been broken and the automated tests that asserted its functionality
have been commented out. I was never content with this resolution.

This commit re-activates Groovy support without invalifating the fix for
JAVASERVERFACES-2491.

Ran the agnostic and web-profile tests with

-Dwebapp.projectStage=Development -Dwebapp.stateSavingMethod=server -Dwebapp.partialStateSaving=false

Ran the new test as well.

SECTION: Modified Files
----------------------------
M jsf-ri/src/main/java/com/sun/faces/scripting/groovy/GroovyHelperImpl.java

- Make some things in here public.

M jsf-ri/src/main/java/com/sun/faces/util/Util.java

- In loadClass() use the newly publicized things to cause the
performance penalty of Groovy support to only be paid if Groovy is actually enabled.

A test/scripting
A test/scripting/groovy
A test/scripting/groovy/basic
A test/scripting/groovy/basic/src
A test/scripting/groovy/basic/src/main
A test/scripting/groovy/basic/src/main/java
A test/scripting/groovy/basic/src/main/java/com
A test/scripting/groovy/basic/src/main/java/com/sun
A test/scripting/groovy/basic/src/main/java/com/sun/faces
A test/scripting/groovy/basic/src/main/java/com/sun/faces/test
A test/scripting/groovy/basic/src/main/java/com/sun/faces/test/scripting
A test/scripting/groovy/basic/src/main/java/com/sun/faces/test/scripting/groovy
A + test/scripting/groovy/basic/src/main/java/com/sun/faces/test/scripting/groovy/Name.java
A test/scripting/groovy/basic/src/main/webapp
A test/scripting/groovy/basic/src/main/webapp/WEB-INF
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/faces-config.xml
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy-taglib.xml
A test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy
A test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/MessageRenderer.groovy
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/AgeComponent.groovy
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/MessageOutput.groovy
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/NameValidator.groovy
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/NameConverter.groovy
A + test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/HelloBean.groovy
A + test/scripting/groovy/basic/src/main/webapp/hello.xhtml
A + test/scripting/groovy/basic/src/main/webapp/submit.xhtml

- Migrated test from old test harness to new area in new harness. This
test will not be run until
is fixed.

Change Path Actions
M branches/MOJARRA_2_1X_ROLLING/jsf-ri/src/main/java/com/sun/faces/scripting/groovy/GroovyHelperImpl.java
M branches/MOJARRA_2_1X_ROLLING/jsf-ri/src/main/java/com/sun/faces/util/Util.java
A branches/MOJARRA_2_1X_ROLLING/test/scripting
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/nbactions.xml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/pom.xml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java/com
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java/com/sun
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java/com/sun/faces
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java/com/sun/faces/test
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java/com/sun/faces/test/scripting
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java/com/sun/faces/test/scripting/groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/java/com/sun/faces/test/scripting/groovy/Name.java
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/resources
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/resources/runtime_overwrites
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/resources/runtime_overwrites/groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/resources/runtime_overwrites/groovy/hello
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/resources/runtime_overwrites/groovy/hello/AgeComponent.groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/faces-config.xml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/glassfish-web.xml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy-taglib.xml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/AgeComponent.groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/HelloBean.groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/MessageOutput.groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/MessageRenderer.groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/NameConverter.groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/groovy/hello/NameValidator.groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/WEB-INF/web.xml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/hello.xhtml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/main/webapp/submit.xhtml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java/com
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java/com/sun
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java/com/sun/faces
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java/com/sun/faces/test
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java/com/sun/faces/test/scripting
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java/com/sun/faces/test/scripting/groovy
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/basic/src/test/java/com/sun/faces/test/scripting/groovy/Issue1655IT.java
A branches/MOJARRA_2_1X_ROLLING/test/scripting/groovy/pom.xml
A branches/MOJARRA_2_1X_ROLLING/test/scripting/pom.xml
 
 
Close
loading
Please Confirm
Close