Adam Heath wrote:
> David E Jones wrote:
>> On Feb 26, 2010, at 10:33 AM, Adam Heath wrote:
>>
>>> Adam Heath (JIRA) wrote:
>>>>     [ 
>>>> https://issues.apache.org/jira/browse/OFBIZ-3520?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
>>>>  ]
>>>>
>>>> Adam Heath updated OFBIZ-3520:
>>>> ------------------------------
>>>>
>>>>    Attachment: 897606-testcase.patch
>>>>
>>>>> revision 897605 breaks certain delegator.find() EntityListIterator calls
>>>>> ------------------------------------------------------------------------
>>>>>
>>>>>                Key: OFBIZ-3520
>>>>>                URL: https://issues.apache.org/jira/browse/OFBIZ-3520
>>>>>            Project: OFBiz
>>>>>         Issue Type: Bug
>>>>>         Components: framework
>>>>>   Affects Versions: SVN trunk
>>>>>           Reporter: Adam Heath
>>>>>           Assignee: David E. Jones
>>>>>        Attachments: 897606-testcase.patch
>>>>>
>>>>>
>>>>> We recently upgraded our internal ofbiz package to a newer trunk version, 
>>>>> one from the end of January.  Ean then deployed that to the server it was 
>>>>> developing on.  This broke requirements processing.  I have reduced it, 
>>>>> however, to a simple patch, that works if I revert 897606, but breaks 
>>>>> when it is applied.
>>>>> Test case will be attached.
>>> David, could you please take a look at this, as it is your commit that
>>> causes this to break.
>> Did you read the discussion about this?

Which discussion?  I don't see any discussion mentioned in the
changelog, I went back to Jan 10, don't see anything near by.

>> Just search for the revision number.

I searched my own mail archive, which goes back to 12-31, and there
are no hits on 897606, except for your original commit mail, and this
issue/thread.

>> I believe it has already been fixed, and you'll either have to update to 
>> something more recent or back-port the fix.

The issue in the requirements system is not the problem.  I have given
a test case that stands by itself, that is broken with your commit
applied, and works without it.  The test case is doing something
completely reasonable, and was supported by previous versions of ofbiz.

I have checked that I didn't miss anything, and I haven't.



Reply via email to