Skip to content

Commit

Permalink
fix nre - Inumedia#109
Browse files Browse the repository at this point in the history
  • Loading branch information
koderhari committed Nov 17, 2017
1 parent 9f7b35e commit 99890d5
Showing 1 changed file with 6 additions and 11 deletions.
17 changes: 6 additions & 11 deletions SlackAPI/Request.cs
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ internal void GotResponse(IAsyncResult result)
if (we.Response == null)
{
var defaultResponse = CreateDefaultResponseForError(we);
callback?.Invoke((K)defaultResponse);
callback?.Invoke(defaultResponse);
return;
}

Expand All @@ -91,14 +91,14 @@ internal void GotResponse(IAsyncResult result)
catch (Exception e)
{
var defaultResponse = CreateDefaultResponseForError(e);
callback?.Invoke((K)defaultResponse);
callback?.Invoke(defaultResponse);
return;
}

K responseObj;
if (response == null)
{
responseObj = (K)CreateDefaultResponseForError(new Exception("Empty response"));
responseObj = CreateDefaultResponseForError(new Exception("Empty response"));
callback?.Invoke(responseObj);
return;
}
Expand All @@ -114,24 +114,19 @@ internal void GotResponse(IAsyncResult result)
}
catch (Exception e)
{
responseObj = (K)CreateDefaultResponseForError(e);
responseObj = CreateDefaultResponseForError(e);
}

callback?.Invoke(responseObj);
}

private Response CreateDefaultResponseForError(Exception e)
private K CreateDefaultResponseForError(Exception e)
{
var defaultResponse = new DefaultResponse();
var defaultResponse = (K)Activator.CreateInstance<K>();
defaultResponse.ok = false;
defaultResponse.error = e.ToString();
return defaultResponse;
}

class DefaultResponse: Response
{

}
}

[AttributeUsage(AttributeTargets.Class, Inherited=false)]
Expand Down

0 comments on commit 99890d5

Please # to comment.