[
https://issues.apache.org/jira/browse/OFBIZ-5308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13772034#comment-13772034
]
Jacques Le Roux edited comment on OFBIZ-5308 at 9/19/13 4:58 PM:
-----------------------------------------------------------------
== TYPO ==
Karim Rahimpur, the path author (OFBIZ-2163) did not answer. But by analogy
with clearAllLines() method, I decided that packageSeq should not be less that
1!
To be backported after OFBIZ-4794
was (Author: jacques.le.roux):
Rhe path's author ((OFBIZ-2163 - Karim Rahimpur) did not answer. But by
analogy with clearAllLines() method I decided that packageSeq should not be
less that 1!
To be backported after OFBIZ-4794
> If you clear the last packed item from from a shipment with two items and
> then complete the pack, you get the exception below
> -----------------------------------------------------------------------------------------------------------------------------
>
> Key: OFBIZ-5308
> URL: https://issues.apache.org/jira/browse/OFBIZ-5308
> Project: OFBiz
> Issue Type: Bug
> Components: product
> Affects Versions: Release Branch 11.04, SVN trunk, Release Branch 12.04,
> Release Branch 13.07
> Environment: All
> Reporter: Skip Dever
> Assignee: Jacques Le Roux
> Priority: Minor
>
> If you clear the last packed item from from a shipment with two items and
> then complete the pack, you get the exception below:
> Exception thrown while creating the "newEntity" ....ShipmentPackageContent
> ... Key ... is not present in table shipment_package
> This is happening because
> org.ofbiz.shipment.packing.PackingSession.clearLine() around line 600 is
> decrementing packageSeq from 1 to zero. This causes createPackages() to not
> generate any ShipmentPackages because of the for statement on packageSeq and
> therefore the subsequent exception.
> I fixed this by just commenting out the block starting with
> "if(line.packageSeq ==". I did not submit a patch because I do not fully
> understand the logic and do not have time now to learn it now.
> Hopefully the author will be better equipped to deal with it properly.
> Skip
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira
