Merge lp:~unifield-team/unifield-addons/addons-utp-872-create-hook into lp:unifield-addons

Proposed by jftempo
Status: Merged
Merged at revision: 4601
Proposed branch: lp:~unifield-team/unifield-addons/addons-utp-872-create-hook
Merge into: lp:unifield-addons
Diff against target: 22 lines (+6/-3)
1 file modified
purchase/purchase.py (+6/-3)
To merge this branch: bzr merge lp:~unifield-team/unifield-addons/addons-utp-872-create-hook
Reviewer Review Type Date Requested Status
UniField Dev Team Pending
Review via email: mp+199549@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'purchase/purchase.py'
--- purchase/purchase.py 2012-08-14 21:31:14 +0000
+++ purchase/purchase.py 2013-12-18 20:24:01 +0000
@@ -314,12 +314,15 @@
314 314
315 return True315 return True
316316
317 #TODO: implement messages system317 #UTP-872: extracted this method for a particularly check with sync
318 def _hook_check_po_no_line(self, po, context):
319 if not po.order_line:
320 raise osv.except_osv(_('Error !'), _('You can not confirm purchase order without Purchase Order Lines.'))
321
318 def wkf_confirm_order(self, cr, uid, ids, context=None):322 def wkf_confirm_order(self, cr, uid, ids, context=None):
319 todo = []323 todo = []
320 for po in self.browse(cr, uid, ids, context=context):324 for po in self.browse(cr, uid, ids, context=context):
321 if not po.order_line:325 self._hook_check_po_no_line(po, context)
322 raise osv.except_osv(_('Error !'),_('You can not confirm purchase order without Purchase Order Lines.'))
323 for line in po.order_line:326 for line in po.order_line:
324 if line.state=='draft':327 if line.state=='draft':
325 todo.append(line.id)328 todo.append(line.id)

Subscribers

People subscribed via source and target branches

to all changes: