[RESOLVIDO] VRaptor - Deserialização XML -> Objeto com XStream - Bug

15 de abril de 2013 9 respostas

alias 15 de abril de 2013

Bom dia, amigos.

Estou com um problema com o VRaptor que creio ser um bugzinho…fiz um workaround pra resolver mas vou postar o cenário aqui e a galera do projeto confirmando sendo um bug, crio uma issue lá pra corrigir (ou eu tento corrigir sem gerar mais bugs :lol: )

Bom basicamente estou recebendo um xml que deve ser deserializado para um objeto que é o argumento da minha action. O problema é que no objeto em questão os alias do XStream foram configurados via annotations (@XStreamAlias), e o objeto que faz a deserialização (esse aqui: https://github.com/caelum/vraptor/blob/master/vraptor-core/src/main/java/br/com/caelum/vraptor/deserialization/XStreamXMLDeserializer.java) não chama o método processAnnotations do XStream…

Pra resolver temporariamente eu criei um componente herdando desse cara ai e sobrescrevi o método getConfiguredXStream, e inclui a chamada pro método do XStream.

É um bugzinho?

Valeu!

9 Respostas

Lucas_Cavalcanti 16 de abr. de 2013

não sei se é um bug, mas talvez seja uma feature faltando

abre uma issue lá por favor

alias 16 de abr. de 2013

A serialização objeto -> xml chama o método processAnnotations, e a desserialização não, pra mim tá mais pra bug…enfim, posso corrigir isso lucas, o que voce acha, esse classe/metodo ai que eu citei é o lugar ideal pra fazer isso?

valeu!

Lucas_Cavalcanti 16 de abr. de 2013

Como ele é feito direto no serializer:

github.com

caelum/vraptor/blob/master/vraptor-core/src/main/java/br/com/caelum/vraptor/serialization/xstream/XStreamSerializer.java

/***
 * Copyright (c) 2009 Caelum - www.caelum.com.br/opensource All rights reserved.
 *
 * Licensed under the Apache License, Version 2.0 (the "License"); you may not
 * use this file except in compliance with the License. You may obtain a copy of
 * the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 * License for the specific language governing permissions and limitations under
 * the License.
 */
package br.com.caelum.vraptor.serialization.xstream;

import static br.com.caelum.vraptor.serialization.xstream.VRaptorClassMapper.isPrimitive;
import static com.google.common.base.Preconditions.checkNotNull;

This file has been truncated. show original

acho que é justo fazer aqui:

github.com

caelum/vraptor/blob/master/vraptor-core/src/main/java/br/com/caelum/vraptor/deserialization/XStreamXMLDeserializer.java#L49


	private final ParameterNameProvider provider;
	private final XStreamBuilder builder;


	public XStreamXMLDeserializer(ParameterNameProvider provider, XStreamBuilder builder) {
		this.provider = provider;
		this.builder = builder;
	}


	public Object[] deserialize(InputStream inputStream, ResourceMethod method) {
		Method javaMethod = method.getMethod();
		Class<?>[] types = javaMethod.getParameterTypes();
		if (types.length == 0) {
			throw new IllegalArgumentException("Methods that consumes xml must receive just one argument: the xml root element");
		}
		XStream xStream = getConfiguredXStream(javaMethod, types);


		Object[] params = new Object[types.length];
		
		chooseParam(types, params, xStream.fromXML(inputStream));


		return params;

linha 49, passando por todos os types e processando as anotações deles.

vc faz lá e manda um pull request, por favor? =)

tenta colocar um teste pra isso, ou usando mocks ou anotando classes e mandando deserializar.

alias 16 de abr. de 2013

Legal lucas, cadastrei a issue lá e hoje já devo fazer isso, como é mesmo tem que fazer o fork da branch master né?

Lucas_Cavalcanti 16 de abr. de 2013

sim, o ideal é fazer o fork a partir do master do VRaptor, de preferencia criando um branch com o nome da feature no seu fork. Aí fica mais fácil de fazer rebase e manter o master sincronizado com o VRaptor, supondo que vc vai criar vários pull requests no VRaptor =)

alias 18 de abr. de 2013

Fiz tudo lá lucas, alterei também o JsonDeserializer como você pediu e incluí os testes para o cenário com anotações.

Pra quem mais precisar, sugiro usar por enquanto essa abordagem que sugeri, que é criar um componente sobrescrevendo o desserializer do VRaptor, e sobrescrever o método que configura o XStream (e aí incluir o processamento das annotations)

Valeu!

Lucas_Cavalcanti 18 de abr. de 2013

Falhou o build =(

Mas acho que não foi da sua feature… vou dar uma olhada lá! valeu =)

alias 19 de abr. de 2013

É eu vi, os testes dos lugares onde mexi (incluindo a nova classe de teste que adicionei) passaram, quebraram alguns testes do LinkToHandler…de repente tá faltando alguma coisa no source que tá na minha máquina, precisa que eu faça alguma outra coisa e mande de novo o código?

Lucas_Cavalcanti 19 de abr. de 2013

Já fiz o merge, valeu!

Criado 15 de abril de 2013

Ultima resposta 19 de abr. de 2013

Respostas 9

Participantes 2

9 Respostas

Topicos relacionados