report_sxw: wrong pageCount and pageNumber when attachments are not kept
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Odoo Server (MOVED TO GITHUB) |
Fix Committed
|
Low
|
OpenERP's Framework R&D | |
| 5.0 |
Invalid
|
Undecided
|
Unassigned |
Bug Description
affects openobject-server
Version Affected:
OpenERP Server 5.0 (stable) - lasted revision
for
Steps to reproduce the problem:
-. I will use the invoice report for example, but it is not specific to
this one.
0. Add <pageCount/> in corporate header
1. Add <pageNumberReset/> just before the </story> tag in invoice.rml
report.
2. Find a record with lot of invoice line (invoice must be draft,
otherwise report_sxw with force report per record and for all group)
3. Now select 5 invoices (but 1st invoice should be the record choosen
at step "2", other invoices must be draft too)
2. Use "Print" button, and choose report "Invoices"
In the the document you will get, the page count will be identical on
all pages, which in not right. It should specific to the document printed.
See patch attached, perhaps it will be better explaining :)
PS: this only affects reports where PDFs are NOT kept as attachments in objects.
--
Xavier ALT
AJM Technologies S.A.
22, rue Goethe / L-1637 Luxembourg
Related branches
- OpenERP Core Team: Pending requested 2011-11-16
- Anup(SerpentCS): Pending requested 2011-11-16
- Xavier ALT: Pending requested 2011-11-16
-
Diff: 100 lines (+34/-8)1 file modifiedbin/report/render/rml2pdf/trml2pdf.py (+34/-8)
- Naresh(OpenERP) (community): Needs Fixing on 2011-10-13
- Raphael Collet (OpenERP) (community): Needs Fixing on 2011-10-13
- Vo Minh Thu: Pending requested 2011-06-10
-
Diff: 63 lines (+19/-14)2 files modifiedopenerp/report/render/rml2pdf/trml2pdf.py (+1/-1)
openerp/report/report_sxw.py (+18/-13)
Xavier ALT (xal-openerp) wrote : report_sxw: pageCount and pageNumberReset not handled correctly with grouped printing | #0 |
Changed in openobject-server: | |
assignee: | nobody → gpa(Open ERP) (gpa-openerp) |
Changed in openobject-server: | |
status: | Incomplete → Confirmed |
Numérigraphe (numerigraphe) wrote : | #2 |
I can confirm that putting <pageNumberReset/> just before the </story> resets the page count in a wrong way. I've just tried this on a custom report of ours.
The patch attached by Xavier ALT (AJM) seems to fix this problem.
To illustrate the misbehavior, Xavier was suggesting you print a report with at least 1 invoice spanning several pages, and watch the pagecount go wrong.
Lionel
Changed in openobject-server: | |
milestone: | none → 5.0.8 |
Hello Lionel and Xavier,
I have printed 3 invoices worth 5 pages.
The Pages were like:
-- page 1-- -- page 2-- -- page 1-- -- page 1-- -- page 2--
--Invoice1- ---Invoice1-- --Invoice2-
It is working pretty normal.
Kindly make the problem clear that you want Paging Object-wse or documnet wise?
i.e. it should go like as I mentioned or as like page 1 2 3 4 5 ?
Thanks.
Changed in openobject-server: | |
assignee: | gpa(Open ERP) (gpa-openerp) → nobody |
Numérigraphe (numerigraphe) wrote : | #4 |
Jay, I just tried with the standard v5.0.7 and it works indeed. What I don't understand yet is why it doesn't work on my developpment branch, where i've merged everything up to 5.0.7...
Numérigraphe (numerigraphe) wrote : | #5 |
Oh sorry, I just clicked on.
First, about the page numbering, you will notice that the behavior is not the same on invoices and sale orders - that's why I added a <pageNumberReset/> in the first place.
Invoices are numbered like you said, but orders are numbered as :
-- page 1/5-- -- page 2/5-- -- page 3/5-- -- page 4/5-- -- page 5/5--
--order1- ---order1-- --order2-
Maybe it's due to the fact that order PDF's are not kept as attachements? - I didn't check this yet.
Now to the main issue: Jay you're right, it works fine as long as you *don't* "Add <pageNumberReset/> just before the </story> tag", which is what Xavier and I did. If ever you do, then the page count will be wrong.
So basically I think the <pageNumberReset/> tag is broken (which may not be such a big deal after all).
Lionel
Numérigraphe (numerigraphe) wrote : | #6 |
Sorry - by "which may not be such a big deal after all" I meant: if we can get the orders to have page numbers like the invoices.
Well Yes Lionel,
I need to figure out why the orders have no similar behaviour?
Thanks.
Hello Guys,
Sale Order Report behave differently , it's due to the fact that order PDF's are not kept as attachements.
It reloads the reporting process object-wise.
Thanks.
Numérigraphe (numerigraphe) wrote : Re: [Bug 516966] Re: report_sxw: pageCount and pageNumberReset not handled correctly with grouped printing | #9 |
Jay (Open ERP) a écrit :
> Hello Guys,
>
> Sale Order Report behave differently , it's due to the fact that order
> PDF's are not kept as attachements.
>
> It reloads the reporting process object-wise.
>
> Thanks.
>
>
Yes, I just tried and I confirm that if you keep attachments, the page
numbering is right.
Lionel.
summary: |
- report_sxw: pageCount and pageNumberReset not handled correctly with - grouped printing + report_sxw: wrong pageCount and pageNumberReset when attachments are not + kept |
description: | updated |
Numérigraphe (numerigraphe) wrote : Re: report_sxw: wrong pageCount and pageNumberReset when attachments are not kept | #10 |
My humble opinion about this issue is:
- </story> should implicitly do a page number reset: it will give a uniform behavior not matter whether attachements are kept or not, and it will solve Bug #546918
- <pageNumberReset> should be fixed, so that we can use it in special cases when we really want to reset the page number in the middle of a story. For example, if I want to print a letter along with each invoice, I want both to based on the same object and be numbered 1/1.
Lionel.
Changed in openobject-server: | |
status: | Confirmed → In Progress |
assignee: | nobody → Anup (Open ERP) (ach-openerp) |
Changed in openobject-server: | |
milestone: | 5.0.8 → 5.0.9 |
Changed in openobject-server: | |
milestone: | 5.0.9 → 5.0.10 |
duplicate of itself.
Changed in openobject-server: | |
milestone: | 5.0.10 → 5.0.11 |
Numérigraphe (numerigraphe) wrote : | #12 |
What do you mean with "duplicate of itself" ?
Lionel
Just To avoid the listing of bug under the same milestone.
Thanks.
summary: |
- report_sxw: wrong pageCount and pageNumberReset when attachments are not - kept + report_sxw: wrong pageCount and pageNumber when attachments are not kept |
Numérigraphe (numerigraphe) wrote : | #14 |
I made an attempt at fixing this, and push my changes to lp:~numerigraphe/openobject-server/5.0-bug516966-wrong-pageCount-pageNumber.
Xavier ALT (AJM) could you please test this branch and comment on the merge proposal?
Lionel
Numérigraphe (numerigraphe) wrote : | #15 |
My fix is not correct yet, I'll give it another try and let you know.
Lionel
Numérigraphe (numerigraphe) wrote : | #16 |
I think it's better now. I'll submit the merge proposal again.
Lionel
Changed in openobject-server: | |
status: | In Progress → Fix Committed |
Hello,
This issue has now been fixed in the development trunk of OpenERP.
Thanks.
Changed in openobject-server: | |
assignee: | Anup (OpenERP) (ach-openerp) → nobody |
milestone: | 5.0.11 → 6.0-rc2 |
status: | Fix Committed → Fix Released |
Xavier ALT (xal-openerp) wrote : | #18 |
Hello,
I'm re-opening this bug as confirmed because on latest trunk the <pageCount/> is always 0 (ZERO). (using "external" header and with <pageCount/> in coporate header).
Actually I've check the trunk code, and a didn't see any merge of the work of Numerigraphe.
Can someone confirm this bug still exist?
Changed in openobject-server: | |
status: | Fix Released → Confirmed |
Xavier ALT (xal-openerp) wrote : | #19 |
Re,
On standard installation <pageCount/> in not present in the "external" coporate header so bug not visible. If we add <pageCount/> inside it will always be 0 whatever the real number of pages.
See patch attached for a "one line" fix :)
Changed in openobject-server: | |
assignee: | nobody → OpenERP's Framework R&D (openerp-dev-framework) |
importance: | Undecided → Low |
josepato (josepato) wrote : | #20 |
Xavier's patch use _pageCount instead of _pageNumber works just fine...
I just check the openerp trunk and this bug is still un fixed..
Changed in openobject-server: | |
status: | Confirmed → Incomplete |
Vo Minh Thu (thu) wrote : | #21 |
josepato,
Why did you change from 'confirmed' to 'incomplete'?
'confirmed' means that this bug report is still in the loop.
'incomplete' means we are waiting for more information from the original person who did post the bug report.
Actually here, we are working on it, and we forgot to put it to 'in progress'.
Please don't change bug reports meta information such as its status unwisely, thanks.
Changed in openobject-server: | |
milestone: | 6.0-rc2 → none |
status: | Incomplete → In Progress |
Hello,
Thanks for reporting.
It has been fixed in lp:~openerp-dev/openobject-server/trunk-bug-516966-ysa.
Revision No:- 3426
Revision ID: <email address hidden> and It will be merged soon
in trunk server.
Thanks,
Changed in openobject-server: | |
status: | In Progress → Fix Committed |
Yan-PCSol (yannick-soldati) wrote : | #23 |
"It will be merged soon"
Does soon means 6 monthes? Are you kidding?
Fix it now! Thanks
I placed a patch for the version revison
lp:~openerp/openobject-server/6.1/
revision 4158
Hello Xavier ALT,
Can you explain me in more in detail ?
Because i cannot understand in following statement:
2. Find a record with lot of invoice line (invoice must be draft,
otherwise report_sxw with force report per record and for all group)
3. Now select 5 invoices (but 1st invoice should be the record choosen
at step "2", other invoices must be draft too).
Can you attach the report pdf ?
Thanks.