Since this is checking a bunch of enum values and then returning immediately, it seems that a switch is a better option here.
Additionally, purple_upnp_discover handles both undiscovered and in-progress discovering, so no need to special case those here.

Compile only.