IMHO a return value set by GET should have priority, this enable third party and site managers to develop better login integrations

also in my notice the old $this->form->getValue('return') can be removed as never valued (i think it's old legacy inheritance, but you should check)