July 2012 - Jaxx-commits - mailman.cloud.codelutin.com

r2382 - trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application
by tchemit＠users.nuiton.org July 5, 2012

July 5, 2012

Author: tchemit Date: 2012-07-05 16:56:58 +0200 (Thu, 05 Jul 2012) New Revision: 2382 Url: http://nuiton.org/repositories/revision/jaxx/2382 Log: fixes #2163 Improve ActionWorker api Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java 2012-07-05 13:51:54 UTC (rev 2381) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java 2012-07-05 14:56:58 UTC (rev 2382) @@ -24,11 +24,13 @@ */ package jaxx.runtime.swing.application; +import com.google.common.base.Preconditions; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.util.StringUtil; import javax.swing.SwingWorker; +import java.util.concurrent.Callable; /** * Action worker to execute a incoming action. @@ -43,7 +45,7 @@ protected final String actionLabel; - protected Runnable target; + protected Callable<R> target; protected ActionStatus status; @@ -58,38 +60,39 @@ } public ActionWorker(String actionLabel, Runnable target) { - this.target = target; + this.target = new RunnableBridge<R>(target); this.actionLabel = actionLabel; } - public Runnable getTarget() { + public Callable<R> getTarget() { return target; } public void setTarget(Runnable target) { - this.target = target; + setTarget(new RunnableBridge<R>(target)); } /** * Set the target to execute. * - * @param targetWithException the target to execute + * @param target the target to execute * @since 2.5.1 */ - public void setTarget(RunnableWithException targetWithException) { - setTarget(new RunnableBridge(targetWithException)); + public void setTarget(Callable<R> target) { + this.target = target; } @Override protected R doInBackground() throws Exception { + Preconditions.checkNotNull(target != null, "Targe field can not be null here."); startTime = System.nanoTime(); if (log.isDebugEnabled()) { log.debug("Action [" + getActionLabel() + "] is starting..."); } + R result = null; try { - getTarget().run(); - } catch (RunnableBridgeException e) { - error = (Exception) e.getCause(); + + result = getTarget().call(); } catch (Exception e) { error = e; } finally { @@ -97,7 +100,7 @@ log.debug("Action [" + getActionLabel() + "] is ending..."); } } - return null; + return result; } public boolean isFailed() { @@ -152,47 +155,23 @@ } /** - * Contract like {@link Runnable} but with exception autorized. + * Transform a {@link Runnable} into a {@link Callable}. * + * @param <R> type of return (used to maintain generic checks in this class). * @since 2.5.1 */ - public static interface RunnableWithException { + private static class RunnableBridge<R> implements Callable<R> { - void run() throws Exception; - } + private final Runnable target; - /** - * This exception is used by a {@link Runnable} when executing a - * {@link RunnableWithException} inside the - * {@link ActionWorker#doInBackground()} - * method. - * - * @since 2.5.1 - */ - private static class RunnableBridgeException extends RuntimeException { - - private static final long serialVersionUID = 1L; - - private RunnableBridgeException(Throwable cause) { - super(cause); - } - } - - public static class RunnableBridge implements Runnable { - - private final RunnableWithException target; - - public RunnableBridge(RunnableWithException target) { + public RunnableBridge(Runnable target) { this.target = target; } @Override - public void run() { - try { - target.run(); - } catch (Exception e) { - throw new RunnableBridgeException(e); - } + public R call() throws Exception { + target.run(); + return null; } }

1 0

r2381 - trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application
by tchemit＠users.nuiton.org July 5, 2012

July 5, 2012

Author: tchemit Date: 2012-07-05 15:51:54 +0200 (Thu, 05 Jul 2012) New Revision: 2381 Url: http://nuiton.org/repositories/revision/jaxx/2381 Log: #2163: Improve ActionWorker api Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java 2012-07-04 16:12:58 UTC (rev 2380) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/application/ActionWorker.java 2012-07-05 13:51:54 UTC (rev 2381) @@ -70,6 +70,16 @@ this.target = target; } + /** + * Set the target to execute. + * + * @param targetWithException the target to execute + * @since 2.5.1 + */ + public void setTarget(RunnableWithException targetWithException) { + setTarget(new RunnableBridge(targetWithException)); + } + @Override protected R doInBackground() throws Exception { startTime = System.nanoTime(); @@ -78,6 +88,8 @@ } try { getTarget().run(); + } catch (RunnableBridgeException e) { + error = (Exception) e.getCause(); } catch (Exception e) { error = e; } finally { @@ -139,4 +151,49 @@ FAIL } + /** + * Contract like {@link Runnable} but with exception autorized. + * + * @since 2.5.1 + */ + public static interface RunnableWithException { + + void run() throws Exception; + } + + /** + * This exception is used by a {@link Runnable} when executing a + * {@link RunnableWithException} inside the + * {@link ActionWorker#doInBackground()} + * method. + * + * @since 2.5.1 + */ + private static class RunnableBridgeException extends RuntimeException { + + private static final long serialVersionUID = 1L; + + private RunnableBridgeException(Throwable cause) { + super(cause); + } + } + + public static class RunnableBridge implements Runnable { + + private final RunnableWithException target; + + public RunnableBridge(RunnableWithException target) { + this.target = target; + } + + @Override + public void run() { + try { + target.run(); + } catch (Exception e) { + throw new RunnableBridgeException(e); + } + } + } + }

1 0

r2380 - in trunk: jaxx-compiler/src/main/java/jaxx/compiler/tags/validator jaxx-demo/src/main/java/jaxx/demo/feature/validation
by tchemit＠users.nuiton.org July 4, 2012

July 4, 2012

