Completed
Push — master ( a7e3b2...3479d6 )
by Tom
03:17
created
src/Hook/BasicProperties.php 1 patch
Braces   +23 added lines, -9 removed lines patch added patch discarded remove patch
@@ -22,8 +22,11 @@  discard block
 block discarded – undo
22 22
 			//Remove the current contents
23 23
 			$this->removeAllChildren($element);
24 24
 			//Now make a text node
25
-			if ($this->getContentMode($rule->getRules()) === 'replace') $this->replaceContent($element, $value);
26
-			else $this->appendContent($element, $value);
25
+			if ($this->getContentMode($rule->getRules()) === 'replace') {
26
+				$this->replaceContent($element, $value);
27
+			} else {
28
+				$this->appendContent($element, $value);
29
+			}
27 30
 		}
28 31
 	}
29 32
 
@@ -69,8 +72,9 @@  discard block
 block discarded – undo
69 72
 				$node = $element->ownerDocument->importNode($node, true);
70 73
 				$element->parentNode->appendChild($node);
71 74
 			}
75
+		} else {
76
+			$element->parentNode->appendChild($element->ownerDocument->createElement('span', implode('', $content)));
72 77
 		}
73
-		else $element->parentNode->appendChild($element->ownerDocument->createElement('span', implode('', $content)));
74 78
 
75 79
 		$element->setAttribute('transphporm', 'remove');
76 80
 	}
@@ -81,22 +85,29 @@  discard block
 block discarded – undo
81 85
 				$node = $element->ownerDocument->importNode($node, true);
82 86
 				$element->appendChild($node);
83 87
 			}
88
+		} else {
89
+			$element->appendChild($element->ownerDocument->createTextNode(implode('', $content)));
84 90
 		}
85
-		else $element->appendChild($element->ownerDocument->createTextNode(implode('', $content)));		
86 91
 	}
87 92
 
88 93
 	private function removeAllChildren($element) {
89
-		while ($element->hasChildNodes()) $element->removeChild($element->firstChild);
94
+		while ($element->hasChildNodes()) {
95
+			$element->removeChild($element->firstChild);
96
+		}
90 97
 	}
91 98
 
92 99
 	private function createHook($newRules, $rule) {
93 100
 		$hook = new Rule($newRules, $rule->getPseudoMatcher(), $this->data);
94
-		foreach ($rule->getProperties() as $obj) $hook->registerProperties($obj);
101
+		foreach ($rule->getProperties() as $obj) {
102
+			$hook->registerProperties($obj);
103
+		}
95 104
 		return $hook;
96 105
 	}
97 106
 
98 107
 	public function repeat($value, $element, $rule) {
99
-		if ($element->getAttribute('transphporm') === 'added') return $element->parentNode->removeChild($element);
108
+		if ($element->getAttribute('transphporm') === 'added') {
109
+			return $element->parentNode->removeChild($element);
110
+		}
100 111
 
101 112
 		foreach ($value as $key => $iteration) {
102 113
 			$clone = $element->cloneNode(true);
@@ -119,8 +130,11 @@  discard block
 block discarded – undo
119 130
 	}
120 131
 
121 132
 	public function display($value, $element) {
122
-		if (strtolower($value[0]) === 'none') $element->setAttribute('transphporm', 'remove');
123
-		else $element->setAttribute('transphporm', 'show');
133
+		if (strtolower($value[0]) === 'none') {
134
+			$element->setAttribute('transphporm', 'remove');
135
+		} else {
136
+			$element->setAttribute('transphporm', 'show');
137
+		}
124 138
 	}
125 139
 
126 140
 	public function bind($value, $element) {
Please login to merge, or discard this patch.