@@ -52,8 +52,12 @@ |
||
| 52 | 52 | { |
| 53 | 53 | $prop = get_object_vars($this); |
| 54 | 54 | unset($prop["__yaml__object__api"]); |
| 55 | - if (count($prop) > 0) return $prop; |
|
| 56 | - if (count($this) > 0) return iterator_to_array($this); |
|
| 55 | + if (count($prop) > 0) { |
|
| 56 | + return $prop; |
|
| 57 | + } |
|
| 58 | + if (count($this) > 0) { |
|
| 59 | + return iterator_to_array($this); |
|
| 60 | + } |
|
| 57 | 61 | return $this->__yaml__object__api->value ?? "Empty YamlObject"; |
| 58 | 62 | } |
| 59 | 63 | } |
@@ -88,7 +88,9 @@ discard block |
||
| 88 | 88 | { |
| 89 | 89 | $source = $this->content ?? preg_split("/\n/m", preg_replace('/(\r\n|\r)/', "\n", $strContent), 0, PREG_SPLIT_DELIM_CAPTURE); |
| 90 | 90 | //TODO : be more permissive on $strContent values |
| 91 | - if (!is_array($source) || !count($source)) throw new \Exception(self::EXCEPTION_LINE_SPLIT); |
|
| 91 | + if (!is_array($source) || !count($source)) { |
|
| 92 | + throw new \Exception(self::EXCEPTION_LINE_SPLIT); |
|
| 93 | + } |
|
| 92 | 94 | $previous = $root = new Node(); |
| 93 | 95 | $emptyLines = []; |
| 94 | 96 | try { //var_dump($source); |
@@ -101,7 +103,9 @@ discard block |
||
| 101 | 103 | $n = new Node($lineString, $lineNb); |
| 102 | 104 | $deepest = $previous->getDeepestNode(); |
| 103 | 105 | if ($n->type & (Y::LITTERALS|Y::BLANK|Y::COMMENT|Y::TAG) || $deepest->type & Y::PARTIAL) { |
| 104 | - if ($this->onSpecialType($n, $previous, $emptyLines, $lineString)) continue; |
|
| 106 | + if ($this->onSpecialType($n, $previous, $emptyLines, $lineString)) { |
|
| 107 | + continue; |
|
| 108 | + } |
|
| 105 | 109 | } |
| 106 | 110 | //Note: 6.6 comments: Note that outside scalar content, a line containing only white space characters is taken to be a comment line. |
| 107 | 111 | foreach ($emptyLines as $blankNode) { |
@@ -130,7 +134,9 @@ discard block |
||
| 130 | 134 | } |
| 131 | 135 | } |
| 132 | 136 | } |
| 133 | - if ($this->onContextType($n, $target, $lineString)) continue; |
|
| 137 | + if ($this->onContextType($n, $target, $lineString)) { |
|
| 138 | + continue; |
|
| 139 | + } |
|
| 134 | 140 | $target->add($n); |
| 135 | 141 | $previous = $n; |
| 136 | 142 | } |
@@ -167,8 +173,12 @@ discard block |
||
| 167 | 173 | return true; |
| 168 | 174 | } |
| 169 | 175 | if ($n->type & Y::BLANK) { |
| 170 | - if ($previous->type & Y::SCALAR) $emptyLines[] = $n->setParent($previous->getParent()); |
|
| 171 | - if ($deepest->type & Y::LITTERALS) $emptyLines[] = $n->setParent($deepest); |
|
| 176 | + if ($previous->type & Y::SCALAR) { |
|
| 177 | + $emptyLines[] = $n->setParent($previous->getParent()); |
|
| 178 | + } |
|
| 179 | + if ($deepest->type & Y::LITTERALS) { |
|
| 180 | + $emptyLines[] = $n->setParent($deepest); |
|
| 181 | + } |
|
| 172 | 182 | return true; |
| 173 | 183 | } elseif ($n->type & Y::COMMENT |
| 174 | 184 | && !($previous->getParent()->value->type & Y::LITTERALS) |
@@ -68,7 +68,9 @@ discard block |
||
| 68 | 68 | if ($this->type === Y::ROOT) { |
| 69 | 69 | return $this; |
| 70 | 70 | } |
| 71 | - if (!is_int($indent)) return $this->parent ?? $this; |
|
| 71 | + if (!is_int($indent)) { |
|
| 72 | + return $this->parent ?? $this; |
|
| 73 | + } |
|
| 72 | 74 | $cursor = $this; |
| 73 | 75 | while ($cursor instanceof Node && $cursor->indent >= $indent) { |
| 74 | 76 | if ($cursor->indent === $indent && $cursor->type !== $type) { |
@@ -99,7 +101,7 @@ discard block |
||
| 99 | 101 | if (is_null($current)) { |
| 100 | 102 | $this->value = $child; |
| 101 | 103 | return; |
| 102 | - }elseif ($current instanceof Node) { |
|
| 104 | + } elseif ($current instanceof Node) { |
|
| 103 | 105 | $this->value = new NodeList(); |
| 104 | 106 | if ($current->type & Y::LITTERALS) { |
| 105 | 107 | $this->value->type = $current->type; |
@@ -109,13 +111,21 @@ discard block |
||
| 109 | 111 | } |
| 110 | 112 | //modify type according to child |
| 111 | 113 | if (is_null($this->value->type)) { |
| 112 | - if ($child->type & Y::SET_KEY) $this->value->type = Y::SET; |
|
| 113 | - if ($child->type & Y::KEY) $this->value->type = Y::MAPPING; |
|
| 114 | - if ($child->type & Y::ITEM) $this->value->type = Y::SEQUENCE; |
|
| 114 | + if ($child->type & Y::SET_KEY) { |
|
| 115 | + $this->value->type = Y::SET; |
|
| 116 | + } |
|
| 117 | + if ($child->type & Y::KEY) { |
|
| 118 | + $this->value->type = Y::MAPPING; |
|
| 119 | + } |
|
| 120 | + if ($child->type & Y::ITEM) { |
|
| 121 | + $this->value->type = Y::SEQUENCE; |
|
| 122 | + } |
|
| 115 | 123 | } |
| 116 | 124 | $this->value->push($child); |
| 117 | 125 | |
| 118 | - if ($this->type & Y::LITTERALS) $this->value->type = $this->type; |
|
| 126 | + if ($this->type & Y::LITTERALS) { |
|
| 127 | + $this->value->type = $this->type; |
|
| 128 | + } |
|
| 119 | 129 | } |
| 120 | 130 | |
| 121 | 131 | /** |
@@ -40,7 +40,9 @@ discard block |
||
| 40 | 40 | */ |
| 41 | 41 | public static function toString($dataType, int $options = null):string |
| 42 | 42 | { |
| 43 | - if (is_null($dataType)) throw new \Exception(self::class.": No content to convert to Yaml", 1); |
|
| 43 | + if (is_null($dataType)) { |
|
| 44 | + throw new \Exception(self::class.": No content to convert to Yaml", 1); |
|
| 45 | + } |
|
| 44 | 46 | self::$options = is_int($options) ? $options : self::OPTIONS; |
| 45 | 47 | self::$result = new DLL; |
| 46 | 48 | if ($dataType instanceof YamlObject) { |
@@ -76,9 +78,13 @@ discard block |
||
| 76 | 78 | if (is_scalar($dataType)) { |
| 77 | 79 | switch (gettype($dataType)) { |
| 78 | 80 | case 'boolean': return $dataType ? 'true' : 'false'; |
| 79 | - case 'float': if (is_infinite($dataType)) return $dataType > 0 ? '.inf' : '-.inf'; |
|
| 81 | + case 'float': if (is_infinite($dataType)) { |
|
| 82 | + return $dataType > 0 ? '.inf' : '-.inf'; |
|
| 83 | + } |
|
| 80 | 84 | return sprintf('%.2F', $dataType); |
| 81 | - case 'double': if (is_nan((float) $dataType)) return '.nan'; |
|
| 85 | + case 'double': if (is_nan((float) $dataType)) { |
|
| 86 | + return '.nan'; |
|
| 87 | + } |
|
| 82 | 88 | default: |
| 83 | 89 | return $dataType; |
| 84 | 90 | } |
@@ -91,7 +97,9 @@ discard block |
||
| 91 | 97 | |
| 92 | 98 | private static function dumpYamlObject(YamlObject $dataType) |
| 93 | 99 | { |
| 94 | - if ($dataType->hasDocStart() && self::$result instanceof DLL) self::$result->push("---"); |
|
| 100 | + if ($dataType->hasDocStart() && self::$result instanceof DLL) { |
|
| 101 | + self::$result->push("---"); |
|
| 102 | + } |
|
| 95 | 103 | // self::dump($dataType, 0); |
| 96 | 104 | if (count($dataType) > 0) { |
| 97 | 105 | self::dumpSequence($dataType->getArrayCopy(), 0); |
@@ -142,9 +150,13 @@ discard block |
||
| 142 | 150 | self::add(self::dump($obj->value, $indent + self::INDENT), $indent + self::INDENT); |
| 143 | 151 | } |
| 144 | 152 | } |
| 145 | - if ($obj instanceof Compact) return self::dumpCompact($obj, $indent); |
|
| 153 | + if ($obj instanceof Compact) { |
|
| 154 | + return self::dumpCompact($obj, $indent); |
|
| 155 | + } |
|
| 146 | 156 | //TODO: consider dumping datetime as date strings according to a format provided by user or default |
| 147 | - if ($obj instanceof \DateTime) return $obj->format('Y-m-d'); |
|
| 157 | + if ($obj instanceof \DateTime) { |
|
| 158 | + return $obj->format('Y-m-d'); |
|
| 159 | + } |
|
| 148 | 160 | // if ($obj instanceof \SplString) {var_dump('splstrin',$obj);return '"'.$obj.'"';} |
| 149 | 161 | $propList = get_object_vars($obj); |
| 150 | 162 | foreach ($propList as $property => $value) { |
@@ -23,8 +23,12 @@ discard block |
||
| 23 | 23 | */ |
| 24 | 24 | public static function get(Node $n) |
| 25 | 25 | { |
| 26 | - if (is_null($n->value)) return null; |
|
| 27 | - if ($n->type & (Y::REF_CALL|Y::SCALAR)) return self::getScalar((string) $n->value); |
|
| 26 | + if (is_null($n->value)) { |
|
| 27 | + return null; |
|
| 28 | + } |
|
| 29 | + if ($n->type & (Y::REF_CALL|Y::SCALAR)) { |
|
| 30 | + return self::getScalar((string) $n->value); |
|
| 31 | + } |
|
| 28 | 32 | if ($n->type & Y::JSON) { |
| 29 | 33 | return $n->value; |
| 30 | 34 | } |
@@ -43,8 +47,12 @@ discard block |
||
| 43 | 47 | */ |
| 44 | 48 | private static function getScalar(string $v) |
| 45 | 49 | { |
| 46 | - if (R::isDate($v)) return date_create($v); |
|
| 47 | - if (R::isNumber($v)) return self::getNumber($v); |
|
| 50 | + if (R::isDate($v)) { |
|
| 51 | + return date_create($v); |
|
| 52 | + } |
|
| 53 | + if (R::isNumber($v)) { |
|
| 54 | + return self::getNumber($v); |
|
| 55 | + } |
|
| 48 | 56 | $types = ['yes' => true, |
| 49 | 57 | 'no' => false, |
| 50 | 58 | 'true' => true, |
@@ -66,8 +74,12 @@ discard block |
||
| 66 | 74 | */ |
| 67 | 75 | private static function getNumber(string $v) |
| 68 | 76 | { |
| 69 | - if (preg_match("/^(0o\d+)$/i", $v)) return intval(base_convert($v, 8, 10)); |
|
| 70 | - if (preg_match("/^(0x[\da-f]+)$/i", $v)) return intval(base_convert($v, 16, 10)); |
|
| 77 | + if (preg_match("/^(0o\d+)$/i", $v)) { |
|
| 78 | + return intval(base_convert($v, 8, 10)); |
|
| 79 | + } |
|
| 80 | + if (preg_match("/^(0x[\da-f]+)$/i", $v)) { |
|
| 81 | + return intval(base_convert($v, 16, 10)); |
|
| 82 | + } |
|
| 71 | 83 | return is_bool(strpos($v, '.')) ? intval($v) : floatval($v); |
| 72 | 84 | } |
| 73 | 85 | } |
| 74 | 86 | \ No newline at end of file |
@@ -33,8 +33,12 @@ |
||
| 33 | 33 | public function jsonSerialize():array |
| 34 | 34 | { |
| 35 | 35 | $prop = get_object_vars($this); |
| 36 | - if (count($prop) > 0) return $prop; |
|
| 37 | - if (count($this) > 0) return iterator_to_array($this); |
|
| 36 | + if (count($prop) > 0) { |
|
| 37 | + return $prop; |
|
| 38 | + } |
|
| 39 | + if (count($this) > 0) { |
|
| 40 | + return iterator_to_array($this); |
|
| 41 | + } |
|
| 38 | 42 | return new \Stdclass; |
| 39 | 43 | } |
| 40 | 44 | |
@@ -42,10 +42,14 @@ discard block |
||
| 42 | 42 | } |
| 43 | 43 | $_root->value instanceof NodeList && $_root->value->setIteratorMode(NodeList::IT_MODE_DELETE); |
| 44 | 44 | foreach ($_root->value as $child) { |
| 45 | - if ($child->type & Y::DOC_START) $totalDocStart++; |
|
| 45 | + if ($child->type & Y::DOC_START) { |
|
| 46 | + $totalDocStart++; |
|
| 47 | + } |
|
| 46 | 48 | //if 0 or 1 DOC_START = we are still in first document |
| 47 | 49 | $currentDoc = $totalDocStart > 1 ? $totalDocStart - 1 : 0; |
| 48 | - if (!isset($documents[$currentDoc])) $documents[$currentDoc] = new NodeList(); |
|
| 50 | + if (!isset($documents[$currentDoc])) { |
|
| 51 | + $documents[$currentDoc] = new NodeList(); |
|
| 52 | + } |
|
| 49 | 53 | $documents[$currentDoc]->push($child); |
| 50 | 54 | } |
| 51 | 55 | $content = []; |
@@ -70,7 +74,9 @@ discard block |
||
| 70 | 74 | */ |
| 71 | 75 | private static function build(object $node, &$parent = null) |
| 72 | 76 | { |
| 73 | - if ($node instanceof NodeList) return self::buildNodeList($node, $parent); |
|
| 77 | + if ($node instanceof NodeList) { |
|
| 78 | + return self::buildNodeList($node, $parent); |
|
| 79 | + } |
|
| 74 | 80 | return self::buildNode($node, $parent); |
| 75 | 81 | } |
| 76 | 82 | |
@@ -133,13 +139,17 @@ discard block |
||
| 133 | 139 | } else { |
| 134 | 140 | $tmp = Node2PHP::get($node); |
| 135 | 141 | } |
| 136 | - if ($type === Y::REF_DEF) self::$_root->addReference($identifier, $tmp); |
|
| 142 | + if ($type === Y::REF_DEF) { |
|
| 143 | + self::$_root->addReference($identifier, $tmp); |
|
| 144 | + } |
|
| 137 | 145 | return self::$_root->getReference($identifier); |
| 138 | 146 | } |
| 139 | 147 | if ($type & (Y::COMPACT_MAPPING|Y::COMPACT_SEQUENCE)) { |
| 140 | 148 | return self::buildNodeList($node->value, $parent); |
| 141 | 149 | } |
| 142 | - if ($type & Y::COMMENT) self::$_root->addComment($node->line, $node->value); |
|
| 150 | + if ($type & Y::COMMENT) { |
|
| 151 | + self::$_root->addComment($node->line, $node->value); |
|
| 152 | + } |
|
| 143 | 153 | $typesActions = [Y::DIRECTIVE => 'buildDirective', |
| 144 | 154 | Y::ITEM => 'buildItem', |
| 145 | 155 | Y::KEY => 'buildKey', |
@@ -254,18 +264,24 @@ discard block |
||
| 254 | 264 | } |
| 255 | 265 | $result = ''; |
| 256 | 266 | $separator = ''; |
| 257 | - if ($type & Y::LITT) $separator = "\n"; |
|
| 258 | - if ($type & Y::LITT_FOLDED) $separator = ' '; |
|
| 267 | + if ($type & Y::LITT) { |
|
| 268 | + $separator = "\n"; |
|
| 269 | + } |
|
| 270 | + if ($type & Y::LITT_FOLDED) { |
|
| 271 | + $separator = ' '; |
|
| 272 | + } |
|
| 259 | 273 | foreach ($list as $child) { |
| 260 | 274 | if ($child->value instanceof NodeList) { |
| 261 | 275 | $result .= self::buildLitteral($child->value, $type).$separator; |
| 262 | 276 | } else { |
| 263 | 277 | $val = $child->type & (Y::SCALAR|Y::BLANK) ? $child->value : substr($child->raw, $refIndent); |
| 264 | 278 | if ($type & Y::LITT_FOLDED && ($child->indent > $refIndent || ($child->type & Y::BLANK))) { |
| 265 | - if ($result[-1] === $separator) |
|
| 266 | - $result[-1] = "\n"; |
|
| 267 | - if ($result[-1] === "\n") |
|
| 268 | - $result .= $val; |
|
| 279 | + if ($result[-1] === $separator) { |
|
| 280 | + $result[-1] = "\n"; |
|
| 281 | + } |
|
| 282 | + if ($result[-1] === "\n") { |
|
| 283 | + $result .= $val; |
|
| 284 | + } |
|
| 269 | 285 | continue; |
| 270 | 286 | } |
| 271 | 287 | $result .= $val.$separator; |