Author: tchemit Date: 2012-07-04 18:12:58 +0200 (Wed, 04 Jul 2012) New Revision: 2380 Url: http://nuiton.org/repositories/revision/jaxx/2380 Log: fixes #2161: Can't use 'errors' id for SwingValidatorMessageTableModel Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/BeanValidatorHandler.java trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationListDemo.jaxx trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationTableDemo.jaxx Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/BeanValidatorHandler.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/BeanValidatorHandler.java 2012-07-04 16:02:22 UTC (rev 2379) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/BeanValidatorHandler.java 2012-07-04 16:12:58 UTC (rev 2380) @@ -71,12 +71,10 @@ public static final String ERROR_LIST_ATTRIBUTE = "errorList"; public static final String ERROR_TABLE_ATTRIBUTE = "errorTable"; - //TODO-TC20091024 should change this default value to errorListModel - public static final String ERROR_LIST_MODEL_DEFAULT = "errors"; - //TODO-TC20091024 should change this default value to errorTableModel + public static final String ERROR_LIST_MODEL_DEFAULT = "errorListModel"; - public static final String ERROR_TABLE_MODEL_DEFAULT = "errors2"; + public static final String ERROR_TABLE_MODEL_DEFAULT = "errorTableModel"; public static final String ERROR_LIST_DEFAULT = "errorList"; Modified: trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationListDemo.jaxx =================================================================== --- trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationListDemo.jaxx 2012-07-04 16:02:22 UTC (rev 2379) +++ trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationListDemo.jaxx 2012-07-04 16:12:58 UTC (rev 2380) @@ -57,10 +57,8 @@ <Identity id='identity'/>  -  -  - <SwingValidatorMessageListModel id='errors' - onContentsChanged='ok.setEnabled(errors.isEmpty())'/> + <SwingValidatorMessageListModel id='errorListModel' + onContentsChanged='ok.setEnabled(errorListModel.isEmpty())'/>  <BeanValidator id='validator' bean='model1' @@ -353,7 +351,7 @@ layout='{new GridLayout()}' height='200' width='500'> <JScrollPane> - <JList id='errorList' model='{errors}' + <JList id='errorList' model='{errorListModel}' cellRenderer='{new SwingValidatorMessageListRenderer()}'/> </JScrollPane> </JPanel> Modified: trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationTableDemo.jaxx =================================================================== --- trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationTableDemo.jaxx 2012-07-04 16:02:22 UTC (rev 2379) +++ trunk/jaxx-demo/src/main/java/jaxx/demo/feature/validation/ValidationTableDemo.jaxx 2012-07-04 16:12:58 UTC (rev 2380) @@ -58,8 +58,8 @@ <Identity id='identity'/>  - <SwingValidatorMessageTableModel id='errors2' - onTableChanged='ok.setEnabled(errors2.getRowCount()==0)'/> + <SwingValidatorMessageTableModel id='errorTableModel' + onTableChanged='ok.setEnabled(errorTableModel.getRowCount()==0)'/>  <BeanValidator id='validator' bean='model1' @@ -354,7 +354,7 @@ layout='{new GridLayout()}' height='200' width='500'> <JScrollPane columnHeaderView='{errorTable.getTableHeader()}'> - <JTable id='errorTable' model='{errors2}' rowSelectionAllowed='true' + <JTable id='errorTable' model='{errorTableModel}' rowSelectionAllowed='true' autoCreateRowSorter='true' autoResizeMode='2' cellSelectionEnabled='false' selectionMode='0'/>

1 0

July 4, 2012

