Notes |
|
(0102172)
|
hgbot
|
2018-02-05 10:22
|
|
Repository: erp/devel/pi
Changeset: fceb92c459574b09d41b2ec9577985eb27629a49
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Feb 05 10:04:17 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/fceb92c459574b09d41b2ec9577985eb27629a49 [^]
Fixed issue 37827: [ri] Added Referenced Inventory support
---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/utility/FIN_Utility.java
M src-db/database/model/functions/M_MOVEMENT_POST.xml
M src-db/database/model/functions/M_UPDATE_INVENTORY.xml
M src-db/database/model/tables/M_ATTRIBUTESETINSTANCE.xml
M src-db/database/model/tables/M_MOVEMENTLINE.xml
M src-db/database/model/tables/M_STORAGE_DETAIL.xml
M src-db/database/model/triggers/M_MOVEMENTLINE_TRG.xml
M src-db/database/sourcedata/AD_CALLOUT.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src-db/database/sourcedata/AD_MENU.xml
M src-db/database/sourcedata/AD_MESSAGE.xml
M src-db/database/sourcedata/AD_MODEL_OBJECT.xml
M src-db/database/sourcedata/AD_MODEL_OBJECT_MAPPING.xml
M src-db/database/sourcedata/AD_REFERENCE.xml
M src-db/database/sourcedata/AD_TAB.xml
M src-db/database/sourcedata/AD_TABLE.xml
M src-db/database/sourcedata/AD_TREENODE.xml
M src-db/database/sourcedata/AD_WINDOW.xml
M src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
M src-db/database/sourcedata/OBUIAPP_PROCESS.xml
M src-db/database/sourcedata/OBUIAPP_REF_WINDOW.xml
M src-test/src/org/openbravo/test/AllAntTaskTests.java
M src/org/openbravo/erpCommon/info/AttributeSetInstance.java
M src/org/openbravo/erpCommon/utility/AttributeSetInstanceValue.java
A src-db/database/model/tables/M_REFINVENTORY.xml
A src-db/database/model/tables/M_REFINVENTORY_TYPE.xml
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxForcedReservation.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxFullReservationTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservationTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservationTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxSeveralStorageDetailsTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryExceptionTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullBoxTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxFullReservation.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxPartialReservation.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialBoxTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxFullReservation.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventorySequenceTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestSuite.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestUtils.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxReservationTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryUnboxTest.java
A src/org/openbravo/common/actionhandler/ReferencedInventoryBoxHandler.java
A src/org/openbravo/common/actionhandler/ReferencedInventoryUnBoxHandler.java
A src/org/openbravo/erpCommon/ad_callouts/SE_RefInventory_RefInvType.java
A src/org/openbravo/event/ReferenceInventoryEventHandler.java
A src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java
A src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
A src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
A src/org/openbravo/materialmgmt/refinventory/ReservationManager.java
A src/org/openbravo/materialmgmt/refinventory/UnboxProcessor.java
---
|
|
|
(0102173)
|
hgbot
|
2018-02-05 10:22
|
|
Repository: erp/devel/pi
Changeset: e4712f47df4392974d1bb925f3a005d16fea3d15
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Feb 05 10:16:55 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/e4712f47df4392974d1bb925f3a005d16fea3d15 [^]
Fixed issue 37827: [ri] Removed TODO as no need of hooks
---
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
---
|
|
|
(0102185)
|
hgbot
|
2018-02-05 12:43
|
|
Repository: erp/devel/pi
Changeset: 53ef76edf8452311b8f13cb746f101e71d1872ea
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Feb 05 12:42:34 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/53ef76edf8452311b8f13cb746f101e71d1872ea [^]
Fixed issue 37827: [ri] updated copyright year
---
M src/org/openbravo/erpCommon/ad_callouts/SE_RefInventory_RefInvType.java
M src/org/openbravo/event/ReferenceInventoryEventHandler.java
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
---
|
|
|
(0102209)
|
hgbot
|
2018-02-06 10:55
|
|
Repository: erp/devel/pi
Changeset: fcdc7c1db65bceab7753c85ae69bfce370a4533d
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Tue Feb 06 10:54:44 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/fcdc7c1db65bceab7753c85ae69bfce370a4533d [^]
Fixed issue 37827: [ri] clear session after each test execution
---
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTest.java
---
|
|
|
(0102216)
|
hgbot
|
2018-02-06 13:01
|
|
Repository: erp/devel/pi
Changeset: 756a8987324d41af1e54c5cc5d1c419767c39821
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Tue Feb 06 12:59:15 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/756a8987324d41af1e54c5cc5d1c419767c39821 [^]
Fixed issue 37827: [ri] clear session after each individual test execution
---
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxForcedReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxFullReservationTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservationTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservationTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxSeveralStorageDetailsTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullBoxTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxFullReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxPartialReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialBoxTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxFullReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxTest.java
---
|
|
|
(0102275)
|
hgbot
|
2018-02-08 17:48
|
|
Repository: erp/devel/pi
Changeset: 967f7b6068eedef0e146bb97f70345ac36ecbe5f
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Thu Feb 08 17:46:57 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/967f7b6068eedef0e146bb97f70345ac36ecbe5f [^]
Fixed issue 37827: [ri] Added message to Application Dictionary
---
M src-db/database/sourcedata/AD_MESSAGE.xml
M src/org/openbravo/materialmgmt/refinventory/UnboxProcessor.java
---
|
|
|
(0102391)
|
hgbot
|
2018-02-14 10:23
|
|
Repository: erp/devel/pi
Changeset: de2414ba537927d64d10009227fa146791a0ec50
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Feb 12 15:19:05 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/de2414ba537927d64d10009227fa146791a0ec50 [^]
Fixed issue 37827: [ri] Reservation Management Refactor
Removed new reservation management initially implemented for Referenced Inventory, and adapted the Core's engine to work with RI.
M_Movement_Post:
1. The new storage detail passed to the M_RESERVE_STOCK_MANUAL takes into account the new m_attributesetinstanceto_id
2. The check for unique bin in referenced inventory is only executed for RI movement, thus improving performance
M_RESERVE_STOCK_MANUAL:
1. The code that gets the storage detail is able to work with reference inventory
ReferencedInventoryProcessor:
1. Removed ReservationManager stuff as it is now directly executed in the existing Core's code
2. The movement lines are created in 2 steps: first without reservation and, if necessary, moving reservations.
3. Some util methods are moved to ReferencedInventoryUtil class.
Automatic tests modifications to adapt to new logic:
1. Now we always work in just one reservation, and we don't create a new one releasing from the existing one anymore.
2. The goods movement might create several lines (depending on reservation quantities)
3. Split some tests to make them more readable
---
M src-db/database/model/functions/M_MOVEMENT_POST.xml
M src-db/database/model/functions/M_RESERVE_STOCK_MANUAL.xml
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxForcedReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullUnboxPartialReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestSuite.java
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservation1MovementLineTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservation2MovementLinesTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservation1MovementLineTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservation2MovementLinesTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation1MovementLineUnboxTest.java
A src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation2MovementLinesUnboxTest.java
R src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxOverReservationTest.java
R src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxPartialReservationTest.java
R src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialUnboxPartialReservation.java
R src/org/openbravo/materialmgmt/refinventory/ReservationManager.java
---
|
|
|
(0102392)
|
hgbot
|
2018-02-14 10:24
|
|
Repository: erp/devel/pi
Changeset: e9356b180aac683c6d1cecdf9056bb0e0c2cd313
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Feb 12 19:29:47 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/e9356b180aac683c6d1cecdf9056bb0e0c2cd313 [^]
Fixed issue 37827: [ri] declare method as public to be used by AWO
---
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
---
|
|
|
(0102564)
|
hgbot
|
2018-02-21 19:05
|
|
Repository: erp/devel/pi
Changeset: 9cb6170994b12df7d36d7ff6fb53b33d1cde11aa
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Tue Feb 13 18:05:20 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/9cb6170994b12df7d36d7ff6fb53b33d1cde11aa [^]
Fixed issue 37827: [ri] redesign M_REFINVENTORY_VALUE_UN unique constraint
For AWO purposes it's better to ensure a unique referenced inventory per client.
---
M src-db/database/model/tables/M_REFINVENTORY.xml
---
|
|
|
(0102565)
|
hgbot
|
2018-02-21 19:05
|
|
Repository: erp/devel/pi
Changeset: 499ed06c9d18daaf5f81314a39e71629a3495a39
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Sat Feb 17 09:33:17 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/499ed06c9d18daaf5f81314a39e71629a3495a39 [^]
Fixed issue 37827: [ri] set original attributesetinstance=0 if null
---
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
---
|
|
|
(0102566)
|
hgbot
|
2018-02-21 19:05
|
|
Repository: erp/devel/pi
Changeset: 526633fb1fd0db848ff139ce8450bbdc8f6dc2e2
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Tue Feb 20 17:38:26 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/526633fb1fd0db848ff139ce8450bbdc8f6dc2e2 [^]
Fixed issue 37827: [ri] Moved validation over storage bin in boxing to Core
---
M src-db/database/sourcedata/AD_VAL_RULE.xml
M src-db/database/sourcedata/OBUIAPP_PARAMETER.xml
---
|
|
|
(0102567)
|
hgbot
|
2018-02-21 19:05
|
|
|
|
(0102568)
|
hgbot
|
2018-02-21 19:05
|
|
Repository: erp/devel/pi
Changeset: b6a93ba857f3e251039dab3a0a54830e90104686
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Wed Feb 21 10:12:00 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b6a93ba857f3e251039dab3a0a54830e90104686 [^]
Fixed issue 37827: [ri] Added RefInventory to M_PRODUCT_STOCK_V and related selectors
---
M src-db/database/model/views/M_PRODUCT_STOCK_V.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/OBUISEL_SELECTOR_FIELD.xml
---
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
(0102837)
|
hgbot
|
2018-02-27 09:08
|
|
Repository: erp/devel/pi
Changeset: 5c21517ea155534712ce7ced27fa91049b0790e6
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Feb 26 17:29:18 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/5c21517ea155534712ce7ced27fa91049b0790e6 [^]
Fixed issue 37827: [ri] M_Movement_Post referenced inventory validation improvement
The validation was previously executed only for a boxing activity, however we also need to validate any movement where a referenced inventory is involved.
This avoids partial movements of boxes
---
M src-db/database/model/functions/M_MOVEMENT_POST.xml
---
|
|
|
|
|
|
(0102987)
|
hgbot
|
2018-03-05 17:09
|
|
Repository: erp/devel/pi
Changeset: 00def91a508f1542966d141415ddc73ff9cd38cf
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Fri Mar 02 13:33:57 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/00def91a508f1542966d141415ddc73ff9cd38cf [^]
Fixed issue 37827: [ri] [cr] M_Movement_Post validation at document level
The validation that ensures the same referenced inventory is not located in two different bins have been moved from line level to document level.
This is necessary to ensure we can fully move all the storage details within a referenced inventory to a different bin.
---
M src-db/database/model/functions/M_MOVEMENT_POST.xml
---
|
|
|
(0102989)
|
hgbot
|
2018-03-05 17:09
|
|
Repository: erp/devel/pi
Changeset: 66d89284c6608040f4c3992f315d9f68fbcd8501
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Mar 05 13:33:07 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/66d89284c6608040f4c3992f315d9f68fbcd8501 [^]
Fixed issue 37827: [ri] [cr] Code review improvements
M_RESERVE_STOCK_MANUAL:
Removed useless COALESCE
M_UPDATE_INVENTORY:
Added comment to clarify code
AD:
Added help/description where missed
Declared Referenced Inventory windows as Purchase to force on hand selector to not filter by on hand warehouses.
ReferencedInventoryUnBoxHandler:
Removed useles throw JSONException
SE_RefInventory_RefInvType:
Use org.slf4j.Logger
AttributeSetInstance:
Use && instead of &
BoxProcessor:
Add to cache map the found attribute set instance so the process is faster
Removed useless override of method createAndProcessGoodsMovement(). This would create a false positive
ReferencedInventoryProcessor:
Do not check reservationStockScroll != null
Do not create variable to inmediately return it
ReferencedInventoryUtil:
Added Javadoc in public method
Automatic Tests:
Declared static constants where missed
Clarify if condition by creating private methods
Declare private methods where useful
Use org.slf4j.Logger
---
M src-db/database/model/functions/M_RESERVE_STOCK_MANUAL.xml
M src-db/database/model/functions/M_UPDATE_INVENTORY.xml
M src-db/database/sourcedata/AD_CALLOUT.xml
M src-db/database/sourcedata/AD_REFERENCE.xml
M src-db/database/sourcedata/AD_TAB.xml
M src-db/database/sourcedata/AD_VAL_RULE.xml
M src-db/database/sourcedata/AD_WINDOW.xml
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxForcedReservation.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryBoxTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryFullBoxTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryPartialBoxTest.java
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventorySequenceTest.java
M src/org/openbravo/common/actionhandler/ReferencedInventoryBoxHandler.java
M src/org/openbravo/common/actionhandler/ReferencedInventoryUnBoxHandler.java
M src/org/openbravo/erpCommon/ad_callouts/SE_RefInventory_RefInvType.java
M src/org/openbravo/erpCommon/info/AttributeSetInstance.java
M src/org/openbravo/materialmgmt/refinventory/BoxProcessor.java
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
---
|
|
|
(0102990)
|
hgbot
|
2018-03-05 17:09
|
|
Repository: erp/devel/pi
Changeset: a59ae9cabeb052c512eecae6c42df48b1e978aa3
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Mar 05 13:45:45 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/a59ae9cabeb052c512eecae6c42df48b1e978aa3 [^]
Fixed issue 37827: [ri] [cr] Code review improvements
Reordered and hide some fields
Added UOM display logic where missed
Fixed javadoc typo
Control NPE in ReferencedInventoryUtil.getProposedValueFromSequenceOrNull()
---
M src-db/database/sourcedata/AD_FIELD.xml
M src-test/src/org/openbravo/test/referencedinventory/ReferencedInventoryTestSuite.java
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryProcessor.java
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
---
|
|
|
(0102991)
|
hgbot
|
2018-03-05 17:09
|
|
Repository: erp/devel/pi
Changeset: 9179fe0a1c2c85006cb657db954a21b0f276ff20
Author: Víctor Martínez Romanos <victor.martinez <at> openbravo.com>
Date: Mon Mar 05 13:54:53 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/9179fe0a1c2c85006cb657db954a21b0f276ff20 [^]
Fixed issue 37827: [ri] [cr] Added unique constraint.
This will create a false positive in PI
---
M src-db/database/model/tables/M_REFINVENTORY_TYPE.xml
---
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
(0103032)
|
hgbot
|
2018-03-06 16:55
|
|
Repository: erp/devel/pi
Changeset: f71fb18c45380f8be585dde53b509558945e1981
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Tue Mar 06 16:55:15 2018 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/f71fb18c45380f8be585dde53b509558945e1981 [^]
Related to Issue 37827: [ri][cr] Changed "x" expression for one that explains
why we compare an id to a literal that is not going to match any Id
---
M src/org/openbravo/materialmgmt/refinventory/ReferencedInventoryUtil.java
---
|
|
|
|
|
|
|
|