Fix ClassCastException for ParameterizedType

[dipold]

Fix ClassCastException for ParameterizedType as discussed here:
https://groups.google.com/forum/ #! topic/caelum-vraptor4/HQOwfMuLbCY

[garcia-jj]

Can you write a test for this case?

[dipold]

Sure

[lucascs]

why not change getActualType to return Type? (Class<?> implements Type)

[dipold]

because registerTypeHierarchyAdapter is expecting a Class<?>

[lucascs]

You can use it: http://google-gson.googlecode.com/svn/trunk/gson/docs/javadocs/com/google/gson/GsonBuilder.html#registerTypeAdapter(java.lang.reflect.Type, java.lang.Object)

[dipold]

I just tested your suggestion here and unfortunately doesn´t work as expected.

When I register a JsonSerializer<Enum<?>> the behavior expected is that all enums has intercept by this serializer, but none is intercepted.

Using registerTypeAdapter() Gson probably expect the type to be intercepted is exactly equals to Enum<?>

Any other suggestion?

[lucascs]

No, 🚢

On Fri, Oct 4, 2013 at 11:56 AM, Rafael Dipold notifications@gitpro.ttaallkk.topwrote:

In
vraptor-core/src/main/java/br/com/caelum/vraptor/serialization/gson/VRaptorGsonBuilder.java:

@@ -85,8 +85,11 @@ public Gson create() {
Type[] genericInterfaces = adapter.getClass().getGenericInterfaces();
ParameterizedType type = (ParameterizedType) genericInterfaces[0];

Type actualType = type.getActualTypeArguments()[0];

•   return (Class<?>) actualType;
  

•   if (actualType instanceof ParameterizedType)
  

•       return (Class<?>) ((ParameterizedType) actualType).getRawType();
  

•   else  
  

•       return (Class<?>) actualType;
  

  }

I just tested your suggestion here and unfortunately doesn´t work as
expected.

When I register a JsonSerializer<Enum<?>> the behavior expected is that
all enums has intercept by this serializer, but none is intercepted.

Using registerTypeAdapter() Gson probably expect the type to be
intercepted is exactly equals to Enum<?>

Any other suggestion?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/171/files#r6769781
.

[dipold]

How I can improve this test? @garcia-jj any suggestion?

I should do the same with JsonDeserializer<> in GsonDeserialization?

[garcia-jj]

@dipold, this pull request is cleared to :takeoff:?

[dipold]

@garcia-jj As I said in another topic, everything sounds good here

[garcia-jj]

Oh, sorry. i've missed something. You can ship if you want. Lucas already did your approval.

[dipold]

In fact I am still waiting because I haven´t received feedback...

[garcia-jj]