Author: tchemit Date: 2012-07-04 18:02:22 +0200 (Wed, 04 Jul 2012) New Revision: 2379 Url: http://nuiton.org/repositories/revision/jaxx/2379 Log: fixes #2162: Updates to commons-lang3 Modified: trunk/jaxx-compiler/pom.xml trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/DefaultCompilerConfiguration.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataListener.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaFile.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/DefaultObjectHandler.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/JScrollPaneHandler.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/TabHandler.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/FieldValidatorHandler.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/PrintTagInfo.java trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/jaxxcapture/JAXXCapture.java trunk/jaxx-demo/pom.xml trunk/jaxx-demo/src/main/java/jaxx/demo/entities/Movie.java trunk/jaxx-demo/src/main/java/jaxx/demo/entities/People.java trunk/jaxx-maven-plugin/pom.xml trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/AbstractGenerateHelpMojo.java trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/GenerateMojo.java trunk/jaxx-runtime/pom.xml trunk/jaxx-runtime/src/main/java/jaxx/runtime/context/JAXXInitialContext.java trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/MultiDecoratorTableCelleRenderer.java trunk/jaxx-tutorial-validation/pom.xml trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/Movie.java trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/People.java trunk/jaxx-widgets/pom.xml trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/editor/NumberEditorHandler.java Modified: trunk/jaxx-compiler/pom.xml =================================================================== --- trunk/jaxx-compiler/pom.xml 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/pom.xml 2012-07-04 16:02:22 UTC (rev 2379) @@ -77,11 +77,16 @@ </dependency> <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> + <groupId>org.apache.commons</groupId> + <artifactId>commons-lang3</artifactId> </dependency> <dependency> + <groupId>com.google.guava</groupId> + <artifactId>guava</artifactId> + </dependency> + + <dependency> <groupId>commons-collections</groupId> <artifactId>commons-collections</artifactId> </dependency> Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/CompiledObject.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -33,7 +33,7 @@ import jaxx.compiler.tags.TagManager; import jaxx.compiler.types.TypeManager; import jaxx.runtime.JAXXUtil; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import java.awt.Container; import java.util.ArrayList; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/DefaultCompilerConfiguration.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/DefaultCompilerConfiguration.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/DefaultCompilerConfiguration.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -29,8 +29,8 @@ import jaxx.compiler.spi.Initializer; import jaxx.runtime.JAXXContext; import jaxx.runtime.JAXXObject; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/JAXXCompiler.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -46,7 +46,7 @@ import jaxx.runtime.JAXXObjectDescriptor; import jaxx.runtime.css.Rule; import jaxx.runtime.css.Stylesheet; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.eugene.java.JavaGeneratorUtil; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataBinding.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -31,8 +31,8 @@ import jaxx.compiler.java.JavaFileGenerator; import jaxx.compiler.java.JavaMethod; import jaxx.compiler.types.TypeManager; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataListener.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataListener.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataListener.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -24,8 +24,8 @@ */ package jaxx.compiler.binding; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; /** * Created: 5 déc. 2009 Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/binding/DataSource.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -45,9 +45,9 @@ import jaxx.compiler.tags.TagManager; import jaxx.compiler.types.TypeManager; import jaxx.runtime.JAXXUtil; -import org.apache.commons.lang.StringUtils; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.StringUtils; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/decorators/DefaultCompiledObjectDecorator.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -36,7 +36,7 @@ import jaxx.compiler.java.JavaMethod; import jaxx.compiler.script.ScriptInitializer; import jaxx.compiler.types.TypeManager; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/finalizers/DefaultFinalizer.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -51,7 +51,7 @@ import jaxx.runtime.JAXXObjectDescriptor; import jaxx.runtime.JAXXUtil; import jaxx.runtime.SwingUtil; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaFile.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaFile.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaFile.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -28,7 +28,7 @@ import jaxx.compiler.finalizers.JAXXCompilerFinalizer; import jaxx.compiler.types.TypeManager; import org.apache.commons.collections.CollectionUtils; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.nuiton.eugene.java.extension.ImportsManager; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/DefaultObjectHandler.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/DefaultObjectHandler.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/DefaultObjectHandler.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -45,7 +45,7 @@ import jaxx.runtime.JAXXObject; import jaxx.runtime.JAXXObjectDescriptor; import jaxx.runtime.css.Stylesheet; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.w3c.dom.Attr; import org.w3c.dom.Element; import org.w3c.dom.NamedNodeMap; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/JScrollPaneHandler.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/JScrollPaneHandler.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/JScrollPaneHandler.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -31,7 +31,7 @@ import jaxx.compiler.reflect.ClassDescriptor; import jaxx.compiler.reflect.ClassDescriptorHelper; import jaxx.compiler.tags.DefaultComponentHandler; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import javax.swing.JScrollPane; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/TabHandler.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/TabHandler.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/swing/TabHandler.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -34,7 +34,7 @@ import jaxx.compiler.tags.TagHandler; import jaxx.compiler.types.TypeManager; import jaxx.runtime.swing.TabInfo; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.w3c.dom.Attr; import org.w3c.dom.Element; import org.w3c.dom.NamedNodeMap; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/FieldValidatorHandler.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/FieldValidatorHandler.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tags/validator/FieldValidatorHandler.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -33,7 +33,7 @@ import jaxx.compiler.tags.TagHandler; import jaxx.compiler.tags.validator.BeanValidatorHandler.CompiledBeanValidator; import jaxx.runtime.validator.swing.SwingValidator; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.w3c.dom.Element; Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/PrintTagInfo.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/PrintTagInfo.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/PrintTagInfo.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -33,6 +33,7 @@ import jaxx.compiler.reflect.ClassDescriptor; import jaxx.compiler.reflect.ClassDescriptorHelper; import jaxx.compiler.reflect.MethodDescriptor; +import org.apache.commons.lang3.StringUtils; import java.io.BufferedWriter; import java.io.FileWriter; @@ -109,7 +110,7 @@ w.append("|"); w.append(className.replace('.', '/')); w.append("|set"); - w.append(org.apache.commons.lang.StringUtils.capitalize(name)); + w.append(StringUtils.capitalize(name)); w.append("|"); w.append(JAXXCompiler.getCanonicalName(property.getPropertyType())); w.append("}}"); Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/jaxxcapture/JAXXCapture.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/jaxxcapture/JAXXCapture.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/tools/jaxxcapture/JAXXCapture.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -34,7 +34,7 @@ import jaxx.compiler.tools.jaxxcapture.handlers.ObjectHandler; import jaxx.compiler.tools.jaxxcapture.handlers.TableHandler; import jaxx.compiler.types.TypeManager; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.w3c.dom.Document; import org.w3c.dom.Element; import org.w3c.dom.Node; Modified: trunk/jaxx-demo/pom.xml =================================================================== --- trunk/jaxx-demo/pom.xml 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-demo/pom.xml 2012-07-04 16:02:22 UTC (rev 2379) @@ -89,8 +89,8 @@ </dependency> <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> + <groupId>org.apache.commons</groupId> + <artifactId>commons-lang3</artifactId> </dependency> <dependency> Modified: trunk/jaxx-demo/src/main/java/jaxx/demo/entities/Movie.java =================================================================== --- trunk/jaxx-demo/src/main/java/jaxx/demo/entities/Movie.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-demo/src/main/java/jaxx/demo/entities/Movie.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -25,8 +25,8 @@ package jaxx.demo.entities; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import java.util.ArrayList; import java.util.List; Modified: trunk/jaxx-demo/src/main/java/jaxx/demo/entities/People.java =================================================================== --- trunk/jaxx-demo/src/main/java/jaxx/demo/entities/People.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-demo/src/main/java/jaxx/demo/entities/People.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -25,8 +25,8 @@ package jaxx.demo.entities; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import static org.nuiton.i18n.I18n.n_; Modified: trunk/jaxx-maven-plugin/pom.xml =================================================================== --- trunk/jaxx-maven-plugin/pom.xml 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-maven-plugin/pom.xml 2012-07-04 16:02:22 UTC (rev 2379) @@ -124,8 +124,8 @@ </dependency> <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> + <groupId>org.apache.commons</groupId> + <artifactId>commons-lang3</artifactId> </dependency> <dependency> Modified: trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/AbstractGenerateHelpMojo.java =================================================================== --- trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/AbstractGenerateHelpMojo.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/AbstractGenerateHelpMojo.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -25,7 +25,7 @@ package org.nuiton.jaxx.plugin; -import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang3.StringUtils; import org.apache.maven.plugin.MojoFailureException; import org.nuiton.i18n.I18nUtil; Modified: trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/GenerateMojo.java =================================================================== --- trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/GenerateMojo.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-maven-plugin/src/main/java/org/nuiton/jaxx/plugin/GenerateMojo.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -39,8 +39,8 @@ import jaxx.runtime.JAXXContext; import jaxx.runtime.JAXXObject; import jaxx.runtime.swing.help.JAXXHelpBroker; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import org.apache.maven.plugin.MojoExecutionException; import org.nuiton.i18n.I18n; import org.nuiton.io.MirroredFileUpdater; Modified: trunk/jaxx-runtime/pom.xml =================================================================== --- trunk/jaxx-runtime/pom.xml 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-runtime/pom.xml 2012-07-04 16:02:22 UTC (rev 2379) @@ -51,11 +51,6 @@ <groupId>org.nuiton.i18n</groupId> <artifactId>nuiton-i18n</artifactId> </dependency> - - <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> - </dependency> <dependency> <groupId>commons-logging</groupId> Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/context/JAXXInitialContext.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/context/JAXXInitialContext.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/context/JAXXInitialContext.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -26,7 +26,6 @@ import jaxx.runtime.JAXXContext; import jaxx.runtime.JAXXObject; -import org.apache.commons.lang.NotImplementedException; import java.util.Map.Entry; @@ -102,21 +101,21 @@ @Override public void setContextValue(Object o) { - throw new NotImplementedException(); + throw new UnsupportedOperationException(); } @Override public void setContextValue(Object o, String name) { - throw new NotImplementedException(); + throw new UnsupportedOperationException(); } @Override public <T> void removeContextValue(Class<T> klazz) { - throw new NotImplementedException(); + throw new UnsupportedOperationException(); } @Override public <T> void removeContextValue(Class<T> klazz, String name) { - throw new NotImplementedException(); + throw new UnsupportedOperationException(); } } Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/MultiDecoratorTableCelleRenderer.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/MultiDecoratorTableCelleRenderer.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/MultiDecoratorTableCelleRenderer.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -64,12 +64,12 @@ types = new ArrayList<Class<?>>(); List<Decorator<?>> tmp = new ArrayList<Decorator<?>>(); for (JXPathDecorator<?> d : decorator) { - if (types.contains(d.getInternalClass())) { + if (types.contains(d.getType())) { throw new IllegalArgumentException( "can not have twice a decorator of type " + - d.getInternalClass()); + d.getType()); } - types.add(d.getInternalClass()); + types.add(d.getType()); tmp.add(d); } decorators = tmp.toArray(new Decorator<?>[tmp.size()]); Modified: trunk/jaxx-tutorial-validation/pom.xml =================================================================== --- trunk/jaxx-tutorial-validation/pom.xml 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-tutorial-validation/pom.xml 2012-07-04 16:02:22 UTC (rev 2379) @@ -85,8 +85,8 @@ </dependency> <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> + <groupId>org.apache.commons</groupId> + <artifactId>commons-lang3</artifactId> </dependency> <dependency> Modified: trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/Movie.java =================================================================== --- trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/Movie.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/Movie.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -25,8 +25,8 @@ package jaxx.demo.validation.model; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import java.util.ArrayList; import java.util.List; Modified: trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/People.java =================================================================== --- trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/People.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-tutorial-validation/src/main/java/jaxx/demo/validation/model/People.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -25,8 +25,8 @@ package jaxx.demo.validation.model; -import org.apache.commons.lang.builder.ToStringBuilder; -import org.apache.commons.lang.builder.ToStringStyle; +import org.apache.commons.lang3.builder.ToStringBuilder; +import org.apache.commons.lang3.builder.ToStringStyle; import static org.nuiton.i18n.I18n.n_; Modified: trunk/jaxx-widgets/pom.xml =================================================================== --- trunk/jaxx-widgets/pom.xml 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-widgets/pom.xml 2012-07-04 16:02:22 UTC (rev 2379) @@ -72,8 +72,8 @@ </dependency> <dependency> - <groupId>commons-lang</groupId> - <artifactId>commons-lang</artifactId> + <groupId>org.apache.commons</groupId> + <artifactId>commons-lang3</artifactId> </dependency> <dependency> Modified: trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/editor/NumberEditorHandler.java =================================================================== --- trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/editor/NumberEditorHandler.java 2012-07-04 09:44:48 UTC (rev 2378) +++ trunk/jaxx-widgets/src/main/java/jaxx/runtime/swing/editor/NumberEditorHandler.java 2012-07-04 16:02:22 UTC (rev 2379) @@ -42,8 +42,8 @@ import javax.swing.SwingUtilities; import javax.swing.text.BadLocationException; import org.apache.commons.beanutils.PropertyUtils; -import org.apache.commons.lang.StringUtils; -import org.apache.commons.lang.math.NumberUtils; +import org.apache.commons.lang3.StringUtils; +import org.apache.commons.lang3.math.NumberUtils; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory;

