Passed
Push — develop ( 72cbea...47a228 )
by Paul
02:45
created
src/Facades/Log.php 2 patches
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -6,13 +6,13 @@
 block discarded – undo
6 6
 
7 7
 class Log extends Facade
8 8
 {
9
-    /**
10
-     * Get the fully qualified class name of the component.
11
-     *
12
-     * @return string
13
-     */
14
-    protected static function getFacadeAccessor()
15
-    {
16
-        return \GeminiLabs\Castor\Helpers\Log::class;
17
-    }
9
+	/**
10
+	 * Get the fully qualified class name of the component.
11
+	 *
12
+	 * @return string
13
+	 */
14
+	protected static function getFacadeAccessor()
15
+	{
16
+		return \GeminiLabs\Castor\Helpers\Log::class;
17
+	}
18 18
 }
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,8 +4,7 @@
 block discarded – undo
4 4
 
5 5
 use GeminiLabs\Castor\Facade;
6 6
 
7
-class Log extends Facade
8
-{
7
+class Log extends Facade {
9 8
     /**
10 9
      * Get the fully qualified class name of the component.
11 10
      *
Please login to merge, or discard this patch.
src/Controller.php 2 patches
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -8,8 +8,7 @@
 block discarded – undo
8 8
 use GeminiLabs\Castor\Facades\Utility;
9 9
 use WP_Customize_Manager;
10 10
 
11
-class Controller
12
-{
11
+class Controller {
13 12
 	/**
14 13
 	 * @return void
15 14
 	 * @action after_setup_theme
Please login to merge, or discard this patch.
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -16,11 +16,11 @@  discard block
 block discarded – undo
16 16
 	 */
17 17
 	public function afterSetupTheme()
18 18
 	{
19
-		castor_app()->cssDir = trailingslashit( (string) apply_filters( 'castor/assets/styles/dir', 'css' ));
20
-		castor_app()->imgDir = trailingslashit( (string) apply_filters( 'castor/assets/images/dir', 'img' ));
21
-		castor_app()->jsDir = trailingslashit( (string) apply_filters( 'castor/assets/scripts/dir', 'js' ));
19
+		castor_app()->cssDir = trailingslashit( (string) apply_filters( 'castor/assets/styles/dir', 'css' ) );
20
+		castor_app()->imgDir = trailingslashit( (string) apply_filters( 'castor/assets/images/dir', 'img' ) );
21
+		castor_app()->jsDir = trailingslashit( (string) apply_filters( 'castor/assets/scripts/dir', 'js' ) );
22 22
 
23
-		add_editor_style( Theme::assetUri( castor_app()->cssDir.'editor.css' ));
23
+		add_editor_style( Theme::assetUri( castor_app()->cssDir.'editor.css' ) );
24 24
 		add_theme_support( 'customize-selective-refresh-widgets' );
25 25
 		add_theme_support( 'html5', ['caption', 'comment-form', 'comment-list', 'gallery', 'search-form'] );
26 26
 		add_theme_support( 'post-thumbnails' );
@@ -30,11 +30,11 @@  discard block
 block discarded – undo
30 30
 		add_theme_support( 'soil-nice-search' );
31 31
 		add_theme_support( 'soil-relative-urls' );
32 32
 		add_theme_support( 'title-tag' );
33
-		load_theme_textdomain( 'castor', Theme::paths( 'dir.template' ) . '/languages' );
33
+		load_theme_textdomain( 'castor', Theme::paths( 'dir.template' ).'/languages' );
34 34
 
35 35
 		$menus = apply_filters( 'castor/register/nav_menus', [
36 36
 			'main_menu' => __( 'Main Menu', 'castor' ),
37
-		]);
37
+		] );
38 38
 
39 39
 		foreach( $menus as $location => $description ) {
40 40
 			register_nav_menu( $location, $description );
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 		if( Theme::displaySidebar() ) {
51 51
 			$classes[] = 'has-sidebar';
52 52
 		}
53
-		return array_keys( array_flip( $classes ));
53
+		return array_keys( array_flip( $classes ) );
54 54
 	}
55 55
 
56 56
 	/**
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 	 */
78 78
 	public function filterTemplate( $template )
79 79
 	{
80
-		if( is_string( $template )) {
80
+		if( is_string( $template ) ) {
81 81
 			$template = Template::setLayout( $template );
82 82
 			Development::storeTemplatePath( $template );
83 83
 		}
@@ -102,8 +102,8 @@  discard block
 block discarded – undo
102 102
 	 */
103 103
 	public function loadAdminFavicon()
104 104
 	{
105
-		if( file_exists( Theme::assetPath( 'favicon/favicon-admin.ico' ))) {
106
-			printf( '<link rel="shortcut icon" href="%s">', Theme::assetUri( 'favicon/favicon-admin.ico' ));
105
+		if( file_exists( Theme::assetPath( 'favicon/favicon-admin.ico' ) ) ) {
106
+			printf( '<link rel="shortcut icon" href="%s">', Theme::assetUri( 'favicon/favicon-admin.ico' ) );
107 107
 		}
108 108
 	}
109 109
 
@@ -113,8 +113,8 @@  discard block
 block discarded – undo
113 113
 	 */
114 114
 	public function login()
115 115
 	{
116
-		if( file_exists( Theme::assetPath( castor_app()->cssDir.'login.css' ))) {
117
-			printf( '<link rel="stylesheet" href="%s">', Theme::assetUri( castor_app()->cssDir.'login.css' ));
116
+		if( file_exists( Theme::assetPath( castor_app()->cssDir.'login.css' ) ) ) {
117
+			printf( '<link rel="stylesheet" href="%s">', Theme::assetUri( castor_app()->cssDir.'login.css' ) );
118 118
 		}
119 119
 	}
120 120
 
@@ -124,14 +124,14 @@  discard block
 block discarded – undo
124 124
 	 */
125 125
 	public function registerAdminAssets()
126 126
 	{
127
-		if( file_exists( Theme::assetPath( castor_app()->cssDir.'admin.css' ))) {
127
+		if( file_exists( Theme::assetPath( castor_app()->cssDir.'admin.css' ) ) ) {
128 128
 			wp_enqueue_style( 'castor/admin.css',
129 129
 				Theme::assetUri( castor_app()->cssDir.'admin.css' ),
130 130
 				apply_filters( 'castor/enqueue/admin/css/deps', [] ),
131 131
 				null
132 132
 			);
133 133
 		}
134
-		if( file_exists( Theme::assetPath( castor_app()->jsDir.'admin.js' ))) {
134
+		if( file_exists( Theme::assetPath( castor_app()->jsDir.'admin.js' ) ) ) {
135 135
 			wp_enqueue_script( 'castor/admin.js',
136 136
 				Theme::assetUri( castor_app()->jsDir.'admin.js' ),
137 137
 				apply_filters( 'castor/enqueue/admin/js/deps', [] ),
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 		wp_localize_script( 'castor/main.js', apply_filters( 'castor/enqueue/js/localize/variable', 'globals' ),
162 162
 			apply_filters( 'castor/enqueue/js/localize/variables', [
163 163
 				'ajax' => admin_url( 'admin-ajax.php' ),
164
-			])
164
+			] )
165 165
 		);
166 166
 		do_action( 'castor/register/assets' );
167 167
 		wp_enqueue_style( 'castor/main.css' );
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
 			'render_callback' => function() {
181 181
 				bloginfo( 'name' );
182 182
 			},
183
-		]);
183
+		] );
184 184
 	}
185 185
 
186 186
 	/**
@@ -203,15 +203,15 @@  discard block
 block discarded – undo
203 203
 			'after_widget'  => '</div>',
204 204
 			'before_title'  => '<h4>',
205 205
 			'after_title'   => '</h4>',
206
-		]);
206
+		] );
207 207
 
208 208
 		$sidebars = apply_filters( 'castor/register/sidebars', [
209 209
 			'sidebar-primary' => __( 'Primary Sidebar', 'castor' ),
210 210
 			'sidebar-footer'  => __( 'Footer Widgets', 'castor' ),
211
-		]);
211
+		] );
212 212
 
213 213
 		foreach( $sidebars as $id => $name ) {
214
-			register_sidebar([
214
+			register_sidebar( [
215 215
 				'id'   => $id,
216 216
 				'name' => $name,
217 217
 			] + $defaults );
Please login to merge, or discard this patch.
src/Exceptions/BindingResolutionException.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -4,6 +4,5 @@
 block discarded – undo
4 4
 
5 5
 use Exception;
6 6
 
7
-class BindingResolutionException extends Exception
8
-{
7
+class BindingResolutionException extends Exception {
9 8
 }
Please login to merge, or discard this patch.
src/AliasLoader.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@  discard block
 block discarded – undo
40 40
 	 */
41 41
 	public static function getInstance( array $aliases = [] )
42 42
 	{
43
-		if( is_null( static::$instance )) {
43
+		if( is_null( static::$instance ) ) {
44 44
 			return static::$instance = new static( $aliases );
45 45
 		}
46 46
 
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
 	 */
61 61
 	public function load( $alias )
62 62
 	{
63
-		if( isset( $this->aliases[$alias] )) {
63
+		if( isset( $this->aliases[$alias] ) ) {
64 64
 			return class_alias( $this->aliases[$alias], $alias );
65 65
 		}
66 66
 	}
Please login to merge, or discard this patch.
Braces   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -2,8 +2,7 @@  discard block
 block discarded – undo
2 2
 
3 3
 namespace GeminiLabs\Castor;
4 4
 
5
-class AliasLoader
6
-{
5
+class AliasLoader {
7 6
 	/**
8 7
 	 * The singleton instance of the loader.
9 8
 	 *
@@ -31,7 +30,8 @@  discard block
 block discarded – undo
31 30
 	}
32 31
 
33 32
 	private function __clone()
34
-	{}
33
+	{
34
+}
35 35
 
36 36
 	/**
37 37
 	 * Get or create the singleton alias loader instance.
Please login to merge, or discard this patch.
src/Facade.php 2 patches
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -102,11 +102,11 @@
 block discarded – undo
102 102
 	 */
103 103
 	protected static function resolveFacadeInstance( $name )
104 104
 	{
105
-		if( is_object( $name )) {
105
+		if( is_object( $name ) ) {
106 106
 			return $name;
107 107
 		}
108 108
 
109
-		if( isset( static::$resolvedInstance[$name] )) {
109
+		if( isset( static::$resolvedInstance[$name] ) ) {
110 110
 			return static::$resolvedInstance[$name];
111 111
 		}
112 112
 
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -5,8 +5,7 @@
 block discarded – undo
5 5
 use GeminiLabs\Castor\Application;
6 6
 use RuntimeException;
7 7
 
8
-abstract class Facade
9
-{
8
+abstract class Facade {
10 9
 	/**
11 10
 	 * The application instance being facaded.
12 11
 	 *
Please login to merge, or discard this patch.
src/Image.php 2 patches
Braces   +4 added lines, -3 removed lines patch added patch discarded remove patch
@@ -5,8 +5,7 @@  discard block
 block discarded – undo
5 5
 use GeminiLabs\Castor\Helpers\PostMeta;
6 6
 use GeminiLabs\Castor\Helpers\Utility;
7 7
 
8
-class Image
9
-{
8
+class Image {
10 9
 	public $image;
11 10
 
12 11
 	protected $postmeta;
@@ -59,7 +58,9 @@  discard block
 block discarded – undo
59 58
 
60 59
 		$attachment = get_post( $attachmentId );
61 60
 
62
-		if( is_null( $attachment ) || $attachment->post_type != 'attachment' )return;
61
+		if( is_null( $attachment ) || $attachment->post_type != 'attachment' ) {
62
+			return;
63
+		}
63 64
 
64 65
 		return $attachment->ID;
65 66
 	}
Please login to merge, or discard this patch.
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@  discard block
 block discarded – undo
26 26
 	public function get( $attachment )
27 27
 	{
28 28
 		$attachment = $this->normalize( $attachment );
29
-		if( $attachment && $thumbnail = wp_get_attachment_image_src( $attachment, 'thumbnail' )) {
29
+		if( $attachment && $thumbnail = wp_get_attachment_image_src( $attachment, 'thumbnail' ) ) {
30 30
 			$medium = $this->normalizeSrc( wp_get_attachment_image_src( $attachment, 'medium' ), $thumbnail );
31 31
 			$large = $this->normalizeSrc( wp_get_attachment_image_src( $attachment, 'large' ), $medium );
32 32
 
@@ -53,7 +53,7 @@  discard block
 block discarded – undo
53 53
 
54 54
 	protected function normalize( $attachmentId )
55 55
 	{
56
-		if( !filter_var( $attachmentId, FILTER_VALIDATE_INT )) {
56
+		if( !filter_var( $attachmentId, FILTER_VALIDATE_INT ) ) {
57 57
 			$attachmentId = $this->postmeta->get( $attachmentId );
58 58
 		}
59 59
 
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 	 */
72 72
 	protected function normalizeSrc( array $image, $fallback = false )
73 73
 	{
74
-		if( is_array( $fallback ) && count( array_diff( $image, $fallback )) < 2 ) {
74
+		if( is_array( $fallback ) && count( array_diff( $image, $fallback ) ) < 2 ) {
75 75
 			$image = $fallback;
76 76
 		}
77 77
 		$image = array_pad( $image, 3, '' );
Please login to merge, or discard this patch.
src/Services/Email.php 2 patches
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
 		$this->to          = $email['to'];
56 56
 
57 57
 		add_action( 'phpmailer_init', function( PHPMailer $phpmailer ) {
58
-			if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
58
+			if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ) )return;
59 59
 			$phpmailer->AltBody = $this->buildPlainTextMessage( $phpmailer->Body );
60 60
 		});
61 61
 
@@ -106,11 +106,11 @@  discard block
 block discarded – undo
106 106
 			'reply-to',
107 107
 		];
108 108
 
109
-		$headers = array_intersect_key( $email, array_flip( $allowed ));
109
+		$headers = array_intersect_key( $email, array_flip( $allowed ) );
110 110
 		$headers = array_filter( $headers );
111 111
 
112 112
 		foreach( $headers as $key => $value ) {
113
-			unset( $headers[ $key ] );
113
+			unset( $headers[$key] );
114 114
 			$headers[] = sprintf( '%s: %s', $key, $value );
115 115
 		}
116 116
 
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
 			? $this->renderTemplate( $email['template'], $email['template-tags'] )
131 131
 			: $email['message'];
132 132
 
133
-		$message = $this->filterHtml( $email['before'] . $message . $email['after'] );
133
+		$message = $this->filterHtml( $email['before'].$message.$email['after'] );
134 134
 		$message = str_replace( '{message}', $message, $body );
135 135
 
136 136
 		return apply_filters( 'castor/email/message', $message, 'html', $this );
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
177 177
 	 */
178 178
 	protected function normalize( $email )
179 179
 	{
180
-		$defaults = array_fill_keys([
180
+		$defaults = array_fill_keys( [
181 181
 			'after', 'attachments', 'bcc', 'before', 'cc', 'from', 'message', 'reply-to', 'subject',
182 182
 			'template', 'template-tags', 'to',
183 183
 		], '' );
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
188 188
 		foreach( ['attachments', 'template-tags'] as $key ) {
189 189
 			$email[$key] = array_filter( (array) $email[$key] );
190 190
 		}
191
-		if( empty( $email['reply-to'] )) {
191
+		if( empty( $email['reply-to'] ) ) {
192 192
 			$email['reply-to'] = $email['from'];
193 193
 		}
194 194
 
@@ -204,12 +204,12 @@  discard block
 block discarded – undo
204 204
 	 */
205 205
 	protected function renderTemplate( $templatePath, array $args = [] )
206 206
 	{
207
-		$file = $this->template->get( sprintf( 'castor/%s', $templatePath ));
207
+		$file = $this->template->get( sprintf( 'castor/%s', $templatePath ) );
208 208
 
209
-		if( !file_exists( $file )) {
209
+		if( !file_exists( $file ) ) {
210 210
 			$file = sprintf( '%s/templates/%s.php', dirname( __DIR__ ), $templatePath );
211 211
 		}
212
-		if( !file_exists( $file ))return;
212
+		if( !file_exists( $file ) )return;
213 213
 
214 214
 		ob_start();
215 215
 		include $file;
Please login to merge, or discard this patch.
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -5,8 +5,7 @@  discard block
 block discarded – undo
5 5
 use GeminiLabs\Castor\Helpers\Template;
6 6
 use PHPMailer;
7 7
 
8
-class Email
9
-{
8
+class Email {
10 9
 	/**
11 10
 	 * @var Template
12 11
 	 */
@@ -56,7 +55,9 @@  discard block
 block discarded – undo
56 55
 		$this->to          = $email['to'];
57 56
 
58 57
 		add_action( 'phpmailer_init', function( PHPMailer $phpmailer ) {
59
-			if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody ))return;
58
+			if( $phpmailer->ContentType === 'text/plain' || !empty( $phpmailer->AltBody )) {
59
+				return;
60
+			}
60 61
 			$phpmailer->AltBody = $this->buildPlainTextMessage( $phpmailer->Body );
61 62
 		});
62 63
 
@@ -80,7 +81,9 @@  discard block
 block discarded – undo
80 81
 	 */
81 82
 	public function send()
82 83
 	{
83
-		if( !$this->message || !$this->subject || !$this->to )return;
84
+		if( !$this->message || !$this->subject || !$this->to ) {
85
+			return;
86
+		}
84 87
 
85 88
 		$sent = wp_mail(
86 89
 			$this->to,
@@ -210,7 +213,9 @@  discard block
 block discarded – undo
210 213
 		if( !file_exists( $file )) {
211 214
 			$file = sprintf( '%s/templates/%s.php', dirname( __DIR__ ), $templatePath );
212 215
 		}
213
-		if( !file_exists( $file ))return;
216
+		if( !file_exists( $file )) {
217
+			return;
218
+		}
214 219
 
215 220
 		ob_start();
216 221
 		include $file;
Please login to merge, or discard this patch.
src/Services/Normalizer.php 2 patches
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 		foreach( $attributes as $key => $value ) {
102 102
 			// if data attributes, use single quotes in case of json encoded values
103 103
 			$quotes = false !== stripos( $key, 'data-' ) ? "'" : '"';
104
-			if( is_array( $value )) {
104
+			if( is_array( $value ) ) {
105 105
 				$value = json_encode( $value );
106 106
 				$quotes = "'";
107 107
 			}
@@ -144,29 +144,29 @@  discard block
 block discarded – undo
144 144
 	 */
145 145
 	protected function filterAttributes( array $attributeKeys )
146 146
 	{
147
-		$filtered = array_intersect_key( $this->args, array_flip( $attributeKeys ));
147
+		$filtered = array_intersect_key( $this->args, array_flip( $attributeKeys ) );
148 148
 
149 149
 		// normalize truthy boolean attributes
150 150
 		foreach( $filtered as $key => $value ) {
151
-			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES ))continue;
151
+			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES ) )continue;
152 152
 
153 153
 			if( $value !== false ) {
154
-				$filtered[ $key ] = '';
154
+				$filtered[$key] = '';
155 155
 				continue;
156 156
 			}
157 157
 
158
-			unset( $filtered[ $key ] );
158
+			unset( $filtered[$key] );
159 159
 		}
160 160
 
161 161
 		$filteredKeys = array_filter( array_keys( $filtered ), function( $key ) use ( $filtered ) {
162 162
 			return !(
163
-				empty( $filtered[ $key ] )
164
-				&& !is_numeric( $filtered[ $key ] )
163
+				empty( $filtered[$key] )
164
+				&& !is_numeric( $filtered[$key] )
165 165
 				&& !in_array( $key, self::BOOLEAN_ATTRIBUTES )
166 166
 			);
167 167
 		});
168 168
 
169
-		return array_intersect_key( $filtered, array_flip( $filteredKeys ));
169
+		return array_intersect_key( $filtered, array_flip( $filteredKeys ) );
170 170
 	}
171 171
 
172 172
 	/**
@@ -183,20 +183,20 @@  discard block
 block discarded – undo
183 183
 			foreach( $this->args as $key => $value ) {
184 184
 
185 185
 				$length = strlen( $wildcard );
186
-				$result = substr( $key, 0, $length) === $wildcard;
186
+				$result = substr( $key, 0, $length ) === $wildcard;
187 187
 
188 188
 				if( $result ) {
189 189
 					// only allow data attributes to have an empty value
190
-					if( $wildcard != 'data-' && empty( $value ))continue;
190
+					if( $wildcard != 'data-' && empty( $value ) )continue;
191 191
 
192
-					if( is_array( $value )) {
192
+					if( is_array( $value ) ) {
193 193
 
194 194
 						if( $wildcard != 'data-' )continue;
195 195
 
196 196
 						$value = json_encode( $value );
197 197
 					}
198 198
 
199
-					$wildcards[ $key ] = $value;
199
+					$wildcards[$key] = $value;
200 200
 				}
201 201
 			}
202 202
 		}
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 	 */
210 210
 	protected function filterInputType()
211 211
 	{
212
-		if( !isset( $this->args['type'] ) || !in_array( $this->args['type'], self::INPUT_TYPES )) {
212
+		if( !isset( $this->args['type'] ) || !in_array( $this->args['type'], self::INPUT_TYPES ) ) {
213 213
 			$this->args['type'] = 'text';
214 214
 		}
215 215
 	}
@@ -219,7 +219,7 @@  discard block
 block discarded – undo
219 219
 	 */
220 220
 	protected function parseAttributes( array $attributes, array $args = [] )
221 221
 	{
222
-		if( !empty( $args )) {
222
+		if( !empty( $args ) ) {
223 223
 			$this->args = array_change_key_case( $args );
224 224
 		}
225 225
 
Please login to merge, or discard this patch.
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -2,8 +2,7 @@  discard block
 block discarded – undo
2 2
 
3 3
 namespace GeminiLabs\Castor\Services;
4 4
 
5
-class Normalizer
6
-{
5
+class Normalizer {
7 6
 	const BOOLEAN_ATTRIBUTES = [
8 7
 		'autofocus', 'capture', 'checked', 'disabled', 'draggable', 'formnovalidate', 'hidden',
9 8
 		'multiple', 'novalidate', 'readonly', 'required', 'selected', 'spellcheck',
@@ -148,7 +147,9 @@  discard block
 block discarded – undo
148 147
 
149 148
 		// normalize truthy boolean attributes
150 149
 		foreach( $filtered as $key => $value ) {
151
-			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES ))continue;
150
+			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES )) {
151
+				continue;
152
+			}
152 153
 
153 154
 			if( $value !== false ) {
154 155
 				$filtered[ $key ] = '';
@@ -187,11 +188,15 @@  discard block
 block discarded – undo
187 188
 
188 189
 				if( $result ) {
189 190
 					// only allow data attributes to have an empty value
190
-					if( $wildcard != 'data-' && empty( $value ))continue;
191
+					if( $wildcard != 'data-' && empty( $value )) {
192
+						continue;
193
+					}
191 194
 
192 195
 					if( is_array( $value )) {
193 196
 
194
-						if( $wildcard != 'data-' )continue;
197
+						if( $wildcard != 'data-' ) {
198
+							continue;
199
+						}
195 200
 
196 201
 						$value = json_encode( $value );
197 202
 					}
Please login to merge, or discard this patch.
src/Video.php 2 patches
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
41 41
 
42 42
 	public function render()
43 43
 	{
44
-		if( !isset( $this->video->html ))return;
44
+		if( !isset( $this->video->html ) )return;
45 45
 		return sprintf(
46 46
 			'<div class="video embed">%s%s</div>',
47 47
 			$this->renderScreenshot(),
@@ -52,11 +52,11 @@  discard block
 block discarded – undo
52 52
 	public function renderPlayButton()
53 53
 	{
54 54
 		return sprintf(
55
-			'<div class="video-play">' .
56
-				'<div class="video-play-pulse pulse1"></div>' .
57
-				'<div class="video-play-pulse pulse2"></div>' .
58
-				'<div class="video-play-pulse pulse3"></div>' .
59
-				'<a href="%s" class="video-play-btn">%s</a>' .
55
+			'<div class="video-play">'.
56
+				'<div class="video-play-pulse pulse1"></div>'.
57
+				'<div class="video-play-pulse pulse2"></div>'.
58
+				'<div class="video-play-pulse pulse3"></div>'.
59
+				'<a href="%s" class="video-play-btn">%s</a>'.
60 60
 			'</div>',
61 61
 			$this->args['url'],
62 62
 			$this->theme->svg( 'play.svg' )
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 	public function renderScreenshot()
67 67
 	{
68 68
 		if( $this->args['image']
69
-			&& in_array( strtolower( $this->video->provider_name ), $this->supported )) {
69
+			&& in_array( strtolower( $this->video->provider_name ), $this->supported ) ) {
70 70
 			return sprintf( '%s<div class="video-poster" style="background-image: url(%s)">%s</div>',
71 71
 				$this->renderSpinner(),
72 72
 				$this->args['image'],
@@ -78,10 +78,10 @@  discard block
 block discarded – undo
78 78
 	public function renderSpinner()
79 79
 	{
80 80
 		return sprintf(
81
-			'<div class="video-spinner">' .
82
-				'<div class="spinner"><div class="spinner-dots">%s</div></div>' .
81
+			'<div class="video-spinner">'.
82
+				'<div class="spinner"><div class="spinner-dots">%s</div></div>'.
83 83
 			'</div>',
84
-			implode( '', array_fill( 0, 8, '<div class="spinner-dot"></div>' ))
84
+			implode( '', array_fill( 0, 8, '<div class="spinner-dot"></div>' ) )
85 85
 		);
86 86
 	}
87 87
 
@@ -102,11 +102,11 @@  discard block
 block discarded – undo
102 102
 
103 103
 	protected function normalize( $args )
104 104
 	{
105
-		if( is_string( $args )) {
105
+		if( is_string( $args ) ) {
106 106
 			$args = ['url' => $args];
107 107
 		}
108 108
 
109
-		$this->args = shortcode_atts([
109
+		$this->args = shortcode_atts( [
110 110
 			'image'  => '', // string || int
111 111
 			'player' => '', // string || array
112 112
 			'url'    => '', // string
@@ -114,8 +114,8 @@  discard block
 block discarded – undo
114 114
 
115 115
 		foreach( $this->args as $key => $value ) {
116 116
 			$method = $this->utility->buildMethodName( $key, 'set' );
117
-			if( !method_exists( $this, $method ))continue;
118
-			call_user_func([ $this, $method ], $value );
117
+			if( !method_exists( $this, $method ) )continue;
118
+			call_user_func( [$this, $method], $value );
119 119
 		}
120 120
 		return $this->args;
121 121
 	}
Please login to merge, or discard this patch.
Braces   +7 added lines, -4 removed lines patch added patch discarded remove patch
@@ -8,8 +8,7 @@  discard block
 block discarded – undo
8 8
 use GeminiLabs\Castor\Image;
9 9
 use GeminiLabs\Castor\Oembed;
10 10
 
11
-class Video
12
-{
11
+class Video {
13 12
 	public $video;
14 13
 
15 14
 	protected $args;
@@ -41,7 +40,9 @@  discard block
 block discarded – undo
41 40
 
42 41
 	public function render()
43 42
 	{
44
-		if( !isset( $this->video->html ))return;
43
+		if( !isset( $this->video->html )) {
44
+			return;
45
+		}
45 46
 		return sprintf(
46 47
 			'<div class="video embed">%s%s</div>',
47 48
 			$this->renderScreenshot(),
@@ -114,7 +115,9 @@  discard block
 block discarded – undo
114 115
 
115 116
 		foreach( $this->args as $key => $value ) {
116 117
 			$method = $this->utility->buildMethodName( $key, 'set' );
117
-			if( !method_exists( $this, $method ))continue;
118
+			if( !method_exists( $this, $method )) {
119
+				continue;
120
+			}
118 121
 			call_user_func([ $this, $method ], $value );
119 122
 		}
120 123
 		return $this->args;
Please login to merge, or discard this patch.