Merge pull request #49 from moorereason/earlyreturn

Remove unnecessary else clause
This commit is contained in:
Adnan Hajdarević 2015-11-03 21:14:32 +01:00
commit 8544653787

View file

@ -265,23 +265,23 @@ func (h *Hook) ParseJSONParameters(headers, query, payload *map[string]interface
if err != nil { if err != nil {
return &ParseError{err} return &ParseError{err}
}
var source *map[string]interface{}
switch h.JSONStringParameters[i].Source {
case SourceHeader:
source = headers
case SourcePayload:
source = payload
case SourceQuery:
source = query
}
if source != nil {
ReplaceParameter(h.JSONStringParameters[i].Name, source, newArg)
} else { } else {
var source *map[string]interface{} return &SourceError{h.JSONStringParameters[i]}
switch h.JSONStringParameters[i].Source {
case SourceHeader:
source = headers
case SourcePayload:
source = payload
case SourceQuery:
source = query
}
if source != nil {
ReplaceParameter(h.JSONStringParameters[i].Name, source, newArg)
} else {
return &SourceError{h.JSONStringParameters[i]}
}
} }
} else { } else {
return &ArgumentError{h.JSONStringParameters[i]} return &ArgumentError{h.JSONStringParameters[i]}