Delete foto in database e cartella

[Alex_70]

niente, pagina vuota

secondo me la logica nello script e' sbagliata,
mi deve permettere di selezionare quale foto voglio cancellare
poi eliminare l'immagine dal database
e poi dalla cartella

invece....

 

[macus_adi]

Una relazione polimorfa e un listener per la delete asincrona del file forse sarebbe migliore di ...

Ma questa è solo una supposizione, nulla è certo, scontato, funzionante e testato, solo chi copia può essere sicuro dell'errore...

Leggi qui ,forse anche questa per te risulterà errata, MA solitamente funziona immediatamente.

 

[Alex_70]

Una relazione polimorfa e un listener per la delete asincrona del file forse sarebbe migliore di ...

Ma questa è solo una supposizione, nulla è certo, scontato, funzionante e testato, solo chi copia può essere sicuro dell'errore...

Leggi qui ,forse anche questa per te risulterà errata, MA solitamente funziona immediatamente.

macus_adi grazie, ma qua siamo a livelli avanzati,

lo script che ho postato andrebbe bene se solo qualcuno riuscisse a fixarlo

vedo gente che si presentano con CV da fare invidia alla Microsoft e poi nessuno che riesce a risolvere questo semplice script (per loro) ma difficile per me

va beh.. pazienza

 

[Max 1]

vedo gente che si presentano con CV da fare invidia alla Microsoft e poi nessuno che riesce a risolvere questo semplice script (per loro) ma difficile per me

va beh.. pazienza

Niente polemiche o critiche sul forum!! Devi ringraziare chi ti risponde anche perché qui nessuno p obbligato!!!
Se non ti vanno bene le risposte non ne tieni conto o non fai domande! Ma niente polemiche

Chiediti anche perché le tue discussioni sono sempre lunghe molte pagine!

 

[Alex_70]

Niente polemiche o critiche sul forum!! Devi ringraziare chi ti risponde anche perché qui nessuno p obbligato!!!
Se non ti vanno bene le risposte non ne tieni conto o non fai domande! Ma niente polemiche

Chiediti anche perché le tue discussioni sono sempre lunghe molte pagine!

Non e' una polemica, e' una riflessione, succede cosi anche nella vita
non obbligo nessuno ad aiutarmi, ringrazio invece tutti quellli che lo hanno fatto

le mie discussioni sono lunghe? forse perche' voglio capire fino in fondo l'argomento, non essendo competente in questa materia,

inoltre non mi sembra che stia violando nessun regolamento su limiti di pagine che scrivo, se si, allora mi limitero' a 2 righe.

Grazie

 

[Max 1]

No! Non c'è limitazioni ma considerato che non vedo nessuno che ad ogni discussione impegna pagine e pagine mi sorge il dubbi ci sia qualche altro problema .....

Comunque fai quello che vuoi nelle regole ma niente polemiche e lamentele

 

[Alex_70]

No! Non c'è limitazioni ma considerato che non vedo nessuno che ad ogni discussione impegna pagine e pagine mi sorge il dubbi ci sia qualche altro problema .....

Comunque fai quello che vuoi nelle regole ma niente polemiche e lamentele

Il motivo e' semplice perche' nessuno scrive quanto me, perche' sono tutti bravi e preparati ,
io (e lo specificato ogni volta e lo ripeto) non sono un programmatore, solo passione e voglia d'imparare, quindi per me e' difficile capire a volte il codice, sto cercando d'imparare dai maestri, sperimentando, e leggendo qualcosa online.

Quindi abbiate pazienza, quando smettero' di scrivere qui vuol dire che allora non avro' piu' bisogno del vostro aiuto.
D'altronde i forum sono nati proprio per questo, condividere la conoscenza e il sapere.
Concludo, non siete obbligati ad aiutarmi, ma lo spero.

grazie a tutti

 

[macus_adi]

sto cercando d'imparare

Questo il problema... Per come stai approcciando i problemi saranno sempre dolori...

Ripeto e ribadisco il concetto di qualche post fa...
Fai uno schema logico funzionale dell'applicazione, vedi ogni azione a cosa deve corrispondere, PROVA TU a creare lo script senza copiare, a questo punto se non funziona poi chiedere aiuto su un problema che effettivamente sai manipolare con le tue mani....

vedo gente che si presentano con CV da fare invidia alla Microsoft

Magari quelli con un buon CV forse hanno studiato (non sui forum)...

Detto questo, dovresti sanatizzare la stringa prima di creare il path altrimenti non ne verrai mai fuori...
Ripeto.... Vedi MediaLibrary non su come si usa ma il codice!

 

[Alex_70]

Buongiorno a tutti

sono riuscito a mettere un po' di logica allo script,

il problema e' che ricevo errore quando vado a cancellare la foto

Warning: mysqli_fetch_assoc() expects parameter 1 to be mysqli_result, boolean given in E:\OpenServer\domains\localhost\cinema\crud\gallery_delete.php on line 29

