schmittjoh / serializer

src/JMS/Serializer/YamlSerializationVisitor.php

Spacing +11 added lines, -11 removed lines

@@ -85,7 +85,7 @@ 
         $isHash = isset($type['params'][1]);
 
         $count = $this->writer->changeCount;
-        $isList = (isset($type['params'][0]) && !isset($type['params'][1]))
+        $isList = (isset($type['params'][0]) && ! isset($type['params'][1]))
             || array_keys($data) === range(0, \count($data) - 1);
 
         foreach ($data as $k => $v) {
@@ -93,10 +93,10 @@ 
                 continue;
             }
 
-            if ($isList && !$isHash) {
+            if ($isList && ! $isHash) {
                 $this->writer->writeln('-');
             } else {
-                $this->writer->writeln(Inline::dump($k) . ':');
+                $this->writer->writeln(Inline::dump($k).':');
             }
 
             $this->writer->indent();
@@ -104,7 +104,7 @@ 
             if (null !== $v = $this->navigator->accept($v, $this->getElementType($type), $context)) {
                 $this->writer
                     ->rtrim(false)
-                    ->writeln(' ' . $v);
+                    ->writeln(' '.$v);
             }
 
             $this->writer->outdent();
@@ -134,7 +134,7 @@ 
 
     public function visitDouble($data, array $type, Context $context)
     {
-        $v = (string)$data;
+        $v = (string) $data;
 
         if ('' === $this->writer->content) {
             $this->writer->writeln($v);
@@ -145,7 +145,7 @@ 
 
     public function visitInteger($data, array $type, Context $context)
     {
-        $v = (string)$data;
+        $v = (string) $data;
 
         if ('' === $this->writer->content) {
             $this->writer->writeln($v);
@@ -172,9 +172,9 @@ 
             $name = $this->namingStrategy->translateName($metadata);
         }
 
-        if (!$metadata->inline) {
+        if ( ! $metadata->inline) {
             $this->writer
-                ->writeln(Inline::dump($name) . ':')
+                ->writeln(Inline::dump($name).':')
                 ->indent();
         }
 
@@ -185,12 +185,12 @@ 
         if (null !== $v = $this->navigator->accept($v, $metadata->type, $context)) {
             $this->writer
                 ->rtrim(false)
-                ->writeln(' ' . $v);
-        } elseif ($count === $this->writer->changeCount && !$metadata->inline) {
+                ->writeln(' '.$v);
+        } elseif ($count === $this->writer->changeCount && ! $metadata->inline) {
             $this->writer->revert();
         }
 
-        if (!$metadata->inline) {
+        if ( ! $metadata->inline) {
             $this->writer->outdent();
         }
         $this->revertCurrentMetadata();

src/JMS/Serializer/XmlSerializationVisitor.php

Spacing +24 added lines, -24 removed lines

@@ -125,24 +125,24 @@ 
 
         if (null === $this->document) {
             $this->document = $this->createDocument(null, null, true);
-            $this->currentNode->appendChild($doCData ? $this->document->createCDATASection($data) : $this->document->createTextNode((string)$data));
+            $this->currentNode->appendChild($doCData ? $this->document->createCDATASection($data) : $this->document->createTextNode((string) $data));
 
             return;
         }
 
-        return $doCData ? $this->document->createCDATASection($data) : $this->document->createTextNode((string)$data);
+        return $doCData ? $this->document->createCDATASection($data) : $this->document->createTextNode((string) $data);
     }
 
     public function visitSimpleString($data, array $type, Context $context)
     {
         if (null === $this->document) {
             $this->document = $this->createDocument(null, null, true);
-            $this->currentNode->appendChild($this->document->createTextNode((string)$data));
+            $this->currentNode->appendChild($this->document->createTextNode((string) $data));
 
             return;
         }
 
-        return $this->document->createTextNode((string)$data);
+        return $this->document->createTextNode((string) $data);
     }
 
     public function visitBoolean($data, array $type, Context $context)
@@ -190,7 +190,7 @@ 
             $this->setCurrentNode($entryNode);
 
             if (null !== $keyAttributeName) {
-                $entryNode->setAttribute($keyAttributeName, (string)$k);
+                $entryNode->setAttribute($keyAttributeName, (string) $k);
             }
 
             if (null !== $node = $this->navigator->accept($v, $this->getElementType($type), $context)) {
@@ -242,7 +242,7 @@ 
             $node = $this->navigator->accept($v, $metadata->type, $context);
             $this->revertCurrentMetadata();
 
-            if (!$node instanceof \DOMCharacterData) {
+            if ( ! $node instanceof \DOMCharacterData) {
                 throw new RuntimeException(sprintf('Unsupported value for XML attribute for %s. Expected character data, but got %s.', $metadata->name, json_encode($v)));
             }
             if ($this->namingStrategy instanceof AdvancedNamingStrategyInterface) {
@@ -256,7 +256,7 @@ 
         }
 
         if (($metadata->xmlValue && $this->currentNode->childNodes->length > 0)
-            || (!$metadata->xmlValue && $this->hasValue)
+            || ( ! $metadata->xmlValue && $this->hasValue)
         ) {
             throw new RuntimeException(sprintf('If you make use of @XmlValue, all other properties in the class must have the @XmlAttribute annotation. Invalid usage detected in class %s.', $metadata->class));
         }
@@ -268,7 +268,7 @@ 
             $node = $this->navigator->accept($v, $metadata->type, $context);
             $this->revertCurrentMetadata();
 
-            if (!$node instanceof \DOMCharacterData) {
+            if ( ! $node instanceof \DOMCharacterData) {
                 throw new RuntimeException(sprintf('Unsupported value for property %s::$%s. Expected character data, but got %s.', $metadata->reflection->class, $metadata->reflection->name, \is_object($node) ? \get_class($node) : \gettype($node)));
             }
 
@@ -278,7 +278,7 @@ 
         }
 
         if ($metadata->xmlAttributeMap) {
-            if (!\is_array($v)) {
+            if ( ! \is_array($v)) {
                 throw new RuntimeException(sprintf('Unsupported value type for XML attribute map. Expected array but got %s.', \gettype($v)));
             }
 
@@ -287,7 +287,7 @@ 
                 $node = $this->navigator->accept($value, null, $context);
                 $this->revertCurrentMetadata();
 
-                if (!$node instanceof \DOMCharacterData) {
+                if ( ! $node instanceof \DOMCharacterData) {
                     throw new RuntimeException(sprintf('Unsupported value for a XML attribute map value. Expected character data, but got %s.', json_encode($v)));
                 }
 
@@ -297,7 +297,7 @@ 
             return;
         }
 
-        if ($addEnclosingElement = !$this->isInLineCollection($metadata) && !$metadata->inline) {
+        if ($addEnclosingElement = ! $this->isInLineCollection($metadata) && ! $metadata->inline) {
             if ($this->namingStrategy instanceof AdvancedNamingStrategyInterface) {
                 $elementName = $this->namingStrategy->getPropertyName($metadata, $context);
             } else {
@@ -344,7 +344,7 @@ 
 
     private function isSkippableEmptyObject($node, PropertyMetadata $metadata)
     {
-        return $node === null && !$metadata->xmlCollection && $metadata->skipWhenEmpty;
+        return $node === null && ! $metadata->xmlCollection && $metadata->skipWhenEmpty;
     }
 
     private function isSkippableCollection(PropertyMetadata $metadata)
@@ -354,7 +354,7 @@ 
 
     private function isElementEmpty(\DOMElement $element)
     {
-        return !$element->hasChildNodes() && !$element->hasAttributes();
+        return ! $element->hasChildNodes() && ! $element->hasAttributes();
     }
 
     public function endVisitingObject(ClassMetadata $metadata, $data, array $type, Context $context)
@@ -433,12 +433,12 @@ 
     {
         if (null === $this->document) {
             $this->document = $this->createDocument(null, null, true);
-            $this->currentNode->appendChild($textNode = $this->document->createTextNode((string)$data));
+            $this->currentNode->appendChild($textNode = $this->document->createTextNode((string) $data));
 
             return $textNode;
         }
 
-        return $this->document->createTextNode((string)$data);
+        return $this->document->createTextNode((string) $data);
     }
 
     /**
@@ -455,7 +455,7 @@ 
 
     private function attachNullNamespace()
     {
-        if (!$this->nullWasVisited) {
+        if ( ! $this->nullWasVisited) {
             $this->document->documentElement->setAttributeNS(
                 'http://www.w3.org/2000/xmlns/',
                 'xmlns:xsi',
@@ -476,7 +476,7 @@ 
         foreach ($metadata->xmlNamespaces as $prefix => $uri) {
             $attribute = 'xmlns';
             if ($prefix !== '') {
-                $attribute .= ':' . $prefix;
+                $attribute .= ':'.$prefix;
             } elseif ($element->namespaceURI === $uri) {
                 continue;
             }
@@ -492,19 +492,19 @@ 
         if ($this->currentNode->isDefaultNamespace($namespace)) {
             return $this->document->createElementNS($namespace, $tagName);
         }
-        if (!($prefix = $this->currentNode->lookupPrefix($namespace)) && !($prefix = $this->document->lookupPrefix($namespace))) {
-            $prefix = 'ns-' . substr(sha1($namespace), 0, 8);
+        if ( ! ($prefix = $this->currentNode->lookupPrefix($namespace)) && ! ($prefix = $this->document->lookupPrefix($namespace))) {
+            $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
         }
-        return $this->document->createElementNS($namespace, $prefix . ':' . $tagName);
+        return $this->document->createElementNS($namespace, $prefix.':'.$tagName);
     }
 
     private function setAttributeOnNode(\DOMElement $node, $name, $value, $namespace = null)
     {
         if (null !== $namespace) {
-            if (!$prefix = $node->lookupPrefix($namespace)) {
-                $prefix = 'ns-' . substr(sha1($namespace), 0, 8);
+            if ( ! $prefix = $node->lookupPrefix($namespace)) {
+                $prefix = 'ns-'.substr(sha1($namespace), 0, 8);
             }
-            $node->setAttributeNS($namespace, $prefix . ':' . $name, $value);
+            $node->setAttributeNS($namespace, $prefix.':'.$name, $value);
         } else {
             $node->setAttribute($name, $value);
         }
@@ -528,6 +528,6 @@ 
      */
     public function setFormatOutput($formatOutput)
     {
-        $this->formatOutput = (boolean)$formatOutput;
+        $this->formatOutput = (boolean) $formatOutput;
     }
 }

src/JMS/Serializer/Metadata/Driver/YamlDriver.php

Spacing +42 added lines, -42 removed lines

@@ -35,7 +35,7 @@ 
     {
         $config = Yaml::parse(file_get_contents($file));
 
-        if (!isset($config[$name = $class->name])) {
+        if ( ! isset($config[$name = $class->name])) {
             throw new RuntimeException(sprintf('Expected metadata for class %s to be defined in %s.', $class->name, $file));
         }
 
@@ -44,9 +44,9 @@ 
         $metadata->fileResources[] = $file;
         $metadata->fileResources[] = $class->getFileName();
         $exclusionPolicy = isset($config['exclusion_policy']) ? strtoupper($config['exclusion_policy']) : 'NONE';
-        $excludeAll = isset($config['exclude']) ? (Boolean)$config['exclude'] : false;
+        $excludeAll = isset($config['exclude']) ? (Boolean) $config['exclude'] : false;
         $classAccessType = isset($config['access_type']) ? $config['access_type'] : PropertyMetadata::ACCESS_TYPE_PROPERTY;
-        $readOnlyClass = isset($config['read_only']) ? (Boolean)$config['read_only'] : false;
+        $readOnlyClass = isset($config['read_only']) ? (Boolean) $config['read_only'] : false;
         $this->addClassProperties($metadata, $config);
 
         $propertiesMetadata = array();
@@ -58,8 +58,8 @@ 
 
                 } else {
 
-                    if (!$class->hasMethod($methodName)) {
-                        throw new RuntimeException('The method ' . $methodName . ' not found in class ' . $class->name);
+                    if ( ! $class->hasMethod($methodName)) {
+                        throw new RuntimeException('The method '.$methodName.' not found in class '.$class->name);
                     }
                     $virtualPropertyMetadata = new VirtualPropertyMetadata($name, $methodName);
                 }
@@ -68,7 +68,7 @@ 
             }
         }
 
-        if (!$excludeAll) {
+        if ( ! $excludeAll) {
             foreach ($class->getProperties() as $property) {
                 if ($property->class !== $name || (isset($property->info) && $property->info['class'] !== $name)) {
                     continue;
@@ -88,7 +88,7 @@ 
                     $pConfig = $config['properties'][$pName];
 
                     if (isset($pConfig['exclude'])) {
-                        $isExclude = (Boolean)$pConfig['exclude'];
+                        $isExclude = (Boolean) $pConfig['exclude'];
                     }
 
                     if ($isExclude) {
@@ -96,35 +96,35 @@ 
                     }
 
                     if (isset($pConfig['expose'])) {
-                        $isExpose = (Boolean)$pConfig['expose'];
+                        $isExpose = (Boolean) $pConfig['expose'];
                     }
 
                     if (isset($pConfig['skip_when_empty'])) {
-                        $pMetadata->skipWhenEmpty = (Boolean)$pConfig['skip_when_empty'];
+                        $pMetadata->skipWhenEmpty = (Boolean) $pConfig['skip_when_empty'];
                     }
 
                     if (isset($pConfig['since_version'])) {
-                        $pMetadata->sinceVersion = (string)$pConfig['since_version'];
+                        $pMetadata->sinceVersion = (string) $pConfig['since_version'];
                     }
 
                     if (isset($pConfig['until_version'])) {
-                        $pMetadata->untilVersion = (string)$pConfig['until_version'];
+                        $pMetadata->untilVersion = (string) $pConfig['until_version'];
                     }
 
                     if (isset($pConfig['exclude_if'])) {
-                        $pMetadata->excludeIf = (string)$pConfig['exclude_if'];
+                        $pMetadata->excludeIf = (string) $pConfig['exclude_if'];
                     }
 
                     if (isset($pConfig['expose_if'])) {
-                        $pMetadata->excludeIf = "!(" . $pConfig['expose_if'] . ")";
+                        $pMetadata->excludeIf = "!(".$pConfig['expose_if'].")";
                     }
 
                     if (isset($pConfig['serialized_name'])) {
-                        $pMetadata->serializedName = (string)$pConfig['serialized_name'];
+                        $pMetadata->serializedName = (string) $pConfig['serialized_name'];
                     }
 
                     if (isset($pConfig['type'])) {
-                        $pMetadata->setType((string)$pConfig['type']);
+                        $pMetadata->setType((string) $pConfig['type']);
                     }
 
                     if (isset($pConfig['groups'])) {
@@ -136,21 +136,21 @@ 
 
                         $colConfig = $pConfig['xml_list'];
                         if (isset($colConfig['inline'])) {
-                            $pMetadata->xmlCollectionInline = (Boolean)$colConfig['inline'];
+                            $pMetadata->xmlCollectionInline = (Boolean) $colConfig['inline'];
                         }
 
                         if (isset($colConfig['entry_name'])) {
-                            $pMetadata->xmlEntryName = (string)$colConfig['entry_name'];
+                            $pMetadata->xmlEntryName = (string) $colConfig['entry_name'];
                         }
 
                         if (isset($colConfig['skip_when_empty'])) {
-                            $pMetadata->xmlCollectionSkipWhenEmpty = (Boolean)$colConfig['skip_when_empty'];
+                            $pMetadata->xmlCollectionSkipWhenEmpty = (Boolean) $colConfig['skip_when_empty'];
                         } else {
                             $pMetadata->xmlCollectionSkipWhenEmpty = true;
                         }
 
                         if (isset($colConfig['namespace'])) {
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig['namespace'];
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig['namespace'];
                         }
                     }
 
@@ -159,15 +159,15 @@ 
 
                         $colConfig = $pConfig['xml_map'];
                         if (isset($colConfig['inline'])) {
-                            $pMetadata->xmlCollectionInline = (Boolean)$colConfig['inline'];
+                            $pMetadata->xmlCollectionInline = (Boolean) $colConfig['inline'];
                         }
 
                         if (isset($colConfig['entry_name'])) {
-                            $pMetadata->xmlEntryName = (string)$colConfig['entry_name'];
+                            $pMetadata->xmlEntryName = (string) $colConfig['entry_name'];
                         }
 
                         if (isset($colConfig['namespace'])) {
-                            $pMetadata->xmlEntryNamespace = (string)$colConfig['namespace'];
+                            $pMetadata->xmlEntryNamespace = (string) $colConfig['namespace'];
                         }
 
                         if (isset($colConfig['key_attribute_name'])) {
@@ -179,33 +179,33 @@ 
                     if (isset($pConfig['xml_element'])) {
                         $colConfig = $pConfig['xml_element'];
                         if (isset($colConfig['cdata'])) {
-                            $pMetadata->xmlElementCData = (Boolean)$colConfig['cdata'];
+                            $pMetadata->xmlElementCData = (Boolean) $colConfig['cdata'];
                         }
 
                         if (isset($colConfig['namespace'])) {
-                            $pMetadata->xmlNamespace = (string)$colConfig['namespace'];
+                            $pMetadata->xmlNamespace = (string) $colConfig['namespace'];
                         }
                     }
 
                     if (isset($pConfig['xml_attribute'])) {
-                        $pMetadata->xmlAttribute = (Boolean)$pConfig['xml_attribute'];
+                        $pMetadata->xmlAttribute = (Boolean) $pConfig['xml_attribute'];
                     }
 
                     if (isset($pConfig['xml_attribute_map'])) {
-                        $pMetadata->xmlAttributeMap = (Boolean)$pConfig['xml_attribute_map'];
+                        $pMetadata->xmlAttributeMap = (Boolean) $pConfig['xml_attribute_map'];
                     }
 
                     if (isset($pConfig['xml_value'])) {
-                        $pMetadata->xmlValue = (Boolean)$pConfig['xml_value'];
+                        $pMetadata->xmlValue = (Boolean) $pConfig['xml_value'];
                     }
 
                     if (isset($pConfig['xml_key_value_pairs'])) {
-                        $pMetadata->xmlKeyValuePairs = (Boolean)$pConfig['xml_key_value_pairs'];
+                        $pMetadata->xmlKeyValuePairs = (Boolean) $pConfig['xml_key_value_pairs'];
                     }
 
                     //we need read_only before setter and getter set, because that method depends on flag being set
                     if (isset($pConfig['read_only'])) {
-                        $pMetadata->readOnly = (Boolean)$pConfig['read_only'];
+                        $pMetadata->readOnly = (Boolean) $pConfig['read_only'];
                     } else {
                         $pMetadata->readOnly = $pMetadata->readOnly || $readOnlyClass;
                     }
@@ -217,14 +217,14 @@ 
                     );
 
                     if (isset($pConfig['inline'])) {
-                        $pMetadata->inline = (Boolean)$pConfig['inline'];
+                        $pMetadata->inline = (Boolean) $pConfig['inline'];
                     }
 
                     if (isset($pConfig['max_depth'])) {
-                        $pMetadata->maxDepth = (int)$pConfig['max_depth'];
+                        $pMetadata->maxDepth = (int) $pConfig['max_depth'];
                     }
                 }
-                if ((ExclusionPolicy::NONE === $exclusionPolicy && !$isExclude)
+                if ((ExclusionPolicy::NONE === $exclusionPolicy && ! $isExclude)
                     || (ExclusionPolicy::ALL === $exclusionPolicy && $isExpose)
                 ) {
                     $metadata->addPropertyMetadata($pMetadata);
@@ -265,7 +265,7 @@ 
 
     private function addClassProperties(ClassMetadata $metadata, array $config)
     {
-        if (isset($config['custom_accessor_order']) && !isset($config['accessor_order'])) {
+        if (isset($config['custom_accessor_order']) && ! isset($config['accessor_order'])) {
             $config['accessor_order'] = 'custom';
         }
 
@@ -274,11 +274,11 @@ 
         }
 
         if (isset($config['xml_root_name'])) {
-            $metadata->xmlRootName = (string)$config['xml_root_name'];
+            $metadata->xmlRootName = (string) $config['xml_root_name'];
         }
 
         if (isset($config['xml_root_namespace'])) {
-            $metadata->xmlRootNamespace = (string)$config['xml_root_namespace'];
+            $metadata->xmlRootNamespace = (string) $config['xml_root_namespace'];
         }
 
         if (array_key_exists('xml_namespaces', $config)) {
@@ -293,25 +293,25 @@ 
             if (isset($config['discriminator']['disabled']) && true === $config['discriminator']['disabled']) {
                 $metadata->discriminatorDisabled = true;
             } else {
-                if (!isset($config['discriminator']['field_name'])) {
+                if ( ! isset($config['discriminator']['field_name'])) {
                     throw new RuntimeException('The "field_name" attribute must be set for discriminators.');
                 }
 
-                if (!isset($config['discriminator']['map']) || !\is_array($config['discriminator']['map'])) {
+                if ( ! isset($config['discriminator']['map']) || ! \is_array($config['discriminator']['map'])) {
                     throw new RuntimeException('The "map" attribute must be set, and be an array for discriminators.');
                 }
                 $groups = isset($config['discriminator']['groups']) ? $config['discriminator']['groups'] : array();
                 $metadata->setDiscriminator($config['discriminator']['field_name'], $config['discriminator']['map'], $groups);
 
                 if (isset($config['discriminator']['xml_attribute'])) {
-                    $metadata->xmlDiscriminatorAttribute = (bool)$config['discriminator']['xml_attribute'];
+                    $metadata->xmlDiscriminatorAttribute = (bool) $config['discriminator']['xml_attribute'];
                 }
                 if (isset($config['discriminator']['xml_element'])) {
                     if (isset($config['discriminator']['xml_element']['cdata'])) {
-                        $metadata->xmlDiscriminatorCData = (bool)$config['discriminator']['xml_element']['cdata'];
+                        $metadata->xmlDiscriminatorCData = (bool) $config['discriminator']['xml_element']['cdata'];
                     }
                     if (isset($config['discriminator']['xml_element']['namespace'])) {
-                        $metadata->xmlDiscriminatorNamespace = (string)$config['discriminator']['xml_element']['namespace'];
+                        $metadata->xmlDiscriminatorNamespace = (string) $config['discriminator']['xml_element']['namespace'];
                     }
                 }
 
@@ -323,13 +323,13 @@ 
     {
         if (\is_string($config)) {
             $config = array($config);
-        } elseif (!\is_array($config)) {
+        } elseif ( ! \is_array($config)) {
             throw new RuntimeException(sprintf('callback methods expects a string, or an array of strings that represent method names, but got %s.', json_encode($config['pre_serialize'])));
         }
 
         $methods = array();
         foreach ($config as $name) {
-            if (!$class->hasMethod($name)) {
+            if ( ! $class->hasMethod($name)) {
                 throw new RuntimeException(sprintf('The method %s does not exist in class %s.', $name, $class->name));
             }
 

src/JMS/Serializer/Metadata/PropertyMetadata.php

Spacing +13 added lines, -13 removed lines

@@ -59,7 +59,7 @@ 
     public function __construct($class, $name)
     {
         parent::__construct($class, $name);
-        $this->closureAccessor = \Closure::bind(function ($o, $name) {
+        $this->closureAccessor = \Closure::bind(function($o, $name) {
             return $o->$name;
         }, null, $class);
     }
@@ -69,22 +69,22 @@ 
             $class = $this->reflection->getDeclaringClass();
 
             if (empty($getter)) {
-                if ($class->hasMethod('get' . $this->name) && $class->getMethod('get' . $this->name)->isPublic()) {
-                    $getter = 'get' . $this->name;
-                } elseif ($class->hasMethod('is' . $this->name) && $class->getMethod('is' . $this->name)->isPublic()) {
-                    $getter = 'is' . $this->name;
-                } elseif ($class->hasMethod('has' . $this->name) && $class->getMethod('has' . $this->name)->isPublic()) {
-                    $getter = 'has' . $this->name;
+                if ($class->hasMethod('get'.$this->name) && $class->getMethod('get'.$this->name)->isPublic()) {
+                    $getter = 'get'.$this->name;
+                } elseif ($class->hasMethod('is'.$this->name) && $class->getMethod('is'.$this->name)->isPublic()) {
+                    $getter = 'is'.$this->name;
+                } elseif ($class->hasMethod('has'.$this->name) && $class->getMethod('has'.$this->name)->isPublic()) {
+                    $getter = 'has'.$this->name;
                 } else {
-                    throw new RuntimeException(sprintf('There is neither a public %s method, nor a public %s method, nor a public %s method in class %s. Please specify which public method should be used for retrieving the value of the property %s.', 'get' . ucfirst($this->name), 'is' . ucfirst($this->name), 'has' . ucfirst($this->name), $this->class, $this->name));
+                    throw new RuntimeException(sprintf('There is neither a public %s method, nor a public %s method, nor a public %s method in class %s. Please specify which public method should be used for retrieving the value of the property %s.', 'get'.ucfirst($this->name), 'is'.ucfirst($this->name), 'has'.ucfirst($this->name), $this->class, $this->name));
                 }
             }
 
-            if (empty($setter) && !$this->readOnly) {
-                if ($class->hasMethod('set' . $this->name) && $class->getMethod('set' . $this->name)->isPublic()) {
-                    $setter = 'set' . $this->name;
+            if (empty($setter) && ! $this->readOnly) {
+                if ($class->hasMethod('set'.$this->name) && $class->getMethod('set'.$this->name)->isPublic()) {
+                    $setter = 'set'.$this->name;
                 } else {
-                    throw new RuntimeException(sprintf('There is no public %s method in class %s. Please specify which public method should be used for setting the value of the property %s.', 'set' . ucfirst($this->name), $this->class, $this->name));
+                    throw new RuntimeException(sprintf('There is no public %s method in class %s. Please specify which public method should be used for setting the value of the property %s.', 'set'.ucfirst($this->name), $this->class, $this->name));
                 }
             }
         }
@@ -199,7 +199,7 @@ 
 
         parent::unserialize($parentStr);
 
-        $this->closureAccessor = \Closure::bind(function ($o, $name) {
+        $this->closureAccessor = \Closure::bind(function($o, $name) {
             return $o->$name;
         }, null, $this->class);
     }