Merge lp:~openerp-dev/openobject-addons/6.1-opw-574998-rha into lp:openobject-addons/6.1

Proposed by Rifakat Husen (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-574998-rha
Merge into: lp:openobject-addons/6.1
Diff against target: 20 lines (+0/-8)
1 file modified
account/account_move_line.py (+0/-8)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-574998-rha
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+106781@code.launchpad.net

Description of the change

Hello,

I have corrected return type of _invoice() for field invoice on journal items. It was returning
tuple and generating error for Journal Items list view, opened from wizard 'Manual Reconciliation'. Also removed unneccessary code.

Steps to reproduce it,
1. Create bank statement and encode manual statement line
2. Open wizard 'Manual Reconciliation' from Accounting > Periodical Processing > Reconciliation
3. Edit that line, for field 'Invoice', we can only click on 'Open'
4. Receive error while clikcing on Open

Please review this fix.

Regards,
Rifakat Haradwala

To post a comment you must log in.
Revision history for this message
Naresh(OpenERP) (nch-openerp) wrote :

Hello,

This bug was qualified as Not Relevant on Trunk (means that architecture on trunk has changed and so this bug has no meaning anymore). If this Merge Proposal could not be merged in v6.1 at the release of v7.0, it will be closed.

Thanks,
Naresh Soni

Unmerged revisions

6807. By Rifakat Husen (OpenERP)

[FIX] account: it generates error while opening record for invoice field, wrong return value of function

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'account/account_move_line.py'
--- account/account_move_line.py 2012-03-06 16:10:40 +0000
+++ account/account_move_line.py 2012-05-22 11:04:31 +0000
@@ -383,16 +383,8 @@
383 'WHERE l.move_id = i.move_id ' \383 'WHERE l.move_id = i.move_id ' \
384 'AND l.id IN %s',384 'AND l.id IN %s',
385 (tuple(ids),))385 (tuple(ids),))
386 invoice_ids = []
387 for line_id, invoice_id in cursor.fetchall():386 for line_id, invoice_id in cursor.fetchall():
388 res[line_id] = invoice_id387 res[line_id] = invoice_id
389 invoice_ids.append(invoice_id)
390 invoice_names = {False: ''}
391 for invoice_id, name in invoice_obj.name_get(cursor, user, invoice_ids, context=context):
392 invoice_names[invoice_id] = name
393 for line_id in res.keys():
394 invoice_id = res[line_id]
395 res[line_id] = (invoice_id, invoice_names[invoice_id])
396 return res388 return res
397389
398 def name_get(self, cr, uid, ids, context=None):390 def name_get(self, cr, uid, ids, context=None):