Merge lp:~openerp-dev/openobject-addons/trunk-bug-1159797-chandni into lp:openobject-addons

Proposed by Chandni Machchhar(OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1159797-chandni
Merge into: lp:openobject-addons
Diff against target: 12 lines (+1/-1)
1 file modified
account/report/account_general_ledger_landscape.rml (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1159797-chandni
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+156554@code.launchpad.net

Description of the change

Hello,

 I have fixed the problem of Columns overlap in General Ledger report.

Thanks,
Chandni.

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

Hi Chandni,
strip_name with 9 makes the sting a bit shorter where we still use some more space of the column.
strip_name(line['move'],13) should be good enough. I tested it for 7.0, may you test it for trunk
and adapt the fix *if* needed.

Thanks,
Rifakat

Unmerged revisions

8659. By Chandni Machchhar(OpenERP)

[FIX] Columns overlap in General ledger pdf file.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'account/report/account_general_ledger_landscape.rml'
2--- account/report/account_general_ledger_landscape.rml 2012-06-13 09:06:00 +0000
3+++ account/report/account_general_ledger_landscape.rml 2013-04-02 13:23:29 +0000
4@@ -532,7 +532,7 @@
5 <para style="terp_default_7">[[ line['lref'] and strip_name(line['lref'],17) ]]</para>
6 </td>
7 <td>
8- <para style="terp_default_7">[[ line['move'] ]]</para>
9+ <para style="terp_default_7">[[ strip_name(line['move'],9) ]]</para>
10 </td>
11 <td>
12 <para style="terp_default_7">[[ strip_name(line['lname'],22) ]]</para>

Subscribers

People subscribed via source and target branches

to all changes: