Odoo Server (MOVED TO GITHUB)

missing import in osv.fields

Bug #523971 reported by Pieter J. Kersten (EduSense BV)
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Server (MOVED TO GITHUB)
Fix Released
Low
Unassigned
Odoo Server (MOVED TO GITHUB) 5.0.8
Nominated for 5.0 by Numérigraphe

Bug Description

The osv/fields.py is missing the 'from tools.translate import _' line. Errors now trigger a common traceback instead of a controlled one.

Related branches

lp:openobject-server/5.0
Revision history for this message
Numérigraphe (numerigraphe) wrote :

bin/tools/convert.py doesn't have it either, is it needed?
Lionel

Revision history for this message
Stephane Wirtel (OpenERP) (stephane-openerp) wrote :

Trunk or Stable ?

Revision history for this message
Pieter J. Kersten (EduSense BV) (pieterj) wrote : Re: [Bug 523971] Re: missing import in osv.fields

It does when you use _('whatever') constructs. Once used, it will
trigger an exception based on the missing _ function, not the original
error it tried to signal.
If bin/tools/convert.py uses _('whatever'), then this bug extends to
that file as well.

Op vrijdag 19-02-2010 om 08:18 uur [tijdzone +0000], schreef
Numérigraphe:

> bin/tools/convert.py doesn't have it either, is it needed?
> Lionel
>

Stephane Wirtel (OpenERP) (stephane-openerp)
Changed in openobject-server:
milestone: none → 5.2
Revision history for this message
Numérigraphe (numerigraphe) wrote :

If the import is missing in 5.0 it has to be fixed too. Stephane can you please check ?

Jay Vora (Serpent Consulting Services) (jayvora)
Changed in openobject-server:
status: New → Confirmed
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Tools/convert.py doesn't use any such constructs.
It has been fixed by revision 1980 <email address hidden>.

Many thanks.

Changed in openobject-server:
milestone: 5.2 → 5.0.8
status: Confirmed → Fix Released
Stephane Wirtel (OpenERP) (stephane-openerp)
Changed in openobject-server:
importance: Undecided → Low
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.