Make iDempiere safer using saveEx(trx) instead of save(trx)

Description

When make new madatory column in m_movementline_confirm.
I complete a move inventory with confirm.
=> move inventory document is switch status inprogress success, but new m_movementline_confirm is not create (because madatory is not input)
=> can't complete move inventory document.
root cause is by in MMovementConfirm use cLine.save(move.get_TrxName()) but not process return result.
I think it should switch to cLine.saveex(move.get_TrxName())
I also search and see many code use method po.save(move.get_TrxName()) without process return result
we should switch "all po.save(move.get_TrxName()) without process return result" to cLine.saveex(move.get_TrxName())
or only switch when encounter issue

Environment

None

Attachments

1

Activity

Show:
Fixed

Details

Assignee

Reporter

Components

Affects versions

Priority

Created January 12, 2014 at 3:39 AM
Updated April 7, 2014 at 4:08 AM
Resolved March 5, 2014 at 3:09 PM