pryley / castor-framework

src/Facades/Log.php

Indentation +9 added lines, -9 removed lines

@@ -6,13 +6,13 @@
 
 class Log extends Facade
 {
-    /**
-     * Get the fully qualified class name of the component.
-     *
-     * @return string
-     */
-    protected static function getFacadeAccessor()
-    {
-        return \GeminiLabs\Castor\Helpers\Log::class;
-    }
+	/**
+	 * Get the fully qualified class name of the component.
+	 *
+	 * @return string
+	 */
+	protected static function getFacadeAccessor()
+	{
+		return \GeminiLabs\Castor\Helpers\Log::class;
+	}
 }

Braces +1 added lines, -2 removed lines

@@ -4,8 +4,7 @@
 
 use GeminiLabs\Castor\Facade;
 
-class Log extends Facade
-{
+class Log extends Facade {
     /**
      * Get the fully qualified class name of the component.
      *

src/Controller.php

Braces +1 added lines, -2 removed lines

@@ -8,8 +8,7 @@
 use GeminiLabs\Castor\Facades\Utility;
 use WP_Customize_Manager;
 
-class Controller
-{
+class Controller {
 	/**
 	 * @return void
 	 * @action after_setup_theme

Spacing +19 added lines, -19 removed lines

@@ -16,11 +16,11 @@ 
 	 */
 	public function afterSetupTheme()
 	{
-		castor_app()->cssDir = trailingslashit( (string) apply_filters( 'castor/assets/styles/dir', 'css' ));
-		castor_app()->imgDir = trailingslashit( (string) apply_filters( 'castor/assets/images/dir', 'img' ));
-		castor_app()->jsDir = trailingslashit( (string) apply_filters( 'castor/assets/scripts/dir', 'js' ));
+		castor_app()->cssDir = trailingslashit( (string) apply_filters( 'castor/assets/styles/dir', 'css' ) );
+		castor_app()->imgDir = trailingslashit( (string) apply_filters( 'castor/assets/images/dir', 'img' ) );
+		castor_app()->jsDir = trailingslashit( (string) apply_filters( 'castor/assets/scripts/dir', 'js' ) );
 
-		add_editor_style( Theme::assetUri( castor_app()->cssDir.'editor.css' ));
+		add_editor_style( Theme::assetUri( castor_app()->cssDir.'editor.css' ) );
 		add_theme_support( 'customize-selective-refresh-widgets' );
 		add_theme_support( 'html5', ['caption', 'comment-form', 'comment-list', 'gallery', 'search-form'] );
 		add_theme_support( 'post-thumbnails' );
@@ -30,11 +30,11 @@ 
 		add_theme_support( 'soil-nice-search' );
 		add_theme_support( 'soil-relative-urls' );
 		add_theme_support( 'title-tag' );
-		load_theme_textdomain( 'castor', Theme::paths( 'dir.template' ) . '/languages' );
+		load_theme_textdomain( 'castor', Theme::paths( 'dir.template' ).'/languages' );
 
 		$menus = apply_filters( 'castor/register/nav_menus', [
 			'main_menu' => __( 'Main Menu', 'castor' ),
-		]);
+		] );
 
 		foreach( $menus as $location => $description ) {
 			register_nav_menu( $location, $description );
@@ -50,7 +50,7 @@ 
 		if( Theme::displaySidebar() ) {
 			$classes[] = 'has-sidebar';
 		}
-		return array_keys( array_flip( $classes ));
+		return array_keys( array_flip( $classes ) );
 	}
 
 	/**
@@ -77,7 +77,7 @@ 
 	 */
 	public function filterTemplate( $template )
 	{
-		if( is_string( $template )) {
+		if( is_string( $template ) ) {
 			$template = Template::setLayout( $template );
 			Development::storeTemplatePath( $template );
 		}
@@ -102,8 +102,8 @@ 
 	 */
 	public function loadAdminFavicon()
 	{
-		if( file_exists( Theme::assetPath( 'favicon/favicon-admin.ico' ))) {
-			printf( '<link rel="shortcut icon" href="%s">', Theme::assetUri( 'favicon/favicon-admin.ico' ));
+		if( file_exists( Theme::assetPath( 'favicon/favicon-admin.ico' ) ) ) {
+			printf( '<link rel="shortcut icon" href="%s">', Theme::assetUri( 'favicon/favicon-admin.ico' ) );
 		}
 	}
 
@@ -113,8 +113,8 @@ 
 	 */
 	public function login()
 	{
-		if( file_exists( Theme::assetPath( castor_app()->cssDir.'login.css' ))) {
-			printf( '<link rel="stylesheet" href="%s">', Theme::assetUri( castor_app()->cssDir.'login.css' ));
+		if( file_exists( Theme::assetPath( castor_app()->cssDir.'login.css' ) ) ) {
+			printf( '<link rel="stylesheet" href="%s">', Theme::assetUri( castor_app()->cssDir.'login.css' ) );
 		}
 	}
 
@@ -124,14 +124,14 @@ 
 	 */
 	public function registerAdminAssets()
 	{
-		if( file_exists( Theme::assetPath( castor_app()->cssDir.'admin.css' ))) {
+		if( file_exists( Theme::assetPath( castor_app()->cssDir.'admin.css' ) ) ) {
 			wp_enqueue_style( 'castor/admin.css',
 				Theme::assetUri( castor_app()->cssDir.'admin.css' ),
 				apply_filters( 'castor/enqueue/admin/css/deps', [] ),
 				null
 			);
 		}
-		if( file_exists( Theme::assetPath( castor_app()->jsDir.'admin.js' ))) {
+		if( file_exists( Theme::assetPath( castor_app()->jsDir.'admin.js' ) ) ) {
 			wp_enqueue_script( 'castor/admin.js',
 				Theme::assetUri( castor_app()->jsDir.'admin.js' ),
 				apply_filters( 'castor/enqueue/admin/js/deps', [] ),
@@ -161,7 +161,7 @@ 
 		wp_localize_script( 'castor/main.js', apply_filters( 'castor/enqueue/js/localize/variable', 'globals' ),
 			apply_filters( 'castor/enqueue/js/localize/variables', [
 				'ajax' => admin_url( 'admin-ajax.php' ),
-			])
+			] )
 		);
 		do_action( 'castor/register/assets' );
 		wp_enqueue_style( 'castor/main.css' );
@@ -180,7 +180,7 @@ 
 			'render_callback' => function() {
 				bloginfo( 'name' );
 			},
-		]);
+		] );
 	}
 
 	/**
@@ -203,15 +203,15 @@ 
 			'after_widget'  => '</div>',
 			'before_title'  => '<h4>',
 			'after_title'   => '</h4>',
-		]);
+		] );
 
 		$sidebars = apply_filters( 'castor/register/sidebars', [
 			'sidebar-primary' => __( 'Primary Sidebar', 'castor' ),
 			'sidebar-footer'  => __( 'Footer Widgets', 'castor' ),
-		]);
+		] );
 
 		foreach( $sidebars as $id => $name ) {
-			register_sidebar([
+			register_sidebar( [
 				'id'   => $id,
 				'name' => $name,
 			] + $defaults );

src/Exceptions/BindingResolutionException.php

Braces +1 added lines, -2 removed lines

@@ -4,6 +4,5 @@
 
 use Exception;
 
-class BindingResolutionException extends Exception
-{
+class BindingResolutionException extends Exception {
 }

src/AliasLoader.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
 	 */
 	public static function getInstance( array $aliases = [] )
 	{
-		if( is_null( static::$instance )) {
+		if( is_null( static::$instance ) ) {
 			return static::$instance = new static( $aliases );
 		}
 
@@ -60,7 +60,7 @@ 
 	 */
 	public function load( $alias )
 	{
-		if( isset( $this->aliases[$alias] )) {
+		if( isset( $this->aliases[$alias] ) ) {
 			return class_alias( $this->aliases[$alias], $alias );
 		}
 	}

Braces +3 added lines, -3 removed lines

@@ -2,8 +2,7 @@ 
 
 namespace GeminiLabs\Castor;
 
-class AliasLoader
-{
+class AliasLoader {
 	/**
 	 * The singleton instance of the loader.
 	 *
@@ -31,7 +30,8 @@ 
 	}
 
 	private function __clone()
-	{}
+	{
+}
 
 	/**
 	 * Get or create the singleton alias loader instance.

src/Facade.php

Spacing +2 added lines, -2 removed lines

@@ -102,11 +102,11 @@
 	 */
 	protected static function resolveFacadeInstance( $name )
 	{
-		if( is_object( $name )) {
+		if( is_object( $name ) ) {
 			return $name;
 		}
 
-		if( isset( static::$resolvedInstance[$name] )) {
+		if( isset( static::$resolvedInstance[$name] ) ) {
 			return static::$resolvedInstance[$name];
 		}
 

Braces +1 added lines, -2 removed lines

@@ -5,8 +5,7 @@
 use GeminiLabs\Castor\Application;
 use RuntimeException;
 
-abstract class Facade
-{
+abstract class Facade {
 	/**
 	 * The application instance being facaded.
 	 *

src/Image.php

Braces +4 added lines, -3 removed lines

@@ -5,8 +5,7 @@ 
 use GeminiLabs\Castor\Helpers\PostMeta;
 use GeminiLabs\Castor\Helpers\Utility;
 
-class Image
-{
+class Image {
 	public $image;
 
 	protected $postmeta;
@@ -59,7 +58,9 @@ 
 
 		$attachment = get_post( $attachmentId );
 
-		if( is_null( $attachment ) || $attachment->post_type != 'attachment' )return;
+		if( is_null( $attachment ) || $attachment->post_type != 'attachment' ) {
+			return;
+		}
 
 		return $attachment->ID;
 	}

Spacing +3 added lines, -3 removed lines

@@ -26,7 +26,7 @@ 
 	public function get( $attachment )
 	{
 		$attachment = $this->normalize( $attachment );
-		if( $attachment && $thumbnail = wp_get_attachment_image_src( $attachment, 'thumbnail' )) {
+		if( $attachment && $thumbnail = wp_get_attachment_image_src( $attachment, 'thumbnail' ) ) {
 			$medium = $this->normalizeSrc( wp_get_attachment_image_src( $attachment, 'medium' ), $thumbnail );
 			$large = $this->normalizeSrc( wp_get_attachment_image_src( $attachment, 'large' ), $medium );
 
@@ -53,7 +53,7 @@ 
 
 	protected function normalize( $attachmentId )
 	{
-		if( !filter_var( $attachmentId, FILTER_VALIDATE_INT )) {
+		if( !filter_var( $attachmentId, FILTER_VALIDATE_INT ) ) {
 			$attachmentId = $this->postmeta->get( $attachmentId );
 		}
 
@@ -71,7 +71,7 @@ 
 	 */
 	protected function normalizeSrc( array $image, $fallback = false )
 	{
-		if( is_array( $fallback ) && count( array_diff( $image, $fallback )) < 2 ) {
+		if( is_array( $fallback ) && count( array_diff( $image, $fallback ) ) < 2 ) {
 			$image = $fallback;
 		}
 		$image = array_pad( $image, 3, '' );

src/Services/Normalizer.php

Spacing +14 added lines, -14 removed lines

@@ -101,7 +101,7 @@ 
 		foreach( $attributes as $key => $value ) {
 			// if data attributes, use single quotes in case of json encoded values
 			$quotes = false !== stripos( $key, 'data-' ) ? "'" : '"';
-			if( is_array( $value )) {
+			if( is_array( $value ) ) {
 				$value = json_encode( $value );
 				$quotes = "'";
 			}
@@ -144,29 +144,29 @@ 
 	 */
 	protected function filterAttributes( array $attributeKeys )
 	{
-		$filtered = array_intersect_key( $this->args, array_flip( $attributeKeys ));
+		$filtered = array_intersect_key( $this->args, array_flip( $attributeKeys ) );
 
 		// normalize truthy boolean attributes
 		foreach( $filtered as $key => $value ) {
-			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES ))continue;
+			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES ) )continue;
 
 			if( $value !== false ) {
-				$filtered[ $key ] = '';
+				$filtered[$key] = '';
 				continue;
 			}
 
-			unset( $filtered[ $key ] );
+			unset( $filtered[$key] );
 		}
 
 		$filteredKeys = array_filter( array_keys( $filtered ), function( $key ) use ( $filtered ) {
 			return !(
-				empty( $filtered[ $key ] )
-				&& !is_numeric( $filtered[ $key ] )
+				empty( $filtered[$key] )
+				&& !is_numeric( $filtered[$key] )
 				&& !in_array( $key, self::BOOLEAN_ATTRIBUTES )
 			);
 		});
 
-		return array_intersect_key( $filtered, array_flip( $filteredKeys ));
+		return array_intersect_key( $filtered, array_flip( $filteredKeys ) );
 	}
 
 	/**
@@ -183,20 +183,20 @@ 
 			foreach( $this->args as $key => $value ) {
 
 				$length = strlen( $wildcard );
-				$result = substr( $key, 0, $length) === $wildcard;
+				$result = substr( $key, 0, $length ) === $wildcard;
 
 				if( $result ) {
 					// only allow data attributes to have an empty value
-					if( $wildcard != 'data-' && empty( $value ))continue;
+					if( $wildcard != 'data-' && empty( $value ) )continue;
 
-					if( is_array( $value )) {
+					if( is_array( $value ) ) {
 
 						if( $wildcard != 'data-' )continue;
 
 						$value = json_encode( $value );
 					}
 
-					$wildcards[ $key ] = $value;
+					$wildcards[$key] = $value;
 				}
 			}
 		}
@@ -209,7 +209,7 @@ 
 	 */
 	protected function filterInputType()
 	{
-		if( !isset( $this->args['type'] ) || !in_array( $this->args['type'], self::INPUT_TYPES )) {
+		if( !isset( $this->args['type'] ) || !in_array( $this->args['type'], self::INPUT_TYPES ) ) {
 			$this->args['type'] = 'text';
 		}
 	}
@@ -219,7 +219,7 @@ 
 	 */
 	protected function parseAttributes( array $attributes, array $args = [] )
 	{
-		if( !empty( $args )) {
+		if( !empty( $args ) ) {
 			$this->args = array_change_key_case( $args );
 		}
 

Braces +10 added lines, -5 removed lines

@@ -2,8 +2,7 @@ 
 
 namespace GeminiLabs\Castor\Services;
 
-class Normalizer
-{
+class Normalizer {
 	const BOOLEAN_ATTRIBUTES = [
 		'autofocus', 'capture', 'checked', 'disabled', 'draggable', 'formnovalidate', 'hidden',
 		'multiple', 'novalidate', 'readonly', 'required', 'selected', 'spellcheck',
@@ -148,7 +147,9 @@ 
 
 		// normalize truthy boolean attributes
 		foreach( $filtered as $key => $value ) {
-			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES ))continue;
+			if( !in_array( $key, self::BOOLEAN_ATTRIBUTES )) {
+				continue;
+			}
 
 			if( $value !== false ) {
 				$filtered[ $key ] = '';
@@ -187,11 +188,15 @@ 
 
 				if( $result ) {
 					// only allow data attributes to have an empty value
-					if( $wildcard != 'data-' && empty( $value ))continue;
+					if( $wildcard != 'data-' && empty( $value )) {
+						continue;
+					}
 
 					if( is_array( $value )) {
 
-						if( $wildcard != 'data-' )continue;
+						if( $wildcard != 'data-' ) {
+							continue;
+						}
 
 						$value = json_encode( $value );
 					}

src/Video.php

Spacing +14 added lines, -14 removed lines

@@ -41,7 +41,7 @@ 
 
 	public function render()
 	{
-		if( !isset( $this->video->html ))return;
+		if( !isset( $this->video->html ) )return;
 		return sprintf(
 			'<div class="video embed">%s%s</div>',
 			$this->renderScreenshot(),
@@ -52,11 +52,11 @@ 
 	public function renderPlayButton()
 	{
 		return sprintf(
-			'<div class="video-play">' .
-				'<div class="video-play-pulse pulse1"></div>' .
-				'<div class="video-play-pulse pulse2"></div>' .
-				'<div class="video-play-pulse pulse3"></div>' .
-				'<a href="%s" class="video-play-btn">%s</a>' .
+			'<div class="video-play">'.
+				'<div class="video-play-pulse pulse1"></div>'.
+				'<div class="video-play-pulse pulse2"></div>'.
+				'<div class="video-play-pulse pulse3"></div>'.
+				'<a href="%s" class="video-play-btn">%s</a>'.
 			'</div>',
 			$this->args['url'],
 			$this->theme->svg( 'play.svg' )
@@ -66,7 +66,7 @@ 
 	public function renderScreenshot()
 	{
 		if( $this->args['image']
-			&& in_array( strtolower( $this->video->provider_name ), $this->supported )) {
+			&& in_array( strtolower( $this->video->provider_name ), $this->supported ) ) {
 			return sprintf( '%s<div class="video-poster" style="background-image: url(%s)">%s</div>',
 				$this->renderSpinner(),
 				$this->args['image'],
@@ -78,10 +78,10 @@ 
 	public function renderSpinner()
 	{
 		return sprintf(
-			'<div class="video-spinner">' .
-				'<div class="spinner"><div class="spinner-dots">%s</div></div>' .
+			'<div class="video-spinner">'.
+				'<div class="spinner"><div class="spinner-dots">%s</div></div>'.
 			'</div>',
-			implode( '', array_fill( 0, 8, '<div class="spinner-dot"></div>' ))
+			implode( '', array_fill( 0, 8, '<div class="spinner-dot"></div>' ) )
 		);
 	}
 
@@ -102,11 +102,11 @@ 
 
 	protected function normalize( $args )
 	{
-		if( is_string( $args )) {
+		if( is_string( $args ) ) {
 			$args = ['url' => $args];
 		}
 
-		$this->args = shortcode_atts([
+		$this->args = shortcode_atts( [
 			'image'  => '', // string || int
 			'player' => '', // string || array
 			'url'    => '', // string
@@ -114,8 +114,8 @@ 
 
 		foreach( $this->args as $key => $value ) {
 			$method = $this->utility->buildMethodName( $key, 'set' );
-			if( !method_exists( $this, $method ))continue;
-			call_user_func([ $this, $method ], $value );
+			if( !method_exists( $this, $method ) )continue;
+			call_user_func( [$this, $method], $value );
 		}
 		return $this->args;
 	}

Braces +7 added lines, -4 removed lines

@@ -8,8 +8,7 @@ 
 use GeminiLabs\Castor\Image;
 use GeminiLabs\Castor\Oembed;
 
-class Video
-{
+class Video {
 	public $video;
 
 	protected $args;
@@ -41,7 +40,9 @@ 
 
 	public function render()
 	{
-		if( !isset( $this->video->html ))return;
+		if( !isset( $this->video->html )) {
+			return;
+		}
 		return sprintf(
 			'<div class="video embed">%s%s</div>',
 			$this->renderScreenshot(),
@@ -114,7 +115,9 @@ 
 
 		foreach( $this->args as $key => $value ) {
 			$method = $this->utility->buildMethodName( $key, 'set' );
-			if( !method_exists( $this, $method ))continue;
+			if( !method_exists( $this, $method )) {
+				continue;
+			}
 			call_user_func([ $this, $method ], $value );
 		}
 		return $this->args;

src/Gallery.php

Spacing +28 added lines, -28 removed lines

@@ -34,7 +34,7 @@ 
 	{
 		$this->normalizeArgs( $args );
 
-		$this->gallery = new WP_Query([
+		$this->gallery = new WP_Query( [
 			'orderby'        => 'post__in',
 			'paged'          => $this->getPaged(),
 			'post__in'       => $this->args['media'],
@@ -42,7 +42,7 @@ 
 			'post_type'      => 'attachment',
 			'post_status'    => 'inherit',
 			'posts_per_page' => $this->args['images_per_page'],
-		]);
+		] );
 
 		return $this;
 	}
@@ -52,9 +52,9 @@ 
 	 */
 	public function render()
 	{
-		if( empty( $this->args['media'] ))return;
+		if( empty( $this->args['media'] ) )return;
 		$images = array_reduce( $this->gallery->posts, function( $images, $attachment ) {
-			return $images . $this->renderImage( $attachment );
+			return $images.$this->renderImage( $attachment );
 		});
 		return sprintf( '<div class="gallery-images" itemscope itemtype="http://schema.org/ImageGallery">%s</div>%s',
 			$images,
@@ -71,8 +71,8 @@ 
 
 		if( !$image )return;
 		return sprintf(
-			'<figure class="gallery-image" data-w="%s" data-h="%s" data-ps=\'%s\' itemprop="associatedMedia" itemscope itemtype="http://schema.org/ImageObject">' .
-				'%s%s' .
+			'<figure class="gallery-image" data-w="%s" data-h="%s" data-ps=\'%s\' itemprop="associatedMedia" itemscope itemtype="http://schema.org/ImageObject">'.
+				'%s%s'.
 			'</figure>',
 			$image->thumbnail['width'],
 			$image->thumbnail['height'],
@@ -88,14 +88,14 @@ 
 	public function renderPagination()
 	{
 		if( !$this->args['pagination'] )return;
-		return paginate_links([
-			'before_page_number' => '<span class="screen-reader-text">' . __( 'Page', 'castor' ) . ' </span>',
+		return paginate_links( [
+			'before_page_number' => '<span class="screen-reader-text">'.__( 'Page', 'castor' ).' </span>',
 			'current'            => $this->gallery->query['paged'],
 			'mid_size'           => 1,
 			'next_text'          => __( 'Next', 'castor' ),
 			'prev_text'          => __( 'Previous', 'castor' ),
 			'total'              => $this->gallery->max_num_pages,
-		]);
+		] );
 	}
 
 	/**
@@ -108,7 +108,7 @@ 
 	{
 		$bool = $this->getValue( $key, $value );
 
-		if( is_null( filter_var( $bool, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE ))) {
+		if( is_null( filter_var( $bool, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE ) ) ) {
 			$bool = $this->postmeta->get( $bool );
 		}
 		return wp_validate_boolean( $bool );
@@ -123,10 +123,10 @@ 
 	{
 		$gallery = $this->getValue( 'gallery', $value );
 
-		if( !is_numeric( $gallery ) && is_string( $gallery )) {
-			$gallery = intval( $this->postmeta->get( $gallery ));
+		if( !is_numeric( $gallery ) && is_string( $gallery ) ) {
+			$gallery = intval( $this->postmeta->get( $gallery ) );
 		}
-		return !is_null( get_post( $gallery ))
+		return !is_null( get_post( $gallery ) )
 			? intval( $gallery )
 			: 0;
 	}
@@ -140,7 +140,7 @@ 
 	{
 		$perPage = $this->getValue( 'images_per_page', $value );
 
-		if( !is_numeric( $perPage ) && is_string( $perPage )) {
+		if( !is_numeric( $perPage ) && is_string( $perPage ) ) {
 			$perPage = $this->postmeta->get( $perPage );
 		}
 		return !!intval( $perPage )
@@ -167,11 +167,11 @@ 
 	{
 		$media = $this->getValue( 'media', $value );
 
-		if( is_string( $media )) {
+		if( is_string( $media ) ) {
 			$media = $this->postmeta->get( $media, [
 				'ID'     => $this->getGalleryArg(),
 				'single' => false,
-			]);
+			] );
 		}
 		return is_array( $media )
 			? wp_parse_id_list( $media )
@@ -183,7 +183,7 @@ 
 	 */
 	protected function getPaged()
 	{
-		return intval( get_query_var(( is_front_page() ? 'page' : 'paged' ))) ?: 1;
+		return intval( get_query_var( ( is_front_page() ? 'page' : 'paged' ) ) ) ?: 1;
 	}
 
 	/**
@@ -229,7 +229,7 @@ 
 	 */
 	protected function getValue( $key, $value = null )
 	{
-		if( is_null( $value ) && isset( $this->args[$key] )) {
+		if( is_null( $value ) && isset( $this->args[$key] ) ) {
 			$value = $this->args[$key];
 		}
 		return $value;
@@ -241,20 +241,20 @@ 
 	protected function normalizeArgs( array $args = [] )
 	{
 		$defaults = [
-			'gallery',         // (string) meta_key | (int) post_id
-			'lazyload',        // (string) meta_key | (bool)
-			'media',           // (string) meta_key | (array) post_ids
-			'pagination',      // (string) meta_key | (bool)
+			'gallery', // (string) meta_key | (int) post_id
+			'lazyload', // (string) meta_key | (bool)
+			'media', // (string) meta_key | (array) post_ids
+			'pagination', // (string) meta_key | (bool)
 			'images_per_page', // (string) meta_key | (int) number
-			'permalinks',      // (string) meta_key | (bool)
+			'permalinks', // (string) meta_key | (bool)
 		];
 
 		$this->args = shortcode_atts( array_combine( $defaults, $defaults ), $args );
 
 		array_walk( $this->args, function( &$value, $key ) {
-			$method = $this->utility->buildMethodName( $key . '_arg' );
-			if( method_exists( $this, $method )) {
-				$value = call_user_func([ $this, $method ], $value );
+			$method = $this->utility->buildMethodName( $key.'_arg' );
+			if( method_exists( $this, $method ) ) {
+				$value = call_user_func( [$this, $method], $value );
 			}
 		});
 
@@ -267,10 +267,10 @@ 
 	 */
 	protected function renderImageCaption( $image )
 	{
-		if( !empty( $image->copyright )) {
+		if( !empty( $image->copyright ) ) {
 			$image->caption .= sprintf( ' <span itemprop="copyrightHolder">%s</span>', $image->copyright );
 		}
-		if( empty( $image->caption ))return;
+		if( empty( $image->caption ) )return;
 		return sprintf( '<figcaption itemprop="caption description">%s</figcaption>', $image->caption );
 	}
 

Braces +13 added lines, -6 removed lines

@@ -9,8 +9,7 @@ 
 use WP_Post;
 use WP_Query;
 
-class Gallery
-{
+class Gallery {
 	public $gallery;
 
 	protected $args;
@@ -52,7 +51,9 @@ 
 	 */
 	public function render()
 	{
-		if( empty( $this->args['media'] ))return;
+		if( empty( $this->args['media'] )) {
+			return;
+		}
 		$images = array_reduce( $this->gallery->posts, function( $images, $attachment ) {
 			return $images . $this->renderImage( $attachment );
 		});
@@ -69,7 +70,9 @@ 
 	{
 		$image = $this->image->get( $attachment->ID )->image;
 
-		if( !$image )return;
+		if( !$image ) {
+			return;
+		}
 		return sprintf(
 			'<figure class="gallery-image" data-w="%s" data-h="%s" data-ps=\'%s\' itemprop="associatedMedia" itemscope itemtype="http://schema.org/ImageObject">' .
 				'%s%s' .
@@ -87,7 +90,9 @@ 
 	 */
 	public function renderPagination()
 	{
-		if( !$this->args['pagination'] )return;
+		if( !$this->args['pagination'] ) {
+			return;
+		}
 		return paginate_links([
 			'before_page_number' => '<span class="screen-reader-text">' . __( 'Page', 'castor' ) . ' </span>',
 			'current'            => $this->gallery->query['paged'],
@@ -270,7 +275,9 @@ 
 		if( !empty( $image->copyright )) {
 			$image->caption .= sprintf( ' <span itemprop="copyrightHolder">%s</span>', $image->copyright );
 		}
-		if( empty( $image->caption ))return;
+		if( empty( $image->caption )) {
+			return;
+		}
 		return sprintf( '<figcaption itemprop="caption description">%s</figcaption>', $image->caption );
 	}