Shouldn't use unicode() when exception used in msgs
Bug #1380806 reported by
James Carey
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Low
|
Ivan Kolodyazhny | ||
OpenStack Compute (nova) |
Fix Released
|
Undecided
|
James Carey |
Bug Description
There are cases (identified by mriedem) where an exception is used as replacement text and is coerced using unicode():
http://
reason=_("Driver Error: %s") % unicode(e))
http://
LOG.
doing this can interfere with translation by causing things to be prematurely translated.
Need to scan for and correct any occurrences. Also need to look at adding/updating a hacking check.
Changed in nova: | |
assignee: | nobody → James Carey (jecarey) |
Changed in nova: | |
status: | New → Confirmed |
Changed in nova: | |
milestone: | none → kilo-1 |
status: | Fix Committed → Fix Released |
Changed in cinder: | |
assignee: | nobody → lvdongbing (dbcocle) |
Changed in cinder: | |
status: | New → Confirmed |
importance: | Undecided → Low |
Changed in cinder: | |
milestone: | none → kilo-rc1 |
Changed in cinder: | |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | kilo-1 → 2015.1.0 |
Changed in cinder: | |
milestone: | kilo-rc1 → 2015.1.0 |
To post a comment you must log in.
Is there a way to have a hacking check for this?