1 0

r2378 - trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model
by sletellier＠users.nuiton.org July 4, 2012

July 4, 2012

Author: sletellier Date: 2012-07-04 11:44:48 +0200 (Wed, 04 Jul 2012) New Revision: 2378 Url: http://nuiton.org/repositories/revision/jaxx/2378 Log: fixes #2159 : Create GenericListSelectionModel extracted from GenericListModel Added: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListSelectionModel.java Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListModel.java Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListModel.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListModel.java 2012-07-03 13:39:06 UTC (rev 2377) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListModel.java 2012-07-04 09:44:48 UTC (rev 2378) @@ -24,32 +24,19 @@ package jaxx.runtime.swing.model; import com.google.common.collect.Lists; -import java.beans.PropertyChangeListener; -import java.beans.PropertyChangeSupport; import java.util.Collection; import java.util.List; import javax.swing.ComboBoxModel; import javax.swing.DefaultListModel; -import javax.swing.DefaultListSelectionModel; -import javax.swing.event.EventListenerList; import javax.swing.event.ListDataListener; /** * @author sletellier <letellier(a)codelutin.com> */ -public class GenericListModel extends DefaultListSelectionModel implements ComboBoxModel { +public class GenericListModel extends GenericListSelectionModel implements ComboBoxModel { - protected EventListenerList listenerList = new EventListenerList(); - protected final PropertyChangeSupport pcs = new PropertyChangeSupport(this); - - public static final String PROPERTY_SELECTED_VALUE = "selectedValues"; - - protected DefaultListModel delegate = new DefaultListModel(); - - protected List selectedValues; - public GenericListModel() { - this.selectedValues = Lists.newArrayList(); + super(new DefaultListModel()); } public GenericListModel(Collection values) { @@ -67,7 +54,7 @@ fireValuesRemoved(oldValues); for (B value : values) { - delegate.addElement(value); + getListModel().addElement(value); } fireSelectionAdded(values); @@ -75,184 +62,64 @@ public void clearElements() { Collection elements = getElements(); - delegate.clear(); + getListModel().clear(); fireValuesRemoved(elements); } public Collection getElements() { - int size = delegate.getSize(); + int size = getListModel().getSize(); Collection result = Lists.newArrayList(); for (int i=0;i<size;i++) { - result.add((B) delegate.get(i)); + result.add((B) getListModel().get(i)); } return result; } public void addElement(int index, B valueToAdd) { - delegate.add(index, valueToAdd); + getListModel().add(index, valueToAdd); fireValuesAdded(Lists.newArrayList(valueToAdd)); } public void addElement(B valueToAdd) { - delegate.addElement(valueToAdd); + getListModel().addElement(valueToAdd); fireValuesAdded(Lists.newArrayList(valueToAdd)); } public void addElements(Collection valuesToAdd) { for (B value : valuesToAdd) { - delegate.addElement(value); + getListModel().addElement(value); } fireValuesAdded(valuesToAdd); } - public B getSelectedValue() { - return selectedValues.get(0); - } - - public Collection getSelectedValues() { - return Lists.newArrayList(selectedValues); - } - public void removeElements(Collection values) { for (B value : values) { - delegate.removeElement(value); + getListModel().removeElement(value); } unSelectItems(values); fireValuesRemoved(values); } - protected void unSelectItems(Collection values) { - Collection oldValue = Lists.newArrayList(selectedValues); - for (B value : values) { - int index = selectedValues.indexOf(value); - removeSelectionIntervalWithoutFire(index, index); - } - fireSelectionRemoved(values); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); - } - - protected void unSelectItem(B value) { - Collection oldValue = Lists.newArrayList(selectedValues); - int index = selectedValues.indexOf(value); - removeSelectionIntervalWithoutFire(index, index); - - fireSelectionRemoved(Lists.newArrayList(value)); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); - } - - public void addSelectedItem(B toSelect) { - Collection oldValue = Lists.newArrayList(selectedValues); - selectedValues.add(toSelect); - int index = selectedValues.indexOf(toSelect); - super.addSelectionInterval(index, index); - - fireSelectionAdded(Lists.newArrayList(toSelect)); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); - } - - public boolean hasSelectedIndex() { - return !selectedValues.isEmpty(); - } - @Override - public void addSelectionInterval(int index0, int index1) { - Collection oldValue = Lists.newArrayList(selectedValues); - - addSelectionIntervalWithFire(index0, index1); - super.addSelectionInterval(index0, index1); - - Collection newValue = Lists.newArrayList(selectedValues); - newValue.removeAll(oldValue); - fireSelectionAdded(newValue); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); - } - - @Override - public void setSelectionInterval(int index0, int index1) { - Collection oldValue = Lists.newArrayList(selectedValues); - selectedValues.clear(); - addSelectionIntervalWithFire(index0, index1); - super.setSelectionInterval(index0, index1); - - Collection newValue = Lists.newArrayList(selectedValues); - newValue.removeAll(oldValue); - fireSelectionAdded(newValue); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); - } - - @Override - public void removeSelectionInterval(int index0, int index1) { - Collection oldValue = Lists.newArrayList(selectedValues); - removeSelectionIntervalWithoutFire(index0, index1); - - Collection newValue = Lists.newArrayList(selectedValues); - newValue.removeAll(oldValue); - fireSelectionRemoved(newValue); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); - } - - protected void removeSelectionIntervalWithoutFire(int index0, int index1) { - if (index0 > index1) { - int tmp = index1; - index1 = index0; - index0 = tmp; - } - for (int i=index0;i<=index1;i++) { - if (selectedValues.size() > i && i != -1) { - selectedValues.remove(i); - } - } - super.removeSelectionInterval(index0, index1); - } - - protected void addSelectionIntervalWithFire(int index0, int index1) { - if (index0 > index1) { - int tmp = index1; - index1 = index0; - index0 = tmp; - } - for (int i=index0;i<=index1;i++) { - - if (delegate.size() > i && i != -1) { - B value = (B)delegate.getElementAt(i); - selectedValues.add(value); - } - } - } - - @Override - public void clearSelection() { - Collection oldValue = Lists.newArrayList(selectedValues); - selectedValues.clear(); - super.clearSelection(); - + public void setSelectedItem(Object anItem) { + List oldValue = getSelectedValues(); fireSelectionRemoved(oldValue); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); - } - @Override - public int getSelectionMode() { - return MULTIPLE_INTERVAL_SELECTION; - } + setSelectedValues(Lists.newArrayList((B) anItem)); - @Override - public void setSelectedItem(Object anItem) { - Collection oldValue = this.selectedValues; - fireSelectionRemoved(selectedValues); - - selectedValues = Lists.newArrayList((B)anItem); - - fireSelectionAdded(selectedValues); - firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + List newValues = getSelectedValues(); + fireSelectionAdded(newValues); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, newValues); } @Override public Object getSelectedItem() { + List selectedValues = getSelectedValues(); if (selectedValues.isEmpty()) { return null; } @@ -261,117 +128,22 @@ @Override public int getSize() { - return delegate.size(); + return getListModel().size(); } @Override public Object getElementAt(int index) { - return delegate.get(index); + return getListModel().get(index); } @Override public void addListDataListener(ListDataListener l) { - delegate.addListDataListener(l); + getListModel().addListDataListener(l); } @Override public void removeListDataListener(ListDataListener l) { - delegate.removeListDataListener(l); + getListModel().removeListDataListener(l); } - public void addGenericListListener(GenericListListener l) { - listenerList.add(GenericListListener.class, l); - } - - public void removeGenericListListener(GenericListListener l) { - listenerList.remove(GenericListListener.class, l); - } - - public void addPropertyChangeListener(PropertyChangeListener listener) { - pcs.addPropertyChangeListener(listener); - } - - public void addPropertyChangeListener(String propertyName, PropertyChangeListener listener) { - pcs.addPropertyChangeListener(propertyName, listener); - } - - public void removePropertyChangeListener(PropertyChangeListener listener) { - pcs.removePropertyChangeListener(listener); - } - - public void removePropertyChangeListener(String propertyName, PropertyChangeListener listener) { - pcs.removePropertyChangeListener(propertyName, listener); - } - - protected void fireValuesAdded(Collection values) { - if (values.isEmpty()) { - return; - } - Object[] listeners = listenerList.getListenerList(); - GenericListEvent e = null; - - for (int i = listeners.length - 2; i >= 0; i -= 2) { - if (listeners[i] == GenericListListener.class) { - if (e == null) { - e = new GenericListEvent(this, values); - } - ((GenericListListener)listeners[i+1]).valuesAdded(e); - } - } - } - - protected void fireValuesRemoved(Collection values) { - if (values.isEmpty()) { - return; - } - Object[] listeners = listenerList.getListenerList(); - GenericListEvent e = null; - - for (int i = listeners.length - 2; i >= 0; i -= 2) { - if (listeners[i] == GenericListListener.class) { - if (e == null) { - e = new GenericListEvent(this, values); - } - ((GenericListListener)listeners[i+1]).valuesRemoved(e); - } - } - } - - protected void fireSelectionAdded(Collection selectedValues) { - if (selectedValues.isEmpty()) { - return; - } - Object[] listeners = listenerList.getListenerList(); - GenericListEvent e = null; - - for (int i = listeners.length - 2; i >= 0; i -= 2) { - if (listeners[i] == GenericListListener.class) { - if (e == null) { - e = new GenericListEvent(this, selectedValues); - } - ((GenericListListener)listeners[i+1]).selectionAdded(e); - } - } - } - - protected void fireSelectionRemoved(Collection selectedValues) { - if (selectedValues.isEmpty()) { - return; - } - Object[] listeners = listenerList.getListenerList(); - GenericListEvent e = null; - - for (int i = listeners.length - 2; i >= 0; i -= 2) { - if (listeners[i] == GenericListListener.class) { - if (e == null) { - e = new GenericListEvent(this, selectedValues); - } - ((GenericListListener)listeners[i+1]).selectionAdded(e); - } - } - } - - protected void firePropertyChange(String propertyName, Object oldValue, Object newValue) { - pcs.firePropertyChange(propertyName, oldValue, newValue); - } } \ No newline at end of file Added: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListSelectionModel.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListSelectionModel.java (rev 0) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/model/GenericListSelectionModel.java 2012-07-04 09:44:48 UTC (rev 2378) @@ -0,0 +1,255 @@ +package jaxx.runtime.swing.model; + +import com.google.common.collect.Lists; +import java.beans.PropertyChangeListener; +import java.beans.PropertyChangeSupport; +import java.util.Collection; +import java.util.List; +import javax.swing.DefaultListModel; +import javax.swing.DefaultListSelectionModel; +import javax.swing.event.EventListenerList; + +/** + * @author sletellier <letellier(a)codelutin.com> + */ +public class GenericListSelectionModel extends DefaultListSelectionModel { + + public static final String PROPERTY_SELECTED_VALUE = "selectedValues"; + + protected EventListenerList listenerList = new EventListenerList(); + protected final PropertyChangeSupport pcs = new PropertyChangeSupport(this); + + protected List selectedValues; + protected DefaultListModel listModel; + + public GenericListSelectionModel(DefaultListModel listModel) { + this.selectedValues = Lists.newArrayList(); + this.listModel = listModel; + } + + public DefaultListModel getListModel() { + return listModel; + } + + public B getSelectedValue() { + return selectedValues.get(0); + } + + public List getSelectedValues() { + return Lists.newArrayList(selectedValues); + } + + public void setSelectedValues(List selectedValues) { + this.selectedValues = selectedValues; + } + + protected void unSelectItems(Collection values) { + Collection oldValue = Lists.newArrayList(selectedValues); + for (B value : values) { + int index = selectedValues.indexOf(value); + removeSelectionIntervalWithoutFire(index, index); + } + fireSelectionRemoved(values); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + } + + protected void unSelectItem(B value) { + Collection oldValue = Lists.newArrayList(selectedValues); + int index = selectedValues.indexOf(value); + removeSelectionIntervalWithoutFire(index, index); + + fireSelectionRemoved(Lists.newArrayList(value)); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + } + + public void addSelectedItem(B toSelect) { + Collection oldValue = Lists.newArrayList(selectedValues); + selectedValues.add(toSelect); + int index = selectedValues.indexOf(toSelect); + super.addSelectionInterval(index, index); + + fireSelectionAdded(Lists.newArrayList(toSelect)); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + } + + public boolean hasSelectedIndex() { + return !selectedValues.isEmpty(); + } + + @Override + public void addSelectionInterval(int index0, int index1) { + Collection oldValue = Lists.newArrayList(selectedValues); + + addSelectionIntervalWithFire(index0, index1); + super.addSelectionInterval(index0, index1); + + Collection newValue = Lists.newArrayList(selectedValues); + newValue.removeAll(oldValue); + fireSelectionAdded(newValue); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + } + + @Override + public void setSelectionInterval(int index0, int index1) { + Collection oldValue = Lists.newArrayList(selectedValues); + selectedValues.clear(); + addSelectionIntervalWithFire(index0, index1); + super.setSelectionInterval(index0, index1); + + Collection newValue = Lists.newArrayList(selectedValues); + newValue.removeAll(oldValue); + fireSelectionAdded(newValue); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + } + + @Override + public void removeSelectionInterval(int index0, int index1) { + Collection oldValue = Lists.newArrayList(selectedValues); + removeSelectionIntervalWithoutFire(index0, index1); + + Collection newValue = Lists.newArrayList(selectedValues); + newValue.removeAll(oldValue); + fireSelectionRemoved(newValue); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + } + + protected void removeSelectionIntervalWithoutFire(int index0, int index1) { + if (index0 > index1) { + int tmp = index1; + index1 = index0; + index0 = tmp; + } + for (int i=index0;i<=index1;i++) { + if (selectedValues.size() > i && i != -1) { + selectedValues.remove(i); + } + } + super.removeSelectionInterval(index0, index1); + } + + protected void addSelectionIntervalWithFire(int index0, int index1) { + if (index0 > index1) { + int tmp = index1; + index1 = index0; + index0 = tmp; + } + for (int i=index0;i<=index1;i++) { + + if (listModel.size() > i && i != -1) { + B value = (B)listModel.getElementAt(i); + selectedValues.add(value); + } + } + } + + @Override + public void clearSelection() { + Collection oldValue = Lists.newArrayList(selectedValues); + selectedValues.clear(); + super.clearSelection(); + + fireSelectionRemoved(oldValue); + firePropertyChange(PROPERTY_SELECTED_VALUE, oldValue, selectedValues); + } + + @Override + public int getSelectionMode() { + return MULTIPLE_INTERVAL_SELECTION; + } + + protected void fireValuesAdded(Collection values) { + if (values.isEmpty()) { + return; + } + Object[] listeners = listenerList.getListenerList(); + GenericListEvent e = null; + + for (int i = listeners.length - 2; i >= 0; i -= 2) { + if (listeners[i] == GenericListListener.class) { + if (e == null) { + e = new GenericListEvent(this, values); + } + ((GenericListListener)listeners[i+1]).valuesAdded(e); + } + } + } + + protected void fireValuesRemoved(Collection values) { + if (values.isEmpty()) { + return; + } + Object[] listeners = listenerList.getListenerList(); + GenericListEvent e = null; + + for (int i = listeners.length - 2; i >= 0; i -= 2) { + if (listeners[i] == GenericListListener.class) { + if (e == null) { + e = new GenericListEvent(this, values); + } + ((GenericListListener)listeners[i+1]).valuesRemoved(e); + } + } + } + + protected void fireSelectionAdded(Collection selectedValues) { + if (selectedValues.isEmpty()) { + return; + } + Object[] listeners = listenerList.getListenerList(); + GenericListEvent e = null; + + for (int i = listeners.length - 2; i >= 0; i -= 2) { + if (listeners[i] == GenericListListener.class) { + if (e == null) { + e = new GenericListEvent(this, selectedValues); + } + ((GenericListListener)listeners[i+1]).selectionAdded(e); + } + } + } + + protected void fireSelectionRemoved(Collection selectedValues) { + if (selectedValues.isEmpty()) { + return; + } + Object[] listeners = listenerList.getListenerList(); + GenericListEvent e = null; + + for (int i = listeners.length - 2; i >= 0; i -= 2) { + if (listeners[i] == GenericListListener.class) { + if (e == null) { + e = new GenericListEvent(this, selectedValues); + } + ((GenericListListener)listeners[i+1]).selectionAdded(e); + } + } + } + + public void addGenericListListener(GenericListListener l) { + listenerList.add(GenericListListener.class, l); + } + + public void removeGenericListListener(GenericListListener l) { + listenerList.remove(GenericListListener.class, l); + } + + public void addPropertyChangeListener(PropertyChangeListener listener) { + pcs.addPropertyChangeListener(listener); + } + + public void addPropertyChangeListener(String propertyName, PropertyChangeListener listener) { + pcs.addPropertyChangeListener(propertyName, listener); + } + + public void removePropertyChangeListener(PropertyChangeListener listener) { + pcs.removePropertyChangeListener(listener); + } + + public void removePropertyChangeListener(String propertyName, PropertyChangeListener listener) { + pcs.removePropertyChangeListener(propertyName, listener); + } + + protected void firePropertyChange(String propertyName, Object oldValue, Object newValue) { + pcs.firePropertyChange(propertyName, oldValue, newValue); + } +}

