Merge branch 'session-php-serialize'

ratchetphp · May 25, 2016 · b3b10e3 · b3b10e3
2 parents a177819 + f5e8a18
commit b3b10e3
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/src/Ratchet/Session/Serialize/PhpHandler.php b/src/Ratchet/Session/Serialize/PhpHandler.php
@@ -3,10 +3,21 @@
 
 class PhpHandler implements HandlerInterface {
     /**
+     * Simply reverse behaviour of unserialize method.
      * {@inheritdoc}
      */
     function serialize(array $data) {
-        throw new \RuntimeException("Serialize PhpHandler:serialize code not written yet, write me!");
+        $preSerialized = array();
+        $serialized = '';
+
+        if (count($data)) {
+            foreach ($data as $bucket => $bucketData) {
+                $preSerialized[] = $bucket . '|' . serialize($bucketData);
+            }
+            $serialized = implode('', $preSerialized);
+        }
+
+        return $serialized;
     }
 
     /**

diff --git a/tests/unit/Session/Serialize/PhpHandlerTest.php b/tests/unit/Session/Serialize/PhpHandlerTest.php
@@ -33,4 +33,11 @@ public function serializedProvider() {
     public function testUnserialize($in, $expected) {
         $this->assertEquals($expected, $this->_handler->unserialize($in));
     }
+
+    /**
+     * @dataProvider serializedProvider
+     */
+    public function testSerialize($serialized, $original) {
+        $this->assertEquals($serialized, $this->_handler->serialize($original));
+    }
 }