Skip to content
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

[10.0][mis_builder] Cannot delete mis report template if it contains sub kpi's #281

Closed
JordiBForgeFlow opened this issue Mar 16, 2017 · 2 comments
Labels

Comments

@JordiBForgeFlow
Copy link
Sponsor Member

Testing #275
If I define a mis report template with sub-kpi's, when I try to delete the whole report template I get the error:

`The operation cannot be completed, probably due to the following:

  • deletion: you may be trying to delete a record while other records still reference it
  • creation/update: a mandatory field is not correctly set

[object with reference: mis.report.subkpi - mis.report.subkpi]
`

Perhaps a delete=on_cascade should be added

cc @sbidoul

@sbidoul
Copy link
Member

sbidoul commented Mar 16, 2017

Should be fixed in 1d480d2. Thanks for reporting!

@sbidoul sbidoul closed this as completed Mar 16, 2017
@sbidoul sbidoul reopened this Mar 16, 2017
sbidoul added a commit to acsone/account-financial-reporting that referenced this issue Mar 16, 2017
I don't remember why we did the manual unlink back then in Sorrento.

fixes OCA#281
@sbidoul sbidoul added the bug label Mar 16, 2017
@JordiBForgeFlow
Copy link
Sponsor Member Author

Works correctly now. Thanks!

sbidoul added a commit to acsone/account-financial-reporting that referenced this issue May 6, 2017
I don't remember why we did the manual unlink back then in Sorrento.

fixes OCA#281
sbidoul added a commit to acsone/account-financial-reporting that referenced this issue Jun 16, 2017
I don't remember why we did the manual unlink back then in Sorrento.

fixes OCA#281
sbidoul added a commit to acsone/account-financial-reporting that referenced this issue Aug 30, 2017
I don't remember why we did the manual unlink back then in Sorrento.

fixes OCA#281
sbidoul added a commit to acsone/account-financial-reporting that referenced this issue Sep 26, 2017
I don't remember why we did the manual unlink back then in Sorrento.

fixes OCA#281
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants