Converter CDI Extension working

forge · Dec 20, 2012 · 576f50c · 576f50c
1 parent 1b3b7c9
commit 576f50c
Show file tree

Hide file tree

Showing 5 changed files with 93 additions and 1 deletion.
diff --git a/ui/impl/src/main/java/org/jboss/forge/ui/impl/converter/ConverterRegistryImpl.java b/ui/impl/src/main/java/org/jboss/forge/ui/impl/converter/ConverterRegistryImpl.java
@@ -65,6 +65,7 @@ public void removeConverter(Class<?> sourceType, Class<?> targetType)
       customConverters.remove(key);
    }
 
+   // TODO: Do comparison based on the class hierarchy
    private static class ClassPairEntry
    {
       private final String sourceClassName;

diff --git a/ui/impl/src/main/java/org/jboss/forge/ui/impl/converter/cdi/ConverterExtension.java b/ui/impl/src/main/java/org/jboss/forge/ui/impl/converter/cdi/ConverterExtension.java
@@ -0,0 +1,68 @@
+/*
+ * Copyright 2012 Red Hat, Inc. and/or its affiliates.
+ *
+ * Licensed under the Eclipse Public License version 1.0, available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ */
+
+/*
+ * Copyright 2012 Red Hat, Inc. and/or its affiliates.
+ *
+ * Licensed under the Eclipse Public License version 1.0, available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ */
+package org.jboss.forge.ui.impl.converter.cdi;
+
+import java.lang.reflect.Method;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
+import javax.enterprise.event.Observes;
+import javax.enterprise.inject.spi.AnnotatedType;
+import javax.enterprise.inject.spi.Extension;
+import javax.enterprise.inject.spi.ProcessAnnotatedType;
+
+import org.jboss.forge.ui.converter.Converter;
+import org.jboss.forge.ui.impl.converter.ConverterRegistryImpl;
+
+/**
+ * Registers all the converters into the {@link ConverterRegistryImpl}
+ *
+ * @author <a href="mailto:ggastald@redhat.com">George Gastaldi</a>
+ *
+ */
+public class ConverterExtension implements Extension
+{
+
+   @SuppressWarnings("unchecked")
+   <S, T> void observesConverters(@Observes ProcessAnnotatedType<? extends Converter<S, T>> pat) throws Exception
+   {
+      AnnotatedType<? extends Converter<S, T>> annotatedType = pat.getAnnotatedType();
+      Class<?> javaClass = annotatedType.getJavaClass();
+      try
+      {
+         Converter<S, T> newInstance = (Converter<S, T>) javaClass.newInstance();
+
+         for (Method m : javaClass.getMethods())
+         {
+            // TODO: Ugly hack. It shoulld have a better way to discover the generic type used
+            if ("convert".equals(m.getName()))
+            {
+               // TODO: Find the best way to create this object
+               Class<S> sourceClass = (Class<S>) m.getParameterTypes()[0];
+               Class<T> targetClass = (Class<T>) m.getReturnType();
+               ConverterRegistryImpl.INSTANCE.addConverter(sourceClass, targetClass, newInstance);
+
+               break;
+            }
+         }
+      }
+      catch (Exception e)
+      {
+         Logger.getLogger(getClass().getSimpleName())
+                  .log(Level.FINE, "Error while instantiating " + javaClass, e);
+      }
+      // Always veto the instance
+      pat.veto();
+   }
+}
diff --git a/.../ui/impl/converter/ConverterProducer.java → ...impl/converter/cdi/ConverterProducer.java b/.../ui/impl/converter/ConverterProducer.java → ...impl/converter/cdi/ConverterProducer.java
@@ -5,7 +5,7 @@
  * http://www.eclipse.org/legal/epl-v10.html
  */
 
-package org.jboss.forge.ui.impl.converter;
+package org.jboss.forge.ui.impl.converter.cdi;
 
 import java.lang.reflect.ParameterizedType;
 import java.lang.reflect.Type;
@@ -15,6 +15,7 @@
 import javax.enterprise.inject.spi.InjectionPoint;
 
 import org.jboss.forge.ui.converter.Converter;
+import org.jboss.forge.ui.impl.converter.ConverterRegistryImpl;
 
 public class ConverterProducer
 {

diff --git a/ui/impl/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension b/ui/impl/src/main/resources/META-INF/services/javax.enterprise.inject.spi.Extension
@@ -0,0 +1 @@
+org.jboss.forge.ui.impl.converter.cdi.ConverterExtension
diff --git a/ui/impl/src/test/java/org/jboss/forge/ui/impl/converter/MyConverter.java b/ui/impl/src/test/java/org/jboss/forge/ui/impl/converter/MyConverter.java
@@ -0,0 +1,21 @@
+/*
+ * Copyright 2012 Red Hat, Inc. and/or its affiliates.
+ *
+ * Licensed under the Eclipse Public License version 1.0, available at
+ * http://www.eclipse.org/legal/epl-v10.html
+ */
+
+package org.jboss.forge.ui.impl.converter;
+
+import org.jboss.forge.ui.converter.Converter;
+
+public class MyConverter implements Converter<AnotherBean, SimpleBean>
+{
+
+   @Override
+   public SimpleBean convert(AnotherBean source) throws Exception
+   {
+      return new SimpleBean(source.getValue());
+   }
+
+}