Delete query deletes from all usesrs

[SilverDragonTears]

Why does this


function getdeltabs() {
$formcontent = "";

$result = $GLOBALS['adopts']->select("tabs", array("username", "name"), "username = '".$GLOBALS['username']."'");

while ($row = $result->fetchObject()) {
$tabname=$row->name;
$formcontent = $formcontent."<option value='".$tabname."'>".$tabname."</option>";
$i++;
}
return $formcontent;
}

delete all the users tabs when one person deletes a single tab?

[SilverDragonTears]

bump. Ideas, friends? :catfish:

[SilverDragonTears]

The problem is in this code not the above code:

elseif (isset($_POST['deltab'])) {
$name = $_POST['name'];
$query = "SELECT * FROM ".constant("PREFIX")."tabs WHERE username='".$loggedinname."' AND name='".$name."'";
$stmt = $adopts->query($query);
$num = $stmt->fetch(PDO::FETCH_ASSOC);

if($num == 0){
$article_content='This cage does not exist.';
}else{


$adopts->update("owned_adoptables", array("tab" => ''), "owner='{$loggedinname}' and tab='{$name}'");
$adopts->delete("tabs", array(), "username='{$loggedinname}' and name='{$name}'");
$article_content='You successfully deleted the tab. Your hamster was automatically moved to your default cage at the <a href="myadopts.php">My Hamsters</a> page.';
}

[Hall of Famer]

First of all your script has a problem with the usage of $num. It is an array fetched using a PDO method, and you are checking if its value equals to 0? An array cannot equal to 0 unless it contains only one element(which is considered a string instead), the statement will always evaluate to be false...

On the other hand, you are using the delete() method incorrectly. It does not accept an empty array as its second argument, so get rid of it. I am surprised that you do not receive an error message for the way you use delete().

[SilverDragonTears]

So what would that look like?

[Hall of Famer]

This should suffice:


$adopts->delete("tabs", "username='{$loggedinname}' and name='{$name}'");

[SilverDragonTears]

Ahhh works like a dream. Thought it would look like that but I didn't realize you didn't have to have the array() there.

[Hall of Famer]

Well you can always browse through the existing script files, including those in admincp, classes and functions directories. There should be plenty of examples that tell you how to use a database method such as update() and delete().

[SilverDragonTears]

I do.... must not have come across those O.o