[14.0][IMP] sale_order_revision: carry Invoice links to new revision Sales Orders#2930
[14.0][IMP] sale_order_revision: carry Invoice links to new revision Sales Orders#2930
Conversation
If the user created a default afvorite filter on the Sales Order list, it will override the default filter set by the Revisons button. So, the Sale Order menu Action should not be reused for this.
dreispt
force-pushed
the
14-dr-sale_order_revision2
branch
6 times, most recently
from
1f7dfeb to
dc8b751
Compare
dreispt
force-pushed
the
14-dr-sale_order_revision2
branch
from
dc8b751 to
a67186f
Compare
dreispt
force-pushed
the
14-dr-sale_order_revision2
branch
from
a67186f to
6d0aeba
Compare
| def action_cancel_create_revision(self): | ||
| for sale in self: | ||
| sale.action_cancel() | ||
| action = sale.create_revision() | ||
| if len(self) == 1: | ||
| return action | ||
| return {} |
There was a problem hiding this comment.
| def action_cancel_create_revision(self): | |
| for sale in self: | |
| sale.action_cancel() | |
| action = sale.create_revision() | |
| if len(self) == 1: | |
| return action | |
| return {} | |
| def action_cancel_create_revision(self): | |
| self.ensure_one() | |
| sale.action_cancel() | |
| return sale.create_revision() |
If you can only call from the form view for a record, I think it could be simplified like this.
|
@pedrobaeza I understand. I wish I didn't have to do revisions os confirmed SOs, but Portuguese Tax Authority requires SOs to be locked once sent to customers. I'm trying to make abiding to this less painful for PT companies. |
dreispt
changed the title
That sound a bit weird, Anyway, sent to customers don't mean confirmed. You have the |
The forbidding changes is not the issue, that's not hard to enforce. |
|
OK, but the flow in that case is to cancel the sales order, and then issue a new revision. What you can't do is to try to link things that are generated when you confirm the sales order to the new revision. That's something out of scope of any module IMO. |
|
Additional work was on on this feature, implementing it in a separate module. |
4 participants
Includes: