Merge lp://staging/~kangol/openobject-server/6.0-context-report into lp://staging/openobject-server/6.0

Hello Christophe,

if we talk about passing the missing context the merge proposal seems Good ! :)

But at the end you removed the 'internal_header' from the localcontext, doing so will not print the page number formatting on right top corner (page 1 out of 10) which is supposed to be printed when internal header is selected. The condition is needed here

if self.localcontext and not self.localcontext.get('internal_header',False):
    self.localcontext.pop('internal_header', None)

Thanks,

Naresh,

If there is *no* internal_header in the dict, remove it from the dict? This doesn't make any sense.

So we need to know when the internal_header is present and what is the complete behavior. The previous code never entered the if body (or it would have broke), so it seems the internal_header was always present in the dict with a non-False value. We need to know if we can assume the internal_header is always there.

Hello Thu,

Yes, the localcontext dict will always have the key 'internal_header' present with value True / False
depending on the report's parser 'header' argument. if you specify header = 'internal' or 'internal landscape' the key will have value as True else False. If its true You will see a different header format on the final output.
for eg:
report_sxw.report_sxw('report.account.aged_trial_balance', 'res.partner',
        'addons/account/report/account_aged_partner_balance.rml',parser=aged_trial_report, header="internal landscape")

The previous code do enters *IF* body if you print something there you will see the output(but only when you shut your server down dont know why:(

Thanks