Account Payment Report

[Vincent Renaville@camptocamp]

Hello,

Many of our customers complain about the layout of the Account Payment Report, so we have made a patch for it ;)
We have adapt the layout, add Total, enlarge the bank account column,add amount in company currency ,etc ...
I think it's suitable for all OpenERP customers, so can you review it and integrate it in the next stable version ?

You will find a patch attached to this bug

thanks for your precious help,

Vincent

[Ravindra Mekhiya(OpenERP) (rme-openerp)]

Hello Vincent,

I have applied your patch.
The layout seems quite impressive.
All the required fields have been listed.

However, the header line isn't continuous.
I have attached the screen-shot after applying your patch.

Would you please see through it and send a new patch?

Thanks.

[Vincent Renaville@camptocamp (vrenaville-c2c)]

Hi,

Thanks for your comment, I have just applied my patch on the last stable.
I have no problem with layout (see pdf attached),
and I have also noticed that amount and amount currency are inverted (you can also view it in pdf attached).
Can you test it again, it's really change.

Thanks for your precious help,

Vincent

[Vincent Renaville@camptocamp (vrenaville-c2c)]

Hi,

Do you need some more information ?
We are closed to a solution, one more review an we will have a greate payment report ;) .

Vincent

[Ravindra Mekhiya(OpenERP) (rme-openerp)]

Hi Vincent,

The patch can be approved,thanks.

Is it possible to send us an SXW file ?

Thanks.

[Vincent Renaville@camptocamp (vrenaville-c2c)]

Hi,

Thanks for your reply, I have no SXW file.
I have corrected the RML file directly.

Sorry for that,

Vincent

[Vincent Renaville@camptocamp (vrenaville-c2c)]

Hi,

Could you please applied my patch ;) , it's really important to have the SXW ?

Thanks for your help,

Vincent

[Ravindra Mekhiya(OpenERP) (rme-openerp)]

Hello Vincent,

Yes, the file is important because user may want to change the SXW file via OpenOffice Report Designer tool.
There is no other reason we ask you to attach the file.

I hope you understand the importance.

Thanks for the care.

[Vincent Renaville@camptocamp (vrenaville-c2c)]

Hello,

I have take time to give you a order.sxw file, you will find it attached

After review, you can commit the patch :)

Thanks for your precious help

Vincent

[Ravindra Mekhiya(OpenERP) (rme-openerp)]

Hello Vincent,

After conversion of your .SXW file into .RML file, it really doesn't match the RML sent by you earlier.

Plus, the output also differs.

Here I have attached both reports...
  1) First .RML file's report
  2) .SXW file converted into .RML file's report

Thanks for your efforts,
we appreciate.

[Ravindra Mekhiya(OpenERP) (rme-openerp)]

.SXW file's .RML file Account Payment Report.

[Vincent Renaville@camptocamp (vrenaville-c2c)]

Hello,

I'm too impatient to close this bug :

You will find in attachment the an new version of the patch and the new SXW version

Thanks for you help,

Vincent Renaville

[Stephane Wirtel (OpenERP) (stephane-openerp)]

Vincent,

There is a patch for the 6.0 about this report ?

Thanks

[Stephane Wirtel (OpenERP) (stephane-openerp)]

In fact, I have to discuss with the team, because in 5.0, you know we only accept the bugfixes and not the improvements.

In this case, it's between the improvement and the bugfix, because the current layout seems to be wrong.

Stéphane

[Vincent Renaville@camptocamp (vrenaville-c2c)]

Hi Stephane,

I don't have a patch for the 6.0, I can't currently spend more time on this bug.
but I think It will not be a big deal for your team to make the patch ;) .

You are right, it's between a bugfix and an improvement.
But if you can't use a report because you don't have the right information, for me it's a bug even if the report doesn't crash at runtime ;)

Vincent

[Stephane Wirtel (OpenERP) (stephane-openerp)]

I agree with you !

[Jay Vora (Serpent Consulting Services) (jayvora)]

It has been corrected by revision 2816 <email address hidden> authored to Vincent.
Thanks for all your efforts and time.

[Jay Vora (Serpent Consulting Services) (jayvora)]

We will fix/improve it on trunk too.
Thanks.

[Ravindra Mekhiya(OpenERP) (rme-openerp)]

Hello,

It has been fixed into trunk.

Thanks.