X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=lib%2Fqueuemanager.php;h=d42e4b4b57e88ac31f7b35ddd6f22e46088ea467;hb=dd61ae8fbeee64c85f8186672292335592be1ff5;hp=6666a6cb5a69974a61783a842ac3b106860998b4;hpb=50b12e5820e88682c27a75974143dc61159b327c;p=quix0rs-gnu-social.git diff --git a/lib/queuemanager.php b/lib/queuemanager.php index 6666a6cb5a..d42e4b4b57 100644 --- a/lib/queuemanager.php +++ b/lib/queuemanager.php @@ -143,14 +143,16 @@ abstract class QueueManager extends IoManager return "$class $object->id"; } return $class; - } - if (is_string($object)) { + } elseif (is_string($object)) { $len = strlen($object); $fragment = mb_substr($object, 0, 32); if (mb_strlen($object) > 32) { $fragment .= '...'; } return "string '$fragment' ($len bytes)"; + } elseif (is_array($object)) { + return 'array with ' . count($object) . + ' elements (keys:[' . implode(',', array_keys($object)) . '])'; } return strval($object); } @@ -175,7 +177,43 @@ abstract class QueueManager extends IoManager */ protected function decode($frame) { - return unserialize($frame); + $object = unserialize($frame); + + // If it is a string, we really store a JSON object in there + // except if it begins with '<', because then it is XML. + if (is_string($object) && + substr($object, 0, 1) != '<' && + !is_numeric($object)) + { + $json = json_decode($object); + if ($json === null) { + throw new Exception('Bad frame in queue item'); + } + + // The JSON object has a type parameter which contains the class + if (empty($json->type)) { + throw new Exception('Type not specified for queue item'); + } + if (!is_a($json->type, 'Managed_DataObject', true)) { + throw new Exception('Managed_DataObject class does not exist for queue item'); + } + + // And each of these types should have a unique id (or uri) + if (isset($json->id) && !empty($json->id)) { + $object = call_user_func(array($json->type, 'getKV'), 'id', $json->id); + } elseif (isset($json->uri) && !empty($json->uri)) { + $object = call_user_func(array($json->type, 'getKV'), 'uri', $json->uri); + } + + // But if no object was found, there's nothing we can handle + if (!$object instanceof Managed_DataObject) { + throw new Exception('Queue item frame referenced a non-existant object'); + } + } + + // If the frame was not a string, it's either an array or an object. + + return $object; } /** @@ -233,7 +271,6 @@ abstract class QueueManager extends IoManager if (Event::handle('StartInitializeQueueManager', array($this))) { $this->connect('distrib', 'DistribQueueHandler'); - $this->connect('omb', 'OmbQueueHandler'); $this->connect('ping', 'PingQueueHandler'); if (common_config('sms', 'enabled')) { $this->connect('sms', 'SmsQueueHandler'); @@ -241,9 +278,10 @@ abstract class QueueManager extends IoManager // Background user management tasks... $this->connect('deluser', 'DelUserQueueHandler'); - - // Broadcasting profile updates to OMB remote subscribers - $this->connect('profile', 'ProfileQueueHandler'); + $this->connect('feedimp', 'FeedImporter'); + $this->connect('actimp', 'ActivityImporter'); + $this->connect('acctmove', 'AccountMover'); + $this->connect('actmove', 'ActivityMover'); // For compat with old plugins not registering their own handlers. $this->connect('plugin', 'PluginQueueHandler');