Merge lp:~unifield-team/unifield-addons/uf_1204_cancel_availability_button_2 into lp:unifield-addons

Proposed by jftempo
Status: Merged
Merged at revision: 4548
Proposed branch: lp:~unifield-team/unifield-addons/uf_1204_cancel_availability_button_2
Merge into: lp:unifield-addons
Diff against target: 161 lines (+20/-28)
7 files modified
account/report/account_balance_sheet.py (+1/-1)
account/report/account_general_journal.py (+1/-1)
stock/partner_view.xml (+1/-0)
stock/stock.py (+11/-17)
stock/wizard/stock_partial_move.py (+4/-1)
stock/wizard/stock_partial_picking.py (+1/-7)
stock/wizard/stock_partial_picking_view.xml (+1/-1)
To merge this branch: bzr merge lp:~unifield-team/unifield-addons/uf_1204_cancel_availability_button_2
Reviewer Review Type Date Requested Status
UniField Dev Team Pending
Review via email: mp+115752@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 'account/report/account_balance_sheet.py'
--- account/report/account_balance_sheet.py 2011-06-29 05:29:28 +0000
+++ account/report/account_balance_sheet.py 2012-07-19 14:40:35 +0000
@@ -215,4 +215,4 @@
215 'addons/account/report/account_balance_sheet.rml',parser=report_balancesheet_horizontal,215 'addons/account/report/account_balance_sheet.rml',parser=report_balancesheet_horizontal,
216 header='internal')216 header='internal')
217217
218# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
219\ No newline at end of file218\ No newline at end of file
219# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
220220
=== modified file 'account/report/account_general_journal.py'
--- account/report/account_general_journal.py 2011-01-14 00:11:01 +0000
+++ account/report/account_general_journal.py 2012-07-19 14:40:35 +0000
@@ -158,4 +158,4 @@
158158
159report_sxw.report_sxw('report.account.general.journal', 'account.journal.period', 'addons/account/report/general_journal.rml', parser=journal_print, header='internal')159report_sxw.report_sxw('report.account.general.journal', 'account.journal.period', 'addons/account/report/general_journal.rml', parser=journal_print, header='internal')
160160
161# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
162\ No newline at end of file161\ No newline at end of file
162# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:
163163
=== modified file 'stock/partner_view.xml'
--- stock/partner_view.xml 2012-06-14 12:30:17 +0000
+++ stock/partner_view.xml 2012-07-19 14:40:35 +0000
@@ -9,6 +9,7 @@
9 <field name="name">res.partner.stock.property.form.inherit</field>9 <field name="name">res.partner.stock.property.form.inherit</field>
10 <field name="model">res.partner</field>10 <field name="model">res.partner</field>
11 <field name="type">form</field>11 <field name="type">form</field>
12 <field name="priority" eval="15"/>
12 <field name="inherit_id" ref="base.view_partner_form"/>13 <field name="inherit_id" ref="base.view_partner_form"/>
13 <field name="priority" eval="15" />14 <field name="priority" eval="15" />
14 <field name="arch" type="xml">15 <field name="arch" type="xml">
1516
=== modified file 'stock/stock.py'
--- stock/stock.py 2012-07-06 12:59:15 +0000
+++ stock/stock.py 2012-07-19 14:40:35 +0000
@@ -363,7 +363,7 @@
363 def _product_virtual_get(self, cr, uid, id, product_ids=False, context=None, states=['done']):363 def _product_virtual_get(self, cr, uid, id, product_ids=False, context=None, states=['done']):
364 return self._product_all_get(cr, uid, id, product_ids, context, ['confirmed', 'waiting', 'assigned', 'done'])364 return self._product_all_get(cr, uid, id, product_ids, context, ['confirmed', 'waiting', 'assigned', 'done'])
365365
366 def _product_reserve(self, cr, uid, ids, product_id, product_qty, context=None, lock=False):366 def _product_reserve(self, cr, uid, ids, product_id, product_qty, location_dest_id, context=None, lock=False):
367 """367 """
368 Attempt to find a quantity ``product_qty`` (in the product's default uom or the uom passed in ``context``) of product ``product_id``368 Attempt to find a quantity ``product_qty`` (in the product's default uom or the uom passed in ``context``) of product ``product_id``
369 in locations with id ``ids`` and their child locations. If ``lock`` is True, the stock.move lines369 in locations with id ``ids`` and their child locations. If ``lock`` is True, the stock.move lines
@@ -390,7 +390,9 @@
390 if context is None:390 if context is None:
391 context = {}391 context = {}
392 pool_uom = self.pool.get('product.uom')392 pool_uom = self.pool.get('product.uom')
393 for id in self.search(cr, uid, [('location_id', 'child_of', ids)]):393 for id in self.search(cr, uid, [('location_id', 'child_of', ids)], order="parent_left"):
394 if id == location_dest_id:
395 continue
394 if lock:396 if lock:
395 try:397 try:
396 # Must lock with a separate select query because FOR UPDATE can't be used with398 # Must lock with a separate select query because FOR UPDATE can't be used with
@@ -456,13 +458,9 @@
456 if amount > 0:458 if amount > 0:
457 if amount > min(total, product_qty):459 if amount > min(total, product_qty):
458 amount = min(product_qty, total)460 amount = min(product_qty, total)
459 result.append((amount, id))461
460 product_qty -= amount462 return self._hook_proct_reserve(cr,uid,product_qty,result,amount, id, ids)
461 total -= amount463
462 if product_qty <= 0.0:
463 return result
464 if total <= 0.0:
465 continue
466 return False464 return False
467465
468stock_location()466stock_location()
@@ -2029,6 +2027,7 @@
2029 @return: No. of moves done2027 @return: No. of moves done
2030 """2028 """
2031 done = []2029 done = []
2030 notdone = []
2032 count = 02031 count = 0
2033 pickings = {}2032 pickings = {}
2034 if context is None:2033 if context is None:
@@ -2041,7 +2040,7 @@
2041 continue2040 continue
2042 if move.state in ('confirmed', 'waiting'):2041 if move.state in ('confirmed', 'waiting'):
2043 # Important: we must pass lock=True to _product_reserve() to avoid race conditions and double reservations2042 # Important: we must pass lock=True to _product_reserve() to avoid race conditions and double reservations
2044 res = self.pool.get('stock.location')._product_reserve(cr, uid, [move.location_id.id], move.product_id.id, move.product_qty, {'uom': move.product_uom.id}, lock=True)2043 res = self.pool.get('stock.location')._product_reserve(cr, uid, [move.location_id.id], move.product_id.id, move.product_qty, move.location_dest_id.id ,{'uom': move.product_uom.id}, lock=True)
2045 if res:2044 if res:
2046 #_product_available_test depends on the next status for correct functioning2045 #_product_available_test depends on the next status for correct functioning
2047 #the test does not work correctly if the same product occurs multiple times2046 #the test does not work correctly if the same product occurs multiple times
@@ -2053,13 +2052,8 @@
2053 r = res.pop(0)2052 r = res.pop(0)
2054 cr.execute('update stock_move set location_id=%s, product_qty=%s, product_uos_qty=%s where id=%s', (r[1], r[0], r[0] * move.product_id.uos_coeff, move.id))2053 cr.execute('update stock_move set location_id=%s, product_qty=%s, product_uos_qty=%s where id=%s', (r[1], r[0], r[0] * move.product_id.uos_coeff, move.id))
20552054
2056 while res:2055 done, notdone = self._hook_copy_stock_move(cr, uid, res, move, done, notdone)
2057 r = res.pop(0)2056 count = self._hook_write_state_stock_move(cr, uid, done, notdone, count)
2058 move_id = self.copy(cr, uid, move.id, {'product_qty': r[0],'product_uos_qty': r[0] * move.product_id.uos_coeff,'location_id': r[1]})
2059 done.append(move_id)
2060 if done:
2061 count += len(done)
2062 self.write(cr, uid, done, {'state': 'assigned'})
20632057
2064 if count:2058 if count:
2065 for pick_id in pickings:2059 for pick_id in pickings:
20662060
=== modified file 'stock/wizard/stock_partial_move.py'
--- stock/wizard/stock_partial_move.py 2012-05-15 16:56:55 +0000
+++ stock/wizard/stock_partial_move.py 2012-07-19 14:40:35 +0000
@@ -83,6 +83,9 @@
83 return "product_moves_in"83 return "product_moves_in"
84 else:84 else:
85 return "product_moves_out"85 return "product_moves_out"
86
87 def _hook_move_state(self):
88 return ('done', 'cancel')
86 89
87 def view_init(self, cr, uid, fields_list, context=None):90 def view_init(self, cr, uid, fields_list, context=None):
88 res = super(stock_partial_move, self).view_init(cr, uid, fields_list, context=context)91 res = super(stock_partial_move, self).view_init(cr, uid, fields_list, context=context)
@@ -91,7 +94,7 @@
91 if context is None:94 if context is None:
92 context = {}95 context = {}
93 for move in move_obj.browse(cr, uid, context.get('active_ids', []), context=context):96 for move in move_obj.browse(cr, uid, context.get('active_ids', []), context=context):
94 if move.state in ('done', 'cancel'):97 if move.state in self._hook_move_state():
95 raise osv.except_osv(_('Invalid action !'), _('Cannot deliver products which are already delivered !'))98 raise osv.except_osv(_('Invalid action !'), _('Cannot deliver products which are already delivered !'))
96 99
97 return res100 return res
98101
=== modified file 'stock/wizard/stock_partial_picking.py'
--- stock/wizard/stock_partial_picking.py 2012-05-15 16:56:55 +0000
+++ stock/wizard/stock_partial_picking.py 2012-07-19 14:40:35 +0000
@@ -64,13 +64,7 @@
64 if not picking_ids:64 if not picking_ids:
65 return res65 return res
6666
67 result = []67 result = self._hook_default_get(cr, uid, picking_ids, context)
68 for pick in pick_obj.browse(cr, uid, picking_ids, context=context):
69 pick_type = self.get_picking_type(cr, uid, pick, context=context)
70 for m in pick.move_lines:
71 if m.state in ('done', 'cancel'):
72 continue
73 result.append(self.__create_partial_picking_memory(m, pick_type))
7468
75 if 'product_moves_in' in fields:69 if 'product_moves_in' in fields:
76 res.update({'product_moves_in': result})70 res.update({'product_moves_in': result})
7771
=== modified file 'stock/wizard/stock_partial_picking_view.xml'
--- stock/wizard/stock_partial_picking_view.xml 2011-01-14 00:11:01 +0000
+++ stock/wizard/stock_partial_picking_view.xml 2012-07-19 14:40:35 +0000
@@ -8,6 +8,6 @@
8 <field name="view_mode">form</field>8 <field name="view_mode">form</field>
9 <field name="target">new</field>9 <field name="target">new</field>
10 </record>10 </record>
11 </data>
1112
12 </data>
13</openerp>13</openerp>

Subscribers

People subscribed via source and target branches

to all changes: