El problema que tienes es que cuando no existe el parámetro GET CodUsua
en la URL intentas acceder a él igualmente. PHP lanza una advertencia cuando tratassiempre que trates de acceder al índice de una matriz que no existe para advertirte que ésto podría llevarte a un error no controlado en tu aplicación.
Para detectar cuándo un índice de una matriz existe o no antes de acceder a su valor debes hacer uso de isset()
de la siguiente manera:
<?php if (isset($_GET['CodUsua']) && ($_GET['CodUsua'] != $_SESSION['CodUsua'])): ?>
Donde la evaluación fallará si no existe el índice antes de acceder a él. Esto es así porque en una operación booleana &&
(Y
lógico) se detiene la evaluación del resto de operandos (se evalúan de derecha a izquierda) en cuanto hay uno que vale false
.
Otra zona donde no compruebas la validez del parámetro es ésta:
<li><a class="icon-users" class="amigos" href="amigos.php?CodUsua=<?=
htmlspecialchars($_SESSION['CodUsua']isset($_GET['CodUsua']) ?$_GET['CodUsua']:'')
?>">Ver amigos <?php
if(isset($_GET['CodUsua'])) && !empty(amigos::cantidad_amigos($_GET['CodUsua']))) {
echo amigos::cantidad_amigos($_GET['CodUsua'])[0][0];
} else {
echo 0;
}
?> </a></li>
Entiendo que quieres acceder a $_SESSION['CodUsua']
en vez de a $_GET['CodUsua']
para acceder a los amigos de tu código de usuario.
Además, he agregado htmlspecialchars()
para recordarte que toda salida al navegador debería pasar por esta función. El contenido de la variable súper global $_GET
no la controlas, por lo que podrían hacerte XSS fácilmente.
He hecho uso del operador ternario ?:
para que en caso de que el índice no exista devuelva una cadena vacía, pero queda en tu mano adoptar otra solución en caso de que ésta no exista.