GNUsocial/gnu-social
4
0
Fork 1
Code Issues Releases Wiki Activity

Fix for caching with compound keys: add Managed_DataObject::_allCacheKeys() to override the one in Memcached_DataObject.

Browse Source 
Memcached_DataObject doesn't quite fully understand unique indexes, and can't properly build cache keys for compound unique or primary keys.
Managed_DataObject has more information in its schema data, so we can build a proper list.
This commit is contained in:
Brion Vibber 2011-09-28 18:32:43 -07:00
parent 797e187acb
commit 69765a0550
2 changed files with 37 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
classes/Managed_DataObject.php
View File

@ -168,4 +168,41 @@ abstract class Managed_DataObject extends Memcached_DataObject
}
return $links;
}
/**
* Return a list of all primary/unique keys / vals that will be used for
* caching. This will understand compound unique keys, which
* Memcached_DataObject doesn't have enough info to handle properly.
*
* @return array of strings
*/
function _allCacheKeys()
{
$table = call_user_func(array(get_class($this), 'schemaDef'));
$ckeys = array();
if (!empty($table['unique keys'])) {
foreach ($table['unique keys'] as $idx => $fields) {
$val = array();
foreach ($fields as $name) {
$val[] = self::valueString($this->$name);
}
$keys = implode(',', $fields);
$vals = implode(',', $val);
$ckeys[] = $this->cacheKey($this->tableName(), $keys, $vals);
}
}
if (!empty($table['primary key'])) {
$fields = $table['primary key'];
$val = array();
foreach ($fields as $name) {
$val[] = self::valueString($this->$name);
}
$keys = implode(',', $fields);
$vals = implode(',', $val);
$ckeys[] = $this->cacheKey($this->tableName(), $keys, $vals);
}
return $ckeys;
}
}

22
classes/User_im_prefs.php
View File

@ -80,26 +80,4 @@ class User_im_prefs extends Managed_DataObject
);
}
/**
* We have two compound keys with unique constraints:
* (transport, user_id) which is our primary key, and
* (transport, screenname) which is an additional constraint.
*
* Currently there's not a way to represent that second key
* in the general keys list, so we're adding it here to the
* list of keys to use for caching, ensuring that it gets
* cleared as well when we change.
*
* @return array of cache keys
*/
function _allCacheKeys()
{
$ukeys = 'transport,screenname';
$uvals = $this->transport . ',' . $this->screenname;
$ckeys = parent::_allCacheKeys();
$ckeys[] = $this->cacheKey($this->tableName(), $ukeys, $uvals);
return $ckeys;
}
}