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

alias · Abril 15, 2013, 1:41pm

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!

Lucas_Cavalcanti · Abril 16, 2013, 4:18pm

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

abre uma issue lá por favor

alias · Abril 16, 2013, 4:24pm

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 · Abril 16, 2013, 7:51pm

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 · Abril 16, 2013, 8:10pm

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 · Abril 16, 2013, 8:41pm

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 · Abril 18, 2013, 2:01pm

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 · Abril 18, 2013, 6:20pm

Falhou o build =(

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

alias · Abril 19, 2013, 12:45am

É 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 · Abril 19, 2013, 3:04am

Já fiz o merge, valeu!

system · Janeiro 6, 2016, 4:11am

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

Cursos de Mobile

Cursos de Programação

Cursos de Front-end

Cursos de DevOps

Cursos de Design & UX

Cursos de Business

Cursos de Data & BI