It is either getting late or I havn't worked with XOOPS in a month or so. (I was never that good to begin with)

I need to delete a record from a SQL table. I have a class set up and want to add a method:

function deleteClientID($CurrentID)
{
global $xoopsDB;
$query_table = $xoopsDB->prefix('obsclient_clients');
$sql = "DELETE FROM $query_table WHERE ClientID = $CurrentID";
$result = $xoopsDB->queryF($sql);
$ret = $result
return $ret;
}

This is what I have. Will this work? Are the last two line needed? is queryF the right or wrong choice here and why? If some one could walk me through the code step by step and correct it if possible that would be great. I know that it is super simple, but I am unclear.

Any insight would be appreciated.

JMass

$xoopsDB->query() should normally be used instead of $xoopsDB->queryF(). The latter (queryF) should only be used if the script performing the query is executing as a result of a GET request, and you want to override the safety check done by $xoopsDB->query().

I would change the last three lines from



to



The function's return value will be TRUE if the query was successful, and FALSE otherwise.

Thanks!

I will have to go though my class files and replace queryf with query for most.

I thought that the last lines seemed redundant for no reason.

Thanks again,

JMass