IDEMPIERE-5528 IDEMPIERE-5560 Cannot save more than one order line when an _ID field is not displayed

[CarlosRuiz-globalqss]

https://idempiere.atlassian.net/browse/IDEMPIERE-5560#icft=IDEMPIERE-5560

https://idempiere.atlassian.net/browse/IDEMPIERE-5528

Pull Request Checklist

• My code follows the code guidelines of this project
• My code follows the best practices of this project
• I have performed a self-review of my own code
• My code is easy to understand and review.
• I have checked my code and corrected any misspellings
• In hard-to-understand areas, I have commented my code.
• My changes generate no new warnings

Tests

• I have tested the direct scenario that my code is solving
• I checked all collaterals that can be affected by my changes, and tested other potential affected scenarios
• New and existing unit tests pass locally with my changes
• I have added unit tests that prove my fix is effective or that my feature works

[CarlosRuiz-globalqss]

Hi @hengsin - this is a unit test case to illustrate the failure caused by IDEMPIERE-5528 that was described in IDEMPIERE-5560

I needed to add a call to Login.loadPreferences to enable the grid window for sales order to behave as expected filling the default sales order doc type with a preference.

Also, I created an internal DataStatusListener in line 215.
NOTE: I think it would be good to create a dedicated generic class to open a window, set a DataStatusListener, etc.

Also, for some reason the change on AD_Field is not taken unless I commit it, I guess somewhere in the GridWindow/GridTab definition is reading out of transaction.

[hengsin]

looks good