You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by we...@apache.org on 2012/03/09 14:48:15 UTC
svn commit: r1298824 - in
/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting:
core/common/util/ core/context/ core/engine/dependencyScan/loaders/
core/monitor/ core/reloa...
Author: werpu
Date: Fri Mar 9 13:48:14 2012
New Revision: 1298824
URL: http://svn.apache.org/viewvc?rev=1298824&view=rev
Log:
EXTSCRIPT-154: Code Rewrite/Refactoring, save state of affairs, we now have to work on the refresh code, we have a phase listener working and recompiling, but the reloading yet fails on proxy level.
Modified:
myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/common/util/ClassUtils.java
myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/context/WeavingContext.java
myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/engine/dependencyScan/loaders/ScannerClassloader.java
myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/monitor/ClassResource.java
myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/MethodLevelReloadingHandler.java
myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/SimpleReloadingStrategy.java
myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/jsf/dynamicDecorators/implementations/ELResolverProxy.java
Modified: myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/common/util/ClassUtils.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/common/util/ClassUtils.java?rev=1298824&r1=1298823&r2=1298824&view=diff
==============================================================================
--- myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/common/util/ClassUtils.java (original)
+++ myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/common/util/ClassUtils.java Fri Mar 9 13:48:14 2012
@@ -76,6 +76,9 @@ public class ClassUtils
{
String className = relativeFileName.replaceAll("\\\\", ".").replaceAll("\\/", ".");
className = className.substring(0, className.lastIndexOf("."));
+ if(className.startsWith(".")) {
+ className = className.substring(1);
+ }
return className;
}
Modified: myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/context/WeavingContext.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/context/WeavingContext.java?rev=1298824&r1=1298823&r2=1298824&view=diff
==============================================================================
--- myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/context/WeavingContext.java (original)
+++ myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/context/WeavingContext.java Fri Mar 9 13:48:14 2012
@@ -201,7 +201,7 @@ public class WeavingContext
/**
* reload the class dynamically
*/
- public Class reloadClass(Class clazz)
+ public Class reload(Class clazz)
{
if (!isDynamic(clazz)) return clazz;
ClassResource resource = (ClassResource) getResource(clazz.getName());
Modified: myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/engine/dependencyScan/loaders/ScannerClassloader.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/engine/dependencyScan/loaders/ScannerClassloader.java?rev=1298824&r1=1298823&r2=1298824&view=diff
==============================================================================
--- myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/engine/dependencyScan/loaders/ScannerClassloader.java (original)
+++ myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/engine/dependencyScan/loaders/ScannerClassloader.java Fri Mar 9 13:48:14 2012
@@ -80,6 +80,7 @@ public class ScannerClassloader extends
return super.loadClass(className);
}
+
//ClassResource data = WeavingContext.getFileChangedDaemon().getClassMap().get(className);
//if (data != null && !data.getRefreshAttribute().requiresRefresh()) {
// return data.getAClass();
@@ -98,10 +99,13 @@ public class ScannerClassloader extends
iStream = new FileInputStream(target);
int len = iStream.read(fileContent);
if (_logger.isLoggable(Level.FINER)) {
- _logger.log(Level.FINER, "class read {0}� bytes read", String.valueOf(len));
+ _logger.log(Level.FINER, "class read {0} bytes read", String.valueOf(len));
}
}
+ if(className.contains("TestResourceHandler")){
+ System.out.println("debugpoint found");
+ }
//we have to do it here because just in case
//a dependent class is loaded as well we run into classcast exceptions
Class retVal = super.defineClass(className, fileContent, 0, fileLength);
Modified: myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/monitor/ClassResource.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/monitor/ClassResource.java?rev=1298824&r1=1298823&r2=1298824&view=diff
==============================================================================
--- myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/monitor/ClassResource.java (original)
+++ myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/monitor/ClassResource.java Fri Mar 9 13:48:14 2012
@@ -140,6 +140,7 @@ public class ClassResource extends Watch
{
String targetDir = WeavingContext.getInstance().getConfiguration().getCompileTarget().getAbsolutePath();
String className = ClassUtils.relativeFileToClassName(getSourceFile());
+
return className;
}
Modified: myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/MethodLevelReloadingHandler.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/MethodLevelReloadingHandler.java?rev=1298824&r1=1298823&r2=1298824&view=diff
==============================================================================
--- myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/MethodLevelReloadingHandler.java (original)
+++ myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/MethodLevelReloadingHandler.java Fri Mar 9 13:48:14 2012
@@ -85,7 +85,7 @@ public class MethodLevelReloadingHandler
if (_delegate == null) {
//stateless or lost state due to a lifecycle iteration we trigger anew
- _delegate = (WeavingContext.getInstance().reloadClass(_loadedClass)).newInstance();
+ _delegate = (WeavingContext.getInstance().reload(_loadedClass)).newInstance();
} else {
//if we are stateful only a tainted artifact is reloaded
_delegate = WeavingContext.getInstance().reload(_delegate, _artifactType);
Modified: myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/SimpleReloadingStrategy.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/SimpleReloadingStrategy.java?rev=1298824&r1=1298823&r2=1298824&view=diff
==============================================================================
--- myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/SimpleReloadingStrategy.java (original)
+++ myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/core/reloading/SimpleReloadingStrategy.java Fri Mar 9 13:48:14 2012
@@ -61,7 +61,7 @@ public class SimpleReloadingStrategy imp
public Object reload(Object scriptingInstance, int artifactType) {
//reload the class to get new static content if needed
- Class aclass = WeavingContext.getInstance().reloadClass(scriptingInstance.getClass());
+ Class aclass = WeavingContext.getInstance().reload(scriptingInstance.getClass());
if (aclass.hashCode() == scriptingInstance.getClass().hashCode()) {
//class of this object has not changed although
// reload is enabled we can skip the rest now
Modified: myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/jsf/dynamicDecorators/implementations/ELResolverProxy.java
URL: http://svn.apache.org/viewvc/myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/jsf/dynamicDecorators/implementations/ELResolverProxy.java?rev=1298824&r1=1298823&r2=1298824&view=diff
==============================================================================
--- myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/jsf/dynamicDecorators/implementations/ELResolverProxy.java (original)
+++ myfaces/extensions/scripting/trunk/extscript-core-root/extscript-core/src/main/java/rewrite/org/apache/myfaces/extensions/scripting/jsf/dynamicDecorators/implementations/ELResolverProxy.java Fri Mar 9 13:48:14 2012
@@ -77,7 +77,7 @@ public class ELResolverProxy extends ELR
public Class<?> getType(ELContext elContext, Object o, Object o1) throws NullPointerException, ELException {
Class<?> retVal = _delegate.getType(elContext, o, o1);
if (retVal != null && WeavingContext.getInstance().isDynamic(retVal)) {
- return WeavingContext.getInstance().reloadClass(retVal);
+ return WeavingContext.getInstance().reload(retVal);
}
return retVal;
}