pryley / castor-framework

src/Forms/Form.php

Spacing +40 added lines, -40 removed lines

@@ -33,7 +33,7 @@ 
 	 */
 	protected $normalize;
 
-	public function __construct( Field $field, Normalizer $normalize )
+	public function __construct(Field $field, Normalizer $normalize)
 	{
 		$this->args         = [];
 		$this->dependencies = [];
@@ -48,15 +48,15 @@ 
 	 * @return mixed
 	 * @throws Exception
 	 */
-	public function __get( $property )
+	public function __get($property)
 	{
-		switch( $property ) {
+		switch ($property) {
 			case 'args':
 			case 'dependencies':
 			case 'fields':
 				return $this->$property;
 		}
-		throw new Exception( sprintf( 'Invalid %s property: %s', __CLASS__, $property ));
+		throw new Exception(sprintf('Invalid %s property: %s', __CLASS__, $property));
 	}
 
 	/**
@@ -66,16 +66,16 @@ 
 	 * @return void
 	 * @throws Exception
 	 */
-	public function __set( $property, $value )
+	public function __set($property, $value)
 	{
-		switch( $property ) {
+		switch ($property) {
 			case 'args':
 			case 'dependencies':
 			case 'fields':
 				$this->$property = $value;
 				break;
 			default:
-				throw new Exception( sprintf( 'Invalid %s property: %s', __CLASS__, $property ));
+				throw new Exception(sprintf('Invalid %s property: %s', __CLASS__, $property));
 		}
 	}
 
@@ -84,13 +84,13 @@ 
 	 *
 	 * @return Form
 	 */
-	public function addField( array $args = [] )
+	public function addField(array $args = [])
 	{
-		$field = $this->field->normalize( $args );
+		$field = $this->field->normalize($args);
 
-		if( $field->args['render'] !== false ) {
+		if ($field->args['render'] !== false) {
 			$this->dependencies = array_unique(
-				array_merge( $field->dependencies, $this->dependencies )
+				array_merge($field->dependencies, $this->dependencies)
 			);
 			$this->fields[] = $field;
 		}
@@ -103,7 +103,7 @@ 
 	 *
 	 * @return Form
 	 */
-	public function normalize( array $args = [] )
+	public function normalize(array $args = [])
 	{
 		$defaults = [
 			'action'     => '',
@@ -113,12 +113,12 @@ 
 			'enctype'    => 'multipart/form-data',
 			'method'     => 'post',
 			'nonce'      => '',
-			'submit'     => __( 'Submit', 'site-reviews' ),
+			'submit'     => __('Submit', 'site-reviews'),
 		];
 
-		$this->args = array_merge( $defaults, $args );
+		$this->args = array_merge($defaults, $args);
 
-		$attributes = $this->normalize->form( $this->args, 'implode' );
+		$attributes = $this->normalize->form($this->args, 'implode');
 
 		$this->args['attributes'] = $attributes;
 
@@ -132,15 +132,15 @@ 
 	 *
 	 * @return string|void
 	 */
-	public function render( $print = true )
+	public function render($print = true)
 	{
-		$rendered = sprintf( '<form %s>%s%s</form>',
+		$rendered = sprintf('<form %s>%s%s</form>',
 			$this->args['attributes'],
 			$this->generateFields(),
 			$this->generateSubmitButton()
 		);
 
-		if( !!$print && $print !== 'return' ) {
+		if (!!$print && $print !== 'return') {
 			echo $rendered;
 		}
 
@@ -169,7 +169,7 @@ 
 	{
 		$hiddenFields = '';
 
-		$fields = array_reduce( $this->fields, function( $carry, $formField ) use ( &$hiddenFields ) {
+		$fields = array_reduce($this->fields, function($carry, $formField) use (&$hiddenFields) {
 
 			$stringLegend    = '<legend class="screen-reader-text"><span>%s</span></legend>';
 			$stringFieldset  = '<fieldset%s>%s%s</fieldset>';
@@ -185,30 +185,30 @@ 
 			$rendered = $field->render();
 
 			// render hidden inputs outside the table
-			if( $field->args['type'] === 'hidden' ) {
+			if ($field->args['type'] === 'hidden') {
 				$hiddenFields .= $rendered;
 				return $carry;
 			}
 
-			$hiddenClass = $this->isFieldHidden( $formField ) ? 'hidden' : '';
+			$hiddenClass = $this->isFieldHidden($formField) ? 'hidden' : '';
 
-			if( $multi ) {
-				if( $depends = $formField->getDataDepends() ) {
-					$depends = sprintf( ' data-depends=\'%s\'', json_encode( $depends ));
+			if ($multi) {
+				if ($depends = $formField->getDataDepends()) {
+					$depends = sprintf(' data-depends=\'%s\'', json_encode($depends));
 				}
 
-				$legend = $label ? sprintf( $stringLegend, $label ) : '';
-				$rendered = sprintf( $stringFieldset, $depends, $legend, $rendered );
+				$legend = $label ? sprintf($stringLegend, $label) : '';
+				$rendered = sprintf($stringFieldset, $depends, $legend, $rendered);
 			}
 
 			$renderedField = $field->outside
-				? sprintf( $outsideRendered, $rendered )
-				: sprintf( $stringRendered, $hiddenClass, $label, $rendered );
+				? sprintf($outsideRendered, $rendered)
+				: sprintf($stringRendered, $hiddenClass, $label, $rendered);
 
 			return $carry . $renderedField;
 		});
 
-		return sprintf( '<table class="form-table"><tbody>%s</tbody></table>%s', $fields, $hiddenFields );
+		return sprintf('<table class="form-table"><tbody>%s</tbody></table>%s', $fields, $hiddenFields);
 	}
 
 	/**
@@ -220,19 +220,19 @@ 
 	{
 		$args = $this->args['submit'];
 
-		is_array( $args ) ?: $args = ['text' => $args ];
+		is_array($args) ?: $args = ['text' => $args];
 
 		$args = shortcode_atts([
-			'text' => __( 'Save Changes', 'site-reviews' ),
+			'text' => __('Save Changes', 'site-reviews'),
 			'type' => 'primary',
 			'name' => 'submit',
 			'wrap' => true,
 			'other_attributes' => null,
-		], $args );
+		], $args);
 
-		if( is_admin() ) {
+		if (is_admin()) {
 			ob_start();
-			submit_button( $args['text'], $args['type'], $args['name'], $args['wrap'], $args['other_attributes'] );
+			submit_button($args['text'], $args['type'], $args['name'], $args['wrap'], $args['other_attributes']);
 			return ob_get_clean();
 		}
 	}
@@ -242,15 +242,15 @@ 
 	 *
 	 * @return bool|null
 	 */
-	protected function isFieldHidden( $field )
+	protected function isFieldHidden($field)
 	{
-		if( !( $dependsOn = $field->getDataDepends() ))return;
+		if (!($dependsOn = $field->getDataDepends()))return;
 
-		foreach( $this->fields as $formField ) {
-			if( $dependsOn['name'] !== $formField->args['name'] )continue;
+		foreach ($this->fields as $formField) {
+			if ($dependsOn['name'] !== $formField->args['name'])continue;
 
-			if( is_array( $dependsOn['value'] )) {
-				return !in_array( $formField->args['value'], $dependsOn['value'] );
+			if (is_array($dependsOn['value'])) {
+				return !in_array($formField->args['value'], $dependsOn['value']);
 			}
 
 			return $dependsOn['value'] != $formField->args['value'];

Braces +6 added lines, -2 removed lines

@@ -244,10 +244,14 @@
 	 */
 	protected function isFieldHidden( $field )
 	{
-		if( !( $dependsOn = $field->getDataDepends() ))return;
+		if( !( $dependsOn = $field->getDataDepends() )) {
+			return;
+		}
 
 		foreach( $this->fields as $formField ) {
-			if( $dependsOn['name'] !== $formField->args['name'] )continue;
+			if( $dependsOn['name'] !== $formField->args['name'] ) {
+				continue;
+			}
 
 			if( is_array( $dependsOn['value'] )) {
 				return !in_array( $formField->args['value'], $dependsOn['value'] );

src/Forms/Field.php

Spacing +65 added lines, -65 removed lines

@@ -26,14 +26,14 @@ 
 	 * @return mixed
 	 * @throws Exception
 	 */
-	public function __get( $property )
+	public function __get($property)
 	{
-		switch( $property ) {
+		switch ($property) {
 			case 'args':
 			case 'dependencies':
 				return $this->$property;
 		}
-		throw new Exception( sprintf( 'Invalid %s property: %s', __CLASS__, $property ));
+		throw new Exception(sprintf('Invalid %s property: %s', __CLASS__, $property));
 	}
 
 
@@ -42,19 +42,19 @@ 
 	 *
 	 * @return mixed GeminiLabs\SiteReviews\Html\Fields\*
 	 */
-	public function getField( array $args = [] )
+	public function getField(array $args = [])
 	{
-		if( empty( $args )) {
+		if (empty($args)) {
 			$args = $this->args;
 		}
 
-		$className = sprintf( 'GeminiLabs\Castor\Forms\Fields\%s', ucfirst( $args['type'] ));
+		$className = sprintf('GeminiLabs\Castor\Forms\Fields\%s', ucfirst($args['type']));
 
-		if( !class_exists( $className )) {
-			throw new ReflectionException( "Class does not exist: {$className}" );
+		if (!class_exists($className)) {
+			throw new ReflectionException("Class does not exist: {$className}");
 		}
 
-		return (new $className( $args ));
+		return (new $className($args));
 	}
 
 	/**
@@ -62,7 +62,7 @@ 
 	 *
 	 * @return $this
 	 */
-	public function normalize( array $args = [] )
+	public function normalize(array $args = [])
 	{
 		$defaults = [
 			'after'       => '',
@@ -86,23 +86,23 @@ 
 			'value'       => '',
 		];
 
-		$args = $atts = wp_parse_args( $args, $defaults );
+		$args = $atts = wp_parse_args($args, $defaults);
 
-		$args['attributes'] = $this->parseAttributes( $atts );
-		$args['id']         = $this->parseId( $atts );
-		$args['inline']     = $this->parseInline( $atts );
-		$args['type']       = $this->parseType( $atts );
-		$args['name']       = $this->parseName( $atts );
+		$args['attributes'] = $this->parseAttributes($atts);
+		$args['id']         = $this->parseId($atts);
+		$args['inline']     = $this->parseInline($atts);
+		$args['type']       = $this->parseType($atts);
+		$args['name']       = $this->parseName($atts);
 		$args['options']    = (array) $atts['options']; // make sure this is always an array
 		$args['path']       = $atts['name'];
-		$args['prefix']     = $this->parsePrefix( $atts );
-		$args['value']      = $this->parseValue( $atts );
+		$args['prefix']     = $this->parsePrefix($atts);
+		$args['value']      = $this->parseValue($atts);
 
 		$this->args = $args;
-		$this->dependencies = $this->getField( $args )->dependencies;
+		$this->dependencies = $this->getField($args)->dependencies;
 
 		$this->setDataDepends();
-		$this->checkForErrors( $atts );
+		$this->checkForErrors($atts);
 
 		return $this;
 	}
@@ -114,9 +114,9 @@ 
 	 *
 	 * @return string|void
 	 */
-	public function render( $print = true )
+	public function render($print = true)
 	{
-		if( $this->args['render'] === false )return;
+		if ($this->args['render'] === false)return;
 
 		$field = $this->getField();
 
@@ -125,16 +125,16 @@ 
 
 		$renderedString = '%s';
 
-		if( ( isset( $field->args['required'] ) && $field->args['required'] )
-			|| ( isset( $field->args['attributes']['required'] ) || in_array( 'required', $field->args['attributes'] )) ) {
+		if ((isset($field->args['required']) && $field->args['required'])
+			|| (isset($field->args['attributes']['required']) || in_array('required', $field->args['attributes']))) {
 			$class .= ' glsr-required';
 		}
 
-		if( $field->args['type'] !== 'hidden' ) {
-			$renderedString = sprintf( '<div class="%s">%%s</div>', $class );
+		if ($field->args['type'] !== 'hidden') {
+			$renderedString = sprintf('<div class="%s">%%s</div>', $class);
 		}
 
-		$rendered = sprintf( $renderedString,
+		$rendered = sprintf($renderedString,
 			$this->args['before'] .
 			$field->generateLabel() .
 			$field->render() .
@@ -142,9 +142,9 @@ 
 			$this->args['errors']
 		);
 
-		$rendered = apply_filters( 'castor/rendered/field', $rendered, $field->args['type'] );
+		$rendered = apply_filters('castor/rendered/field', $rendered, $field->args['type']);
 
-		if( !!$print && $print !== 'return' ) {
+		if (!!$print && $print !== 'return') {
 			echo $rendered;
 		}
 
@@ -168,22 +168,22 @@ 
 	 *
 	 * @return void
 	 */
-	protected function checkForErrors( array $atts )
+	protected function checkForErrors(array $atts)
 	{
 		$args = $this->args;
 
-		if( !array_key_exists( $atts['name'], $args['errors'] )) {
+		if (!array_key_exists($atts['name'], $args['errors'])) {
 			$this->args['errors'] = ''; // set to an empty string
 			return;
 		}
 
-		$field_errors = $args['errors'][ $atts['name'] ];
+		$field_errors = $args['errors'][$atts['name']];
 
-		$errors = array_reduce( $field_errors['errors'], function( $carry, $error ) {
-			return $carry . sprintf( '<span>%s</span> ', $error );
+		$errors = array_reduce($field_errors['errors'], function($carry, $error) {
+			return $carry . sprintf('<span>%s</span> ', $error);
 		});
 
-		$this->args['errors'] = sprintf( '<span class="glsr-field-errors">%s</span>', $errors );
+		$this->args['errors'] = sprintf('<span class="glsr-field-errors">%s</span>', $errors);
 	}
 
 	/**
@@ -191,19 +191,19 @@ 
 	 *
 	 * @return array
 	 */
-	protected function parseAttributes( array $args )
+	protected function parseAttributes(array $args)
 	{
-		if( empty( $args['attributes'] )) {
+		if (empty($args['attributes'])) {
 			return [];
 		}
 
 		$attributes = (array) $args['attributes'];
 
-		foreach( $attributes as $key => $value ) {
-			if( is_string( $key ))continue;
-			unset( $attributes[ $key ] );
-			if( !isset( $attributes[ $value ] )) {
-				$attributes[ $value ] = '';
+		foreach ($attributes as $key => $value) {
+			if (is_string($key))continue;
+			unset($attributes[$key]);
+			if (!isset($attributes[$value])) {
+				$attributes[$value] = '';
 			}
 		}
 
@@ -215,13 +215,13 @@ 
 	 *
 	 * @return null|string
 	 */
-	protected function parseId( array $args )
+	protected function parseId(array $args)
 	{
-		if( isset( $args['id'] ) && !$args['id'] )return;
+		if (isset($args['id']) && !$args['id'])return;
 
 		!$args['suffix'] ?: $args['suffix'] = "-{$args['suffix']}";
 
-		return str_replace( ['[]','[',']','.'], ['','-','','-'], $this->parseName( $args ) . $args['suffix'] );
+		return str_replace(['[]', '[', ']', '.'], ['', '-', '', '-'], $this->parseName($args) . $args['suffix']);
 	}
 
 	/**
@@ -229,9 +229,9 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function parseInline( array $args )
+	protected function parseInline(array $args)
 	{
-		return false !== stripos( $args['type'], '_inline' )
+		return false !== stripos($args['type'], '_inline')
 			? true
 			: $args['inline'];
 	}
@@ -241,20 +241,20 @@ 
 	 *
 	 * @return string
 	 */
-	protected function parseName( array $args )
+	protected function parseName(array $args)
 	{
 		$name   = $args['name'];
-		$prefix = $this->parsePrefix( $args );
+		$prefix = $this->parsePrefix($args);
 
-		if( $prefix === false ) {
+		if ($prefix === false) {
 			return $name;
 		}
 
-		$paths = explode( '.', $name );
+		$paths = explode('.', $name);
 
-		return array_reduce( $paths, function( $result, $value ) {
+		return array_reduce($paths, function($result, $value) {
 			return $result .= "[$value]";
-		}, $prefix );
+		}, $prefix);
 	}
 
 	/**
@@ -262,7 +262,7 @@ 
 	 *
 	 * @return string|false
 	 */
-	protected function parsePrefix( array $args )
+	protected function parsePrefix(array $args)
 	{
 		return $args['prefix'];
 	}
@@ -272,12 +272,12 @@ 
 	 *
 	 * @return string
 	 */
-	protected function parseType( array $args )
+	protected function parseType(array $args)
 	{
 		$type = $args['type'];
 
-		return false !== stripos( $type, '_inline' )
-			? str_replace( '_inline', '', $type )
+		return false !== stripos($type, '_inline')
+			? str_replace('_inline', '', $type)
 			: $type;
 	}
 
@@ -286,18 +286,18 @@ 
 	 *
 	 * @return string
 	 */
-	protected function parseValue( array $args )
+	protected function parseValue(array $args)
 	{
 		$default = $args['default'];
 		$name    = $args['name'];
 		$prefix  = $args['prefix'];
 		$value   = $args['value'];
 
-		if( $default == ':placeholder' ) {
+		if ($default == ':placeholder') {
 			$default = '';
 		}
 
-		return ( !empty( $value ) || !$name || $prefix === false )
+		return (!empty($value) || !$name || $prefix === false)
 			? $value
 			: $default;
 	}
@@ -329,15 +329,15 @@ 
 	 */
 	protected function setDataDepends()
 	{
-		if( !( $depends = $this->args['depends'] ))return;
+		if (!($depends = $this->args['depends']))return;
 
 		$name  = $depends;
 		$value = true;
 
-		if( is_array( $depends )) {
-			reset( $depends );
-			$name  = key( $depends );
-			$value = $depends[ $name ];
+		if (is_array($depends)) {
+			reset($depends);
+			$name  = key($depends);
+			$value = $depends[$name];
 		}
 
 		$name = $this->parseName([

Braces +12 added lines, -4 removed lines

@@ -116,7 +116,9 @@ 
 	 */
 	public function render( $print = true )
 	{
-		if( $this->args['render'] === false )return;
+		if( $this->args['render'] === false ) {
+			return;
+		}
 
 		$field = $this->getField();
 
@@ -200,7 +202,9 @@ 
 		$attributes = (array) $args['attributes'];
 
 		foreach( $attributes as $key => $value ) {
-			if( is_string( $key ))continue;
+			if( is_string( $key )) {
+				continue;
+			}
 			unset( $attributes[ $key ] );
 			if( !isset( $attributes[ $value ] )) {
 				$attributes[ $value ] = '';
@@ -217,7 +221,9 @@ 
 	 */
 	protected function parseId( array $args )
 	{
-		if( isset( $args['id'] ) && !$args['id'] )return;
+		if( isset( $args['id'] ) && !$args['id'] ) {
+			return;
+		}
 
 		!$args['suffix'] ?: $args['suffix'] = "-{$args['suffix']}";
 
@@ -329,7 +335,9 @@ 
 	 */
 	protected function setDataDepends()
 	{
-		if( !( $depends = $this->args['depends'] ))return;
+		if( !( $depends = $this->args['depends'] )) {
+			return;
+		}
 
 		$name  = $depends;
 		$value = true;

src/Helpers/Utility.php

Spacing +36 added lines, -36 removed lines

@@ -7,22 +7,22 @@ 
 	/**
 	 * @return string
 	 */
-	public function buildAttributes( array $atts = [] )
+	public function buildAttributes(array $atts = [])
 	{
 		$attributes = [];
-		foreach( $atts as $key => $value ) {
-			$attributes[] = sprintf( '%s="%s"', $key, $value );
+		foreach ($atts as $key => $value) {
+			$attributes[] = sprintf('%s="%s"', $key, $value);
 		}
-		return implode( ' ', $attributes );
+		return implode(' ', $attributes);
 	}
 
 	/**
 	 * @return string
 	 */
-	public function buildAttributesFor( $tag, array $atts = [] )
+	public function buildAttributesFor($tag, array $atts = [])
 	{
 		return $this->buildAttributes(
-			wp_parse_args( $atts, apply_filters( "castor/render/$tag/attributes", [] ))
+			wp_parse_args($atts, apply_filters("castor/render/$tag/attributes", []))
 		);
 	}
 
@@ -32,13 +32,13 @@ 
 	 *
 	 * @return string
 	 */
-	public function buildClassName( $name, $path = '' )
+	public function buildClassName($name, $path = '')
 	{
-		$className = array_map( 'ucfirst', array_map( 'strtolower', preg_split( '/[-_]/', $name )));
-		$className = implode( '', $className );
+		$className = array_map('ucfirst', array_map('strtolower', preg_split('/[-_]/', $name)));
+		$className = implode('', $className);
 
-		return !empty( $path )
-			? str_replace( '\\\\', '\\', sprintf( '%s\%s', $path, $className ))
+		return !empty($path)
+			? str_replace('\\\\', '\\', sprintf('%s\%s', $path, $className))
 			: $className;
 	}
 
@@ -48,9 +48,9 @@ 
 	 *
 	 * @return string
 	 */
-	public function buildMethodName( $name, $prefix = 'get' )
+	public function buildMethodName($name, $prefix = 'get')
 	{
-		return lcfirst( $this->buildClassName( $prefix . '-' . $name ));
+		return lcfirst($this->buildClassName($prefix . '-' . $name));
 	}
 
 	/**
@@ -60,9 +60,9 @@ 
 	 *
 	 * @return string
 	 */
-	public function endWith( $suffix, $string, $unique = true )
+	public function endWith($suffix, $string, $unique = true)
 	{
-		return $unique && $this->endsWith( $suffix, $string )
+		return $unique && $this->endsWith($suffix, $string)
 			? $string
 			: $string . $suffix;
 	}
@@ -73,11 +73,11 @@ 
 	 *
 	 * @return string
 	 */
-	public function endsWith( $needle, $haystack )
+	public function endsWith($needle, $haystack)
 	{
-		$length = strlen( $needle );
+		$length = strlen($needle);
 		return $length != 0
-			? substr( $haystack, -$length ) === $needle
+			? substr($haystack, -$length) === $needle
 			: true;
 	}
 
@@ -87,12 +87,12 @@ 
 	 *
 	 * @return void
 	 */
-	public function printTag( $tag, $value, array $attributes = [] )
+	public function printTag($tag, $value, array $attributes = [])
 	{
-		$attributes = $this->buildAttributesFor( $tag, $attributes );
+		$attributes = $this->buildAttributesFor($tag, $attributes);
 
-		printf( '<%s>%s</%s>',
-			rtrim( sprintf( '%s %s', $tag, $attributes )),
+		printf('<%s>%s</%s>',
+			rtrim(sprintf('%s %s', $tag, $attributes)),
 			$value,
 			$tag
 		);
@@ -105,9 +105,9 @@ 
 	 *
 	 * @return string
 	 */
-	public function startWith( $prefix, $string, $unique = true )
+	public function startWith($prefix, $string, $unique = true)
 	{
-		return $unique && $this->startsWith( $prefix, $string )
+		return $unique && $this->startsWith($prefix, $string)
 			? $string
 			: $prefix . $string;
 	}
@@ -118,9 +118,9 @@ 
 	 *
 	 * @return string
 	 */
-	public function startsWith( $needle, $haystack )
+	public function startsWith($needle, $haystack)
 	{
-		return substr( $haystack, 0, strlen( $needle )) === $needle;
+		return substr($haystack, 0, strlen($needle)) === $needle;
 	}
 
 	/**
@@ -128,12 +128,12 @@ 
 	 *
 	 * @return array
 	 */
-	public function toArray( $value )
+	public function toArray($value)
 	{
-		if( is_string( $value )) {
-			$value = trim( $value );
+		if (is_string($value)) {
+			$value = trim($value);
 		}
-		return array_filter((array) $value );
+		return array_filter((array) $value);
 	}
 
 	/**
@@ -143,11 +143,11 @@ 
 	 *
 	 * @return string
 	 */
-	public function trimLeft( $string, $needle, $caseSensitive = true )
+	public function trimLeft($string, $needle, $caseSensitive = true)
 	{
 		$strPos = $caseSensitive ? "strpos" : "stripos";
-		if( $strPos( $string, $needle ) === 0 ) {
-			$string = substr( $string, strlen( $needle ));
+		if ($strPos($string, $needle) === 0) {
+			$string = substr($string, strlen($needle));
 		}
 		return $string;
 	}
@@ -159,11 +159,11 @@ 
 	 *
 	 * @return string
 	 */
-	public function trimRight( $string, $needle, $caseSensitive = true )
+	public function trimRight($string, $needle, $caseSensitive = true)
 	{
 		$strPos = $caseSensitive ? "strpos" : "stripos";
-		if( $strPos( $string, $needle, strlen( $string ) - strlen( $needle )) !== false ) {
-			$string = substr( $string, 0, -strlen( $needle ));
+		if ($strPos($string, $needle, strlen($string) - strlen($needle)) !== false) {
+			$string = substr($string, 0, -strlen($needle));
 		}
 		return $string;
 	}

src/Facade.php

Spacing +11 added lines, -11 removed lines

@@ -30,15 +30,15 @@ 
 	 * @return mixed
 	 * @throws \RuntimeException
 	 */
-	public static function __callStatic( $method, $args )
+	public static function __callStatic($method, $args)
 	{
 		$instance = static::getFacadeRoot();
 
-		if( !$instance ) {
-			throw new RuntimeException( 'A facade root has not been set.' );
+		if (!$instance) {
+			throw new RuntimeException('A facade root has not been set.');
 		}
 
-		return $instance->$method( ...$args );
+		return $instance->$method(...$args);
 	}
 
 	/**
@@ -68,7 +68,7 @@ 
 	 */
 	public static function getFacadeRoot()
 	{
-		return static::resolveFacadeInstance( static::getFacadeAccessor() );
+		return static::resolveFacadeInstance(static::getFacadeAccessor());
 	}
 
 	/**
@@ -76,7 +76,7 @@ 
 	 *
 	 * @return void
 	 */
-	public static function setFacadeApplication( Application $app )
+	public static function setFacadeApplication(Application $app)
 	{
 		static::$app = $app;
 	}
@@ -90,7 +90,7 @@ 
 	 */
 	protected static function getFacadeAccessor()
 	{
-		throw new RuntimeException( 'Facade does not implement getFacadeAccessor method.' );
+		throw new RuntimeException('Facade does not implement getFacadeAccessor method.');
 	}
 
 	/**
@@ -100,16 +100,16 @@ 
 	 *
 	 * @return mixed
 	 */
-	protected static function resolveFacadeInstance( $name )
+	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];
 		}
 
-		return static::$resolvedInstance[$name] = static::$app->make( $name );
+		return static::$resolvedInstance[$name] = static::$app->make($name);
 	}
 }

src/Controller.php

Spacing +48 added lines, -48 removed lines

@@ -16,24 +16,24 @@ 
 	 */
 	public function afterSetupTheme()
 	{
-		add_editor_style( Theme::assetUri( 'css/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' );
-		add_theme_support( 'soil-clean-up' );
-		add_theme_support( 'soil-jquery-cdn' );
-		add_theme_support( 'soil-nav-walker' );
-		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' );
-
-		$menus = apply_filters( 'castor/register/nav_menus', [
-			'main_menu' => __( 'Main Menu', 'castor' ),
+		add_editor_style(Theme::assetUri('css/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');
+		add_theme_support('soil-clean-up');
+		add_theme_support('soil-jquery-cdn');
+		add_theme_support('soil-nav-walker');
+		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');
+
+		$menus = apply_filters('castor/register/nav_menus', [
+			'main_menu' => __('Main Menu', 'castor'),
 		]);
 
-		foreach( $menus as $location => $description ) {
-			register_nav_menu( $location, $description );
+		foreach ($menus as $location => $description) {
+			register_nav_menu($location, $description);
 		}
 	}
 
@@ -43,7 +43,7 @@ 
 	 */
 	public function filterLoginTitle()
 	{
-		return get_bloginfo( 'name' );
+		return get_bloginfo('name');
 	}
 
 	/**
@@ -52,18 +52,18 @@ 
 	 */
 	public function filterLoginUrl()
 	{
-		return get_bloginfo( 'url' );
+		return get_bloginfo('url');
 	}
 
 	/**
 	 * @return string
 	 * @filter template_include
 	 */
-	public function filterTemplate( $template )
+	public function filterTemplate($template)
 	{
-		if( is_string( $template )) {
-			$template = Template::setLayout( $template );
-			Development::storeTemplatePath( $template );
+		if (is_string($template)) {
+			$template = Template::setLayout($template);
+			Development::storeTemplatePath($template);
 		}
 		return $template;
 	}
@@ -72,11 +72,11 @@ 
 	 * @return array
 	 * @filter {$type}_template_hierarchy
 	 */
-	public function filterTemplateHierarchy( array $templates )
+	public function filterTemplateHierarchy(array $templates)
 	{
-		return array_map( function( $template ) {
-			return Utility::startWith( 'templates/', $template );
-		}, $templates );
+		return array_map(function($template) {
+			return Utility::startWith('templates/', $template);
+		}, $templates);
 	}
 
 	/**
@@ -85,8 +85,8 @@ 
 	 */
 	public function login()
 	{
-		if( file_exists( Theme::assetPath( 'css/login.css' ))) {
-			printf( '<link rel="stylesheet" href="%s">', Theme::assetUri( 'css/login.css' ));
+		if (file_exists(Theme::assetPath('css/login.css'))) {
+			printf('<link rel="stylesheet" href="%s">', Theme::assetUri('css/login.css'));
 		}
 	}
 
@@ -96,20 +96,20 @@ 
 	 */
 	public function registerAssets()
 	{
-		wp_enqueue_style( 'castor/main.css',
-			Theme::assetUri( 'css/main.css' ),
-			apply_filters( 'castor/enqueue/css/deps', [] ),
+		wp_enqueue_style('castor/main.css',
+			Theme::assetUri('css/main.css'),
+			apply_filters('castor/enqueue/css/deps', []),
 			null
 		);
-		wp_enqueue_script( 'castor/main.js',
-			Theme::assetUri( 'js/main.js' ),
-			apply_filters( 'castor/enqueue/js/deps', [] ),
+		wp_enqueue_script('castor/main.js',
+			Theme::assetUri('js/main.js'),
+			apply_filters('castor/enqueue/js/deps', []),
 			null,
 			true
 		);
-		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' ),
+		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'),
 			])
 		);
 	}
@@ -118,13 +118,13 @@ 
 	 * @return void
 	 * @action customize_register
 	 */
-	public function registerCustomizer( WP_Customize_Manager $manager )
+	public function registerCustomizer(WP_Customize_Manager $manager)
 	{
-		$manager->get_setting( 'blogname' )->transport = 'postMessage';
-		$manager->selective_refresh->add_partial( 'blogname', [
+		$manager->get_setting('blogname')->transport = 'postMessage';
+		$manager->selective_refresh->add_partial('blogname', [
 			'selector'        => '.brand',
 			'render_callback' => function() {
-				bloginfo( 'name' );
+				bloginfo('name');
 			},
 		]);
 	}
@@ -135,7 +135,7 @@ 
 	 */
 	public function registerCustomizerAssets()
 	{
-		wp_enqueue_script( 'castor/customizer.js', Theme::assetUri( 'js/customizer.js' ), ['customize-preview'], null, true );
+		wp_enqueue_script('castor/customizer.js', Theme::assetUri('js/customizer.js'), ['customize-preview'], null, true);
 	}
 
 	/**
@@ -144,23 +144,23 @@ 
 	 */
 	public function registerSidebars()
 	{
-		$defaults = apply_filters( 'castor/register/sidebars/defaults', [
+		$defaults = apply_filters('castor/register/sidebars/defaults', [
 			'before_widget' => '<div class="widget %1$s %2$s">',
 			'after_widget'  => '</div>',
 			'before_title'  => '<h4>',
 			'after_title'   => '</h4>',
 		]);
 
-		$sidebars = apply_filters( 'castor/register/sidebars', [
-			'sidebar-primary' => __( 'Primary Sidebar', 'castor' ),
-			'sidebar-footer'  => __( 'Footer Sidebar', 'castor' ),
+		$sidebars = apply_filters('castor/register/sidebars', [
+			'sidebar-primary' => __('Primary Sidebar', 'castor'),
+			'sidebar-footer'  => __('Footer Sidebar', 'castor'),
 		]);
 
-		foreach( $sidebars as $id => $name ) {
+		foreach ($sidebars as $id => $name) {
 			register_sidebar([
 				'id'   => $id,
 				'name' => $name,
-			] + $defaults );
+			] + $defaults);
 		}
 	}
 }

src/Facades/SiteMeta.php

Indentation +9 added lines, -9 removed lines

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

src/Helpers/SiteMeta.php

Spacing +12 added lines, -12 removed lines

@@ -11,17 +11,17 @@ 
 	 *
 	 * @return mixed
 	 */
-	public function get( $group, $key = false, $fallback = '' )
+	public function get($group, $key = false, $fallback = '')
 	{
-		$metaKey = sprintf( '%ssettings-%s', apply_filters( 'castor/sitemeta/prefix', 'pollux_' ), $group );
-		$options = get_option( $metaKey, false );
+		$metaKey = sprintf('%ssettings-%s', apply_filters('castor/sitemeta/prefix', 'pollux_'), $group);
+		$options = get_option($metaKey, false);
 
-		if( !$options || !is_array( $options )) {
+		if (!$options || !is_array($options)) {
 			return $fallback;
 		}
 
-		return is_string( $key )
-			? $this->normalize( $options, $key, $fallback )
+		return is_string($key)
+			? $this->normalize($options, $key, $fallback)
 			: $options;
 	}
 
@@ -31,19 +31,19 @@ 
 	 *
 	 * @return mixed
 	 */
-	protected function normalize( array $options, $key, $fallback )
+	protected function normalize(array $options, $key, $fallback)
 	{
-		if( !array_key_exists( $key, $options )) {
+		if (!array_key_exists($key, $options)) {
 			return $fallback;
 		}
 
 		$option = $options[$key];
 
-		$option = is_array( $option )
-			? array_filter( $option )
-			: trim( $option );
+		$option = is_array($option)
+			? array_filter($option)
+			: trim($option);
 
-		return empty( $option )
+		return empty($option)
 			? $fallback
 			: $option;
 	}

src/Helpers/Development.php

Spacing +23 added lines, -23 removed lines

@@ -9,25 +9,25 @@ 
 	public function capture()
 	{
 		ob_start();
-		call_user_func_array( [$this, 'printF'], func_get_args() );
+		call_user_func_array([$this, 'printF'], func_get_args());
 		return ob_get_clean();
 	}
 
 	public function className()
 	{
-		return $this->isDev() && in_array( DEV, ['css', true] )
+		return $this->isDev() && in_array(DEV, ['css', true])
 			? 'dev'
 			: '';
 	}
 
 	public function debug()
 	{
-		call_user_func_array( [$this, 'printF'], func_get_args() );
+		call_user_func_array([$this, 'printF'], func_get_args());
 	}
 
 	public function isDev()
 	{
-		return defined( 'DEV' ) && !!DEV && WP_ENV == 'development';
+		return defined('DEV') && !!DEV && WP_ENV == 'development';
 	}
 
 	public function isProduction()
@@ -35,30 +35,30 @@ 
 		return WP_ENV == 'production';
 	}
 
-	public function printFiltersFor( $hook = '' )
+	public function printFiltersFor($hook = '')
 	{
 		global $wp_filter;
-		if( empty( $hook ) || !isset( $wp_filter[$hook] ))return;
-		$this->printF( $wp_filter[ $hook ] );
+		if (empty($hook) || !isset($wp_filter[$hook]))return;
+		$this->printF($wp_filter[$hook]);
 	}
 
 	public function printTemplatePaths()
 	{
-		if( $this->isDev() && ( DEV == 'templates' || DEV === true )) {
-			$templates = array_keys( array_flip( $this->templatePaths ));
-			$templates = array_map( function( $key, $value ) {
-				return sprintf( '[%s] => %s', $key, $value );
-			}, array_keys( $templates ), $templates );
+		if ($this->isDev() && (DEV == 'templates' || DEV === true)) {
+			$templates = array_keys(array_flip($this->templatePaths));
+			$templates = array_map(function($key, $value) {
+				return sprintf('[%s] => %s', $key, $value);
+			}, array_keys($templates), $templates);
 
-			$this->printF( implode( "\n", $templates ));
+			$this->printF(implode("\n", $templates));
 		}
 	}
 
-	public function storeTemplatePath( $template )
+	public function storeTemplatePath($template)
 	{
-		if( is_string( $template )) {
-			$themeName = basename( strstr( $template, '/templates/', true ));
-			$this->templatePaths[] = $themeName . strstr( $template, '/templates/' );
+		if (is_string($template)) {
+			$themeName = basename(strstr($template, '/templates/', true));
+			$this->templatePaths[] = $themeName . strstr($template, '/templates/');
 		}
 	}
 
@@ -66,15 +66,15 @@ 
 	{
 		$args = func_num_args();
 
-		if( $args == 1 ) {
-			printf( '<div class="print__r"><pre>%s</pre></div>',
-				htmlspecialchars( print_r( func_get_arg(0), true ), ENT_QUOTES, 'UTF-8' )
+		if ($args == 1) {
+			printf('<div class="print__r"><pre>%s</pre></div>',
+				htmlspecialchars(print_r(func_get_arg(0), true), ENT_QUOTES, 'UTF-8')
 			);
 		}
-		else if( $args > 1 ) {
+		else if ($args > 1) {
 			echo '<div class="print__r_group">';
-			foreach( func_get_args() as $value ) {
-				$this->printF( $value );
+			foreach (func_get_args() as $value) {
+				$this->printF($value);
 			}
 			echo '</div>';
 		}

Braces +4 added lines, -3 removed lines

@@ -38,7 +38,9 @@ 
 	public function printFiltersFor( $hook = '' )
 	{
 		global $wp_filter;
-		if( empty( $hook ) || !isset( $wp_filter[$hook] ))return;
+		if( empty( $hook ) || !isset( $wp_filter[$hook] )) {
+			return;
+		}
 		$this->printF( $wp_filter[ $hook ] );
 	}
 
@@ -70,8 +72,7 @@ 
 			printf( '<div class="print__r"><pre>%s</pre></div>',
 				htmlspecialchars( print_r( func_get_arg(0), true ), ENT_QUOTES, 'UTF-8' )
 			);
-		}
-		else if( $args > 1 ) {
+		} else if( $args > 1 ) {
 			echo '<div class="print__r_group">';
 			foreach( func_get_args() as $value ) {
 				$this->printF( $value );

src/Gallery.php

Spacing +64 added lines, -64 removed lines

@@ -19,7 +19,7 @@ 
 	protected $theme;
 	protected $utility;
 
-	public function __construct( Image $image, PostMeta $postmeta, Theme $theme, Utility $utility )
+	public function __construct(Image $image, PostMeta $postmeta, Theme $theme, Utility $utility)
 	{
 		$this->image    = $image;
 		$this->postmeta = $postmeta;
@@ -30,9 +30,9 @@ 
 	/**
 	 * @return WP_Query
 	 */
-	public function get( array $args = [] )
+	public function get(array $args = [])
 	{
-		$this->normalizeArgs( $args );
+		$this->normalizeArgs($args);
 
 		$this->gallery = new WP_Query([
 			'orderby'        => 'post__in',
@@ -52,11 +52,11 @@ 
 	 */
 	public function render()
 	{
-		if( empty( $this->args['media'] ))return;
-		$images = array_reduce( $this->gallery->posts, function( $images, $attachment ) {
-			return $images . $this->renderImage( $attachment );
+		if (empty($this->args['media']))return;
+		$images = array_reduce($this->gallery->posts, function($images, $attachment) {
+			return $images . $this->renderImage($attachment);
 		});
-		return sprintf( '<div class="gallery-images" itemscope itemtype="http://schema.org/ImageGallery">%s</div>%s',
+		return sprintf('<div class="gallery-images" itemscope itemtype="http://schema.org/ImageGallery">%s</div>%s',
 			$images,
 			$this->renderPagination()
 		);
@@ -65,11 +65,11 @@ 
 	/**
 	 * @return null|string
 	 */
-	public function renderImage( WP_Post $attachment )
+	public function renderImage(WP_Post $attachment)
 	{
-		$image = $this->image->get( $attachment->ID )->image;
+		$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<figcaption itemprop="caption description">' .
@@ -78,8 +78,8 @@ 
 			'</figure>',
 			$image->thumbnail['width'],
 			$image->thumbnail['height'],
-			$this->getPhotoswipeData( $image ),
-			$this->renderImageTag( $image ),
+			$this->getPhotoswipeData($image),
+			$this->renderImageTag($image),
 			$image->caption,
 			$image->copyright
 		);
@@ -90,13 +90,13 @@ 
 	 */
 	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>',
+			'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' ),
+			'next_text'          => __('Next', 'castor'),
+			'prev_text'          => __('Previous', 'castor'),
 			'total'              => $this->gallery->max_num_pages,
 		]);
 	}
@@ -107,14 +107,14 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function getBoolValue( $key, $value = null )
+	protected function getBoolValue($key, $value = null)
 	{
-		$bool = $this->getValue( $key, $value );
+		$bool = $this->getValue($key, $value);
 
-		if( is_null( filter_var( $bool, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE ))) {
-			$bool = $this->postmeta->get( $bool );
+		if (is_null(filter_var($bool, FILTER_VALIDATE_BOOLEAN, FILTER_NULL_ON_FAILURE))) {
+			$bool = $this->postmeta->get($bool);
 		}
-		return wp_validate_boolean( $bool );
+		return wp_validate_boolean($bool);
 	}
 
 	/**
@@ -122,15 +122,15 @@ 
 	 *
 	 * @return int
 	 */
-	protected function getGalleryArg( $value = null )
+	protected function getGalleryArg($value = null)
 	{
-		$gallery = $this->getValue( 'gallery', $value );
+		$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 ))
-			? intval( $gallery )
+		return !is_null(get_post($gallery))
+			? intval($gallery)
 			: 0;
 	}
 
@@ -139,14 +139,14 @@ 
 	 *
 	 * @return int
 	 */
-	protected function getImagesPerPageArg( $value = null )
+	protected function getImagesPerPageArg($value = null)
 	{
-		$perPage = $this->getValue( 'images_per_page', $value );
+		$perPage = $this->getValue('images_per_page', $value);
 
-		if( !is_numeric( $perPage ) && is_string( $perPage )) {
-			$perPage = $this->postmeta->get( $perPage );
+		if (!is_numeric($perPage) && is_string($perPage)) {
+			$perPage = $this->postmeta->get($perPage);
 		}
-		return !!intval( $perPage )
+		return !!intval($perPage)
 			? $perPage
 			: -1;
 	}
@@ -156,9 +156,9 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function getLazyloadArg( $value = null )
+	protected function getLazyloadArg($value = null)
 	{
-		return $this->getBoolValue( 'lazyload', $value );
+		return $this->getBoolValue('lazyload', $value);
 	}
 
 	/**
@@ -166,18 +166,18 @@ 
 	 *
 	 * @return array
 	 */
-	protected function getMediaArg( $value = null )
+	protected function getMediaArg($value = null)
 	{
-		$media = $this->getValue( 'media', $value );
+		$media = $this->getValue('media', $value);
 
-		if( is_string( $media )) {
-			$media = $this->postmeta->get( $media, [
+		if (is_string($media)) {
+			$media = $this->postmeta->get($media, [
 				'ID'     => $this->getGalleryArg(),
 				'single' => false,
 			]);
 		}
-		return is_array( $media )
-			? wp_parse_id_list( $media )
+		return is_array($media)
+			? wp_parse_id_list($media)
 			: [];
 	}
 
@@ -186,7 +186,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;
 	}
 
 	/**
@@ -194,9 +194,9 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function getPaginationArg( $value = null )
+	protected function getPaginationArg($value = null)
 	{
-		return $this->getBoolValue( 'pagination', $value );
+		return $this->getBoolValue('pagination', $value);
 	}
 
 	/**
@@ -204,17 +204,17 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function getPermalinksArg( $value = null )
+	protected function getPermalinksArg($value = null)
 	{
-		return $this->getBoolValue( 'permalinks', $value );
+		return $this->getBoolValue('permalinks', $value);
 	}
 
 	/**
 	 * @return string
 	 */
-	protected function getPhotoswipeData( $image )
+	protected function getPhotoswipeData($image)
 	{
-		return sprintf( '{"l":{"src":"%s","w":%d,"h":%d},"m":{"src":"%s","w":%d,"h":%d}}',
+		return sprintf('{"l":{"src":"%s","w":%d,"h":%d},"m":{"src":"%s","w":%d,"h":%d}}',
 			$image->large['url'],
 			$image->large['width'],
 			$image->large['height'],
@@ -230,9 +230,9 @@ 
 	 *
 	 * @return mixed
 	 */
-	protected function getValue( $key, $value = null )
+	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,23 +241,23 @@ 
 	/**
 	 * @return array
 	 */
-	protected function normalizeArgs( array $args = [] )
+	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 );
+		$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 );
+		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);
 			}
 		});
 
@@ -269,20 +269,20 @@ 
 	 *
 	 * @return null|string
 	 */
-	protected function renderImageTag( $image )
+	protected function renderImageTag($image)
 	{
 		$imgSrc = $this->getLazyloadArg()
-			? $this->theme->imageUri( 'blank.gif' )
+			? $this->theme->imageUri('blank.gif')
 			: $image->thumbnail['url'];
 
-		$imgTag = sprintf( '<img src="%s" data-src="%s" itemprop="thumbnail" alt="%s"/>',
+		$imgTag = sprintf('<img src="%s" data-src="%s" itemprop="thumbnail" alt="%s"/>',
 			$imgSrc,
 			$image->thumbnail['url'],
 			$image->alt
 		);
 
 		return $this->getPermalinksArg()
-			? sprintf( '<a href="%s" itemprop="contentUrl">%s</a>', $image->permalink, $imgTag )
+			? sprintf('<a href="%s" itemprop="contentUrl">%s</a>', $image->permalink, $imgTag)
 			: $imgTag;
 	}
 }

Braces +9 added lines, -3 removed lines

@@ -52,7 +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 );
 		});
@@ -69,7 +71,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<figcaption itemprop="caption description">' .
@@ -90,7 +94,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'],