Abrir sub_enlace

[El_Astur]

Hola
he hecho una lista y quiero que debajo de cada enlace se abra con php un sub_enlace. Lo he hecho, pero me da el problema que en vez de abrir de uno en uno abren todos, es decir, si pincho en un enlace me abre el sub_enlace del mismo y a la vez el de todos los demás. Al cerrar pasa exactamente lo mismo.
En el html tengo:
<ul>
<li><a href="coyuntura.php?page=info&section=ver">Enlace1 </a>
</li>
<?php AbrirEnlace(); ?>
<li><a href="coyuntura.php?page=info&section=ver">Enlace2 </a>
</li>
<?php AbrirEnlace(); ?>
</ul>

En php he puesto lo siguiente:
function AbrirEnlace() //Enlaces que abren: vídeo, link, información o PDF
{
?>
<?php
if(isset($_GET['page']))
{
$page = $_GET['page'];
$section = "include";
if(isset($_GET['section']))
{
$section = $_GET['section'];
if(!strpos($page,"/") && !strpos($section,"/") &&
!strpos($page,".") && !strpos($section,"."))
{
$path = $section."/".$page.".php";
if(file_exists($path))
{
include($path);
}
}
}
}
?>

El problema que tengo es que cuando pincho en un enlace me abre automáticamente los otros también. ¿Sabríais cómo podría hacer que abrieran de uno en uno?, es decir, que pinche, por ejemplo, en enlace1 y me abra sólo ese no también el enlace2.

Un saludo.

[w0rldart]

Pon primero, el codigo, entre las etiquetas CODE, PHP ... para que se pueda entender mejor el codigo

[El_Astur]

HTML:


<ul>
<li><a href="coyuntura.php?page=info&section=ver">Enlace1 </a>
</li>
<?php AbrirEnlace(); ?>
<li><a href="coyuntura.php?page=info&section=ver">Enlace2 </a>
</li>
<?php AbrirEnlace(); ?>
</ul>


PHP:

function AbrirEnlace()
{
?>
<?php
if(isset($_GET['page']))
{
$page = $_GET['page'];
$section = "include";
if(isset($_GET['section']))
{
$section = $_GET['section'];
if(!strpos($page,"/") && !strpos($section,"/") &&
!strpos($page,".") && !strpos($section,"."))
{
$path = $section."/".$page.".php";
if(file_exists($path))
{
include($path);
}
}
}
}
?>

El archivo PHP forma parte de una librería php, desde cada enlace se hace una llamada para que despliegue.

[historiasdemaria]

muy mala practica para la seguridad:
definir al path hacia un include a partir de la variable $_GET
alguien podria inyectar cualquier cosa..

[El_Astur]

Ya he arreglado lo del tema de la seguridad, ¿será mejor así?:

<?php
if(isset($_GET['page']))
{
$page = $_GET['page'];
$section = "include";
if(isset($_GET['section']))
{
$section = $_GET['section'];
if(!strpos($page,"/") && !strpos($section,"/") &&
!strpos($page,".") && !strpos($section,"."))
{
$path = $section."/".$page.".php";
if(file_exists($path))
{
include($path);
}
else
{
echo "La pagina solicitada no existe"; //no acceso
}
}
else
{
echo "No se permite ingresar a esta página"; //no acceso
}
}
}
?>

[historiasdemaria]

Mmm, haz pruebas, yo hubiera puesto un filtro logico, por si puede inyectar desde otro servidor

O sea un variable array que contenga el nombre de las secciones permitidas y luego comprobar si la seccion es valida, o sea limpia

Código PHP:

if (in_array($seccion, $arraySeccionesLimpias))
{
 // Hago esto o lo otro
}
 

:)

[El_Astur]

Ok, muchas gracias!!!!