UPDATE zwraca wartość jakby działało, ale nie aktualizuje danych

Question

Problem wygląda tak, że loguje się na testowe konto na stronce (localhost), i próbuję dodać zdjęcie do folderu w ten sposób:

<?php
    session_start();
	if (!isset($_SESSION['online'])) {
		header('Location: index.php');
		exit();
	}

	$msg = "";
	
    if  (isset($_POST['upload'])) {
        $target = "images/".basename($_FILES['image']['name']);
        require_once "connect.php";
        $db_connect = new mysqli($host, $db_user, $db_password, $db_name);
        
        $image = $_FILES['image']['name'];
		
		$sql = "INSERT INTO dane (zdjecie_i) VALUES ('$image')";
		mysqli_query($db_connect, $sql);
		
		if (move_uploaded_file($_FILES['image']['tmp_name'], $target)) {
			$msg = "Zdjęcie dodane!";
		} else {
			$msg = "Błąd podczas wgrywania zdjęcia.";
		}
    }
?>

Tak wygląda formularz:

<div class="userProfileEdit">
    <form method="post" action="user_profile.php" enctype="multipart/form-data">
        <input type="file" name="image">
        <input type="submit" name="upload" value="Zmień zdjęcie">
        <?php 
            echo $msg;
        ?>
    </form>
</div>

Kiedy zamiast UPDATE, jest INSERT INTO, to zdjęcie zostaje dodane do folderu, ale tworzy się nowy użytkownik w bazie danych, a w obu przypadkach $msg wyskakuje jako "Zdjęcie dodane".
I teraz moje pytanie brzmi, czy muszę jakoś wyciągnąć wcześniej z bazy ID użytkownika który się przed chwilą zalogował, żeby dodało do właściwej kolumny? Czy mam coś nie tak w kodzie?  A może coś nie tak zrozumiałem?

Comments

Pokaż strukturę bazy i te twoje zapytanie UPDATE, bo ciężko cokolwiek zrozumieć w z Twojej wypowiedzi.

Pozdrawiam.

---

To samo, tylko zamiast INSERT INTO, jest UPDATE, tak jak napisałem wcześniej.

 

---

<?php
    session_start();
    if (!isset($_SESSION['online'])) {
        header('Location: index.php');
        exit();
    }
 
    $msg = "";
     
    if  (isset($_POST['upload'])) {
        $target = "images/".basename($_FILES['image']['name']);
        require_once "connect.php";
        $db_connect = new mysqli($host, $db_user, $db_password, $db_name);
         
        $image = $_FILES['image']['name'];
         
        $sql = "UPDATE dane (zdjecie_i) VALUES ('$image')";
        mysqli_query($db_connect, $sql);
         
        if (move_uploaded_file($_FILES['image']['tmp_name'], $target)) {
            $msg = "Zdjęcie dodane!";
        } else {
            $msg = "Błąd podczas wgrywania zdjęcia.";
        }
    }
?>

 

---

Dobra, co do struktury bazy: po kiego grzyba tyle kolumn? Wystarczy kolumna z id. Zamiast wszystkich kolumn ze zdjęciami robisz tabelę z pojedynczymi zdjęciami i id, a potem łączysz relacjami. Tak samo z ofertą.

Co do zapytania: dlaczego nie dałeś tam WHERE? Co chciałeś przez to zapytanie osiągnąć? Nic nie rozumiem... :D

---

Czyli tak jak pytałem, muszę wyciągnąć jakoś z bazy ID zalogowanego użytkownika?

---

Ok, zrobiłem tak:
 

<?php
    session_start();
	if (!isset($_SESSION['online'])) {
		header('Location: index.php');
		exit();
	}

	$msg = "";
	
    if  (isset($_POST['upload'])) {
        $target = "images/".basename($_FILES['image']['name']);
        require_once "connect.php";
        $db_connect = new mysqli($host, $db_user, $db_password, $db_name);
        
        $image = $_FILES['image']['name'];
		
		$sql = "UPDATE dane (zdjecie_i) VALUES ('$image') WHERE login='".$_SESSION['login']."'";
		mysqli_query($db_connect, $sql);
		
		if (move_uploaded_file($_FILES['image']['tmp_name'], $target)) {
			$msg = "Zdjęcie dodane!";
		} else {
			$msg = "Błąd podczas wgrywania zdjęcia.";
		}
    }
?>

I dalej $msg wyświetla, że zdjęcie dodane, ale w bazie nic się nie zmienia.

---

Musisz po czymś identyfikować gdzie wstawić te dane, czy to po id, czy po loginie, to twój interes.

$msg wyświetla, że zdjęcie dodane, bo jest w ofierze który sprawdza, czy zdjęcie zostało przeniesione do folderu. Czy ty  wogóle sam wiesz co piszesz?

---

Napisałem przecież, że zrobiłem tak:

 $sql = "UPDATE dane (zdjecie_i) VALUES ('$image') WHERE login='".$_SESSION['login']."'";

Czyli powinno mi wziąć login z sesji logowania prawda?

---

Nawet nie powinno, ale tak robi. Poza tym, wcześniej napisałeś to:

I teraz moje pytanie brzmi, czy muszę jakoś wyciągnąć wcześniej z bazy ID użytkownika który się przed chwilą zalogował, żeby dodało do właściwej kolumny?

Co trochę się kłóci z tym, co piszesz, bo login =/= id, ale pomińmy to :D 

---

A jaka jest różnica? Kiedy wyciągam login, czy id, to nie powinno mi włożyć wartości przypisanej do kolumny, gdzie te wartości się zgadzają?

---

Zapomniałeś użyć SET w tym zapytaniu, czemu nie zainstalujesz sobie jakiegoś IDE które od razu Ci powie, że odpierdzielasz manianę w tym zapytaniu? :D

---

Nie jestem omnibusem i pewnie dlatego pytam co robię nie tak. Uczę się jak zapewne widzisz. IDE to coś takiego jak Emmet w Brackets? Zrobiłem :
 

$sql = "UPDATE dane SET zdjecie_i = '$image' WHERE id ='".$_SESSION['id']."'";

I działa, dziękuję.

---

Dobra już przeczytałem czym jest IDE.

---

Polecam PHPstorm, jak jesteś uczniem/ studentem to możesz się zarejsetrować na stronie JetBrains i masz wszystkie ich produkty za free :D

---

Nie jestem. Ściągnąłem Eclipse przed chwilą, będzie testowane ;d