On Inventory Move, Adding Warehouse and Warehouse To field on header

Description

For better user experience when working with multiple warehouse, It is good idea to select from and to warehouse on header and restrict locator at line level.

Expected design should be

  1. On header ad M_Warehouse_ID and M_WarehouseTo_ID column and field as Non Mandatory

  2. Locator field filter locators based on M_Warehouse_ID and Locator To field filter based on M_WarehouseTo_ID

  3. If either of warehouse null on header, respective locater on line shows Locator from all warehouse

  4. Write call out to set M_WarehouseTo_ID to be M_Warehouse_ID when M_WarehouseTo_ID is null. (Making default same warehouse)

 

xref mattermost design discussion → https://mattermost.idempiere.org/idempiere/pl/ie9z7mnnpbbit8pn47qydmjmfe

Environment

None

Attachments

1

Activity

Carlos Ruiz 
November 15, 2021 at 12:44 PM

Hi

I would suggest to add the following patch:

 

During my tests I found these:

  • warehouse and warehouse_to must default to null - this is what people is get used to and it works just fine, defaulting the warehouse changes drastically how this window is managed

  • why is the warehouse field not updatable? I would say these fields are read-only when there are movement lines, otherwise, it can be changed

  • warehouse must be filtered to match the org - when is read-only it cannot be changed and the document is stuck

  • I thin the callout is unnecessary - it is assuming that the movements will mostly happen within the same warehouse - which can be true in some implementations, but different in others

The patch solves these issues and also generates the X/I model classes.

Regards,

Carlos Ruiz

Deepak Pansheriya 
November 12, 2021 at 12:17 PM

Create PR

Fixed

Details

Assignee

Reporter

Components

Fix versions

Affects versions

Priority

Created July 16, 2021 at 10:48 AM
Updated February 1, 2022 at 12:53 PM
Resolved December 23, 2021 at 9:29 AM