-
-
Notifications
You must be signed in to change notification settings - Fork 63
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
8.0 remove Purchase Requisition creation wizard #73
Conversation
e684a12
to
f249c3d
Compare
👍 |
procurement_method = 'procurement' contained | ||
in the Line | ||
""" | ||
source_lines = self.source_ids | ||
if not source_lines: | ||
sources = self.mapped('source_ids') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you use self.mapped('source_ids')
instead of self.source_ids
? I would understand if self
could be a recordset with many records but here it is decorated with @api.one
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In fact it should be a @api.multi as we want only one po_requisition at the end.
f1ab38e
to
01c984e
Compare
@guewen Thanks for the review. Fixed. |
'please create one.')) | ||
pricelist = self.requisition_id.pricelist_id or None | ||
source_lines._action_create_po_requisition(pricelist=pricelist) | ||
_('No sourcing line with a Tender procurement ' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still a Tender?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still as it is the term in procurement_method Selection field
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Weird but okay!
👍 |
1 similar comment
👍 |
01c984e
to
6033124
Compare
The action should be done from LRL or done with a multi selection of LRS
…ding one click for nothing.
6033124
to
e658bc2
Compare
👍 |
8.0 remove Purchase Requisition creation wizard
view: Logistics Requisition Line Form
action: Button Create Purchase Requisition on Form
Aim: Create a Single Purchase requisition for all Sources in Tender
view: LR Source List
action: More -> Create Purchase Requisition on Form
Aim: Select precisely which Source you want in same Purchase Requisition