Hi Patric,
Can you try the latest CVS version. I checked in a fix that works fine
for me.
If the FieldDescriptor was also the Identity, then the conversion code
wasn't getting executed.
The fix checked in for Lewis Lin didn't cause this bug, but rather
exposed it, because before that fix two descriptors were being created
for the same field. One that was marked as the identity descriptor, and
the other that was treated as an ordinary field.
Thanks for reporting this.
--Keith
Patric Bechtel wrote:
>
> Hello Castor team (especially Keith!),
>
> we just discovered a new bug, this time in Castor XML.
>
> It's easily reproducable: Just take an Object with an
> Integer (or Long) identity field and try to unmarshal
> it. The exception states that it was not possible to
> convert a String into an Integer (or Long).
> As we found out, the bug was introduced on July 20
> (we made a checkout for Juli 19. and it worked).
> As far as the CVS logs state, there was only one
> relevant change that day, in XMLClassDescriptorAdapter,
> which changed to version 1.10 (which is broken).
>
> We tried to make sense WHY it doesn't work anymore,
> but so far, we analyzed JDO pretty much, but not
> the XML part. Additionally, the next day there so
> many changes to this part (and FieldDescriptor),
> which seem to depend on each other, that we can not
> get it to work again AFTER that (so to speak on the
> actual CVS).
> The initial patch was from Guillaume Rousse and Lewis
> Lin; that part seems to start the break.
> <speculation>
> As far as I understand it, it was a fix to suppress
> a double identity on marshalling. So far, it suppresses
> ANY identity field (at least the proper init of its
> FieldDescriptor), with the result of a broken instance
> of a FieldDescriptor which isn't capable of converting
> any values to anything else then Strings.
> </speculation>
>
> Maybe someone (Keith?) can help at that point?
>
> thanks a lot in advance,
>
> Patric
>
> sincerely,
>
> Patric Bechtel
> IPCON Informationssysteme
>
> PGP Public Key Fingerprint: 5579 8D11 C4A4 DD84 1CC0 D2D1 112F A924
>
> -----------------------------------------------------------
> If you wish to unsubscribe from this mailing, send mail to
> [EMAIL PROTECTED] with a subject of:
> unsubscribe castor-dev
-----------------------------------------------------------
If you wish to unsubscribe from this mailing, send mail to
[EMAIL PROTECTED] with a subject of:
unsubscribe castor-dev
