[FIX] Exception when scheduling a phonecall from another

[antespi]

This PR fix a bug when scheduling a phonecall from another. Steps to reproduce:

1. Install addon 'crm_phonecall_summary_predefined'
2. Go to Sales > Logged phones and clean filters
3. Click at Phone icon (icons on the right in each call row) to 'Schedule Other Call'
4. Press 'Schedule Call' button

An IntegrityError is raised:

Integrity 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: summary_id - summary.id]

This PR adds summary_id field to crm.phonecall2phonecall model and view in order to create a new call with that field assigned and avoid integrity error.

This PR depends on these ones:

• [8.0][IMP][crm] Add hook for preparing scheduled phonecall values and allow inheritance OCB#482
• [8.0][IMP][crm] Add hook for preparing scheduled phonecall values and allow inheritance odoo/odoo#11791

[pedrobaeza]

You should do res['summary_id'] = self.env.context.get('summary_id', res.get('summary_id', False)).

[antespi]

Until OCA/OCB#482 or odoo/odoo#11791 merged, travis and runbot will be red

[pedrobaeza]

This can't be merged on OCB, so the only chance is Odoo.

[antespi]

Well, I hope Odoo HQ takes my PR into account. If not, we can see another approach to solve this bug.

[coveralls]

Coverage decreased (-1.4%) to 77.581% when pulling 83671f2 on Antiun:fix-schedule_phonecall into a45c3ad on OCA:8.0.

[pedrobaeza]

Can you add a test for the new behavior?

[antespi]

Odoo is ignoring us and OCB fix is not an option, so I have to change strategy. Then when I have a new PR for this (without no core changes) I'll add test for it.

[rafaelbn]

Hi @antespi @pedrobaeza , none of the dependencies of this PR are going to be merged. So Please I think it's better to close this. Anyway we are migrating this problem to v9, so better we must find a way in v9.

[antespi]

I'm closing this because requirements are not going to be merged.