New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
IDEMPIERE-5893 Completed invoices are being updated in MOrder.afterSave #2075
IDEMPIERE-5893 Completed invoices are being updated in MOrder.afterSave #2075
Conversation
- test for date and ID
COLUMNNAME_PaymentRule, | ||
COLUMNNAME_C_PaymentTerm_ID, | ||
COLUMNNAME_DateAcct}; | ||
boolean propagateColChanged = false; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what would be the options available if an implementation wish to change the auto propagate columns ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
more columns can be propagated with an EventHandler
less columns or a different document status would not be possible, maybe an EventHandler on MInvoice to capture the change and reset it back to the old value?
any suggestion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps as a new sysconfig entry ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps as a new sysconfig entry ?
ready, changed the approach to use SysConfig ORDER_COLUMNS_TO_COPY_TO_NOT_COMPLETED_INVOICES
… as suggested by Heng Sin
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
…ve (#2075) * IDEMPIERE-5893 Completed invoices are being updated in MOrder.afterSave * - refactor - test for date and ID * - implement SysConfig ORDER_COLUMNS_TO_COPY_TO_NOT_COMPLETED_INVOICES as suggested by Heng Sin
…ve (idempiere#2075) * IDEMPIERE-5893 Completed invoices are being updated in MOrder.afterSave * - refactor - test for date and ID * - implement SysConfig ORDER_COLUMNS_TO_COPY_TO_NOT_COMPLETED_INVOICES as suggested by Heng Sin
https://idempiere.atlassian.net/browse/IDEMPIERE-5893
Pull Request Checklist
Tests