Mark as read/unread button data sheet php/mysql [closed] - php

Question

Closed. This question needs to be more focused. It is not currently accepting answers.
Want to improve this question? Update the question so it focuses on one problem only by editing this post.
Closed 5 years ago.
Improve this question
I'm new to coding and I'm using php and mysql to make a admin panel to show clients data from a form.
Now I want to make a button to show wether the data has been processed or not.
The idea is, I have a column named star and if star == read that display "green" and if star =! read display "red".
Then if the button is pressed, if star == read update to unread but if star =! read update to read.
I have the button like this:
<td>star</td>
And read.php like this:
<?php
include("db.php");
$id = $_GET['id'];
$query = "UPDATE users SET star='read' WHERE id = '$id'";
mysqli_query($link, $query) or die('Database error!');
header('location:admin_main.php');
?>
This just updates the row to read and doesn't have the ability to become unread again.
But I don't know how to formulate the if statements.
If anyone has any suggestions, that is much appreciated.
EDIT
To show a bit of what I tried:
I added a new column to the data sheet:
<td>".$star."</td>
And then I tried to use the code below to check the database:
$query = "SELECT star FROM users";
$selectie = mysqli_query($link, $query) or die($query."<br>".mysql_error());
if($selectie == 'read') {
$star = 'read';
} else {
$star = 'unread';
}
And for the read.php:
<?php
include("inc/verbinden.php");
$id = $_GET['id'];
$query = "SELECT star FROM users";
$selectie = mysqli_query($link, $query) or die($query." <br>".mysql_error());
if($selectie == 'read') {
$query = "UPDATE users SET star='unread' WHERE id = '$id'";
mysqli_query($link, $query) or die('Database error!');
} else {
$query = "UPDATE users SET star='read' WHERE id = '$id'";
mysqli_query($link, $query) or die('Database error!');
}
header('location:admin_main.php');
?>
But I realize that the if/else is wrong.

Answer 1

You're almost there. What you missed is looping over (successful) results, such as using a while loop for instance:
Side note: I added (int) for the GET array which helps to safeguard against a possible SQL injection and corrected the use of mysql_error(). That api does not intermix with the mysqli_* api.
<?php
include("inc/verbinden.php");
if(!empty($_GET['id'])){
$id = (int)$_GET['id'];
}else{
echo "The GET array is empty.";
exit; // Stops further execution.
}
$query = "SELECT star FROM users";
$selectie = mysqli_query($link, $query) or die($query." <br>".mysqli_error($link));
while($row = mysqli_fetch_array($selectie)){
if($row['star'] == 'read') {
$query = "UPDATE users SET star='unread' WHERE id = '$id'";
mysqli_query($link, $query) or die('Database error!'); // use mysqli_error($link)
}else{
$query = "UPDATE users SET star='read' WHERE id = '$id'";
mysqli_query($link, $query) or die('Database error!'); // use mysqli_error($link)
}
}
header('location:admin_main.php');
exit; // Stops further execution.
Note: You could substitute mysqli_fetch_array() with mysqli_fetch_assoc().
Also, it's best to use mysqli_affected_rows() when using UPDATE in order to get actual truthness.
You can read up on those functions:
http://php.net/manual/en/mysqli-result.fetch-array.php
http://php.net/manual/en/mysqli-result.fetch-assoc.php
http://php.net/manual/en/mysqli.affected-rows.php
One thing to note though is that read and Read, as well as unread and Unread are two different animals. So make absolutely sure that those are indeed the values in your database as well is what is going in the database.