line 29

while($row = mysqli_fetch_assoc($rs))

codice completo modificato

gallery_delete.php

View and download from Yandex.Disk

yadi.sk

 

[Tommy03]

hai provato ad eseguire la query che ti da errore su phpmyadmin?

 

[Alex_70]

le query sono molte, il codice si ferma qui

$actor_id = $_GET["id"];
    
    
    
    $sql = "
select *
FROM
actor a
INNER JOIN
gallery g
on
g.actor_id_fk = a.actor_id
WHERE
a.actor_id=". $actor_id;
    
    
    
    $rs = mysqli_query($conn, $sql);
    
    while($row = mysqli_fetch_assoc($rs))  // QUI ERRORE
        {
    
    $nome= $row ['nome'];
    
    $getIamgeName = $row['photo'];
        
        
        
        
        $createDeletePath = "../image_upload/actor/uploads/$nome ($actor_id)/".$getIamgeName;
    
        
    
    if(isset($_GET['deleteid']))

 

[Alex_70]

questa funziona

$sql = "
select *
FROM
actor a
INNER JOIN
gallery g
on
g.actor_id_fk = a.actor_id
WHERE
a.actor_id=". $actor_id;

e questa pure

cliccando sul bottone

<a href="?deleteid=<?php echo $row["gallery_id"]?>" class="btn btn-primary">Delete image</a>

dovrebbe partire la sequenza che cancella la riga gallery_id e poi dalla cartella, ma non succede

qualcosa mi sfugge

 

[Tommy03]

La connessione al DB sei sicuro che funziona? Perché non vedo errori che potrebbero causare l'errore alla riga 29.
Al limite fai una prova sostituendo la query dell'errore (la prima) con una tipo

$sql="SELECT * FROM actor";

Solo per provare

 

[Tommy03]

dovrebbe partire la sequenza che cancella la riga gallery_id e poi dalla cartella, ma non succede

Intanto l'URL cambia correttamente quando lo clicchi? Dovrebbe diventare ?deleteid=[gallery_id]
Cmq credo che tu debba sostituire questo:

$deleteSql = "delete from gallery where gallery_id = ".$getRow['deleteid'];

Con questo:

$deleteSql = "delete from gallery where gallery_id = ".$_GET['deleteid'];

 

[Alex_70]

si la connessione funziona, se provo a modificare i dati di connessione mi da errore, quindi il problema non e' li

se scrivo la query che hai proposto ottengo zero risultati

 

[Alex_70]

con questa

$deleteSql = "delete from gallery where gallery_id = ".$_GET['deleteid'];

stesso risultato, errore ,

ma l'url pero 'e' corretto

line 29 e' quella sopra

 

[Tommy03]

Tutta questa parte:

if(isset($_GET['deleteid']))
    {
        
        $selectSql = "
        
        select * from gallery WHERE actor_id_fk = ".$_GET['id'];
        $rsSelect = mysqli_query($conn,$selectSql);
        $getRow = mysqli_fetch_assoc($rsSelect);
        
        
        
        
        
        if(unlink($createDeletePath))
        {
            $deleteSql = "delete from gallery where gallery_id = ".$getRow['deleteid'];
            $rsDelete = mysqli_query($conn, $deleteSql);    
            
            if($rsDelete)
            {
                header('location:cinema.php?success=true');
                exit();
            }
        }
        else
        {
            $errorMsg = "Unable to delete Image";
        }
        
    }

Sposta la fuori dal while, mettila subito dopo

$actor_id = $_GET["id"];

 

[Alex_70]

spostato cosi

<?php
    
    require_once("cineteca.include");

    if(!$conn)
    {
        die(mysqli_error());
    }
    
    $actor_id = $_GET["id"];
    
    
    if(isset($_GET['deleteid']))
    {
        
        $selectSql = "
        
        select * from gallery WHERE actor_id_fk = ".$_GET['id'];
        $rsSelect = mysqli_query($conn,$selectSql);
        $getRow = mysqli_fetch_assoc($rsSelect);
        
        
        
        
        
        if(unlink($createDeletePath))
        {
            $deleteSql = "delete from gallery where gallery_id = ".$getRow['deleteid'];
            $rsDelete = mysqli_query($conn, $deleteSql);   
            
            if($rsDelete)
            {
                header('location:cinema.php?success=true');
                exit();
            }
        }
        else
        {
            $errorMsg = "Unable to delete Image";
        }
        
    }
    
    
    
    
    $sql = "
select *
FROM
actor a
INNER JOIN
gallery g
on
g.actor_id_fk = a.actor_id
WHERE
a.actor_id=". $actor_id;
    
    
    
    $rs = mysqli_query($conn, $sql);
    
    while($row = mysqli_fetch_assoc($rs))
        {
    
    $nome= $row ['nome'];
    
    $getIamgeName = $row['photo'];
        
        
        
        
        $createDeletePath = "../image_upload/actor/uploads/$nome ($actor_id)/".$getIamgeName;
    
        
    
    

    
?>

risultato pessimo

 

[Tommy03]

Intanto prova a cambiare così, forse il problema era che $actor_id diventa vuota dopo l'invio, dato che non vedo il parametro id nell'url della foto

<?php 
    
    require_once("cineteca.include");

    if(!$conn)
    {
        die(mysqli_error());
    }
    
    $actor_id = $_GET["id"];
    
    
    
    $sql = "
select *
FROM 
actor a 
INNER JOIN
gallery g 
on 
g.actor_id_fk = a.actor_id
WHERE 
a.actor_id=". $actor_id;
    
    
    
    $rs = mysqli_query($conn, $sql);
    
    while($row = mysqli_fetch_assoc($rs))
        {
    
    $nome= $row ['nome'];
    
    $getIamgeName = $row['photo'];
  }      
        
        
        
        $createDeletePath = "../image_upload/actor/uploads/$nome ($actor_id)/".$getIamgeName;
    
        
    
    if(isset($_GET['deleteid']))
    {
        
        $selectSql = "
        
        select * from gallery WHERE actor_id_fk = ".$_GET['id'];
        $rsSelect = mysqli_query($conn,$selectSql);
        $getRow = mysqli_fetch_assoc($rsSelect);
        
        
        
        
        
        if(unlink($createDeletePath))
        {
            $deleteSql = "delete from gallery where gallery_id = ".$_GET['deleteid'];
            $rsDelete = mysqli_query($conn, $deleteSql);    
            
            if($rsDelete)
            {
                header('location:cinema.php?success=true');
                exit();
            }
        }
        else
        {
            $errorMsg = "Unable to delete Image";
        }
}

    
?>

<!DOCTYPE html>
<html>
<head>
<title>PHP remove uploaded file from folder and database</title>

<!-- Latest compiled and minified CSS -->
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" >

<!-- Optional theme -->
<link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap-theme.min.css" >
</head>



<body>
    <div class="container">
        <h3 class="text-center">Delete Images</h3>
        
        <div class="row">
        
        <?php 
        if(isset($errorMsg))
        {
        ?>
            <div class="alert alert-danger">
                <?php 
                    echo $errorMsg;
                    unset($errorMsg);
                ?>
            </div>
        <?php 
        }
        ?>
        
        <?php 
        if(isset($_GET['success']) && $_GET['success'] == 'true')
        {
        ?>
            <div class="alert alert-success">
                <?php 
                    echo "Images has been deleted sucessfully";
                ?>
            </div>
        <?php 
        }
        ?>
        
        
    <?php 
        
            
            $imgWithPath = "../image_upload/actor/uploads/$nome ($actor_id)/".$row['photo'];
        ?>    
            
                
            <div class="col-md-12 text-center" style="margin-top:10px">
                <img src="<?php echo $imgWithPath ?>" width="100">
                <a href="?id=<?php echo $actor_id;?>&deleteid=<?php echo $row["gallery_id"]?>" class="btn btn-primary">Delete image</a>
            </div>
            
            
    <?php        
        
        }
    ?>
    </div>
    </div>
    
    
<script src="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/js/bootstrap.min.js"></script>    
</body>
</html>

 

[Alex_70]

ho dovuto modificare qui
da cosi

$rs = mysqli_query($conn, $sql);
   
    while($row = mysqli_fetch_assoc($rs))
        {
   
    $nome= $row ['nome'];
   
    $getIamgeName = $row['photo'];
  }

a cosi

$rs = mysqli_query($conn, $sql);
   
    while($row = mysqli_fetch_assoc($rs))
        {                                                                        // <-----------QUESTA ELIMINATA
   
    $nome= $row ['nome'];
   
    $getIamgeName = $row['photo'];

manca }

inserita qui se no non caricava la foto

?php
       
           
            $imgWithPath = "../image_upload/actor/uploads/$nome ($actor_id)/".$row['photo'];
        ?>  
           
               
            <div class="col-md-12 text-center" style="margin-top:10px">
                <img src="<?php echo $imgWithPath ?>" width="100">
                <a href="?id=<?php echo $actor_id;?>&deleteid=<?php echo $row["gallery_id"]?>" class="btn btn-primary">Delete image</a>
            </div>
           
           
    <?php      
       
        }   // <---------  INSERITA QUI
    ?>

risultato, quasi buono , ha cancellato dal database e dalla cartella, pero'.....

e apparso questo

nota:

ho cambiato questo parametro, quello che puntava a cinema.php mi dava errore uguale

header('location:gallery_delete.php?success=true');

pero' niente messaggi dell'avvenuta cancellazione

dai che ci siamo, manca qualcosa d'aggiustare