Advanced #[serde(rename)] impl about serde-xml-rs HOT 8 OPEN

Looks cool.

Just a question: is it required to une an XPath ?
In your case it can be defined in rust like:

struct Reply {
    #[serde(rename = "$xml:path:Container")]
    container: Container
}

struct Container {
    #[serde(rename = "$xml:path:Container")]
    items: vec<i32>
}

but using the XPath is more generic.

Do you plan to develop this feature ? I need it during next month, and for that I will probably work on it soon.

from serde-xml-rs.

Using such kind of names will break serde for other destination types since they don't know anything about $xml: prefix and it will be treated as part of the field name (which is what rename is for).

from serde-xml-rs.

Yes made sense.
So do you think we can purpose Serde to add the field target to solve that ?

#[rename(rename = "container", target=json)]
#[rename(rename = "$xml:path:Container", target=xml)]
field: Type

or

#[rename(rename = "container"), target=json)]
#[rename(rename = "$xml:path:Container"), target=xml)]
field: Type

from serde-xml-rs.

I don't know, maybe? That issue should go on serde repo.

from serde-xml-rs.

I've just started getting into rust. Reading the response from the serde-rs project, are you still considering how to allow serialization for attributes?

from serde-xml-rs.

Regarding exchange with Serde Core developers, I started a new project to support that.
You can found it here: https://github.com/media-io/yaserde

from serde-xml-rs.

@MarcAntoine-Arnaud Does yaserde already support telling the serializer to serialize a struct field as a xml attribute? Or what is the current progress status of yaserde?

It seems to support it via #[yaserde(attribute)], does it also support something like #[serde(rename = "..")] in addition to that? :)

Can yaserde be used to serialize to all kinds of xml documents that one could want, or are some things unsupported?

from serde-xml-rs.

@Boscop please follow the discussion here.

from serde-xml-rs.