[6.0 trunk] account : period selection not working for BS and PL legal reports

Bug #711886 reported by Bogdan Stanciu
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
OpenERP R&D Addons Team 3

Bug Description

Hi there,

When trying to select some period for BS and PL legal reports the result contains all the periods in the db (all data).

thank you,
Bogdan

Related branches

Revision history for this message
Bogdan Stanciu (bstanciu) wrote :

A bit more detail:

1. Open the wizard for BS or PL reports from Accounting/Reporting/Legal Reports
2. Apply filter 'Periods'
3. Select whatever periods you want to test
4. Print report

Result: filter not applied!!
However, the report contains the filter in the header (Period from-to) with the correct periods.

(date filter is applied - they have a simillar mechanism, but...)

Amit Parik (amit-parik)
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
importance: Undecided → Low
status: New → Confirmed
Revision history for this message
Bogdan Stanciu (bstanciu) wrote : Re: [Bug 711886] Re: [6.0 trunk] account : period selection not working for BS and PL legal reports

On 04. 02. 11 10:13, Amit Parik (OpenERP) wrote:
> ** Changed in: openobject-addons
> Importance: Undecided => Low
>
> ** Changed in: openobject-addons
> Status: New => Confirmed
>
> ** Changed in: openobject-addons
> Assignee: (unassigned) => OpenERP R&D Addons Team 3 (openerp-dev-addons3)
>
any news on this one? thank you...

Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) wrote :

yes we are working on it, in other reports we build query (from account move line => query_get method) and for BS and PL we are not making query we just used periods in context.

Thanks,
mra

Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Bogdan Stanciu (bstanciu) wrote :

On 07. 02. 11 12:05, Mustufa Rangwala (Open ERP) wrote:
> yes we are working on it, in other reports we build query (from account
> move line => query_get method) and for BS and PL we are not making query
> we just used periods in context.
>
> Thanks,
> mra
>
yes i noticed that the reports and wizards are built differently,
unfortunately! it's a pity for a OOL and project to recreate the wheel
each time, and worse, to recreate it differently :-)

thank you for your work! looking forward for the solution!
bogdan

Revision history for this message
Ashvin Rathod (OpenERP) (ara-tinyerp) wrote :

Hello Bogdan,

I have fixed the problem in lp:~openerp-commiter/openobject-addons/trunk-dev-addons3-ara branch, It will be merged soon with trunk-dev-addons3.

Revision ID: <email address hidden>
Revision No: 4696

Thank you,
ara

Changed in openobject-addons:
status: In Progress → Fix Committed
Changed in openobject-addons:
status: Fix Committed → Fix Released
Revision history for this message
Bogdan Stanciu (bstanciu) wrote :

On 07. 02. 11 13:12, Ashvin Rathod (OpenERP) wrote:
> Hello Bogdan,
>
> I have fixed the problem in lp:~openerp-commiter/openobject-addons
> /trunk-dev-addons3-ara branch, It will be merged soon with trunk-dev-
> addons3.
>
> Revision ID: <email address hidden>
> Revision No: 4696
>
>
> Thank you,
> ara
>
> ** Changed in: openobject-addons
> Status: In Progress => Fix Committed
>
Please don't take bad, but did you test it?

Because this is what I already tried, and it didn't work...

ctx['period_from'] = data['form'].get('period_from', False)
ctx['period_to'] = data['form'].get('period_to', False)
I have the feeling that the filter does not influence the browse..

I just tried your change from the merge, for the BS, and I have
identical figures for the following filters:

01/20xx - 12/20xx
01-06
01-01

if on your side it works, we should look for an issue maybe in the Swiss
localisation...

thank you!
Bogdan

Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) wrote :

Hello,

It is working fine at our end.

Thanks,
mra

Revision history for this message
Bogdan Stanciu (bstanciu) wrote :

On 07. 02. 11 14:18, Mustufa Rangwala (Open ERP) wrote:
> Hello,
>
> It is working fine at our end.
>
> Thanks,
> mra
>
hi,

thank you!

it works (almost). I filed another bug for a peculiar case, bug #714787.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.