webadmin: Removed recursion for readability's sake

The attach and detach commands were being run in parallel, so that
when both terminate the dialog is closed and the source tab is
refreshed. However, this was implemented in a bizarre recursion which
wasn't trivial to follow. Hopefully it is more readable now.

Change-Id: I4cb3238dbc82cbf8f41d9bf93889649f8d021f54
Signed-off-by: Lior Vernia <lvernia@redhat.com>