CardknoxResponse parsing issue

[davidfried]

When charging a card twice with the same amount in the same minute cardknox rejects the charge with a 'Duplicate charge' error message

for some reason, the response has in it the xAvsResultCode key but its empty.

now, since the parsing code is not handled for exceptions this line raises an exception

AvsResultCode = (AvsResponseType)Enum.Parse(typeof(AvsResponseType), _values["xAvsResultCode"]);

Please update the CardknoxResponse constructor to handle exceptions like this

here is my 2 cents

` public CardknoxResponse(NameValueCollection _values)
{
if (_values.AllKeys.Contains("xResult"))
{
try
{
Result = (ResultType)Enum.Parse(typeof(ResultType), _values["xResult"]);
}
catch { }
}

        if (_values.AllKeys.Contains("xError"))
            Error = _values["xError"];

        if (_values.AllKeys.Contains("xErrorCode"))
            ErrorCode = _values["xErrorCode"];

        if (_values.AllKeys.Contains("xStatus"))
            Status = (StatusType)Enum.Parse(typeof(StatusType), _values["xStatus"]);

        if (_values.AllKeys.Contains("xRefNum"))
            RefNum = _values["xRefNum"];

        if (_values.AllKeys.Contains("xAuthCode"))
            AuthCode = _values["xAuthCode"];

        if (_values.AllKeys.Contains("xBatch"))
            Batch = _values["xBatch"];

        if (_values.AllKeys.Contains("xAvsResultCode"))
        {
            try
            {
                AvsResultCode = (AvsResponseType)Enum.Parse(typeof(AvsResponseType), _values["xAvsResultCode"]);
            }
            catch { }
        }

        if (_values.AllKeys.Contains("xAvsResult"))
        {
            try
            {
                AvsResult = HttpUtility.UrlDecode(_values["xAvsResult"]);
            }
            catch { }
        }

        if (_values.AllKeys.Contains("xAuthAmount"))
        {
            try
            {
                AuthAmount = Decimal.Parse(_values["xAuthAmount"]);
            }
            catch { }                
        }
        else AuthAmount = null;

        if (_values.AllKeys.Contains("xMaskedCardNumber"))
            MaskedCardNumber = _values["xMaskedCardNumber"];

        if (_values.AllKeys.Contains("xCardType"))
        {
            try
            {
                CardType = (CardType)Enum.Parse(typeof(CardType), _values["xCardType"]);
            }
            catch { }
        }

        if (_values.AllKeys.Contains("xToken"))
            Token = _values["xToken"];

        if (_values.AllKeys.Contains("xEntryMethod"))
            EntryMethod = _values["xEntryMethod"];

} `

[ahwm]

You make a great point!

[ahwm]

The fix for this is now live in 4.5.3.

Thanks!