Been trying to get the following working, but keep getting a unexpected elseif. Does someone know what I do wrong?

/**
* WordPress function for redirecting users on login based on user role
*/
function my_login_redirect( $url, $request, $user ){
if ( $user && is_object( $user ) && is_a( $user, 'WP_User' ) ) {
    if( $user->has_cap( 'administrator' ) ) {
        $url = admin_url();
    elseif( $users->has_cap( 'Subscriber' ) ) {
        $redirect="https://mycustomlinkforsubscribers.com";
    }
    else {
        $redirect="https://thenormallink";
    }
}
return $url;
}

add_filter('login_redirect', 'my_login_redirect', 10, 3 );