enhancements to job status tracking

 - Switch to using a Redis hash instead of a json object
 - add helper methods to support fetching created/updated timestamps
 - do not discard `created` timestamp on update
 - simplify job id mapping
 - sanity check `update()` to accept valid statuses only

Author: Aeon

lib/Resque/Job/Status.php

@@ -39,7 +39,14 @@ class Resque_Job_Status
 	 */
 	public function __construct($id)
 	{
-		$this->id = $id;
+		$this->id = self::generateId($id);
+	}
+
+	/**
+	 * generate job id consistently
+	 */
+	private static function generateId($id) {
+		return 'job:' . $id . ':status';
 	}
 
 	/**
@@ -53,9 +60,9 @@ public static function create($id)
 		$statusPacket = array(
 			'status' => self::STATUS_WAITING,
 			'updated' => time(),
-			'started' => time(),
+			'started' => time()
 		);
-		Resque::redis()->set('job:' . $id . ':status', json_encode($statusPacket));
+		Resque::redis()->hmset(self::generateId($id), $statusPacket);
 	}
 
 	/**
@@ -70,7 +77,7 @@ public function isTracking()
 			return false;
 		}
 
-		if(!Resque::redis()->exists((string)$this)) {
+		if(!Resque::redis()->exists($this->id)) {
 			$this->isTracking = false;
 			return false;
 		}
@@ -86,19 +93,25 @@ public function isTracking()
 	 */
 	public function update($status)
 	{
+		$status = (int)$status;
+
 		if(!$this->isTracking()) {
 			return;
 		}
 
+		if($status < 1 || $status > 4) {
+			return;
+		}
+
 		$statusPacket = array(
 			'status' => $status,
-			'updated' => time(),
+			'updated' => time()
 		);
-		Resque::redis()->set((string)$this, json_encode($statusPacket));
+		Resque::redis()->hmset($this->id, $statusPacket);
 
 		// Expire the status for completed jobs after 24 hours
 		if(in_array($status, self::$completeStatuses)) {
-			Resque::redis()->expire((string)$this, 86400);
+			Resque::redis()->expire($this->id, 86400);
 		}
 	}
 
@@ -110,24 +123,60 @@ public function update($status)
 	 */
 	public function get()
 	{
-		if(!$this->isTracking()) {
-			return false;
-		}
+		return $this->fetch('status');
+	}
 
-		$statusPacket = json_decode(Resque::redis()->get((string)$this), true);
-		if(!$statusPacket) {
-			return false;
-		}
+	/**
+	 * Fetch the status for the job being monitored.
+	 *
+	 * @return mixed False if the status is not being monitored, otherwise the status as
+	 * 	as an integer, based on the Resque_Job_Status constants.
+	 */
+	public function status()
+	{
+		return $this->fetch('status');
+	}
+
+	/**
+	 * Fetch the updated timestamp for the job being monitored.
+	 *
+	 * @return mixed False if the status is not being monitored, otherwise the updated timestamp
+	 */
+	public function updated()
+	{
+		return $this->fetch('updated');
+	}
+
+	/**
+	 * Fetch the created timestamp for the job being monitored.
+	 *
+	 * @return mixed False if the status is not being monitored, otherwise the created timestamp
+	 */
+	public function created()
+	{
+		return $this->fetch('created');
+	}
 
-		return $statusPacket['status'];
+	/**
+	 * Fetch the created timestamp for the job being monitored.
+	 *
+	 * @return mixed False if the status is not being monitored, otherwise the created timestamp
+	 */
+	private function fetch($key)
+	{
+		$val = Resque::redis()->hget($this->id, $key);
+		if($val) {
+			return (int)$val;
+		}
+		return false;
 	}
 
 	/**
 	 * Stop tracking the status of a job.
 	 */
 	public function stop()
 	{
-		Resque::redis()->del((string)$this);
+		Resque::redis()->del($this->id);
 	}
 
 	/**
@@ -137,7 +186,7 @@ public function stop()
 	 */
 	public function __toString()
 	{
-		return 'job:' . $this->id . ':status';
+		return $this->id;
 	}
 }
-?>
+?>