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.

rob


 > (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.

rob

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to