Fixed serialize issue, fixes #63

lib/ILess/Cache/Cache.php

@@ -10,6 +10,7 @@
 namespace ILess\Cache;
 
 use ILess\Configurable;
+use ILess\Util\Serializer;
 use InvalidArgumentException;
 
 /**
@@ -53,7 +54,7 @@ protected function getLifetime($lifetime)
      */
     protected function serialize($data)
     {
-        return serialize($data);
+        return Serializer::serialize($data);
     }
 
     /**
@@ -65,6 +66,6 @@ protected function serialize($data)
      */
     protected function unserialize($data)
     {
-        return unserialize($data);
+        return Serializer::unserialize($data);
     }
 }

lib/ILess/FileInfo.php

@@ -9,6 +9,8 @@
 
 namespace ILess;
 
+use ILess\Util\Serializer;
+
 /**
  * File information.
  */
@@ -85,12 +87,13 @@ public function serialize()
         unset($vars['reference']);
         unset($vars['rootPath']);
 
-        return serialize($vars);
+        return Serializer::serialize($vars);
     }
 
     public function unserialize($serialized)
     {
-        $unserialized = unserialize($serialized);
+        $unserialized = Serializer::unserialize($serialized);
+
         foreach ($unserialized as $var => $val) {
             $this->$var = $val;
         }

lib/ILess/ImportedFile.php

@@ -11,6 +11,7 @@
 
 use ILess\Exception\Exception;
 use ILess\Node\RulesetNode;
+use ILess\Util\Serializer;
 
 /**
  * Import.
@@ -146,17 +147,19 @@ public function getLastModified()
 
     public function serialize()
     {
-        return serialize([
+        return Serializer::serialize([
             $this->path,
             $this->lastModified,
             base64_encode($this->content),
-            $this->ruleset,
+            // we cannot include the parsed ruleset, it contains circular references
+            // Is there any way to handle it?
         ]);
     }
 
     public function unserialize($serialized)
     {
-        list($this->path, $this->lastModified, $this->content, $this->ruleset) = unserialize($serialized);
+        list($this->path, $this->lastModified, $this->content) = Serializer::unserialize($serialized);
+
         $this->content = base64_decode($this->content);
     }
 }

lib/ILess/Node.php

@@ -261,4 +261,5 @@ public function compileFirst()
     {
         return $this->compileFirst;
     }
+
 }

lib/ILess/Node/ImportNode.php

@@ -325,12 +325,12 @@ public function serialize()
         $vars = get_object_vars($this);
         unset($vars['skip'], $vars['error']);
 
-        return serialize($vars);
+        return Util\Serializer::serialize($vars);
     }
 
     public function unserialize($serialized)
     {
-        $unserialized = unserialize($serialized);
+        $unserialized = Util\Serializer::unserialize($serialized);
         foreach ($unserialized as $var => $val) {
             $this->$var = $val;
         }

lib/ILess/Node/RulesetNode.php

@@ -14,6 +14,7 @@
 use ILess\Exception\ParserException;
 use ILess\Node;
 use ILess\Output\OutputInterface;
+use ILess\Util\Serializer;
 use ILess\Visitor\VisitorInterface;
 
 /**
@@ -1001,18 +1002,15 @@ public function isRulesetLike()
     public function serialize()
     {
         $vars = get_object_vars($this);
-        unset($vars['functionRegistry']);
 
-        $serialized = serialize($vars);
-
-        return $serialized;
+        return Serializer::serialize($vars);
     }
 
     public function unserialize($serialized)
     {
-        $unserialized = unserialize($serialized);
+        $unserialized = Serializer::unserialize($serialized);
         foreach ($unserialized as $var => $val) {
             $this->$var = $val;
-        };
+        }
     }
 }

lib/ILess/Util/Serializer.php

@@ -0,0 +1,39 @@
+<?php
+/*
+ * This file is part of the Sift PHP framework.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace ILess\Util;
+
+final class Serializer
+{
+    /**
+     * Serializes variables.
+     *
+     * @param array $vars
+     *
+     * @return string
+     */
+    public static function serialize($vars)
+    {
+        return serialize($vars);
+    }
+
+    /**
+     * @param string $serialized
+     *
+     * @return array
+     */
+    public static function unserialize($serialized)
+    {
+        $unserialized = @unserialize($serialized);
+        if ($unserialized === false) {
+            throw new \RuntimeException('Error unserializing serialized data.');
+        }
+
+        return $unserialized;
+    }
+}