Post also clobbered the indention

/**
* @param integer $id The id of the record in this model
* @param mixed $assoc_name The name of the HABTM association
* @param mixed $save_id The associated id or an array of id’s to be saved
* @param mixed $delete_id The associated id or an array of id’s to be removed
* @return array Data array with current HABTM association intact
*/
function updateAssoc($id,$assoc_name,$save_id=null,$delete_id=null)
{
if ( $save_id || $delete_id ) {
$data=$this->_auxAssoc($id,$assoc_name);
if ($save_id) {
if (!is_array($save_id)) {
$save_id=array($save_id);
}
$data[$assoc_name][$assoc_name]= $data[$assoc_name][$assoc_name] + $save_id;
}
if ($delete_id) {
if (!is_array($delete_id)) {
$delete_id=array($delete_id);
}
} else {
$delete_id = array();
}
$result=array();
foreach ($data[$assoc_name][$assoc_name] as $id)
{
if (!in_array($id, $delete_id)) {
$result[$id]=$id;
}
}
$data[$assoc_name][$assoc_name]=$result;
return $this->save($data, true);
}
}

updateAssoc($id, ‘Tag’, array(id’s to save), array(id’s to delete))

I will do this for my project. Perhaps it’s just due to quirks of my implementation, but I find myself calling both functions back-to-back in many cases. This would permit a single save() call to the DB, rather than a call to remove some items followed by a call to add items.