1 0

r2377 - trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer
by tchemit＠users.nuiton.org July 3, 2012

July 3, 2012

Author: tchemit Date: 2012-07-03 15:39:06 +0200 (Tue, 03 Jul 2012) New Revision: 2377 Url: http://nuiton.org/repositories/revision/jaxx/2377 Log: revert some error commit (2334) Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java 2012-07-03 13:12:49 UTC (rev 2376) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java 2012-07-03 13:39:06 UTC (rev 2377) @@ -80,6 +80,6 @@ } checkBox.setSelected((value != null && (Boolean) value)); setBorder(render.getBorder()); - return this; + return checkBox; } }

1 0

r2376 - in trunk/jaxx-runtime/src/main/java/jaxx/runtime: . swing/renderer
by tchemit＠users.nuiton.org July 3, 2012

July 3, 2012

Author: tchemit Date: 2012-07-03 15:12:49 +0200 (Tue, 03 Jul 2012) New Revision: 2376 Url: http://nuiton.org/repositories/revision/jaxx/2376 Log: refs #2156 (reput old behaviour) Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/SwingUtil.java trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/SwingUtil.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/SwingUtil.java 2012-07-02 23:28:37 UTC (rev 2375) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/SwingUtil.java 2012-07-03 13:12:49 UTC (rev 2376) @@ -24,6 +24,46 @@ */ package jaxx.runtime; +import jaxx.runtime.swing.Item; +import jaxx.runtime.swing.JAXXComboBox; +import jaxx.runtime.swing.renderer.BooleanCellEditor; +import jaxx.runtime.swing.renderer.BooleanCellRenderer; +import jaxx.runtime.swing.renderer.EmptyNumberTableCellRenderer; +import jaxx.runtime.swing.renderer.EnumTableCellRenderer; +import jaxx.runtime.swing.renderer.I18nTableCellRenderer; +import org.apache.commons.logging.Log; +import org.apache.commons.logging.LogFactory; +import org.jdesktop.jxlayer.JXLayer; +import org.jdesktop.jxlayer.plaf.LayerUI; +import org.jdesktop.swingx.JXTreeTable; + +import javax.swing.DefaultComboBoxModel; +import javax.swing.DefaultListModel; +import javax.swing.Icon; +import javax.swing.ImageIcon; +import javax.swing.JComboBox; +import javax.swing.JComponent; +import javax.swing.JLabel; +import javax.swing.JLayeredPane; +import javax.swing.JList; +import javax.swing.JRootPane; +import javax.swing.JTabbedPane; +import javax.swing.JTable; +import javax.swing.JTree; +import javax.swing.SwingUtilities; +import javax.swing.UIManager; +import javax.swing.UnsupportedLookAndFeelException; +import javax.swing.event.HyperlinkEvent; +import javax.swing.event.TreeSelectionEvent; +import javax.swing.event.TreeSelectionListener; +import javax.swing.table.DefaultTableCellRenderer; +import javax.swing.table.TableCellEditor; +import javax.swing.table.TableCellRenderer; +import javax.swing.table.TableColumn; +import javax.swing.text.AbstractDocument; +import javax.swing.text.JTextComponent; +import javax.swing.tree.TreeNode; +import javax.swing.tree.TreePath; import java.awt.Color; import java.awt.Component; import java.awt.Container; @@ -56,44 +96,6 @@ import java.util.Map.Entry; import java.util.NoSuchElementException; import java.util.Properties; -import javax.swing.DefaultComboBoxModel; -import javax.swing.DefaultListModel; -import javax.swing.Icon; -import javax.swing.ImageIcon; -import javax.swing.JComboBox; -import javax.swing.JComponent; -import javax.swing.JLabel; -import javax.swing.JLayeredPane; -import javax.swing.JList; -import javax.swing.JRootPane; -import javax.swing.JTabbedPane; -import javax.swing.JTable; -import javax.swing.JTree; -import javax.swing.SwingUtilities; -import javax.swing.UIManager; -import javax.swing.UnsupportedLookAndFeelException; -import javax.swing.event.HyperlinkEvent; -import javax.swing.event.TreeSelectionEvent; -import javax.swing.event.TreeSelectionListener; -import javax.swing.table.DefaultTableCellRenderer; -import javax.swing.table.TableCellEditor; -import javax.swing.table.TableCellRenderer; -import javax.swing.table.TableColumn; -import javax.swing.text.AbstractDocument; -import javax.swing.text.JTextComponent; -import javax.swing.tree.TreeNode; -import javax.swing.tree.TreePath; -import jaxx.runtime.swing.Item; -import jaxx.runtime.swing.JAXXComboBox; -import jaxx.runtime.swing.renderer.BooleanCellEditor; -import jaxx.runtime.swing.renderer.EmptyNumberTableCellRenderer; -import jaxx.runtime.swing.renderer.EnumTableCellRenderer; -import jaxx.runtime.swing.renderer.I18nTableCellRenderer; -import org.apache.commons.logging.Log; -import org.apache.commons.logging.LogFactory; -import org.jdesktop.jxlayer.JXLayer; -import org.jdesktop.jxlayer.plaf.LayerUI; -import org.jdesktop.swingx.JXTreeTable; import static org.nuiton.i18n.I18n._; @@ -892,6 +894,11 @@ public static TableCellRenderer newBooleanTableCellRenderer( TableCellRenderer renderer) { + return new BooleanCellRenderer(renderer); + } + + public static TableCellRenderer newBooleanTableCellEditorAndRenderer( + TableCellRenderer renderer) { return new BooleanCellEditor(renderer); } @@ -1232,11 +1239,11 @@ /** * Used to edit cell * - * @param table to edit - * @param row of cell to editing + * @param table to edit + * @param row of cell to editing * @param colummn of cell to editing * @return false if for any reason the cell cannot be edited, - * or if the indices are invalid + * or if the indices are invalid */ public static boolean editCell(JTable table, int row, int colummn) { Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java 2012-07-02 23:28:37 UTC (rev 2375) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/renderer/BooleanCellRenderer.java 2012-07-03 13:12:49 UTC (rev 2376) @@ -24,8 +24,6 @@ */ package jaxx.runtime.swing.renderer; -import java.awt.Color; -import java.awt.Component; import javax.swing.Icon; import javax.swing.JCheckBox; import javax.swing.JComponent; @@ -33,13 +31,13 @@ import javax.swing.JPanel; import javax.swing.JTable; import javax.swing.table.TableCellRenderer; +import java.awt.Color; +import java.awt.Component; /** - * @deprecated prefere use {@link BooleanCellEditor} * @author tchemit <chemit(a)codelutin.com> * @since 1.5 */ -@Deprecated public class BooleanCellRenderer extends JPanel implements TableCellRenderer { private static final long serialVersionUID = 1L;

