Odoo Addons (MOVED TO GITHUB)

Freight tax not calculating automatically on invoice

Bug #816138 reported by Kevin McMenamin
10
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
Odoo Addons (MOVED TO GITHUB) 6.1

Bug Description

Freight added as a result of the delivery method calculation (either at sales order entry or despatch) is not having tax added automatically - refer document enclosed

Tags: maintenance

Related branches

lp:~openerp-dev/openobject-addons/trunk-bug-816138-uco
Mustufa Rangwala (Open ERP) (community): Needs Fixing
qdp (OpenERP): Pending requested
Diff: 34 lines (+14/-10)
1 file modified
delivery/stock.py (+14/-10)
lp:~openerp-dev/openobject-addons/6.0-opw-51090-ado
Amit Dodiya (OpenERP) (community): Needs Resubmitting
Jean-Christophe VASSORT (community): Needs Fixing
Naresh(OpenERP) (community): Approve
nel: Pending requested
Diff: 32 lines (+12/-10)
1 file modified
delivery/stock.py (+12/-10)
Revision history for this message
Kevin McMenamin (kevin-mcmenamin) wrote :
Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello Kevin McMenamin,

I have tested your scenario as per your specification but I did't face any problem.

I have attached a video for your more reference, Would you please check it and informed us where you faced the

problem.

Thanks and waiting for your reply.

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :
Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
Kevin McMenamin (kevin-mcmenamin) wrote :

here is my video (ignore the first 30 secs or so - my first attempt).

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello,

Thank for your video.

Here I have found two problem.

(1) Tax calculation will not consider freight amount into calculation.

(2) When I create invoice form delivery order, Invoice contains two lines for freight tax which in not correct.

I have attached video for more reference.

Thanks.

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 1 (openerp-dev-addons1)
importance: Undecided → Low
status: Incomplete → Confirmed
assignee: OpenERP R&D Addons Team 1 (openerp-dev-addons1) → nobody
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
Revision history for this message
Kevin McMenamin (kevin-mcmenamin) wrote :

Yes I had picked up the double freight as well - thought this was a feature rather than a bug :)
Good to have that fixed.

Thanks

Bharat Devnani (Open ERP) (bde-openerp)
Changed in openobject-addons:
status: Confirmed → In Progress
Mustufa Rangwala (Open ERP) (mra-tinyerp)
Changed in openobject-addons:
status: In Progress → Confirmed
Ujjvala Collins (uco-openerp)
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Ujjvala Collins (uco-openerp) wrote :

Hello Kevin,

The fix has been committed on lp:~openerp-dev/openobject-addons/trunk-bug-816138-uco with revision 5293 <email address hidden>.

It will be merged into trunk addons soon.

Thanks for reporting,
Ujjvala

Changed in openobject-addons:
status: In Progress → Fix Committed
Mustufa Rangwala (Open ERP) (mra-tinyerp)
Changed in openobject-addons:
milestone: none → 6.1
Revision history for this message
qdp (OpenERP) (qdp) wrote :

fix released in revision 5395

thanks

Changed in openobject-addons:
status: Fix Committed → Fix Released
Rifakat Husen (OpenERP) (rha-openerp)
tags: added: maintenance
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.