Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0049693
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Openbravo ERP] 07. Sales managementtrivialhave not tried2022-06-30 07:022022-07-11 10:23
ReporterAtulOpenbravoView Statuspublic 
Assigned ToAtulOpenbravo 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0049693: API Order Loader: Return ticket logics to implement

DescriptionSMCP cannot use the originalOrderLineId field for return ticket

In case of a Return Ticket (so with the flag isReturn = true OR if amount negative)

At orderLine level there will be the returned item with the new field originalSalesOrderDocumentNumber

When the system processes the External Order Loader, the system will top the line with the same item of the original document number as “returned” (in the same logic as what is done with originalOrderLineId) in order to not allow to return twice the same item of the original document
Steps To Reproduce-
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0138963)
hgbot (developer)
2022-07-01 20:38

 Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/844 [^]
(0138991)
hgbot (developer)
2022-07-03 20:51

 Merge Request created: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/208 [^]
(0139223)
hgbot (developer)
2022-07-11 10:21

 Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 2cce62b2ce52ecb0c215b165d2eb88f22b93a463
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/2cce62b2ce52ecb0c215b165d2eb88f22b93a463 [^]

Fixes ISSUE-0049693: API Order Loader: Return ticket logics
to implement

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139224)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 80433685a16c8c82f9fc9101ac8b5aebb9be3cc0
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/80433685a16c8c82f9fc9101ac8b5aebb9be3cc0 [^]

Related to ISSUE-0049693: Apply formatting

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139225)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: de2323391c334b72e366c14f03d0a7c95aab3d24
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/de2323391c334b72e366c14f03d0a7c95aab3d24 [^]

Related to ISSUE-0049693: Add check whether the isReturn flag
is true or line quantity is negative inorder to compute the
originalOrderLineId based on originlSalesOrderDocumentNo and
product UPCEAN properties.

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139226)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 045f9d2f62243a425c28e97b19859a3b8dc7ba77
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/045f9d2f62243a425c28e97b19859a3b8dc7ba77 [^]

Related To ISSUE-0049693: Compute returned quantity related
to originalOrderLine in the same return order with multiple
lines having same product, apply formatting

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139227)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 3d0f2581e658e5548a0b7871f8847c658b2c7c87
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/3d0f2581e658e5548a0b7871f8847c658b2c7c87 [^]

Related to ISSUE-0049693:Improve computation of originalOrderLineId
in case Return Order has multiple lines with same product and
they need to be linked to same originalOrderLineId

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139228)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: a0fb0492c842795d998fc50d057c5a1b9ce84fbc
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/a0fb0492c842795d998fc50d057c5a1b9ce84fbc [^]

Related to ISSUE-0049693: Add preference to consider original order
line in multiple return order lines.

---
M src-db/database/sourcedata/AD_REF_LIST.xml
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139229)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: d2c9003b712e5bd10e0dedeaaa17271380b334a5
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/d2c9003b712e5bd10e0dedeaaa17271380b334a5 [^]

Related to ISSUE-0049693: Code review improvements

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139230)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 1ccbaa11f77ff7c0f3082eed6067b4b2780b0cb5
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/1ccbaa11f77ff7c0f3082eed6067b4b2780b0cb5 [^]

Related to ISSUE-0049693: Code Review Improvements

**Remove preference to consider Original Order Line in multiple
return line and its usage.

---
M src-db/database/sourcedata/AD_REF_LIST.xml
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139231)
hgbot (developer)
2022-07-11 10:21

 Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal [^]
Changeset: 7490110d31ee6424b251aaa355c9e63473a0e31c
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:38
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/commit/7490110d31ee6424b251aaa355c9e63473a0e31c [^]

Related to ISSUE-0049693: Code review improvements

---
M src/org/openbravo/retail/posterminal/ExternalOrderLoader.java
---
(0139232)
hgbot (developer)
2022-07-11 10:21

 Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.posterminal/-/merge_requests/844 [^]
(0139233)
hgbot (developer)
2022-07-11 10:23

 Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 72bd7dfcb2c0496329aa3910a805859cb0f1017b
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:56
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/72bd7dfcb2c0496329aa3910a805859cb0f1017b [^]

Related to ISSUE-0049693: Add junit test to verfiy the goods
shipment line in return order lines

