Passed
Pull Request — master (#87)
by Glynn
02:29
created
src/Services/View/PHP_Engine.php 1 patch
Spacing   +48 added lines, -48 removed lines patch added patch discarded remove patch
@@ -51,8 +51,8 @@  discard block
 block discarded – undo
51 51
 	 *
52 52
 	 * @param string $base_view_path
53 53
 	 */
54
-	public function __construct( string $base_view_path ) {
55
-		$this->base_view_path = $this->verify_view_path( $base_view_path );
54
+	public function __construct(string $base_view_path) {
55
+		$this->base_view_path = $this->verify_view_path($base_view_path);
56 56
 	}
57 57
 
58 58
 	/**
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 	 * @param Component_Compiler $compiler
62 62
 	 * @return void
63 63
 	 */
64
-	public function set_component_compiler( Component_Compiler $compiler ): void {
64
+	public function set_component_compiler(Component_Compiler $compiler): void {
65 65
 		$this->component_compiler = $compiler;
66 66
 	}
67 67
 
@@ -73,12 +73,12 @@  discard block
 block discarded – undo
73 73
 	 * @param bool $print
74 74
 	 * @return string|void
75 75
 	 */
76
-	public function render( string $view, iterable $data, bool $print = true ) {
77
-		$view = $this->resolve_file_path( $view );
78
-		if ( $print ) {
79
-			print( $this->render_buffer( $view, $data ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
76
+	public function render(string $view, iterable $data, bool $print = true) {
77
+		$view = $this->resolve_file_path($view);
78
+		if ($print) {
79
+			print($this->render_buffer($view, $data)); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
80 80
 		} else {
81
-			return $this->render_buffer( $view, $data );
81
+			return $this->render_buffer($view, $data);
82 82
 		}
83 83
 	}
84 84
 
@@ -88,20 +88,20 @@  discard block
 block discarded – undo
88 88
 	 * @param Component $component
89 89
 	 * @return string|void
90 90
 	 */
91
-	public function component( Component $component, bool $print = true ) {
91
+	public function component(Component $component, bool $print = true) {
92 92
 
93 93
 		// Throw exception of no compiler passed.
94
-		if ( ! is_a( $this->component_compiler, Component_Compiler::class ) ) {
95
-			throw new Exception( 'No component compiler passed to PHP_Engine' );
94
+		if ( ! is_a($this->component_compiler, Component_Compiler::class)) {
95
+			throw new Exception('No component compiler passed to PHP_Engine');
96 96
 		}
97 97
 
98 98
 		// Compile the component.
99
-		$compiled = $this->component_compiler->compile( $component );
100
-		$view     = sprintf( '%s%s.php', \DIRECTORY_SEPARATOR, $this->clean_filename( $compiled->template() ) );
101
-		if ( $print ) {
102
-			print( $this->render_buffer( $view, $compiled->data() ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
99
+		$compiled = $this->component_compiler->compile($component);
100
+		$view     = sprintf('%s%s.php', \DIRECTORY_SEPARATOR, $this->clean_filename($compiled->template()));
101
+		if ($print) {
102
+			print($this->render_buffer($view, $compiled->data())); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
103 103
 		} else {
104
-			return $this->render_buffer( $view, $compiled->data() );
104
+			return $this->render_buffer($view, $compiled->data());
105 105
 		}
106 106
 	}
107 107
 
@@ -112,8 +112,8 @@  discard block
 block discarded – undo
112 112
 	 * @param View_Model $view_model
113 113
 	 * @return string|void
114 114
 	 */
115
-	public function view_model( View_Model $view_model, bool $print = true ) {
116
-		return $this->render( $view_model->template(), $view_model->data(), $print );
115
+	public function view_model(View_Model $view_model, bool $print = true) {
116
+		return $this->render($view_model->template(), $view_model->data(), $print);
117 117
 	}
118 118
 
119 119
 	/**
@@ -124,11 +124,11 @@  discard block
 block discarded – undo
124 124
 	 * @param bool $print
125 125
 	 * @return string|void
126 126
 	 */
127
-	public function partial( string $view, iterable $data = array(), bool $print = true ) {
128
-		if ( $print ) {
129
-			$this->render( $view, $data, $print );
127
+	public function partial(string $view, iterable $data = array(), bool $print = true) {
128
+		if ($print) {
129
+			$this->render($view, $data, $print);
130 130
 		} else {
131
-			return $this->render( $view, $data, $print );
131
+			return $this->render($view, $data, $print);
132 132
 		}
133 133
 	}
134 134
 
@@ -140,23 +140,23 @@  discard block
 block discarded – undo
140 140
 	 * @return string
141 141
 	 * @throws Exception
142 142
 	 */
143
-	protected function render_buffer( string $view, iterable $__data ): string {
143
+	protected function render_buffer(string $view, iterable $__data): string {
144 144
 
145
-		if ( ! file_exists( $view ) ) {
146
-			throw new Exception( "{$view} doesn't exist" );
145
+		if ( ! file_exists($view)) {
146
+			throw new Exception("{$view} doesn't exist");
147 147
 		}
148 148
 
149 149
 		$output = '';
150 150
 		ob_start();
151 151
 
152 152
 		// Set all the data values a parameters.
153
-		foreach ( $__data as $__key => $__value ) {
154
-			if ( is_string( $__key ) ) {
155
-				${\wp_strip_all_tags( $__key )} = $__value;
153
+		foreach ($__data as $__key => $__value) {
154
+			if (is_string($__key)) {
155
+				${\wp_strip_all_tags($__key)} = $__value;
156 156
 			}
157 157
 
158 158
 			// Unset the key and value.
159
-			unset( $__key, $__value, $__data );
159
+			unset($__key, $__value, $__data);
160 160
 		}
161 161
 
162 162
 		include $view;
@@ -171,10 +171,10 @@  discard block
 block discarded – undo
171 171
 	 * @param string $file
172 172
 	 * @return string
173 173
 	 */
174
-	protected function clean_filename( string $file ): string {
175
-		$file = ltrim( $file, '/' );
176
-		return substr( $file, -4 ) === '.php'
177
-			? substr( $file, 0, -4 )
174
+	protected function clean_filename(string $file): string {
175
+		$file = ltrim($file, '/');
176
+		return substr($file, -4) === '.php'
177
+			? substr($file, 0, -4)
178 178
 			: $file;
179 179
 
180 180
 	}
@@ -185,12 +185,12 @@  discard block
 block discarded – undo
185 185
 	 * @param string $filename
186 186
 	 * @return string
187 187
 	 */
188
-	protected function resolve_file_path( string $filename ): string {
189
-		$filename = $this->maybe_resolve_dot_notation( $filename );
188
+	protected function resolve_file_path(string $filename): string {
189
+		$filename = $this->maybe_resolve_dot_notation($filename);
190 190
 		return sprintf(
191 191
 			'%s%s.php',
192 192
 			$this->base_view_path,
193
-			$this->clean_filename( $filename )
193
+			$this->clean_filename($filename)
194 194
 		);
195 195
 	}
196 196
 
@@ -200,13 +200,13 @@  discard block
 block discarded – undo
200 200
 	 * @param string $filename
201 201
 	 * @return string
202 202
 	 */
203
-	protected function maybe_resolve_dot_notation( string $filename ): string {
204
-		if ( $this->str_ends_with( '.php', $filename ) ) {
205
-			$filename = substr( $filename, 0, -4 );
203
+	protected function maybe_resolve_dot_notation(string $filename): string {
204
+		if ($this->str_ends_with('.php', $filename)) {
205
+			$filename = substr($filename, 0, -4);
206 206
 		}
207 207
 
208
-		$parts    = explode( '.', $filename );
209
-		$filename = implode( DIRECTORY_SEPARATOR, $parts );
208
+		$parts    = explode('.', $filename);
209
+		$filename = implode(DIRECTORY_SEPARATOR, $parts);
210 210
 
211 211
 		return $filename;
212 212
 	}
@@ -218,9 +218,9 @@  discard block
 block discarded – undo
218 218
 	 * @param string $haystack
219 219
 	 * @return bool
220 220
 	 */
221
-	protected function str_ends_with( string $needle, string $haystack ): bool {
222
-		$needle_len = strlen( $needle );
223
-		return ( $needle_len === 0 || 0 === substr_compare( $haystack, $needle, - $needle_len ) );
221
+	protected function str_ends_with(string $needle, string $haystack): bool {
222
+		$needle_len = strlen($needle);
223
+		return ($needle_len === 0 || 0 === substr_compare($haystack, $needle, - $needle_len));
224 224
 	}
225 225
 
226 226
 
@@ -231,12 +231,12 @@  discard block
 block discarded – undo
231 231
 	 * @return string
232 232
 	 * @throws Exception
233 233
 	 */
234
-	protected function verify_view_path( string $path ): string {
234
+	protected function verify_view_path(string $path): string {
235 235
 
236
-		$path = rtrim( $path, '/' ) . '/';
236
+		$path = rtrim($path, '/') . '/';
237 237
 
238
-		if ( ! \is_dir( $path ) ) {
239
-			throw new Exception( "{$path} doesn't exist and cant be used as the base view path." );
238
+		if ( ! \is_dir($path)) {
239
+			throw new Exception("{$path} doesn't exist and cant be used as the base view path.");
240 240
 		}
241 241
 
242 242
 		return $path;
Please login to merge, or discard this patch.