1 0

r2375 - in trunk/jaxx-compiler/src: main/java/jaxx/compiler/java test/java/jaxx/compiler/java
by tchemit＠users.nuiton.org July 2, 2012

July 2, 2012

Author: tchemit Date: 2012-07-03 01:28:37 +0200 (Tue, 03 Jul 2012) New Revision: 2375 Url: http://nuiton.org/repositories/revision/jaxx/2375 Log: refs #2154: Jaxx deos not compile under jdk 7 (field comparator fails) : add test and apply quick fix (but not best...) Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaElement.java trunk/jaxx-compiler/src/test/java/jaxx/compiler/java/JavaFieldTest.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaElement.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaElement.java 2012-07-02 21:52:39 UTC (rev 2374) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/java/JavaElement.java 2012-07-02 23:28:37 UTC (rev 2375) @@ -108,22 +108,23 @@ String n1 = o1.getName(); String n2 = o2.getName(); - Matcher matcher1 = NAME_PATTERN.matcher(n1); - Matcher matcher2 = NAME_PATTERN.matcher(n2); - if (matcher1.matches() && matcher2.matches()) { - // les deux noms finissent par un nombre - String p1 = matcher1.group(1); - String p2 = matcher1.group(2); - int i = p1.compareTo(p2); - if (i != 0) { - // on est sur des noms de prefix différents, donc pas de tri sur les suffixes - return i; - } - // les deux noms doivent être triés sur les suffixes entiers - int i1 = Integer.valueOf(matcher1.group(2)); - int i2 = Integer.valueOf(matcher2.group(2)); - return i1 - i2; - } + //FIXME-tchemit-2012-07-02 (see http://nuiton.org/issues/2154) +// Matcher matcher1 = NAME_PATTERN.matcher(n1); +// Matcher matcher2 = NAME_PATTERN.matcher(n2); +// if (matcher1.matches() && matcher2.matches()) { +// // les deux noms finissent par un nombre +// String p1 = matcher1.group(1); +// String p2 = matcher1.group(2); +// int i = p1.compareTo(p2); +// if (i != 0) { +// // on est sur des noms de prefix différents, donc pas de tri sur les suffixes +// return i; +// } +// // les deux noms doivent être triés sur les suffixes entiers +// int i1 = Integer.valueOf(matcher1.group(2)); +// int i2 = Integer.valueOf(matcher2.group(2)); +// return i1 - i2; +// } // les deux noms sont simplement comparé en alpha return n1.compareTo(n2); Modified: trunk/jaxx-compiler/src/test/java/jaxx/compiler/java/JavaFieldTest.java =================================================================== --- trunk/jaxx-compiler/src/test/java/jaxx/compiler/java/JavaFieldTest.java 2012-07-02 21:52:39 UTC (rev 2374) +++ trunk/jaxx-compiler/src/test/java/jaxx/compiler/java/JavaFieldTest.java 2012-07-02 23:28:37 UTC (rev 2375) @@ -24,13 +24,16 @@ */ package jaxx.compiler.java; +import com.google.common.collect.Lists; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; import org.junit.Assert; import org.junit.Test; import java.lang.reflect.Modifier; +import java.util.Collections; import java.util.EnumSet; +import java.util.List; /** * Created: 3 déc. 2009 @@ -42,9 +45,7 @@ * $Author$ */ public class JavaFieldTest { - /** - * Logger - */ + /** Logger */ private static final Log log = LogFactory.getLog(JavaFieldTest.class); @Test @@ -91,4 +92,36 @@ Assert.assertTrue(constants.contains(JavaField.FieldOrder.internalFields)); } + + /** + * To test http://nuiton.org/issues/2154. + * + * @since 2.5.1 + */ + @Test + public void testCompare() { + + String[] names = new String[]{ + "cf0", + "cf1", + "cf0O1", + "cf0A1", + "ct0O1", "ct0A1", + "cgett0", + "cgett1", "cgett0O1", + "cgett0A1", "cfile", "cgetfile", "t0", "t1", + "cmt0", "cmt1", "cmt0O1", "cmt0A1", "cmgett0", + "cmgett1", "cmgett0O1", "cmgett0A1", + "cMfile", "cMgetfile", + "Mt0", "Mt1", "cit0", "cit1", "cit0O1", "cit0A1", "cigett0", + "bindings" + }; + List<JavaField> theFileds = Lists.newArrayList(); + for (String name : names) { + JavaField field = new JavaField(Modifier.PROTECTED, "String", name, false); + theFileds.add(field); + } + Collections.sort(theFileds); + + } }

