WPGov/wp-spid-italia

Redirect su admin in WP multisite

pkirk opened this issue · 2 comments

pkirk commented

Marco,
su

wp_safe_redirect( apply_filters( 'spid_registration_default_login_redirect', user_admin_url() ) );

usando user_admin_url() in un WP multisite si viene rimandati al sito principale. Non conviene usare admin_url() ?

Grazie.

Ciao @pkirk,
andrebbe verificato il comportamento nel caso di ruoli "subscriber" e "author", non vorrei che admin_url() porti a errori di mancate autorizzazioni.

Questa la logica di self_admin_url utile per capire le varie casistiche:

if ( is_network_admin() ) {
$url = network_admin_url( $path, $scheme );
} elseif ( is_user_admin() ) {
$url = user_admin_url( $path, $scheme );
} else {
$url = admin_url( $path, $scheme );
}

Credo comunque che la tua interpretazione sia quella corretta, se intanto vuoi fare una pull request, tempo di verificare e se tutto ok la integriamo!

A presto,
Marco

pkirk commented

@milesimarco il ruolo più basso, il subscriber, ha comunque accesso all'admin, anche se poi lì dentro non troverà granché da fare. Caso diverso sarebbe se il WP ha un ruolo dedicato, come immagino possa capitare, su installazioni particolari dove bisogna dare accesso agli utenti SPID per fargli fare qualcosa o consultare dei dati.