On 03/19/2012 03:02 PM, Rob Crittenden wrote:
Petr Viktorin wrote:
On 03/16/2012 10:23 PM, Rob Crittenden wrote:
When using *attr we should return the param.name of in the exception and
when using a cli option we should return param.cli_name. This didn't
work consistently in the framework.

This is a bit of a kludge, catching exceptions and re-raising them, but
it is a less invasive way of doing it.

I added some examples of things to test in the ticket.


> (name, error) = err.strerror.split(':')
> raise errors.ConversionError(name=attr, error=error)

AFAIU this will break when the error message is translated. Why not just
use err.kw['error']?

Because the attribute name needs to get pulled out of it.


No, you're ignoring the attribute name.
Anyway, even the English error messages are 'invalid %(name)r: %(error)s', so "name" part starts with 'invalid ', and the "error" part starts with an extra space.

You've also based this on an old version of my patch 0016, which is not ACKed yet. Maybe it's better to wait until that gets in.


Freeipa-devel mailing list

Reply via email to