Odoo Addons (MOVED TO GITHUB)

[6.1] Error trying to create periods using Web Client

Bug #960872 reported by tozm
76
This bug affects 14 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Committed
Medium
OpenERP Publisher's Warranty Team
Odoo Addons (MOVED TO GITHUB) 6.1

Bug Description

Getting a problem with trying to add a periods to a newly created financial year in web client.

I was able to create a new period successfully in GTK client however.

If I then return to web client and call up the period under "Config/Financial Accounting/Periods/Fiscal Years" I get this error:

Unknown field state in domain [["state","in",["done"]]]

I can successfully view/edit in Config/Financial Accounting/Periods/Period view though.

All of the above works fine in GTK client.

I did this on a fresh install, no sample data, and no default chart etc installed.

Related branches

lp://staging/~serpent-consulting-services/openobject-addons/6.1-960872-SerpentCS
Rifakat Husen (OpenERP) (community): Approve
Diff: 36 lines (+4/-3)
1 file modified
account/account_view.xml (+4/-3)
Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello tozm,

I have tested your Issue in web-client, But I did not face any problem. It's working fine at my end. I have attached the video for your reference. Would you please check it and notify us where you have faced the problem.

Thanks and waiting for your reply!

Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
tozm (tozm) wrote :

Thanks for that. I can see a an important difference between what you see and what I see.

When I select the financial years form, in order to create one, I don't have the field displaying for "Company". (Is this because I only have one company configured?) This may be a red herring.

But I copied your steps and that worked fine if I create the periods using the create periods button. But as soon as I select one of those periods by clicking on its link *on that same form*, then I get the error referred to above:

Unknown field state in domain [["state","in",["done"]]]

This is the one step that you didn't do in your video... so you haven't yet actually performed the step yet that triggers my problem.

Note that viewing/editing the periods in the periods form works fine.

Thanks for your prompt feedback - look forward to an update when you have a chance to try this again.

Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello,

I have faced the problem of "Unknown field state in domain [["state","in",["done"]]]" when create periods line from the Fiscal year form. So I have attached the video for the reference.

Thanks!

Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :
affects: openobject-addons → openerp-web
Amit Bhavsar (Open ERP) (amb-openerp)
Changed in openerp-web:
status: Incomplete → Confirmed
assignee: nobody → OpenERP R&D Web Team (openerp-dev-web)
importance: Undecided → Low
Revision history for this message
tozm (tozm) wrote :

Thanks. Glad you could reproduce it.

jack lee (liyupu)
Changed in openerp-web:
status: Confirmed → Incomplete
status: Incomplete → Invalid
status: Invalid → Incomplete
Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello,

As per the comment#4 video, It's produced with following revision number.
web : 2293
server : 4130
addons : 6715
That's why I am again confirming this issue.

Thanks!

Changed in openerp-web:
status: Incomplete → Confirmed
Revision history for this message
Willian Paixao (willianpaixao-deactivatedaccount) wrote :

This bug happens to me too.
The same way that Amit.

Nicolas Vanhoren (OpenERP) (niv-openerp)
Changed in openerp-web:
importance: Low → Medium
Serpent Consulting Services (serpent-consulting-services)
affects: openerp-web → openobject-addons
Changed in openobject-addons:
assignee: OpenERP R&D Web Team (openerp-dev-web) → nobody
Revision history for this message
Bipin Rathod(SerpentCS) (brathod-serpentcs) wrote :

Hello Guys,

As far as the web is concerned, its necessary to have the state field in the view when the other fields depend on it.

See the .py file and make sure the other fields like date_* depend for their readonly status on state.

Hence, the attached branch will do the rest.

Though its in 6.1, I understand that it should go fixed.

Regards,
Bipin Rathod,
Serpent Consulting Services.

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Thanks for the fix Bipin, a good catch.

I am setting the bug as fixed committed, but I let OpenERP Team decide who to assign to(either SerpentCS or team).

I am sure this bug will give a sigh of relief to the affected ones.

Thanks,
Serpent Consulting Services.

Changed in openobject-addons:
status: Confirmed → Fix Committed
milestone: none → 6.1
Bipin Rathod(SerpentCS) (brathod-serpentcs)
Changed in openobject-addons:
status: Fix Committed → Confirmed
status: Confirmed → Fix Committed
Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello,

I have faced the same problem in 6.1 as well as trunk also. So it's will be fix for trunk and backport from the trunk for 6.1.

Thanks!

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

If this is gonna be in 6.1, its best thing to do Amit, good decision.

Thanks,
Serpent Consulting Services.

Changed in openobject-addons:
assignee: OpenERP R&D Addons Team 3 (openerp-dev-addons3) → OpenERP Publisher's Warranty Team (openerp-opw)
Stefan Rijnhart (Opener) (stefan-opener)
no longer affects: therp-backports
Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Dears,

This was a blocking point for many in 6.1. This merge was approved too.

Hope to get this noticed.

Thanks.

Revision history for this message
Muhammad Arsalan Ali Khan (m-arsalan-khan) wrote :

I am effected by this bug.. Is the fix for this bug merged ??

Revision history for this message
Serpent Consulting Services (serpent-consulting-services) wrote :

Muhammad,

The fix is attached already.

If you have an OPW, you may get this quickly merged.

Thanks.

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

Duplicates of this bug

Subscribing...

Other bug subscribers

Remote bug watches

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