Merge lp:~openerp-dev/openobject-addons/6.1-opw-577781-skh into lp:openobject-addons/6.1

Proposed by Somesh Khare
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/6.1-opw-577781-skh
Merge into: lp:openobject-addons/6.1
Diff against target: 35 lines (+7/-1)
1 file modified
account/account_move_line.py (+7/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/6.1-opw-577781-skh
Reviewer Review Type Date Requested Status
Naresh(OpenERP) Pending
Review via email: mp+119522@code.launchpad.net

Description of the change

Hello Sir,

[FIX] account_move_line: journal view on the journal form did not work.

Steps to Reproduce:
1. Create a Journal View "Test View" and select field as "move,ref,state,account" from menu "Accounting/Configuration/Financial Accounting/Journals/Journal Views".
2. Select any journal from "Accounting/Configuration/Financial Accounting/Journals/Journals" (eg: Sales Journal).
3. Now select the Display mode as "Test View" on the Journal form view.
4. From the action window select Journal Item.

Your selected display mode's (Journal Views) fields are not visible on the journal items from here.

Expected result:
It should display only the fields those are defined into the Journal View "Test View".

This branch fixes this issue, Kindly review the branch and please share your views on it.

Thanks,
Somesh Khare

To post a comment you must log in.
6947. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

6948. By Somesh Khare

[FIX]account_move_line: journal view on the journal form did not work (Case: Ref 577781)

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

6948. By Somesh Khare

[FIX]account_move_line: journal view on the journal form did not work (Case: Ref 577781)

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-07-27 11:43:03 +0000
+++ account/account_move_line.py 2012-08-16 11:36:20 +0000
@@ -20,6 +20,7 @@
20##############################################################################20##############################################################################
2121
22import time22import time
23import sys
23from datetime import datetime24from datetime import datetime
24from operator import itemgetter25from operator import itemgetter
2526
@@ -1016,8 +1017,11 @@
1016 fields = {}1017 fields = {}
1017 flds = []1018 flds = []
1018 title = _("Accounting Entries") #self.view_header_get(cr, uid, view_id, view_type, context)1019 title = _("Accounting Entries") #self.view_header_get(cr, uid, view_id, view_type, context)
10191020
1020 ids = journal_pool.search(cr, uid, [])1021 ids = journal_pool.search(cr, uid, [])
1022 journal_id = context.get('active_model')=='account.journal' and context.get('active_id') or context.get('search_default_journal_id') or False
1023 if journal_id:
1024 ids = journal_pool.search(cr, uid, [('id','=',journal_id)])
1021 journals = journal_pool.browse(cr, uid, ids, context=context)1025 journals = journal_pool.browse(cr, uid, ids, context=context)
1022 all_journal = [None]1026 all_journal = [None]
1023 common_fields = {}1027 common_fields = {}
@@ -1035,8 +1039,10 @@
1035 common_fields[field.field] = common_fields[field.field] + 11039 common_fields[field.field] = common_fields[field.field] + 1
1036 fld.append(('period_id', 3))1040 fld.append(('period_id', 3))
1037 fld.append(('journal_id', 10))1041 fld.append(('journal_id', 10))
1042 fld.append(('state',sys.maxint))
1038 flds.append('period_id')1043 flds.append('period_id')
1039 flds.append('journal_id')1044 flds.append('journal_id')
1045 flds.append('state')
1040 fields['period_id'] = all_journal1046 fields['period_id'] = all_journal
1041 fields['journal_id'] = all_journal1047 fields['journal_id'] = all_journal
1042 fld = sorted(fld, key=itemgetter(1))1048 fld = sorted(fld, key=itemgetter(1))