Multiple delete checkbox

[wpeloquin]

ALCON,

I am having trouble getting a multiple select delete (via checkbox) to work properly. I even went so far as to look at how vB4 does it in the pm inbox and did a mirror. Is there an easier way to do this, to delete multiple items at one time?

In my template, i have the following code:

<form id="wp_script" name="wp_script" action="file.php?do=doedit" method="post">
...
<vb:each from="wp_var" value="list">
<div class="ddo_tr top blockrow">
<div><input type="checkbox" name="del_id[{vb:raw list.id}]" id="del_id[{vb:raw list.id}]" value="{vb:raw list.id}" /></div>
<div>{vb:raw list.location}</div>
<div>{vb:raw list.comments}</div>
</div>
</vb:each>
...
<div><input class="button" type="submit" id="delete" name="delete" value="{vb:rawphrase delete}" /></div>

</form>


In my php file, i have the following code:

$vbulletin->input->clean_array_gpc('p', array(
'set_userid' => TYPE_UINT,
'del_id' => TYPE_ARRAY_UINT)
);

foreach (array_keys($vbulletin->GPC['del_id']) AS $id)
{
$id = intval($id);
$wp_ids["$id"] = $id;
}
unset($id);

$sids = $db->query_read_slave("
SELECT id
FROM " . TABLE_PREFIX . "wp_table
WHERE userid = " . $vbulletin->userinfo['userid'] . "
AND id IN(" . implode(', ', $wp_ids) . ")
");


// build the final array of ids to work with
while ($wp_id = $db->fetch_array($sids))
{
$wp_sids[] = $wp_id['id'];
}

// delete from the pm table using the results from above
$delsql = "DELETE FROM " . TABLE_PREFIX . "wp_table WHERE id IN(" . implode(', ', $wp_sids) . ")";
$db->query_write($delsql);

[Badshah93]

Replace this


$sids = $db->query_read_slave("
SELECT id
FROM " . TABLE_PREFIX . "wp_table
WHERE userid = " . $vbulletin->userinfo['userid'] . "
AND id IN(" . implode(', ', $wp_ids) . ")
");


// build the final array of ids to work with
while ($wp_id = $db->fetch_array($sids))
{
$wp_sids[] = $wp_id['id'];
}

// delete from the pm table using the results from above
$delsql = "DELETE FROM " . TABLE_PREFIX . "wp_table WHERE id IN(" . implode(', ', $wp_sids) . ")";
$db->query_write($delsql);

with this

// delete from the pm table using the results from above
$delsql = "DELETE FROM " . TABLE_PREFIX . "wp_table WHERE id IN(" . implode(', ', $wp_ids) . ") AND userid = " . $vbulletin->userinfo['userid'] . "";
$db->query_write($delsql);

[wpeloquin]

That is what i originally had and it was not working. That's why I looked up the private.php method, and it seemed to take a few more steps than what i really needed, but I did a copy anyways and it still wasn't working. I triple checked my syntax and found the error in my template. I still cannot believe i did not find this after my first check before posting this thread, but hopefully it will help someone else.

Much thanks Sherif :)

Apparently the template was:

<input type="checkbox" name="del[{vb:raw list.id}]" id="del[{vb:raw list.id}]" value="{vb:raw list.id}" />

Instead of:

<input type="checkbox" name="del_id[{vb:raw list.id}]" id="del_id[{vb:raw list.id}]" value="{vb:raw list.id}" />