Issue #3445: Clean PublicError conversions - freeipa

freeipa

#3445 Clean PublicError conversions

Opened 11 years ago by mkosek. Modified 7 years ago

There are many places where we convert PublicError to string with unicode(PublicError()). However, this may cause conversion when Gettext is used:

# ipa group-add-member group --external "AD\Administrator"
      member group: AD\Domain Admins: Insufficient access:
Gettext('communication
with trusted domains is allowed for Trusts administrator group members only',
domain='ipa', localedir=None)

We need to use PublicError().strerror to avoid it. Just by running git grep "unicode(e)" I saw many other potential places where this could happen.

mkosek commented 10 years ago

As Ana is moving out of the team, I am moving her unresolved tickets to free-to-take pool.

Metadata Update from @mkosek:
- Issue assigned to someone
- Issue set to the milestone: Future Releases

7 years ago

Metadata

Assignee

someone

Tags

None

Blocking

None

Depending on

None

Priority

normal

Milestone

Future Releases

None

affects_doc

None

source

None

knownissue

None

type

refactoring

blockedby

None

test_case

None

component

IPA

blocking

None

on_review

keywords

None

test_coverage

None

reviewer

None

external_tracker

None

rhbz

todo

tester

None

changelog

None

design

None

freeipa

Source Code

#3445 Clean PublicError conversions Opened 11 years ago by mkosek. Modified 7 years ago

Close issue as:

Metadata

#3445 Clean PublicError conversions

Opened 11 years ago by mkosek. Modified 7 years ago