Not a bug Alert DataWriter

Robbo

Well-known member
I'm assuming alerts are never loaded directly by the datawriter because in XenForo_DataWriter_Alert->_getExistingData($data) on line 60 it makes a call to $this->_getAlertModel() which doesn't exist. I can't think of why you would want to update an alert other than view_date which is handled by a db->update call but if someone wants to in an addon this method needs to be created.
 
I also just noticed a variable being set which does nothing.

Same class, on line 82 you have...
$extraData = '';
return true;

$extraData isn't a reference so this is doing nothing. Maybe it is meant to be $this->set('extra_data', ''); ?

Edit: the set thing is wrong since it calls that function again heh. In my stuff I change $extraData to reference.
 
Oh, I didn't think it would be there. Interesting...

What about my second post? Is it meant to be by reference?
 
Top Bottom