[19.0][ADD] mis_builder_cash_flow: cash-flow forecast source for MIS Builder#7
Open
dnplkndll wants to merge 2 commits into
Open
[19.0][ADD] mis_builder_cash_flow: cash-flow forecast source for MIS Builder#7dnplkndll wants to merge 2 commits into
dnplkndll wants to merge 2 commits into
Conversation
dnplkndll
force-pushed
the
19.0-add-mis_builder_cash_flow
branch
6 times, most recently
from
0646cc0 to
b90d79b
Compare
- mis.cash_flow SQL-view source (unreconciled move-line residuals UNION forecast lines) for MIS Builder, rebuilt clean for 19.0. - account.account.user_type_id removed in 19.0 -> source exposes account_type (granular) for the AEP; ALSO exposes a mapped legacy account_internal_type (asset_cash->liquidity, asset_receivable->receivable, liability_payable-> payable, else other) so pre-19.0 cash-flow reports that filter on account_internal_type keep computing unchanged. - target_move now handled by mis_builder core (parent_state); dropped the old period filter override. company_id->company_ids; removed invalid auto_join.
dnplkndll
force-pushed
the
19.0-add-mis_builder_cash_flow
branch
from
b90d79b to
3b51c35
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Clean 19.0 rebuild of
mis_builder_cash_flow(last shipped on OCA/account-financial-reporting@13.0, ADHOC/Tecnativa). Adds themis.cash_flowSQL-view source so MIS Builder reports can forecast cash flow from unreconciled move-line residuals UNIONed with manual forecast lines.Depends on OCA#752 (mis_builder) — merged into 19.0.
Non-mechanical adaptations worth flagging
user_type_id/account_internal_type. The SQL view now joinsaccount_accountand exposesaccount_typeas a real column (with aSelectionmirror on the model so it is groupable/filterable in the AEP's_read_group); the report KPI domains mapliquidity→asset_cash,receivable→asset_receivable,payable→liability_payable.state→parent_stateso mis_builder core's generic posted/all filter (_supports_target_move_filter+_get_target_move_domain, keyed onparent_state) applies automatically. The 13.0mis.report.instance.period._get_additional_move_line_filteroverride that hard-coded("state","=","posted")is therefore dropped.account.account.company_idis gone in 19.0; the company constraint and view domains usecompany_ids.