Yes, it should. Thanks for cross-checking. Can you please create a new
Jira issue ?

Werner

Paul Croarkin wrote:
> Werner,
> 
> I'm looking at the trunk of IDResolverImpl:
> http://svn.codehaus.org/castor/castor/trunk/src/main/java/org/exolab/castor/xml/IDResolverImpl.java
> 
> and see this in the bind method:
> 
> void bind(final String id, final Object object, final boolean
> isValidating) throws ValidationException {
>   if (id.equals("") || id == null) {
>             throw new ValidationException ("Invalid (empty) ID value
> encountered");
>         }
> ...
> }
> 
> My guess is that the line should look something like:
> 
> if ((isValidating) && (id.equals("") || id == null)) {
>             throw new ValidationException ("Invalid (empty) ID value
> encountered");
>         }
> 
> 
> Is there another branch where I should look?
> 
> thanks,
> Paul
> 
> --- Werner Guttmann <[EMAIL PROTECTED]> wrote:
> 
>> Paul,
>>
>> I do remember that we relaxed some of those 'requirements' already. I
>> wonder whether this has happended before actually releasing 1.1 or
>> after doing so. Have you had a look at the release notes in SVN trunk
>> or the changelog page for 1.1.1 in Jira already ? 
>>
>> Werner
>>
>>> -----Ursprüngliche Nachricht-----
>>> Von: Paul Croarkin [mailto:[EMAIL PROTECTED]
>>> Gesendet: Dienstag, 03. April 2007 22:29
>>> An: [email protected]
>>> Betreff: [castor-user] [XML] New IdResolver implementation
>>>
>>> Hello,
>>>
>>> We are trying to upgrade from castor 1.0.1 to 1.1. When trying to
>>> unmarshall we are getting a MarshalException: "Invalid (empty) ID
>> value
>>> encountered" that we did not get in castor 1.0.1. We are explicitly
>>> setting validation to false:
>>>
>>> Unmarshaller unmarshaller = new Unmarshaller(clazz);
>>> unmarshaller.setValidation(false);
>>>
>>> I thought maybe we could extend IDResolverImpl, but it is not
>> public.
>>> Any assistance would be greatly appreciated.
>>>
>>> Thank you,
>>>
>>> Paul Croarkin
>>>
>>>
>>>
>>>
>>>
>>>
> __________________________________________________________________________
>>> __________
>>> Don't get soaked.  Take a quick peek at the forecast
>>> with the Yahoo! Search weather shortcut.
>>> http://tools.search.yahoo.com/shortcuts/#loc_weather
>>>
>>>
>> ---------------------------------------------------------------------
>>> To unsubscribe from this list please visit:
>>>
>>>     http://xircles.codehaus.org/manage_email
>>
>> ---------------------------------------------------------------------
>> To unsubscribe from this list please visit:
>>
>>     http://xircles.codehaus.org/manage_email
>>
>>
> 
> 
> 
> 
>  
> ____________________________________________________________________________________
> Need Mail bonding?
> Go to the Yahoo! Mail Q&A for great tips from Yahoo! Answers users.
> http://answers.yahoo.com/dir/?link=list&sid=396546091
> 
> ---------------------------------------------------------------------
> To unsubscribe from this list please visit:
> 
>     http://xircles.codehaus.org/manage_email
> 


---------------------------------------------------------------------
To unsubscribe from this list please visit:

    http://xircles.codehaus.org/manage_email

Reply via email to