Use RedirectDestinationTrait instead of parsing destination from query.

During a patch creation I found that the module use "base:" protocol to get URL, the error is caused by #3319509 patch. Anyway I think its a good improvement to use RedirectDestinationTrait, so I keep patch below. Also there is a chance that #3319509 would be fixed in this way.

If base url exists, its added one more time for Url set to form redirect. The easiest way to fix its to return destination to query bag, setting empty 'base_url' in Url object params looks tricky for me.

The last submitted patch, 5: tmgmt-use_redirect_destination_trait-3436477-4.patch, failed testing. View results →

With this changes to use RedirectResponseSubscriber for redirects when destination is set it solves parent issue too.

Also I assume the module has issue with redirects when destination is set on sub-folder installation, looks like test doesn't cover the case. I'll check later and rename a task if needed.