diff --git a/formulaires/editer_auteur.php b/formulaires/editer_auteur.php index 9c039f9..020f503 100644 --- a/formulaires/editer_auteur.php +++ b/formulaires/editer_auteur.php @@ -164,6 +164,7 @@ function formulaires_editer_auteur_verifier_dist( $oblis[] = 'new_login'; } // mais il reste obligatoire si on a rien trouve + $erreurs = []; $erreurs = formulaires_editer_objet_verifier('auteur', $id_auteur, $oblis); if (isset($erreurs['new_login'])) { $erreurs['login'] = $erreurs['new_login']; diff --git a/formulaires/editer_groupe.php b/formulaires/editer_groupe.php index 421f158..6916bf8 100644 --- a/formulaires/editer_groupe.php +++ b/formulaires/editer_groupe.php @@ -176,8 +176,7 @@ function formulaires_editer_groupe_traiter_dist($id_article=0, $redirect=''){ ); $ids_auteurs = sql_allfetsel('id_auteur', 'spip_auteurs_liens', $where); $ids_auteurs = array_column($ids_auteurs, 'id_auteur'); - $Tauteurs = _request('Tauteurs'); - + $Tauteurs = _request('Tauteurs') ?? []; if (count($Tauteurs)) { $associer = array_diff($Tauteurs, $ids_auteurs); $dissocier = array_diff($ids_auteurs, $Tauteurs); @@ -204,7 +203,7 @@ function formulaires_editer_groupe_traiter_dist($id_article=0, $redirect=''){ $ids_mots = sql_allfetsel('M.id_mot AS id_mot', $from, $where); $ids_mots = array_column($ids_mots, 'id_mot'); - $coches = _request('id_groupe_'.$id_groupe); + $coches = _request('id_groupe_'.$id_groupe) ?? []; if (count($coches)) { $associer = array_diff($coches, $ids_mots); $dissocier = array_diff($ids_mots, $coches); diff --git a/formulaires/editer_membre.php b/formulaires/editer_membre.php index 1858db6..2e30a33 100644 --- a/formulaires/editer_membre.php +++ b/formulaires/editer_membre.php @@ -161,6 +161,7 @@ function formulaires_editer_membre_verifier_dist( $oblis[] = 'new_login'; } // mais il reste obligatoire si on a rien trouve + $erreurs = []; $erreurs = formulaires_editer_objet_verifier('auteur', $id_auteur, $oblis); if (isset($erreurs['new_login'])) { $erreurs['login'] = $erreurs['new_login']; diff --git a/formulaires/editer_post.php b/formulaires/editer_post.php index 5d630a8..866f5a1 100644 --- a/formulaires/editer_post.php +++ b/formulaires/editer_post.php @@ -60,7 +60,7 @@ function formulaires_editer_post_charger_dist($id_article=0, $redirect=''){ 'L.objet="article"', 'L.id_gis=G.id_gis' ); - $res = sql_fetsel('G.lat, G.lon', $from, $where); + $res = sql_fetsel('G.lat, G.lon', $from, $where) ?? []; if (count($res)) { $valeurs['lat'] = $res['lat']; $valeurs['lon'] = $res['lon']; @@ -155,7 +155,7 @@ function formulaires_editer_post_traiter_dist($id_article = 0, $redirect = ''){ ); $ids_auteurs = sql_allfetsel('id_auteur', 'spip_auteurs_liens', $where); $ids_auteurs = array_column($ids_auteurs, 'id_auteur'); - $Tauteurs = _request('Tauteurs'); + $Tauteurs = _request('Tauteurs') ?? []; if (count($Tauteurs)) { $associer = array_diff($Tauteurs, $ids_auteurs); $dissocier = array_diff($ids_auteurs, $Tauteurs); @@ -188,8 +188,7 @@ function formulaires_editer_post_traiter_dist($id_article = 0, $redirect = ''){ } } - $coches = _request('id_emplacements'); - + $coches = _request('id_emplacements') ?? []; if (count($coches)){ $associer = array_diff($coches, $T_ids_mots); $dissocier = array_diff($T_ids_mots, $coches); @@ -265,7 +264,8 @@ function formulaires_editer_post_traiter_dist($id_article = 0, $redirect = ''){ $retour = array(); $retour['message_ok'] = "L'article est bien enregistré."; if ($new==0){ - $redirect .= '&id_article=' . $id_article; + //$redirect .= '&id_article=' . $id_article; + $redirect = parametre_url($redirect, 'id_article', $id_article); } $retour['redirect'] = $redirect; diff --git a/formulaires/editer_sujet.php b/formulaires/editer_sujet.php index ea01afa..930615e 100644 --- a/formulaires/editer_sujet.php +++ b/formulaires/editer_sujet.php @@ -107,8 +107,7 @@ function formulaires_editer_sujet_traiter_dist($id_article = 0, $id_rubrique, $r ); $ids_auteurs = sql_allfetsel('id_auteur', 'spip_auteurs_liens', $where); $ids_auteurs = array_column($ids_auteurs, 'id_auteur'); - $Tauteurs = _request('Tauteurs'); - if (count($Tauteurs)) { + $Tauteurs = _request('Tauteurs') ?? []; $associer = array_diff($Tauteurs, $ids_auteurs); $dissocier = array_diff($ids_auteurs, $Tauteurs); diff --git a/formulaires/inscription.php b/formulaires/inscription.php index 71a454e..53d206c 100644 --- a/formulaires/inscription.php +++ b/formulaires/inscription.php @@ -40,7 +40,7 @@ function formulaires_inscription_charger_dist($mode = '', $id = 0, $redirect ='' // Si inscriptions pas autorisees, retourner une chaine d'avertissement function formulaires_inscription_verifier_dist($mode = '', $id = 0, $redirect ='') { - $erreurs = array(); + $erreurs = []; include_spip('inc/config'); set_request("_upgrade_auteur"); // securite diff --git a/paquet.xml b/paquet.xml index 09651bc..0970d71 100644 --- a/paquet.xml +++ b/paquet.xml @@ -3,7 +3,7 @@ categorie="squelette" version="1.0.0" etat="stable" - compatibilite="[3.1.7;3.3.*]" + compatibilite="[4.0.0;4.2.*]" logo="img/balint_64.png" documentation="" schema="1.0.7" diff --git a/urls/propres.php b/urls/propres.php new file mode 100644 index 0000000..3a2c355 --- /dev/null +++ b/urls/propres.php @@ -0,0 +1,85 @@ +