Skip to content

[14.0][IMP] mis_builder_cash_flow: Multi company#1451

Merged
OCA-git-bot merged 1 commit intoOCA:14.0from
PyTech-SRL:14.0-imp-mis_builder_cash_flow-multi_company
Feb 26, 2026
Merged

[14.0][IMP] mis_builder_cash_flow: Multi company#1451
OCA-git-bot merged 1 commit intoOCA:14.0from
PyTech-SRL:14.0-imp-mis_builder_cash_flow-multi_company

Conversation

@SirPyTech
Copy link
Contributor

Add multi company security rules to forecast lines.

I have used the XMLID of the same rule that has been created in 18.0 during the migration:

<record model="ir.rule" id="mis_cash_flow_forecast_line_company_rule">
<field name="name">mis.cash_flow.forecast_line:Company</field>
<field name="model_id" ref="model_mis_cash_flow_forecast_line" />
<field
name="domain_force"
>['|',('company_id','=',False),('company_id', 'parent_of', company_ids)]</field>
<field name="perm_read" eval="1" />
<field name="perm_create" eval="1" />
<field name="perm_write" eval="1" />
<field name="perm_unlink" eval="1" />
</record>
.

@OCA-git-bot
Copy link
Contributor

Hi @jjscarafia,
some modules you are maintaining are being modified, check this out!

@SirPyTech SirPyTech marked this pull request as ready for review February 25, 2026 13:52
@SirPyTech

This comment was marked as outdated.

@SirPyTech SirPyTech force-pushed the 14.0-imp-mis_builder_cash_flow-multi_company branch from 56eab1f to 43e08aa Compare February 25, 2026 14:59
liebana added a commit to liebana/account-financial-reporting that referenced this pull request Feb 26, 2026
Backport ir.rule records for mis.cash_flow and
mis.cash_flow.forecast_line from the 18.0 migration.

Without these rules, in multi-company environments cash flow
data from all companies is visible to all users.

References:
- 18.0 migration: https://github.com/OCA/account-financial-reporting/blob/18.0/mis_builder_cash_flow/security/mis_cash_flow_security.xml#L34-L44
- 14.0 equivalent: OCA#1451
@pedrobaeza pedrobaeza added this to the 14.0 milestone Feb 26, 2026
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-1451-by-pedrobaeza-bump-patch, awaiting test results.

@pedrobaeza pedrobaeza changed the title [IMP] mis_builder_cash_flow: Multi company [14.0][IMP] mis_builder_cash_flow: Multi company Feb 26, 2026
@OCA-git-bot OCA-git-bot merged commit 7131607 into OCA:14.0 Feb 26, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at aa166a9. Thanks a lot for contributing to OCA. ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants