Para corregir el error

[Lorsui]

Hola a todos, estoy teniendo un problema con eliminar un producto de un carrito de compras.
En verCarrito.php tengo lo siguiente:

Código PHP:

  <a href='javascript:eliminar(".$item['id'].");' title='Eliminar producto'><img src='img/delete.gif'></a>  
   

y en la barra del menú, donde está el espacio para el carrito tengo lo siguiente:

Código:

<script>
        function eliminar(id)
        {
        // para no eliminar directamente paso por esta funcion
        if(confirm("Estas seguro de eliminar este producto?"))
            {
            window.location="eliminarItem.php?id="+id;
            }
        }
</script>

Ahora, cuando llamo a eliminarItem.php que contiene:

Código PHP:

  <?php 
session_start(); 
extract($_GET); 
$carro = $_SESSION['carrito']; 
unset($carro($id)); 
$_SESSION['carrito'] = $carro; 
header("location:index.php"); 
?>   

Me aparece en la url: http://localhost/miSitio/eliminarItem.php?id=6 y en la página Fatal error: Can't use function return value in write context in C:\xampp\htdocs\miSitio\eliminarItem.php on line 5, donde la 5 es el unset.
Alguien me puede decir qué está mal?
Gracias, desde ya.

[pateketrueke]

unset no se usa de esa manera..

por favor, lee el manual
http://php.net/unset

Código PHP:

  $foo = array( 
  0 => 'bar', 
  1 => 'candy', 
); 
unset($foo[0]);  
   

pon atención en que tu usas paréntesis, cuando deben ser corchetes: después de todo es un array...

[Lorsui]

estuve viendo el manual, pero al menos mi intención era borrar de una matriz que está en carrito una variable que es el id de un producto.
Puede que esté equivocando la interpretación?

[pateketrueke]

Iniciado por Lorsui

estuve viendo el manual, pero al menos mi intención era borrar de una matriz que está en carrito una variable que es el id de un producto.
Puede que esté equivocando la interpretación?

tu interpretación es correcta: la implementación no

a todo esto... ¿ya te resulto algo o tienes problemas?

[Lorsui]

Sigo sin poder solucionarlo, estuve cambiando paréntesis, pero todo sigue igual, con nuevos errores, con lo cuál no sé para dónde ir.

[pateketrueke]

Iniciado por Lorsui

Sigo sin poder solucionarlo, estuve cambiando paréntesis, pero todo sigue igual, con nuevos errores, con lo cuál no sé para dónde ir.

claro, el problema consiste en el uso de unset() con el cual puedes eliminar variables completas, o índices en caso de arreglos...

ahora, volviendo al ejemplo anterior:

Código PHP:

  $foo = array( 
  0 => 'bar', 
  1 => 'candy', 
); 
unset($foo[0]); 
print_r($foo);  
   

te pido que lo ejecutes, y analices... la clave de tu problema esta ahí mismo, aunque un poco mas simplificado...

otra es que nos muestres lo que llevas hecho, como lo haces, lo que planeas... y por supuesto: los mensajes de error completos...

así podremos orientarte mas sanamente... (:

[Lorsui]

Bueno...aunque estuve probando lo imposible, evidentemente hay algo que no estoy llegando a comprender, así que por eso aquí están los archivos!!!
Gracias y espero...sanamente :)

El archivo catalogo.php:

Código PHP:

 <script> 
function comprarProducto(item) { 
    var pars="item="+item; 
    var myAjax = new Ajax.Updater("divCarrito", "agregarAlCarro.php",{method: 'get', parameters: pars});     
} 
 
</script> 
<?php 
   
    if($_GET["search_query"]!=""){ 
        $catalogo = getCatalogoConEjemplo($_GET["search_query"]); 
    }elseif($_GET["orden"]!=""){ 
        $catalogo = getPrecio($_GET["orden"]); 
    }else{ 
        $catalogo = getCatalogo($_GET["cat"]); 
    } 
    while($registro = mysql_fetch_array($catalogo)) 
    { 
        echo 
    '<div class="contenedor">'. 
        '<a href="imgCarrito/'.$registro["foto"].'"  rel="lightbox[roadtrip]" title="my caption"><img width="135" height="80" src="imgCarrito/'.$registro["foto"].'"></a><br><br>'. 
        '<b>'.$registro["nombre"].'</b><br> 
    Marca: '.$registro["marca"].'<br> 
        Precio: $<b>'.$registro["precio"].'</b><br><br> 
    <a href="javascript:comprarProducto('.$registro["id"].');">Comprar</a> 
    <a href="producto.php?prod='.$registro["id"].'">+Info</a> 
 
    </div>' 
    ; 
    } 
 
