Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0026626Openbravo ERP07. Sales managementpublic2014-05-19 17:072016-12-08 20:16
Reportermaite 
Assigned Tocollazoandy4 
PriorityhighSeverityminorReproducibilityalways
StatusscheduledResolutionopen 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0026626: Improve validation when Voiding a shipment whose order has invoiced amount and Invoice Terms are 'D', 'O', 'S'
DescriptionAt the moment, you are not able to Void a shipment if related order line is invoiced and order's invoice terms is "After delivery" although total invoiced quantity is equals or lower to order's quantity
Steps To Reproduce1. Register Sales Order and add any line with qty=2 units. Process order
2. Register Goods Shipment using "Create lines from" and selecting 1 unit from previous order. Process Shipment
3. Register Sales Invoice using "Create lines from" and selecting previous shipment. Process Invoice
4. Register another Goods Shipment using "Create lines from" and selecting 1 unit from previous order. Process Shipment
5. Try to Void this last shipment (which is not invoiced) and realize that error "'It is not possible to void a shipment of an order that has an invoice rule after delivery having invoiced lines.'" is received
Proposed SolutionInvoiced amount should be considered in global and user should be able to Void receipt always that invoiced quantity is not higher that ordered quantity
Additional Information
TagsNo tags attached.
Relationships
caused by defect 00108452.35MP16 closed rafaroda Generate Invoices process with reactivated orders 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-05-19 17:07maiteNew Issue
2014-05-19 17:07maiteAssigned To => dmiguelez
2014-05-19 17:07maiteModules => Core
2014-05-19 17:07maiteResolution time => 1406844000
2014-05-19 17:07maiteTriggers an Emergency Pack => No
2014-05-19 17:07maiteAssigned Todmiguelez => pramakrishnan
2014-06-16 12:48jonalegriaesarteTarget VersionPR14Q3 => PR14Q4
2014-09-09 13:15AtulOpenbravoAssigned Topramakrishnan => AtulOpenbravo
2014-09-09 13:16AtulOpenbravoStatusnew => scheduled
2014-09-09 13:16AtulOpenbravofix_in_branch => pi
2014-09-26 14:14vmromanosNote Added: 0070436
2014-09-26 14:14vmromanosStatusscheduled => feedback
2014-09-26 14:15vmromanosRelationship addedcaused by 0010845
2014-09-26 14:17vmromanosAssigned ToAtulOpenbravo => vmromanos
2014-09-29 16:12jonalegriaesarteTarget Version3.0PR14Q4 => 3.0PR15Q1
2014-10-14 18:42vmromanosNote Added: 0070938
2014-12-22 12:38vmromanosTarget Version3.0PR15Q1 => 3.0PR15Q2
2014-12-22 12:38vmromanosfix_in_branchpi =>
2015-02-03 17:26SandrahuguetAssigned Tovmromanos => maite
2015-03-03 13:24maiteStatusfeedback => scheduled
2015-03-03 13:24maiteNote Added: 0075116
2015-03-03 13:24maitefix_in_branch => pi
2015-03-03 13:24maiteAssigned Tomaite => vmromanos
2015-03-10 11:54SandrahuguetResolution time1406844000 =>
2015-03-10 11:54Sandrahuguetfix_in_branchpi =>
2015-03-10 11:56SandrahuguetStatusscheduled => resolved
2015-03-10 11:56SandrahuguetFixed in SCM revision => 0
2015-03-10 11:56SandrahuguetResolutionopen => fixed
2015-03-10 11:56SandrahuguetStatusresolved => new
2015-03-10 11:56SandrahuguetResolutionfixed => open
2015-03-13 10:34jonalegriaesarteTarget Version3.0PR15Q2 =>
2015-11-03 13:10ngarciaIssue Monitored: ngarcia
2016-11-15 16:36collazoandy4Assigned Tovmromanos => collazoandy4
2016-11-15 16:36collazoandy4Statusnew => scheduled
2016-12-08 20:16collazoandy4Note Added: 0092229

Notes
(0070436)
vmromanos   
2014-09-26 14:14   
Hi Maite,

As discussed by Skype, please clarify whether the current code is really wrong and we should avoid to reactivate a goods shipment associated to an order with any of its lines already invoiced.

If so, we should remove this check from the M_INOUT_POST, because it should be enough to just check whether any of the lines in the shipment is already linked to an invoice (which is done just before this check)
(0070938)
vmromanos   
2014-10-14 18:42   
Maite, kindly reminder...
(0075116)
maite   
2015-03-03 13:24   
Check from the M_INOUT_POST needs to be reviewed in order to be able to void shipment in explained situation
(0092229)
collazoandy4   
2016-12-08 20:16   
Test plan
  Register a Sales Order and add any line with qty > 1 and process the order
  Register a new Goods Shipment and create the lines from the Create Lines From button
  From the dialog fill out the Warehouse position and select the previous order created, select the product line and click on OK button
  Edit the Shipment Line and set Movement Quantity to 1 and Complete the Shipment
  Register a new Sales Invoice and create a lines from the Create Lines From button
  In the dialog select the previous Shipment Created, select the line and click on OK button. Complete the invoice
  Register a new Goods Shipment and create the lines from the Create Lines From button
  From the dialog fill out the Warehouse position and select the order created, select the product line and click on OK button
  Complete the Shipment and now you can void this shipment created