Skip to content

[9.0][FIX] contract_show_invoice: Error if create invoice with view context - #82

Merged
pedrobaeza merged 2 commits into
OCA:9.0from
Tecnativa:9.0-FIX-contract_show_invoice
Aug 25, 2017
Merged

[9.0][FIX] contract_show_invoice: Error if create invoice with view context#82
pedrobaeza merged 2 commits into
OCA:9.0from
Tecnativa:9.0-FIX-contract_show_invoice

Conversation

@carlosdauden

Copy link
Copy Markdown
Contributor

@pedrobaeza

Copy link
Copy Markdown
Member

Why this change? It was OK before


<record id="act_analytic_invoices" model="ir.actions.act_window">
<field name="context">{'search_default_analytic_account_ids':
[active_id], 'default_analytic_account_ids': active_id}</field>

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@pedrobaeza this context is useless because analytic_account_ids is a computed field and the search is done by domain

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

OK, but don't change the rest...

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The search is needed anyway

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Why is needed?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

For filtering the invoices with that analytic contract. It's the way to do it.

@carlosdauden

carlosdauden commented Aug 3, 2017

Copy link
Copy Markdown
Contributor Author

This module would not be better in analytic branch?

@pedrobaeza

Copy link
Copy Markdown
Member

Well, yes. And with other name even...

@carlosdauden
carlosdauden force-pushed the 9.0-FIX-contract_show_invoice branch from e098b4a to bd64837 Compare August 3, 2017 16:38
@carlosdauden
carlosdauden force-pushed the 9.0-FIX-contract_show_invoice branch from bd64837 to 7af3b76 Compare August 4, 2017 07:22
@dreispt

dreispt commented Aug 10, 2017

Copy link
Copy Markdown
Member

This module would not be better in analytic branch?

That's best done in a next version migration (v10).
You can put that on the README's roadmap if you like.

@rafaelbn rafaelbn added this to the 9.0 milestone Aug 24, 2017
@rafaelbn
rafaelbn self-requested a review August 24, 2017 17:05
@rafaelbn

Copy link
Copy Markdown
Member

hi @pedrobaeza , please take a look again. Thanks!

@rafaelbn rafaelbn self-assigned this Aug 24, 2017
@pedrobaeza
pedrobaeza merged commit 7620353 into OCA:9.0 Aug 25, 2017
@pedrobaeza
pedrobaeza deleted the 9.0-FIX-contract_show_invoice branch August 25, 2017 14:45
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