---
A src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/VerifyReturnOrderLineDetailsIssue49693.java
A src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-order-for-return-order-minimal-1.json
A src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-order-for-return-order-multiline-1.json
A src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-return-order-minimal-1.json
A src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-return-order-multiline-1.json
A src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-return-order-multiline-2.json
M src-test/org/openbravo/test/mobile/retail/pack/webservice/baseIntegration/BaseRetailOrderLoaderTest.java
---
(0139234)
hgbot (developer)
2022-07-11 10:23

 Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 0472787b12d97dde351cc2c52e37a72303b4c1d7
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:56
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/0472787b12d97dde351cc2c52e37a72303b4c1d7 [^]

Related to ISSUE-0049693:Add test data on improved computation
of originalOrderLineId in case Return Order has multiple lines
with same product that needs to be linked to same
originalOrderLineId

---
A src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-order-for-return-order-minimal-2.json
M src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/VerifyReturnOrderLineDetailsIssue49693.java
---
(0139235)
hgbot (developer)
2022-07-11 10:23

 Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: fb07e16d65590f6eb7f64401329726264eafae64
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:56
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/fb07e16d65590f6eb7f64401329726264eafae64 [^]

Related to ISSUE-0049693: Set proper data in the return json,
add data for orphan line.

---
M src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-return-order-multiline-1.json
M src-test/org/openbravo/test/mobile/retail/pack/webservice/tests/orderloader/message-return-order-multiline-2.json
---
(0139236)
hgbot (developer)
2022-07-11 10:23

 Repository: https://gitlab.com/openbravo/ci/mobile-test [^]
Changeset: 16663228931fdfc04a9c9d0f38bd99b8f90f0755
Author: Atul Gaware <atul.gaware@openbravo.com>
Date: 11-07-2022 08:06:56
URL: https://gitlab.com/openbravo/ci/mobile-test/-/commit/16663228931fdfc04a9c9d0f38bd99b8f90f0755 [^]

Related to ISSUE-0049693: Add VerifyReturnOrderLineDetails test
in job003

---
M src-test/org/openbravo/test/mobile/retail/pack/selenium/suites/concurrent/job003/WebServiceSuite.java
---
(0139237)
hgbot (developer)
2022-07-11 10:23

 Merge request merged: https://gitlab.com/openbravo/ci/mobile-test/-/merge_requests/208 [^]

- Issue History
Date Modified Username Field Change
2022-06-30 07:02 AtulOpenbravo New Issue
2022-06-30 07:02 AtulOpenbravo Assigned To => Triage Omni OMS
2022-06-30 07:02 AtulOpenbravo Modules => Core
2022-06-30 07:02 AtulOpenbravo Triggers an Emergency Pack => No
2022-06-30 07:02 AtulOpenbravo Assigned To Triage Omni OMS => AtulOpenbravo
2022-07-01 20:38 hgbot Note Added: 0138963
2022-07-01 20:43 AtulOpenbravo Status new => scheduled
2022-07-03 20:51 hgbot Note Added: 0138991
2022-07-07 07:51 ivancaceres Issue Monitored: ivancaceres
2022-07-11 10:21 hgbot Resolution open => fixed
2022-07-11 10:21 hgbot Status scheduled => closed
2022-07-11 10:21 hgbot Fixed in Version => RR22Q4
2022-07-11 10:21 hgbot Note Added: 0139223
2022-07-11 10:21 hgbot Note Added: 0139224
2022-07-11 10:21 hgbot Note Added: 0139225
2022-07-11 10:21 hgbot Note Added: 0139226
2022-07-11 10:21 hgbot Note Added: 0139227
2022-07-11 10:21 hgbot Note Added: 0139228
2022-07-11 10:21 hgbot Note Added: 0139229
2022-07-11 10:21 hgbot Note Added: 0139230
2022-07-11 10:21 hgbot Note Added: 0139231
2022-07-11 10:21 hgbot Note Added: 0139232
2022-07-11 10:23 hgbot Note Added: 0139233
2022-07-11 10:23 hgbot Note Added: 0139234
2022-07-11 10:23 hgbot Note Added: 0139235
2022-07-11 10:23 hgbot Note Added: 0139236
2022-07-11 10:23 hgbot Note Added: 0139237

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker