Discussion :

[herveD]

Bonjour,
je me suis aperçu que dans les logs j'ai souvent les mêmes messages d'erreurs qui tournent autour des rôles d'un membre d'un site.
Un membre peut avoir entre 1 et plusieurs rôles
Comment l'initialiser ou autre pour ne plus avoir ce warining ?
Merci pour votre aide

ma fonction

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
add_action('wp_head', 'HD_maj_activite');
function HD_maj_activite() {
	$bp_loggedin_user_id = bp_loggedin_user_id();
	$user_info = get_userdata($bp_loggedin_user_id);
	$user_roles = $user_info->roles;
	if (is_super_admin() || in_array('membre2', $user_roles)) 
 	return ;
	$output .= '<style type="text/css">body.activity.directory #activity-friends {display: none;}</style>';
	$output .= '<style type="text/css">body.activity.directory #activity-groups {display: none;}</style>';
   echo $output; // 
}

message d d'erreur

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
04-Dec-2017 09:42:49 UTC] PHP Notice:  Trying to get property of non-object in /home/../wp-content/themes/woffice-am/bp-am/bp-membres.php on line 7 
=> $user_roles = $user_info->roles;
[04-Dec-2017 09:42:49 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in /../wp-content/themes/woffice-am/bp-am/bp-membres.php on line 8 
=> if (is_super_admin() || in_array('membre2', $user_roles))
[04-Dec-2017 09:42:49 UTC] PHP Notice:  Undefined variable: output in /home/../wp-content/themes/woffice-am/bp-am/bp-membres.php on line 10 
=> $output .= '<style type="text/css">body.activity.directory #activity-friends {display: none;}</style>';

[sabotage]

Déjà est-il normal que $user_info ne contiennent rien ?

[Celira]

Traduction des messages :

04-Dec-2017 09:42:49 UTC] PHP Notice: Trying to get property of non-object in /home/../wp-content/themes/woffice-am/bp-am/bp-membres.php on line 7
=> $user_roles = $user_info->roles;

