Odoo Web (MOVED TO GITHUB)

domain not effective in taxes field on invoices

Bug #912814 reported by qdp (OpenERP)
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Web (MOVED TO GITHUB)
Fix Released
Medium
OpenERP R&D Web Team

Bug Description

*install accounting
*generate the generic chart of account
*create a customer invoice: on the invoice line, click to add a tax.

Expected result: you should see only the tax with field type_tax_use in ('sale', 'all'). (i.e. ITAX)
Observed result: all taxes are displayed. (OTAX)

It filters correctly in GTK

Tags: o2m

Related branches

lp:~openerp-dev/openerp-web/trunk-bug-912814-vda
Nicolas Vanhoren (OpenERP) (community): Disapprove
Diff: 12 lines (+1/-1)
1 file modified
addons/web/static/src/js/view_form.js (+1/-1)
lp:openerp-web
Xavier (Open ERP) (xmo-deactivatedaccount)
Changed in openerp-web:
importance: Undecided → Medium
status: New → Confirmed
Xavier (Open ERP) (xmo-deactivatedaccount)
summary: - domain not effective in taxes field on invoices
+ [O2M] domain not effective in taxes field on invoices
Xavier (Open ERP) (xmo-deactivatedaccount)
tags: added: o2m
Xavier (Open ERP) (xmo-deactivatedaccount)
Changed in openerp-web:
assignee: nobody → OpenERP R&D Web Team (openerp-dev-web)
Bhumi Thakkar (Open ERP) (bth-openerp)
Changed in openerp-web:
status: Confirmed → In Progress
Bhumi Thakkar (Open ERP) (bth-openerp)
Changed in openerp-web:
status: In Progress → Confirmed
Vaibhav Darji (vaibhav-openerp)
Changed in openerp-web:
status: Confirmed → In Progress
Revision history for this message
Vaibhav Darji (vaibhav-openerp) wrote : Re: [O2M] domain not effective in taxes field on invoices

Fix Commited in
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-912814-vda
with revID: <email address hidden>

Changed in openerp-web:
status: In Progress → Fix Committed
Nicolas Vanhoren (OpenERP) (niv-openerp)
summary: - [O2M] domain not effective in taxes field on invoices
+ domain not effective in taxes field on invoices
Revision history for this message
Nicolas Vanhoren (OpenERP) (niv-openerp) wrote :

The problem is that the context is not followed correctly. But I think this should be solved in the search view, because the context is correctly passed to it but it does not seem to use it.

Changed in openerp-web:
status: Fix Committed → Confirmed
Revision history for this message
Nicolas Vanhoren (OpenERP) (niv-openerp) wrote :

Ok, after I talked with xmo I think we should modify the SelectCreatePopup class like vda proposed.

Changed in openerp-web:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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