1 0

r2374 - in trunk/jaxx-runtime/src/main/java/jaxx/runtime: . swing
by tchemit＠users.nuiton.org July 2, 2012

July 2, 2012

Author: tchemit Date: 2012-07-02 23:52:39 +0200 (Mon, 02 Jul 2012) New Revision: 2374 Url: http://nuiton.org/repositories/revision/jaxx/2374 Log: remove old directories Removed: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/layer/ trunk/jaxx-runtime/src/main/java/jaxx/runtime/validator/

1 0

r2373 - trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect
by tchemit＠users.nuiton.org July 2, 2012

July 2, 2012

Author: tchemit Date: 2012-07-02 23:36:50 +0200 (Mon, 02 Jul 2012) New Revision: 2373 Url: http://nuiton.org/repositories/revision/jaxx/2373 Log: fixes #2153: ClassDescriptor equals and isAssignableFrom are not correct Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptor.java Modified: trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptor.java =================================================================== --- trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptor.java 2012-07-02 16:10:23 UTC (rev 2372) +++ trunk/jaxx-compiler/src/main/java/jaxx/compiler/reflect/ClassDescriptor.java 2012-07-02 21:36:50 UTC (rev 2373) @@ -251,7 +251,7 @@ return true; } for (ClassDescriptor anInterface : descriptor.getInterfaces()) { - if (equals(anInterface)) { + if (equals(anInterface) || isAssignableFrom(anInterface)) { return true; } } @@ -265,6 +265,22 @@ return "ClassDescriptor[" + getName() + "]"; } + @Override + public boolean equals(Object o) { + if (this == o) return true; + if (!(o instanceof ClassDescriptor)) return false; + + ClassDescriptor that = (ClassDescriptor) o; + + return name.equals(that.name); + + } + + @Override + public int hashCode() { + return name.hashCode(); + } + protected ClassDescriptor getClassDescriptor(String fqn) { if (fqn == null) { return null;

1 0