?> 
     
<? include("sidenav.php");?>   

En agregarAlCarro.php:

Código PHP:

  <? 
include("funciones.php"); 
if($_GET["item"]) 
{ 
    addCarrito($_GET["item"]); 
     
} 
include("verCarrito.php"); 
?>   

En el menu(sidenav) donde se encuentra el carrito sidenav.php:

Código PHP:

  <script> 
 
function eliminar() { 
    var myAjax = new Ajax.Updater("divCarrito", "eliminarItem.php",{method: 'get'});     
} 
 
function verCarrito() { 
    var myAjax = new Ajax.Updater("divCarrito", "verCarrito.php",{method: 'get'});     
} 
 
verCarrito(); 
 
</script> 
 
 
<script> 
        function eliminar(id) 
        { 
        if(confirm("Estas seguro de eliminar este producto?")) 
            { 
        window.location="eliminarItem.php?id="+id; 
            } 
        } 
</script> 
 
<div class="sidenav">  
    <h2>Mi Carrito  </h2> 
    <div id="divCarrito">No hay compras</div> 
    Comprar con:<br> 
   
 
</div>  
  

Y para ver el carrito verCarrito.php:

Código PHP:

  <? 
session_start(); 
if($_SESSION["carrito"]) 
{ 
    echo "<table style='margin-left:20px;'>"; 
    foreach($_SESSION["carrito"] as $item) 
    { 
        echo "<tr> 
            <td>".$item['nombre']."</td> 
            <td align='right'>".$item['precio']."</td> 
            <td><a href='javascript:eliminar(".$item['id'].");' title='Eliminar producto'>X</a></td> 
            </tr>"; 
            $total+= $item["precio"]; 
    } 
    echo "<tr><td>Total</td><td><b>".number_format($total,2)."</b></td></tr>"; 
    echo "</table>"; 
} 
?>   

Y para eliminar tengo el eliminarItem.php:

Código PHP:

  <?php 
session_start(); 
$carro = $_SESSION["carrito"]; 
unset($carro("id"); 
$_SESSION['carrito'] = $carro; 
header("location:index.php"); 
?>   

Creo que no me estoy olvidando nada más. Cualquier cosa avisenme.
Gracias!!!

[Nemutagk]

@pateketrueke ya te habia dicho que lo estabas haciendo mal, de hecho, en el ultimo código publicado esta peor que el primero, la forma correcta de realizar un unset es como el te comento

Código PHP:

$array = array(0=>'hola1',1=>'hola2',2=>'hola2');
 
unset($array[0]); //esta es la forma correcta, ve que el indice esta dentro de corchetes "[ ]" no dentro de paréntesis
print_r($array);
//lo que daria hola2 y hola3
 

[Lorsui]

Bueno, evidentemente el código empeorado es el resultado de buscar una solución que no encuentro. Y aunque aplico los consejos que me dan, no lo llego a resolver.
Gracias igualmente.

[Nemutagk]

Pues no se que problema puede ser, para tu necesidad el código siguiente funciona a la perfección

Código PHP:

<?php
$id = $_GET['id'];
session_start(); 
$carro = $_SESSION["carrito"]; 
unset($carro[$id]); 
$_SESSION['carrito'] = $carro; 
header("location:index.php"); 
?>
 

[Lorsui]

Gracias por tu respuesta Nemutagk, no sé que estaré haciendo mal porque igualmente, con ese cambio me tira el siguiente error:
Fatal error: Can't use function return value in write context in C:\xampp\htdocs\miSitio\eliminarItem.php on line 5
Pero si me decís que así está correcto el código, el problema debe estar en otro lado. Así que voy a investigar, con más seguridad, en los otros php.
Gracias!!!

[Nemutagk]

Pues no tengo idea, el error que indicas solo lo eh visto cuando se implementa mal por ejemplo un IF

Código PHP:

if (strlen($var) = 4) {    //Esto te mandara el error que has comentado
}
 

Sin embargo para el código que tienes no tengo idea que puede ser =/