Passed
Pull Request — master (#99)
by Glynn
04:13
created
src/Services/View/View.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -61,12 +61,12 @@  discard block
 block discarded – undo
61 61
 	 *
62 62
 	 * @param Renderable $engine
63 63
 	 */
64
-	public function __construct( Renderable $engine, Component_Compiler $component_compiler ) {
64
+	public function __construct(Renderable $engine, Component_Compiler $component_compiler) {
65 65
 		$this->engine             = $engine;
66 66
 		$this->component_compiler = $component_compiler;
67 67
 
68 68
 		// Populate engine with compiler.
69
-		$this->engine->set_component_compiler( $component_compiler );
69
+		$this->engine->set_component_compiler($component_compiler);
70 70
 	}
71 71
 
72 72
 	/**
@@ -77,8 +77,8 @@  discard block
 block discarded – undo
77 77
 	 * @param bool $print Print or Return the HTML
78 78
 	 * @return string|void
79 79
 	 */
80
-	public function render( string $view, iterable $view_data = array(), bool $print = self::PRINT_VIEW ) {
81
-		return $this->engine->render( $view, $view_data, $print );
80
+	public function render(string $view, iterable $view_data = array(), bool $print = self::PRINT_VIEW) {
81
+		return $this->engine->render($view, $view_data, $print);
82 82
 	}
83 83
 
84 84
 	/**
@@ -88,8 +88,8 @@  discard block
 block discarded – undo
88 88
 	 * @param bool $print Print or Return the HTML
89 89
 	 * @return string|void
90 90
 	 */
91
-	public function component( Component $component, bool $print = self::PRINT_VIEW ) {
92
-		return $this->engine->component( $component, $print );
91
+	public function component(Component $component, bool $print = self::PRINT_VIEW) {
92
+		return $this->engine->component($component, $print);
93 93
 	}
94 94
 
95 95
 	/**
@@ -99,8 +99,8 @@  discard block
 block discarded – undo
99 99
 	 * @param bool $print Print or Return the HTML
100 100
 	 * @return string|void
101 101
 	 */
102
-	public function view_model( View_Model $view_model, bool $print = self::PRINT_VIEW ) {
103
-		return $this->engine->view_model( $view_model, $print );
102
+	public function view_model(View_Model $view_model, bool $print = self::PRINT_VIEW) {
103
+		return $this->engine->view_model($view_model, $print);
104 104
 	}
105 105
 
106 106
 	/**
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 	 * @param callable $to_buffer
110 110
 	 * @return string
111 111
 	 */
112
-	public static function print_buffer( callable $to_buffer ): string {
112
+	public static function print_buffer(callable $to_buffer): string {
113 113
 		ob_start();
114 114
 		$to_buffer();
115 115
 		$output = ob_get_contents();
Please login to merge, or discard this patch.
src/Services/Registration/Registration_Service.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@  discard block
 block discarded – undo
65 65
 	 * @param DI_Container $di_container
66 66
 	 * @return self
67 67
 	 */
68
-	public function set_container( DI_Container $di_container ): self {
68
+	public function set_container(DI_Container $di_container): self {
69 69
 		$this->di_container = $di_container;
70 70
 		return $this;
71 71
 	}
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
 	 * @param Hook_Loader $loader
77 77
 	 * @return self
78 78
 	 */
79
-	public function set_loader( Hook_Loader $loader ): self {
79
+	public function set_loader(Hook_Loader $loader): self {
80 80
 		$this->loader = $loader;
81 81
 		return $this;
82 82
 	}
@@ -87,8 +87,8 @@  discard block
 block discarded – undo
87 87
 	 * @param Registration_Middleware $middleware
88 88
 	 * @return self
89 89
 	 */
90
-	public function push_middleware( Registration_Middleware $middleware ): self {
91
-		$this->middleware[ \get_class( $middleware ) ] = $middleware;
90
+	public function push_middleware(Registration_Middleware $middleware): self {
91
+		$this->middleware[\get_class($middleware)] = $middleware;
92 92
 		return $this;
93 93
 	}
94 94
 
@@ -98,7 +98,7 @@  discard block
 block discarded – undo
98 98
 	 * @param array<string> $class_list
99 99
 	 * @return self
100 100
 	 */
101
-	public function set_classes( array $class_list ): self {
101
+	public function set_classes(array $class_list): self {
102 102
 		$this->class_list = $class_list;
103 103
 		return $this;
104 104
 	}
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
 	 * @param string $class
110 110
 	 * @return self
111 111
 	 */
112
-	public function push_class( string $class ): self {
112
+	public function push_class(string $class): self {
113 113
 		$this->class_list[] = $class;
114 114
 		return $this;
115 115
 	}
@@ -122,32 +122,32 @@  discard block
 block discarded – undo
122 122
 	public function process(): void {
123 123
 
124 124
 		// Filter all classes, before processing.
125
-		$class_list = apply_filters( Hooks::APP_INIT_REGISTRATION_CLASS_LIST, $this->class_list );
125
+		$class_list = apply_filters(Hooks::APP_INIT_REGISTRATION_CLASS_LIST, $this->class_list);
126 126
 
127
-		foreach ( $this->middleware as $middleware ) {
127
+		foreach ($this->middleware as $middleware) {
128 128
 
129 129
 			// Set the container if requested.
130
-			if ( is_object( $middleware ) && \method_exists( $middleware, 'set_di_container' ) ) {
131
-				$middleware->set_di_container( $this->di_container );
130
+			if (is_object($middleware) && \method_exists($middleware, 'set_di_container')) {
131
+				$middleware->set_di_container($this->di_container);
132 132
 			}
133 133
 
134 134
 			// Set the hook loader if requested.
135
-			if ( is_object( $middleware ) && \method_exists( $middleware, 'set_hook_loader' ) && ! is_null( $this->loader ) ) {
136
-				$middleware->set_hook_loader( $this->loader );
135
+			if (is_object($middleware) && \method_exists($middleware, 'set_hook_loader') && ! is_null($this->loader)) {
136
+				$middleware->set_hook_loader($this->loader);
137 137
 			}
138 138
 
139 139
 			// Run middleware setup
140 140
 			$middleware->setup();
141 141
 
142 142
 			// Pass each class to the middleware.
143
-			foreach ( $class_list as $class ) {
143
+			foreach ($class_list as $class) {
144 144
 
145 145
 				// Construct class using container,
146
-				$class_instance = $this->di_container->create( $class );
146
+				$class_instance = $this->di_container->create($class);
147 147
 
148 148
 				// if valid object process via current middleware
149
-				if ( is_object( $class_instance ) ) {
150
-					$middleware->process( $class_instance );
149
+				if (is_object($class_instance)) {
150
+					$middleware->process($class_instance);
151 151
 				}
152 152
 			}
153 153
 
Please login to merge, or discard this patch.
src/Services/View/Component/Component_Compiler.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -50,9 +50,9 @@  discard block
 block discarded – undo
50 50
 	private $component_aliases = array();
51 51
 
52 52
 	/** @param array<string, string> $component_aliases */
53
-	public function __construct( string $component_base_path = '', array $component_aliases = array() ) {
53
+	public function __construct(string $component_base_path = '', array $component_aliases = array()) {
54 54
 		$this->component_base_path = $component_base_path;
55
-		$this->component_aliases   = \apply_filters( Hooks::COMPONENT_ALIASES, $component_aliases );
55
+		$this->component_aliases   = \apply_filters(Hooks::COMPONENT_ALIASES, $component_aliases);
56 56
 	}
57 57
 
58 58
 	/**
@@ -61,8 +61,8 @@  discard block
 block discarded – undo
61 61
 	 * @param Component $component
62 62
 	 * @return View_Model
63 63
 	 */
64
-	public function compile( Component $component ): View_Model {
65
-		return new View_Model( $this->get_component_path( $component ), $component->get_variables() );
64
+	public function compile(Component $component): View_Model {
65
+		return new View_Model($this->get_component_path($component), $component->get_variables());
66 66
 	}
67 67
 
68 68
 	/**
@@ -71,34 +71,34 @@  discard block
 block discarded – undo
71 71
 	 * @param Component $component
72 72
 	 * @return string
73 73
 	 */
74
-	private function get_component_path( Component $component ): string {
74
+	private function get_component_path(Component $component): string {
75 75
 
76 76
 		// Check aliases.
77
-		$aliases = \apply_filters( Hooks::COMPONENT_ALIASES, $this->component_aliases );
77
+		$aliases = \apply_filters(Hooks::COMPONENT_ALIASES, $this->component_aliases);
78 78
 
79
-		if ( isset( $aliases[ get_class( $component ) ] ) ) {
80
-			return esc_attr( $aliases[ get_class( $component ) ] );
79
+		if (isset($aliases[get_class($component)])) {
80
+			return esc_attr($aliases[get_class($component)]);
81 81
 		}
82 82
 
83
-		$from_annotation = $this->get_annotation( 'view', $component );
83
+		$from_annotation = $this->get_annotation('view', $component);
84 84
 		// If it does have a path defined, use that.
85
-		if ( ! empty( $from_annotation ) ) {
86
-			return \trailingslashit( $this->component_base_path ) . $from_annotation;
85
+		if ( ! empty($from_annotation)) {
86
+			return \trailingslashit($this->component_base_path) . $from_annotation;
87 87
 		}
88 88
 
89 89
 		// If the component has a defined path
90
-		if ( $component->template() ) {
91
-			return \trailingslashit( $this->component_base_path ) . $component->template();
90
+		if ($component->template()) {
91
+			return \trailingslashit($this->component_base_path) . $component->template();
92 92
 		}
93 93
 
94 94
 		// Get path based on class name.
95
-		$reflect    = new \ReflectionClass( $component );
95
+		$reflect    = new \ReflectionClass($component);
96 96
 		$short_name = $reflect->getShortName();
97 97
 		// Add space between capitals, make lowercase and replace underscores with dashes.
98
-		$short_name = strtolower( preg_replace( '/(?<!^)[A-Z]/', '-$0', $short_name ) ?? '' );
99
-		$short_name = str_replace( '_', '-', $short_name );
98
+		$short_name = strtolower(preg_replace('/(?<!^)[A-Z]/', '-$0', $short_name) ?? '');
99
+		$short_name = str_replace('_', '-', $short_name);
100 100
 
101
-		return \trailingslashit( $this->component_base_path ) . $short_name;
101
+		return \trailingslashit($this->component_base_path) . $short_name;
102 102
 	}
103 103
 
104 104
 	/**
@@ -108,18 +108,18 @@  discard block
 block discarded – undo
108 108
 	 * @param Component $component
109 109
 	 * @return string|null
110 110
 	 */
111
-	private function get_annotation( string $annotation, Component $component ): ?string {
112
-		$reflect = new \ReflectionClass( $component );
111
+	private function get_annotation(string $annotation, Component $component): ?string {
112
+		$reflect = new \ReflectionClass($component);
113 113
 		$comment = $reflect->getDocComment();
114 114
 
115 115
 		// if no comment, return null.
116
-		if ( empty( $comment ) ) {
116
+		if (empty($comment)) {
117 117
 			return null;
118 118
 		}
119 119
 
120 120
 		// If comment contains the annotation, return the value.
121
-		return strpos( $comment, "@{$annotation}" ) !== false
122
-			? $this->extract_annotation_value( $comment, $annotation )
121
+		return strpos($comment, "@{$annotation}") !== false
122
+			? $this->extract_annotation_value($comment, $annotation)
123 123
 			: null;
124 124
 	}
125 125
 
@@ -130,9 +130,9 @@  discard block
 block discarded – undo
130 130
 	 * @param string $annotation
131 131
 	 * @return string|null
132 132
 	 */
133
-	private function extract_annotation_value( string $comment, string $annotation ): ?string {
133
+	private function extract_annotation_value(string $comment, string $annotation): ?string {
134 134
 		$pattern = "/@{$annotation}\s+(.*)/";
135
-		preg_match( $pattern, $comment, $matches );
135
+		preg_match($pattern, $comment, $matches);
136 136
 		return $matches[1] ?? null;
137 137
 	}
138 138
 
Please login to merge, or discard this patch.
src/Application/App_Config.php 1 patch
Spacing   +67 added lines, -67 removed lines patch added patch discarded remove patch
@@ -88,9 +88,9 @@  discard block
 block discarded – undo
88 88
 	/**
89 89
 	 * @param array<string, mixed> $settings
90 90
 	 */
91
-	public function __construct( array $settings = array() ) {
92
-		$settings = $this->set_defaults( $settings );
93
-		$this->set_props( $settings );
91
+	public function __construct(array $settings = array()) {
92
+		$settings = $this->set_defaults($settings);
93
+		$this->set_props($settings);
94 94
 	}
95 95
 
96 96
 	/**
@@ -99,8 +99,8 @@  discard block
 block discarded – undo
99 99
 	 * @param array<string, mixed> $settings
100 100
 	 * @return array<string, mixed>
101 101
 	 */
102
-	private function set_defaults( array $settings ): array {
103
-		return array_replace_recursive( $this->settings_defaults(), $settings );
102
+	private function set_defaults(array $settings): array {
103
+		return array_replace_recursive($this->settings_defaults(), $settings);
104 104
 	}
105 105
 
106 106
 	/**
@@ -109,17 +109,17 @@  discard block
 block discarded – undo
109 109
 	 * @param array<string, mixed> $settings
110 110
 	 * @return void
111 111
 	 */
112
-	private function set_props( array $settings ): void {
112
+	private function set_props(array $settings): void {
113 113
 		$this->paths['url']  = $settings['url'];
114 114
 		$this->paths['path'] = $settings['path'];
115
-		$this->namespaces    = $this->filter_key_value_pair( $settings['namespaces'] );
115
+		$this->namespaces    = $this->filter_key_value_pair($settings['namespaces']);
116 116
 		$this->plugin        = $settings['plugin'];
117 117
 		$this->additional    = $settings['additional'];
118
-		$this->db_tables     = $this->filter_key_value_pair( $settings['db_tables'] );
119
-		$this->post_types    = $this->filter_key_value_pair( $settings['post_types'] );
120
-		$this->taxonomies    = $this->filter_key_value_pair( $settings['taxonomies'] );
118
+		$this->db_tables     = $this->filter_key_value_pair($settings['db_tables']);
119
+		$this->post_types    = $this->filter_key_value_pair($settings['post_types']);
120
+		$this->taxonomies    = $this->filter_key_value_pair($settings['taxonomies']);
121 121
 
122
-		$this->set_meta( $settings['meta'] );
122
+		$this->set_meta($settings['meta']);
123 123
 	}
124 124
 
125 125
 	/**
@@ -129,14 +129,14 @@  discard block
 block discarded – undo
129 129
 	 * @param string|null $default The default value to return if not set.
130 130
 	 * @return array<string, mixed>|string|null
131 131
 	 */
132
-	public function path( ?string $path = null, ?string $default = null ) {
132
+	public function path(?string $path = null, ?string $default = null) {
133 133
 
134
-		if ( is_null( $path ) ) {
134
+		if (is_null($path)) {
135 135
 			return $this->paths['path'];
136 136
 		}
137 137
 
138
-		return \array_key_exists( $path, $this->paths['path'] )
139
-			? trailingslashit( $this->paths['path'][ $path ] )
138
+		return \array_key_exists($path, $this->paths['path'])
139
+			? trailingslashit($this->paths['path'][$path])
140 140
 			: $default;
141 141
 	}
142 142
 
@@ -147,14 +147,14 @@  discard block
 block discarded – undo
147 147
 	 * @param string|null $default The default value to return if not set.
148 148
 	 * @return array<string, mixed>|string|null
149 149
 	 */
150
-	public function url( ?string $url = null, ?string $default = null ) {
150
+	public function url(?string $url = null, ?string $default = null) {
151 151
 
152
-		if ( is_null( $url ) ) {
152
+		if (is_null($url)) {
153 153
 			return $this->paths['url'];
154 154
 		}
155 155
 
156
-		return \array_key_exists( $url, $this->paths['url'] )
157
-			? trailingslashit( $this->paths['url'][ $url ] )
156
+		return \array_key_exists($url, $this->paths['url'])
157
+			? trailingslashit($this->paths['url'][$url])
158 158
 			: $default;
159 159
 	}
160 160
 
@@ -183,9 +183,9 @@  discard block
 block discarded – undo
183 183
 	 * @param string|null $default The default value to return if not set.
184 184
 	 * @return string|null
185 185
 	 */
186
-	public function namespace( string $key, ?string $default = null ): ?string {
187
-		return array_key_exists( $key, $this->namespaces )
188
-			? $this->namespaces[ $key ] : $default;
186
+	public function namespace(string $key, ?string $default = null) : ?string {
187
+		return array_key_exists($key, $this->namespaces)
188
+			? $this->namespaces[$key] : $default;
189 189
 	}
190 190
 
191 191
 	/**
@@ -195,9 +195,9 @@  discard block
 block discarded – undo
195 195
 	 * @param string|null $default The default value to return if not set.
196 196
 	 * @return mixed
197 197
 	 */
198
-	public function additional( string $key, ?string $default = null ) {
199
-		return array_key_exists( $key, $this->additional )
200
-			? $this->additional[ $key ] : $default;
198
+	public function additional(string $key, ?string $default = null) {
199
+		return array_key_exists($key, $this->additional)
200
+			? $this->additional[$key] : $default;
201 201
 	}
202 202
 
203 203
 	/**
@@ -216,12 +216,12 @@  discard block
 block discarded – undo
216 216
 	 * @return string
217 217
 	 * @throws OutOfBoundsException
218 218
 	 */
219
-	public function post_types( string $key ) {
220
-		if ( ! array_key_exists( $key, $this->post_types ) ) {
221
-			throw new OutOfBoundsException( "App Config :: \"{$key}\" is not a defined post type" );
219
+	public function post_types(string $key) {
220
+		if ( ! array_key_exists($key, $this->post_types)) {
221
+			throw new OutOfBoundsException("App Config :: \"{$key}\" is not a defined post type");
222 222
 		}
223 223
 
224
-		return $this->post_types[ $key ];
224
+		return $this->post_types[$key];
225 225
 	}
226 226
 
227 227
 	/**
@@ -232,17 +232,17 @@  discard block
 block discarded – undo
232 232
 	 * @return string
233 233
 	 * @throws OutOfBoundsException
234 234
 	 */
235
-	public function meta( string $key, string $type = self::POST_META ): string {
235
+	public function meta(string $key, string $type = self::POST_META): string {
236 236
 		// Check meta type.
237
-		if ( ! array_key_exists( $type, $this->meta ) ) {
238
-			throw new OutOfBoundsException( "App Config :: \"{$type}\" is not a valid meta type and cant be fetched" );
237
+		if ( ! array_key_exists($type, $this->meta)) {
238
+			throw new OutOfBoundsException("App Config :: \"{$type}\" is not a valid meta type and cant be fetched");
239 239
 		}
240 240
 		// Check key.
241
-		if ( ! array_key_exists( $key, $this->meta[ $type ] ) ) {
242
-			throw new OutOfBoundsException( "App Config :: \"{$key}\" is not a defined {$type} meta key" );
241
+		if ( ! array_key_exists($key, $this->meta[$type])) {
242
+			throw new OutOfBoundsException("App Config :: \"{$key}\" is not a defined {$type} meta key");
243 243
 		}
244 244
 
245
-		return $this->meta[ $type ][ $key ];
245
+		return $this->meta[$type][$key];
246 246
 	}
247 247
 
248 248
 	/**
@@ -252,8 +252,8 @@  discard block
 block discarded – undo
252 252
 	 * @param string $key
253 253
 	 * @return string
254 254
 	 */
255
-	public function post_meta( string $key ): string {
256
-		return $this->meta( $key, self::POST_META );
255
+	public function post_meta(string $key): string {
256
+		return $this->meta($key, self::POST_META);
257 257
 	}
258 258
 
259 259
 	/**
@@ -263,8 +263,8 @@  discard block
 block discarded – undo
263 263
 	 * @param string $key
264 264
 	 * @return string
265 265
 	 */
266
-	public function user_meta( string $key ): string {
267
-		return $this->meta( $key, self::USER_META );
266
+	public function user_meta(string $key): string {
267
+		return $this->meta($key, self::USER_META);
268 268
 	}
269 269
 
270 270
 	/**
@@ -274,8 +274,8 @@  discard block
 block discarded – undo
274 274
 	 * @param string $key
275 275
 	 * @return string
276 276
 	 */
277
-	public function term_meta( string $key ): string {
278
-		return $this->meta( $key, self::TERM_META );
277
+	public function term_meta(string $key): string {
278
+		return $this->meta($key, self::TERM_META);
279 279
 	}
280 280
 
281 281
 	/**
@@ -284,15 +284,15 @@  discard block
 block discarded – undo
284 284
 	 * @param array<string, array<string,string>> $meta
285 285
 	 * @return void
286 286
 	 */
287
-	public function set_meta( array $meta ): void {
288
-		$valid_meta_types = array( self::POST_META, self::USER_META, self::TERM_META );
289
-		foreach ( $meta as $meta_type => $pairs ) {
290
-			if ( ! in_array( $meta_type, $valid_meta_types, true ) ) {
291
-				throw new OutOfBoundsException( "App Config :: \"{$meta_type}\" is not a valid meta type and cant be defined" );
287
+	public function set_meta(array $meta): void {
288
+		$valid_meta_types = array(self::POST_META, self::USER_META, self::TERM_META);
289
+		foreach ($meta as $meta_type => $pairs) {
290
+			if ( ! in_array($meta_type, $valid_meta_types, true)) {
291
+				throw new OutOfBoundsException("App Config :: \"{$meta_type}\" is not a valid meta type and cant be defined");
292 292
 			}
293 293
 
294 294
 			// Set all pairs which have both valid key and values.
295
-			$this->meta[ $meta_type ] = $this->filter_key_value_pair( $pairs );
295
+			$this->meta[$meta_type] = $this->filter_key_value_pair($pairs);
296 296
 		}
297 297
 	}
298 298
 
@@ -303,12 +303,12 @@  discard block
 block discarded – undo
303 303
 	 * @return string
304 304
 	 * @throws OutOfBoundsException
305 305
 	 */
306
-	public function taxonomies( string $key ): string {
307
-		if ( ! array_key_exists( $key, $this->taxonomies ) ) {
308
-			throw new OutOfBoundsException( "App Config :: \"{$key}\" is not a defined taxonomy" );
306
+	public function taxonomies(string $key): string {
307
+		if ( ! array_key_exists($key, $this->taxonomies)) {
308
+			throw new OutOfBoundsException("App Config :: \"{$key}\" is not a defined taxonomy");
309 309
 		}
310 310
 
311
-		return $this->taxonomies[ $key ];
311
+		return $this->taxonomies[$key];
312 312
 	}
313 313
 
314 314
 
@@ -319,11 +319,11 @@  discard block
 block discarded – undo
319 319
 	 * @return string
320 320
 	 * @throws OutOfBoundsException
321 321
 	 */
322
-	public function db_tables( string $name ): string {
323
-		if ( ! array_key_exists( $name, $this->db_tables ) ) {
324
-			throw new OutOfBoundsException( "App Config :: \"{$name}\" is not a defined DB table" );
322
+	public function db_tables(string $name): string {
323
+		if ( ! array_key_exists($name, $this->db_tables)) {
324
+			throw new OutOfBoundsException("App Config :: \"{$name}\" is not a defined DB table");
325 325
 		}
326
-		return $this->db_tables[ $name ];
326
+		return $this->db_tables[$name];
327 327
 	}
328 328
 
329 329
 	/**
@@ -332,8 +332,8 @@  discard block
 block discarded – undo
332 332
 	 * @param string $name
333 333
 	 * @return mixed
334 334
 	 */
335
-	public function __get( $name ) {
336
-		return $this->additional( $name );
335
+	public function __get($name) {
336
+		return $this->additional($name);
337 337
 	}
338 338
 
339 339
 	/**
@@ -342,8 +342,8 @@  discard block
 block discarded – undo
342 342
 	 * @return array<string, mixed>
343 343
 	 */
344 344
 	private function settings_defaults(): array {
345
-		$base_path  = \dirname( __DIR__, 2 );
346
-		$plugin_dir = \basename( $base_path );
345
+		$base_path  = \dirname(__DIR__, 2);
346
+		$plugin_dir = \basename($base_path);
347 347
 		$wp_uploads = \wp_upload_dir();
348 348
 
349 349
 		return array(
@@ -358,9 +358,9 @@  discard block
 block discarded – undo
358 358
 				'upload_current' => $wp_uploads['path'],
359 359
 			),
360 360
 			'url'        => array(
361
-				'plugin'         => plugins_url( $plugin_dir ),
362
-				'view'           => plugins_url( $plugin_dir ) . '/views',
363
-				'assets'         => plugins_url( $plugin_dir ) . '/assets',
361
+				'plugin'         => plugins_url($plugin_dir),
362
+				'view'           => plugins_url($plugin_dir) . '/views',
363
+				'assets'         => plugins_url($plugin_dir) . '/assets',
364 364
 				'upload_root'    => $wp_uploads['baseurl'],
365 365
 				'upload_current' => $wp_uploads['url'],
366 366
 			),
@@ -386,14 +386,14 @@  discard block
 block discarded – undo
386 386
 	 * @param array<int|string, mixed> $pairs
387 387
 	 * @return array<string, string>
388 388
 	 */
389
-	private function filter_key_value_pair( array $pairs ): array {
389
+	private function filter_key_value_pair(array $pairs): array {
390 390
 		/** @var array<string, string> (as per filter function)*/
391 391
 		return array_filter(
392 392
 			$pairs,
393
-			function ( $value, $key ): bool {
394
-				return is_string( $value )
395
-				&& \mb_strlen( $value ) > 0
396
-				&& is_string( $key );
393
+			function($value, $key): bool {
394
+				return is_string($value)
395
+				&& \mb_strlen($value) > 0
396
+				&& is_string($key);
397 397
 			},
398 398
 			ARRAY_FILTER_USE_BOTH
399 399
 		);
Please login to merge, or discard this patch.
src/Services/View/PHP_Engine.php 1 patch
Spacing   +47 added lines, -47 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,21 +88,21 @@  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
-		$template = $this->maybe_resolve_dot_notation( $compiled->template() );
101
-		$view     = sprintf( '%s%s.php', \DIRECTORY_SEPARATOR, ltrim( $template ) );
102
-		if ( $print ) {
103
-			print( $this->render_buffer( $view, $compiled->data() ) ); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
99
+		$compiled = $this->component_compiler->compile($component);
100
+		$template = $this->maybe_resolve_dot_notation($compiled->template());
101
+		$view     = sprintf('%s%s.php', \DIRECTORY_SEPARATOR, ltrim($template));
102
+		if ($print) {
103
+			print($this->render_buffer($view, $compiled->data())); // phpcs:ignore WordPress.Security.EscapeOutput.OutputNotEscaped
104 104
 		} else {
105
-			return $this->render_buffer( $view, $compiled->data() );
105
+			return $this->render_buffer($view, $compiled->data());
106 106
 		}
107 107
 	}
108 108
 
@@ -113,8 +113,8 @@  discard block
 block discarded – undo
113 113
 	 * @param View_Model $view_model
114 114
 	 * @return string|void
115 115
 	 */
116
-	public function view_model( View_Model $view_model, bool $print = true ) {
117
-		return $this->render( $view_model->template(), $view_model->data(), $print );
116
+	public function view_model(View_Model $view_model, bool $print = true) {
117
+		return $this->render($view_model->template(), $view_model->data(), $print);
118 118
 	}
119 119
 
120 120
 	/**
@@ -125,11 +125,11 @@  discard block
 block discarded – undo
125 125
 	 * @param bool $print
126 126
 	 * @return string|void
127 127
 	 */
128
-	public function partial( string $view, iterable $data = array(), bool $print = true ) {
129
-		if ( $print ) {
130
-			$this->render( $view, $data, $print );
128
+	public function partial(string $view, iterable $data = array(), bool $print = true) {
129
+		if ($print) {
130
+			$this->render($view, $data, $print);
131 131
 		} else {
132
-			return $this->render( $view, $data, $print );
132
+			return $this->render($view, $data, $print);
133 133
 		}
134 134
 	}
135 135
 
@@ -141,27 +141,27 @@  discard block
 block discarded – undo
141 141
 	 * @return string
142 142
 	 * @throws Exception
143 143
 	 */
144
-	protected function render_buffer( string $view, iterable $__data ): string {
144
+	protected function render_buffer(string $view, iterable $__data): string {
145 145
 
146
-		if ( ! file_exists( $view ) ) {
147
-			throw new Exception( "{$view} doesn't exist" );
146
+		if ( ! file_exists($view)) {
147
+			throw new Exception("{$view} doesn't exist");
148 148
 		}
149 149
 
150 150
 		$output = '';
151 151
 		ob_start();
152 152
 
153 153
 		// Set all the data values a parameters.
154
-		foreach ( $__data as $__key => $__value ) {
155
-			if ( is_string( $__key ) ) {
156
-				${\wp_strip_all_tags( $__key )} = $__value;
154
+		foreach ($__data as $__key => $__value) {
155
+			if (is_string($__key)) {
156
+				${\wp_strip_all_tags($__key)} = $__value;
157 157
 			}
158 158
 
159 159
 			// Unset the key and value.
160
-			unset( $__key, $__value );
160
+			unset($__key, $__value);
161 161
 		}
162 162
 
163 163
 		// Unset the data.
164
-		unset( $__data );
164
+		unset($__data);
165 165
 
166 166
 		include $view;
167 167
 		$output = ob_get_contents();
@@ -175,12 +175,12 @@  discard block
 block discarded – undo
175 175
 	 * @param string $filename
176 176
 	 * @return string
177 177
 	 */
178
-	protected function resolve_file_path( string $filename ): string {
179
-		$filename = $this->maybe_resolve_dot_notation( $filename );
178
+	protected function resolve_file_path(string $filename): string {
179
+		$filename = $this->maybe_resolve_dot_notation($filename);
180 180
 		return sprintf(
181 181
 			'%s%s.php',
182 182
 			$this->base_view_path,
183
-			ltrim( $filename )
183
+			ltrim($filename)
184 184
 		);
185 185
 	}
186 186
 
@@ -190,13 +190,13 @@  discard block
 block discarded – undo
190 190
 	 * @param string $filename
191 191
 	 * @return string
192 192
 	 */
193
-	protected function maybe_resolve_dot_notation( string $filename ): string {
194
-		if ( $this->str_ends_with( '.php', $filename ) ) {
195
-			$filename = substr( $filename, 0, -4 );
193
+	protected function maybe_resolve_dot_notation(string $filename): string {
194
+		if ($this->str_ends_with('.php', $filename)) {
195
+			$filename = substr($filename, 0, -4);
196 196
 		}
197 197
 
198
-		$parts    = explode( '.', $filename );
199
-		$filename = implode( DIRECTORY_SEPARATOR, $parts );
198
+		$parts    = explode('.', $filename);
199
+		$filename = implode(DIRECTORY_SEPARATOR, $parts);
200 200
 
201 201
 		return $filename;
202 202
 	}
@@ -208,9 +208,9 @@  discard block
 block discarded – undo
208 208
 	 * @param string $haystack
209 209
 	 * @return bool
210 210
 	 */
211
-	protected function str_ends_with( string $needle, string $haystack ): bool {
212
-		$needle_len = strlen( $needle );
213
-		return ( $needle_len === 0 || 0 === substr_compare( $haystack, $needle, - $needle_len ) );
211
+	protected function str_ends_with(string $needle, string $haystack): bool {
212
+		$needle_len = strlen($needle);
213
+		return ($needle_len === 0 || 0 === substr_compare($haystack, $needle, - $needle_len));
214 214
 	}
215 215
 
216 216
 
@@ -221,12 +221,12 @@  discard block
 block discarded – undo
221 221
 	 * @return string
222 222
 	 * @throws Exception
223 223
 	 */
224
-	protected function verify_view_path( string $path ): string {
225
-		$path = $this->maybe_resolve_dot_notation( $path );
226
-		$path = rtrim( $path, '/' ) . '/';
224
+	protected function verify_view_path(string $path): string {
225
+		$path = $this->maybe_resolve_dot_notation($path);
226
+		$path = rtrim($path, '/') . '/';
227 227
 
228
-		if ( ! \is_dir( $path ) ) {
229
-			throw new Exception( "{$path} doesn't exist and cant be used as the base view path." );
228
+		if ( ! \is_dir($path)) {
229
+			throw new Exception("{$path} doesn't exist and cant be used as the base view path.");
230 230
 		}
231 231
 
232 232
 		return $path;
Please login to merge, or discard this patch.