$user_info est vide (ou au moins n'est pas un objet) : vérifie ce que fait la fonction get_userdata et ce qu'elle retourne.

[04-Dec-2017 09:42:49 UTC] PHP Warning: in_array() expects parameter 2 to be array, null given in /../wp-content/themes/woffice-am/bp-am/bp-membres.php on line 8
=> if (is_super_admin() || in_array('membre2', $user_roles))

Comme $user_info était vide, on a rien affecté dans $user_roles à la ligne du dessus.

[04-Dec-2017 09:42:49 UTC] PHP Notice: Undefined variable: output in /home/../wp-content/themes/woffice-am/bp-am/bp-membres.php on line 10
=> $output .= '<style type="text/css">body.activity.directory #activity-friends {display: none;}</style>';

$output .= '<style...'; ça revient à faire $output = $output.'<style...';, il faut donc avoir initialiser $output avant. Tu peux essayer d'ajouter $output = ''; au début de la fonction.

[herveD]

Bonsoir
@Celira @sabotage
Ben non il doit contenir quelque chose? Je suis même persuadé qu'il contient qq chose car le test fonctionne.
Je n'ai pas le même rendu en changeant de rôle.
Un peu embêté pour faire un test sur un site en ligne!
Je me demandais s'il ne fallait pas avant que je lui dise que c'est un tableau du genre

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$user_roles = array(); $user_roles = $user_info->roles;  {
	if (bp_is_my_profile() || is_super_admin() || in_array('membre2', $user_roles))

mais en tout les cas si c'est proche ... cela n'a pas fonctionné !

[sabotage]

car le test fonctionne.

quel test ?

[Celira]

Ben non, ya rien dans $user_info, ou au moins ce n'est pas un objet, c'est ce que te dis le message d'erreur.
Je te conseille d'ajouter un var_dump($user_info); juste après $user_info = get_userdata($bp_loggedin_user_id); et de vérifier ce que fait la fonctin get_userdata (tu peux également nous montrer le code de cette fonction, histoire qu'on puisse t'aider)

[herveD]

Bonjour

quel test ?

Effectivement ce n'est pas vraiment un test mais la fonction de ce code
si membre0 => pas d'affichage d'un contenu
si membre2=> affichage du contenu

[herveD]

oui bien sûr un var_dump:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
object(WP_User)#9657 (8) { ["data"]=> object(stdClass)#9658 (10) { ["ID"]=> string(2) "94" ["user_login"]=> string(6) "htest2" ["user_pass"]=> string(34) "$P$BXYrLxe.kvEfbXHSM9JG/F6ioC3iAc1" ["user_nicename"]=> string(6) "htest2" ["user_email"]=> string(13) "htest2@tmp.fr" ["user_url"]=> string(0) "" ["user_registered"]=> string(19) "2017-11-08 23:09:31" ["user_activation_key"]=> string(45) "1510866479:$P$BVPt6sLksW8gxipIWOL05gUDBxLoxm/" ["user_status"]=> string(1) "0" ["display_name"]=> string(6) "htest2" } ["ID"]=> int(94) ["caps"]=> array(2) { ["armember_access_plan_3"]=> bool(true) ["membre0"]=> bool(true) } ["cap_key"]=> string(18) "wpTmp_capabilities" ["roles"]=> array(1) { [1]=> string(7) "membre0" } ["allcaps"]=> array(4) { ["read"]=> bool(true) ["level_0"]=> bool(true) ["armember_access_plan_3"]=> bool(true) ["membre0"]=> bool(true) } ["filter"]=> NULL ["site_id":"WP_User":private]=> int(1) }

https://codex.wordpress.org/Function...e/get_userdata
https://developer.wordpress.org/refe.../get_userdata/

@+

[Celira]

Tu as obtenu ce var_dump dans le cas où tu as aussi le message d'erreur ?

[herveD]

Re

Ben oui je remplis mon fichier de log. Extrait depuis 11h sachant que j'ai le même bout de prog & if à plusieurs endroits:
Bon cela fonctionne mais comment éliminer ce warning si je ne peux pas initialiser en disant que c'est un tableau ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
[08-Dec-2017 11:05:36 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14
[08-Dec-2017 11:05:36 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 54
[08-Dec-2017 11:05:36 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 37
[08-Dec-2017 11:09:36 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14
[08-Dec-2017 11:09:36 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 54
[08-Dec-2017 11:09:36 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 37
[08-Dec-2017 12:40:05 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14
[08-Dec-2017 12:40:05 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 55
[08-Dec-2017 12:40:05 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 37
[08-Dec-2017 13:08:18 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14
[08-Dec-2017 13:08:18 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 55
[08-Dec-2017 13:08:18 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 37
[08-Dec-2017 13:22:24 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14
[08-Dec-2017 13:22:24 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 55
[08-Dec-2017 13:22:24 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 37
[08-Dec-2017 13:22:24 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14
[08-Dec-2017 13:22:24 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 55
[08-Dec-2017 13:22:24 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 37
[08-Dec-2017 13:22:27 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14
[08-Dec-2017 13:22:27 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 55
[08-Dec-2017 13:22:27 UTC] PHP Warning:  in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 37

[Celira]

Ce que tu peux faire pour empêche l'erreur, c'est l'initialiser si il ne l'est pas. Quelque chose du genre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$user_roles = isset($user_info->roles)?$user_info->roles:array();

Mais ça n'explique pas pourquoi le tableau est vide à la base...

[herveD]

Cela a fonctionné. Plus un msg d'erreur dans le debug.log
Néanmoins encore surpris car j'ai ce code à plusieurs endroits.
En navigant j'aurai du généré plusieurs messages d'erreurs car je n'ai inséré ton code qu'à un seul endroit !!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
$user_info = get_userdata($bp_loggedin_user_id);
	//var_dump($user_info);
	$user_roles = isset($user_info->roles)?$user_info->roles:array();
	if (bp_is_my_profile() || is_super_admin() || in_array('membre2', $user_roles)) 
 	return ;
		bp_core_remove_nav_item('activity');
		bp_core_remove_nav_item('friends');
		bp_core_remove_nav_item('groups');
		bp_core_remove_nav_item('notes');
	}

Je continue mes investigations. Sinon, j'ia vu que je n'étais pas le seul et que certains on fait ce test

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

 if (is_array($this->roles)) {

avant le test !
@+

[herveD]

bon finalement j'ai bien à nouveau mes messages d'erreurs dans le fichier de log. C'est ce que je me doutais, il y a un décalage pour que j'y ai accès !!
Ce qui m’inquiète plus c'est que même

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$user_roles = isset($user_info->roles)?$user_info->roles:array();

n’empêche pas la génération de l'erreur dans les lignes qui suivent
Complètement perdu :-(

[rawsrc]

Salut

essaie avec ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if (is_super_admin() || in_array('membre2', (array)$user_roles, true))

[Invité]

Bonjour,..

au lieu de vouloir mettre un emplâtre sur une jambe de bois, il serait plus intéressant de corriger l'origine du problème, non ?

Comment est construit $user_info->roles ??

[herveD]

Bonjour

Je ne sais pas à quoi attribuer cette expression.
Mais il me semble (car ce n'est pas moi qui le construit que $user_info est un tableau fait par WP (voir aussi https://developer.wordpress.org/refe.../get_userdata/)
@+

[Invité]

OK ! (je ne maitrise pas WP).

Dans ce cas, fait un var_dump, pour voir ce que retourne $user_info !

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

var_dump($user_info);

Et montre-nous (au moins) ce qui nous intéresse ici : 'roles'
Il faut vérifier que ça renvoie bien (et systématiquement) un array.


Sinon, pour le correctif :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

	$user_roles = ( !empty($user_info->roles) && is_array($user_info->roles) )? $user_info->roles : array();

Là tu es sûr que $user_roles est un array.

[herveD]

J'avais donné le résultat (plus haut) à @celira

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
var_dump($user_info)
object(WP_User)#9657 (8) { ["data"]=> object(stdClass)#9658 (10) { ["ID"]=> string(2) "94" ["user_login"]=> string(6) "htest2" ["user_pass"]=> string(34) "$P$BXYrLxe.kvEfbXHSM9JG/F6ioC3iAc1" ["user_nicename"]=> string(6) "htest2" ["user_email"]=> string(13) "htest2@tmp.fr" ["user_url"]=> string(0) "" ["user_registered"]=> string(19) "2017-11-08 23:09:31" ["user_activation_key"]=> string(45) "1510866479:$P$BVPt6sLksW8gxipIWOL05gUDBxLoxm/" ["user_status"]=> string(1) "0" ["display_name"]=> string(6) "htest2" } ["ID"]=> int(94) ["caps"]=> array(2) { ["armember_access_plan_3"]=> bool(true) ["membre0"]=> bool(true) } ["cap_key"]=> string(18) "wpTmp_capabilities" ["roles"]=> array(1) { [1]=> string(7) "membre0" } ["allcaps"]=> array(4) { ["read"]=> bool(true) ["level_0"]=> bool(true) ["armember_access_plan_3"]=> bool(true) ["membre0"]=> bool(true) } ["filter"]=> NULL ["site_id":"WP_User":private]=> int(1) }

Ok fait, je laisse tourner un peu pour voir le debug.log

[Invité]

Il suffit de lire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

["roles"]=> array(1) { [1]=> string(7) "membre0" }

On a bien un array, et il n'est pas vide !

Donc, il faut revenir à l'origine du message d'erreur :

in_array() expects parameter 2 to be array, null given in ../bp-membres.php on line 14

On trouve quoi, "ligne 14" ??

[Celira]

Il est possible que, dans le cas où l'utilisateur n'a aucun role, la fonction retourne null au lieu d'un tableau vide. (ce serait fortement dommage, mais on ne maîtrise pas le code de WP) La question est : est-il possible et normal pour un utilisateur de n'avoir aucun role ?