pryley / castor-framework

src/Forms/Fields/Checkbox.php

Spacing +8 added lines, -8 removed lines

@@ -8,11 +8,11 @@ 
 {
 	protected $element = 'input';
 
-	public function __construct( array $args = [] )
+	public function __construct(array $args = [])
 	{
-		parent::__construct( $args );
+		parent::__construct($args);
 
-		if( count( $args['options'] ) > 1 ) {
+		if (count($args['options']) > 1) {
 			$this->multi = true;
 		}
 	}
@@ -24,16 +24,16 @@ 
 	{
 		$inline = $this->args['inline'] ? ' class="inline"' : '';
 
-		if( $this->multi ) {
-			return sprintf( '<ul%s>%s</ul>%s',
+		if ($this->multi) {
+			return sprintf('<ul%s>%s</ul>%s',
 				$inline,
-				$this->implodeOptions( 'multi_input_checkbox' ),
+				$this->implodeOptions('multi_input_checkbox'),
 				$this->generateDescription()
 			);
 		}
 
-		return sprintf( '%s%s',
-			$this->implodeOptions( 'single_input' ),
+		return sprintf('%s%s',
+			$this->implodeOptions('single_input'),
 			$this->generateDescription()
 		);
 	}

src/Forms/Fields/Radio.php

Spacing +3 added lines, -3 removed lines

@@ -11,13 +11,13 @@
 	/**
 	 * @return string
 	 */
-	public function render( $default = null )
+	public function render($default = null)
 	{
 		$inline = $this->args['inline'] ? ' class="inline"' : '';
 
-		return sprintf( '<ul%s>%s</ul>%s',
+		return sprintf('<ul%s>%s</ul>%s',
 			$inline,
-			$this->implodeOptions( 'multi_input', $default ),
+			$this->implodeOptions('multi_input', $default),
 			$this->generateDescription()
 		);
 	}

src/Forms/Fields/Submit.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
 	 */
 	public function render()
 	{
-		if( isset( $this->args['name'] )) {
+		if (isset($this->args['name'])) {
 			$this->args['name'] = 'submit';
 		}
 

src/Forms/Fields/Select.php

Spacing +5 added lines, -5 removed lines

@@ -11,15 +11,15 @@
 	/**
 	 * @return string
 	 */
-	public function render( array $defaults = [] )
+	public function render(array $defaults = [])
 	{
-		$defaults = wp_parse_args( $defaults, [
+		$defaults = wp_parse_args($defaults, [
 			'type' => 'select',
 		]);
 
-		return sprintf( '<select %s>%s</select>%s',
-			$this->implodeAttributes( $defaults ),
-			$this->implodeOptions( 'select_option' ),
+		return sprintf('<select %s>%s</select>%s',
+			$this->implodeAttributes($defaults),
+			$this->implodeOptions('select_option'),
 			$this->generateDescription()
 		);
 	}

src/Forms/Fields/Hidden.php

Spacing +6 added lines, -6 removed lines

@@ -11,16 +11,16 @@
 	 */
 	public function render()
 	{
-		if( isset( $this->args['label'] )) {
-			unset( $this->args['label'] );
+		if (isset($this->args['label'])) {
+			unset($this->args['label']);
 		}
 
-		if( isset( $this->args['desc'] )) {
-			unset( $this->args['desc'] );
+		if (isset($this->args['desc'])) {
+			unset($this->args['desc']);
 		}
 
-		if( isset( $this->args['id'] )) {
-			unset( $this->args['id'] );
+		if (isset($this->args['id'])) {
+			unset($this->args['id']);
 		}
 
 		return parent::render([

src/Forms/Fields/Textarea.php

Spacing +4 added lines, -4 removed lines

@@ -11,16 +11,16 @@
 	/**
 	 * @return string
 	 */
-	public function render( array $defaults = [] )
+	public function render(array $defaults = [])
 	{
-		$defaults = wp_parse_args( $defaults, [
+		$defaults = wp_parse_args($defaults, [
 			'class' => 'large-text',
 			'rows'  => 3,
 			'type'  => 'textarea',
 		]);
 
-		return sprintf( '<textarea %s>%s</textarea>%s',
-			$this->implodeAttributes( $defaults ),
+		return sprintf('<textarea %s>%s</textarea>%s',
+			$this->implodeAttributes($defaults),
 			$this->args['value'],
 			$this->generateDescription()
 		);

src/Forms/Fields/Base.php

Braces +23 added lines, -11 removed lines

@@ -72,7 +72,9 @@ 
 	 */
 	public function generateDescription( $paragraph = true )
 	{
-		if( !isset( $this->args['desc'] ) || !$this->args['desc'] )return;
+		if( !isset( $this->args['desc'] ) || !$this->args['desc'] ) {
+			return;
+		}
 
 		$tag = ( !!$paragraph || $paragraph == 'p' ) ? 'p' : 'span';
 
@@ -86,7 +88,9 @@ 
 	 */
 	public function generateLabel()
 	{
-		if( empty( $this->args['label'] ))return;
+		if( empty( $this->args['label'] )) {
+			return;
+		}
 
 		$for = !!$this->args['id']
 			? " for=\"{$this->args['id']}\""
@@ -145,7 +149,9 @@ 
 
 		if( $method === 'singleInput' ) {
 
-			if( !isset( $this->args['options'] ) || empty( $this->args['options'] ))return;
+			if( !isset( $this->args['options'] ) || empty( $this->args['options'] )) {
+				return;
+			}
 
 			// hack to make sure unset single checkbox values start at 1 instead of 0
 			if( key( $this->args['options'] ) === 0 ) {
@@ -188,7 +194,9 @@ 
 	{
 		// similar to array_merge except overwrite empty values
 		foreach( $defaults as $key => $value ) {
-			if( isset( $this->args[ $key ] ) && !empty( $this->args[ $key ] ))continue;
+			if( isset( $this->args[ $key ] ) && !empty( $this->args[ $key ] )) {
+				continue;
+			}
 			$this->args[ $key ] = $value;
 		}
 
@@ -211,7 +219,9 @@ 
 	{
 		$args = $this->multiInputArgs( $type, $optionKey, $number );
 
-		if( !$args )return;
+		if( !$args ) {
+			return;
+		}
 
 		$attributes = '';
 
@@ -262,7 +272,9 @@ 
 
 		$args = wp_parse_args( $args, $defaults );
 
-		if( !isset( $label ) || $args['name'] === '' )return;
+		if( !isset( $label ) || $args['name'] === '' ) {
+			return;
+		}
 
 		$args['id']   = $this->args['id'] . "-{$number}";
 		$args['name'] = $this->args['name'] . ( $type === 'checkbox' && $this->multi ? '[]' : '' );
@@ -275,11 +287,9 @@ 
 			if( in_array( $args['value'], $this->args['value'] )) {
 				$this->args['default'] = $args['value'];
 			}
-		}
-		else if( $this->args['value'] ) {
+		} else if( $this->args['value'] ) {
 			$this->args['default'] = $this->args['value'];
-		}
-		else if( $type == 'radio' && !$this->args['default'] ) {
+		} else if( $type == 'radio' && !$this->args['default'] ) {
 			$this->args['default'] = 0;
 		}
 
@@ -332,7 +342,9 @@ 
 
 		$args = $this->multiInputArgs( $type, $optionKey, 1 );
 
-		if( !$args )return;
+		if( !$args ) {
+			return;
+		}
 
 		$atts = $this->normalize();
 		$atts = wp_parse_args( $args['attributes'], $atts );

Spacing +69 added lines, -69 removed lines

@@ -38,7 +38,7 @@ 
 	 */
 	protected $element;
 
-	public function __construct( array $args = [] )
+	public function __construct(array $args = [])
 	{
 		$this->args = $args;
 	}
@@ -49,9 +49,9 @@ 
 	 * @return mixed
 	 * @throws Exception
 	 */
-	public function __get( $property )
+	public function __get($property)
 	{
-		switch( $property ) {
+		switch ($property) {
 			case 'args':
 			case 'dependencies':
 			case 'element':
@@ -59,7 +59,7 @@ 
 			case 'outside':
 				return $this->$property;
 		}
-		throw new Exception( sprintf( 'Invalid %s property: %s', __CLASS__, $property ));
+		throw new Exception(sprintf('Invalid %s property: %s', __CLASS__, $property));
 	}
 
 	/**
@@ -69,13 +69,13 @@ 
 	 *
 	 * @return null|string
 	 */
-	public function generateDescription( $paragraph = true )
+	public function generateDescription($paragraph = true)
 	{
-		if( !isset( $this->args['desc'] ) || !$this->args['desc'] )return;
+		if (!isset($this->args['desc']) || !$this->args['desc'])return;
 
-		$tag = ( !!$paragraph || $paragraph == 'p' ) ? 'p' : 'span';
+		$tag = (!!$paragraph || $paragraph == 'p') ? 'p' : 'span';
 
-		return sprintf( '<%1$s class="description">%2$s</%1$s>', $tag, $this->args['desc'] );
+		return sprintf('<%1$s class="description">%2$s</%1$s>', $tag, $this->args['desc']);
 	}
 
 	/**
@@ -85,13 +85,13 @@ 
 	 */
 	public function generateLabel()
 	{
-		if( empty( $this->args['label'] ))return;
+		if (empty($this->args['label']))return;
 
 		$for = !!$this->args['id']
 			? " for=\"{$this->args['id']}\""
 			: '';
 
-		return sprintf( '<label%s>%s</label>', $for, $this->args['label'] );
+		return sprintf('<label%s>%s</label>', $for, $this->args['label']);
 	}
 
 	/**
@@ -108,11 +108,11 @@ 
 	 *
 	 * @return string
 	 */
-	protected function camelCase( $value )
+	protected function camelCase($value)
 	{
-		$value = ucwords( str_replace( ['-', '_'], ' ', $value ));
+		$value = ucwords(str_replace(['-', '_'], ' ', $value));
 
-		return lcfirst( str_replace( ' ', '', $value ));
+		return lcfirst(str_replace(' ', '', $value));
 	}
 
 	/**
@@ -120,9 +120,9 @@ 
 	 *
 	 * @return array
 	 */
-	protected function implodeAttributes( $defaults = [] )
+	protected function implodeAttributes($defaults = [])
 	{
-		return $this->normalize( $defaults, 'implode' );
+		return $this->normalize($defaults, 'implode');
 	}
 
 	/**
@@ -130,24 +130,24 @@ 
 	 *
 	 * @return null|string
 	 */
-	protected function implodeOptions( $method = 'select_option', $default = null )
+	protected function implodeOptions($method = 'select_option', $default = null)
 	{
 		$this->args['default'] ?: $this->args['default'] = $default;
 
-		$method = $this->camelCase( $method );
+		$method = $this->camelCase($method);
 
-		$method = method_exists( $this, $method )
+		$method = method_exists($this, $method)
 			? $method
 			: 'selectOption';
 
 		$i = 0;
 
-		if( $method === 'singleInput' ) {
+		if ($method === 'singleInput') {
 
-			if( !isset( $this->args['options'] ) || empty( $this->args['options'] ))return;
+			if (!isset($this->args['options']) || empty($this->args['options']))return;
 
 			// hack to make sure unset single checkbox values start at 1 instead of 0
-			if( key( $this->args['options'] ) === 0 ) {
+			if (key($this->args['options']) === 0) {
 				$options = ['1' => $this->args['options'][0]];
 				$this->args['options'] = $options;
 			}
@@ -155,8 +155,8 @@ 
 			return $this->singleInput();
 		}
 
-		return array_reduce( array_keys( $this->args['options'] ), function( $carry, $key ) use ( &$i, $method ) {
-			return $carry .= $this->$method( $key, $i++ );
+		return array_reduce(array_keys($this->args['options']), function($carry, $key) use (&$i, $method) {
+			return $carry .= $this->$method($key, $i++);
 		});
 	}
 
@@ -167,15 +167,15 @@ 
 	 *
 	 * @return array
 	 */
-	protected function normalize( array $defaults = [], $implode = false )
+	protected function normalize(array $defaults = [], $implode = false)
 	{
-		$args = $this->mergeAttributesWith( $defaults );
+		$args = $this->mergeAttributesWith($defaults);
 
 		$normalize = new Normalize;
 
-		return ( $this->element && method_exists( $normalize, $this->element ))
-			? $normalize->{$this->element}( $args, $implode )
-			: ( !!$implode ? '' : [] );
+		return ($this->element && method_exists($normalize, $this->element))
+			? $normalize->{$this->element}($args, $implode)
+			: (!!$implode ? '' : []);
 	}
 
 	/**
@@ -183,18 +183,18 @@ 
 	 *
 	 * @return array
 	 */
-	protected function mergeAttributesWith( array $defaults )
+	protected function mergeAttributesWith(array $defaults)
 	{
 		// similar to array_merge except overwrite empty values
-		foreach( $defaults as $key => $value ) {
-			if( isset( $this->args[ $key ] ) && !empty( $this->args[ $key ] ))continue;
-			$this->args[ $key ] = $value;
+		foreach ($defaults as $key => $value) {
+			if (isset($this->args[$key]) && !empty($this->args[$key]))continue;
+			$this->args[$key] = $value;
 		}
 
 		$attributes = $this->args['attributes'];
 
 		// prioritize $attributes over $this->args, don't worry about duplicates
-		return array_merge( $this->args, $attributes );
+		return array_merge($this->args, $attributes);
 	}
 
 	/**
@@ -206,22 +206,22 @@ 
 	 *
 	 * @return null|string
 	 */
-	protected function multiInput( $optionKey, $number, $type = 'radio' )
+	protected function multiInput($optionKey, $number, $type = 'radio')
 	{
-		$args = $this->multiInputArgs( $type, $optionKey, $number );
+		$args = $this->multiInputArgs($type, $optionKey, $number);
 
-		if( !$args )return;
+		if (!$args)return;
 
 		$attributes = '';
 
-		foreach( $args['attributes'] as $key => $val ) {
-			$attributes .= sprintf( '%s="%s" ', $key, $val );
+		foreach ($args['attributes'] as $key => $val) {
+			$attributes .= sprintf('%s="%s" ', $key, $val);
 		}
 
-		return sprintf( '<li><label for="%s"><input %s%s/> %s</label></li>',
+		return sprintf('<li><label for="%s"><input %s%s/> %s</label></li>',
 			$args['attributes']['id'],
 			$attributes,
-			checked( $args['value'], $args['attributes']['value'], false ),
+			checked($args['value'], $args['attributes']['value'], false),
 			$args['label']
 		);
 	}
@@ -235,7 +235,7 @@ 
 	 *
 	 * @return array|null
 	 */
-	protected function multiInputArgs( $type, $optionName, $number )
+	protected function multiInputArgs($type, $optionName, $number)
 	{
 		$defaults = [
 			'class' => '',
@@ -246,39 +246,39 @@ 
 
 		$args = [];
 
-		$value = $this->args['options'][ $optionName ];
+		$value = $this->args['options'][$optionName];
 
-		if( is_array( $value )) {
+		if (is_array($value)) {
 			$args = $value;
 		}
 
-		if( is_string( $value )) {
+		if (is_string($value)) {
 			$label = $value;
 		}
 
-		isset( $args['name'] ) ?: $args['name'] = $optionName;
-		isset( $args['value'] ) ?: $args['value'] = $optionName;
+		isset($args['name']) ?: $args['name'] = $optionName;
+		isset($args['value']) ?: $args['value'] = $optionName;
 
-		$args = wp_parse_args( $args, $defaults );
+		$args = wp_parse_args($args, $defaults);
 
-		if( !isset( $label ) || $args['name'] === '' )return;
+		if (!isset($label) || $args['name'] === '')return;
 
 		$args['id']   = $this->args['id'] . "-{$number}";
-		$args['name'] = $this->args['name'] . ( $type === 'checkbox' && $this->multi ? '[]' : '' );
+		$args['name'] = $this->args['name'] . ($type === 'checkbox' && $this->multi ? '[]' : '');
 
-		$args = array_filter( $args, function( $value ) {
+		$args = array_filter($args, function($value) {
 			return $value !== '';
 		});
 
-		if( is_array( $this->args['value'] )) {
-			if( in_array( $args['value'], $this->args['value'] )) {
+		if (is_array($this->args['value'])) {
+			if (in_array($args['value'], $this->args['value'])) {
 				$this->args['default'] = $args['value'];
 			}
 		}
-		else if( $this->args['value'] ) {
+		else if ($this->args['value']) {
 			$this->args['default'] = $this->args['value'];
 		}
-		else if( $type == 'radio' && !$this->args['default'] ) {
+		else if ($type == 'radio' && !$this->args['default']) {
 			$this->args['default'] = 0;
 		}
 
@@ -297,9 +297,9 @@ 
 	 *
 	 * @return null|string
 	 */
-	protected function multiInputCheckbox( $optionKey, $number )
+	protected function multiInputCheckbox($optionKey, $number)
 	{
-		return $this->multiInput( $optionKey, $number, 'checkbox' );
+		return $this->multiInput($optionKey, $number, 'checkbox');
 	}
 
 	/**
@@ -309,12 +309,12 @@ 
 	 *
 	 * @return string
 	 */
-	protected function selectOption( $optionKey )
+	protected function selectOption($optionKey)
 	{
-		return sprintf( '<option value="%s"%s>%s</option>',
+		return sprintf('<option value="%s"%s>%s</option>',
 			$optionKey,
-			selected( $this->args['value'], $optionKey, false ),
-			$this->args['options'][ $optionKey ]
+			selected($this->args['value'], $optionKey, false),
+			$this->args['options'][$optionKey]
 		);
 	}
 
@@ -325,27 +325,27 @@ 
 	 *
 	 * @return null|string
 	 */
-	protected function singleInput( $type = 'checkbox' )
+	protected function singleInput($type = 'checkbox')
 	{
-		$optionKey = key( $this->args['options'] );
+		$optionKey = key($this->args['options']);
 
-		$args = $this->multiInputArgs( $type, $optionKey, 1 );
+		$args = $this->multiInputArgs($type, $optionKey, 1);
 
-		if( !$args )return;
+		if (!$args)return;
 
 		$atts = $this->normalize();
-		$atts = wp_parse_args( $args['attributes'], $atts );
+		$atts = wp_parse_args($args['attributes'], $atts);
 
 		$attributes = '';
 
-		foreach( $atts as $key => $val ) {
-			$attributes .= sprintf( '%s="%s" ', $key, $val );
+		foreach ($atts as $key => $val) {
+			$attributes .= sprintf('%s="%s" ', $key, $val);
 		}
 
-		return sprintf( '<label for="%s"><input %s%s/> %s</label>',
+		return sprintf('<label for="%s"><input %s%s/> %s</label>',
 			$atts['id'],
 			$attributes,
-			checked( $args['value'], $atts['value'], false ),
+			checked($args['value'], $atts['value'], false),
 			$args['label']
 		);
 	}

src/Forms/Fields/Text.php

Spacing +4 added lines, -4 removed lines

@@ -11,15 +11,15 @@
 	/**
 	 * @return string
 	 */
-	public function render( array $defaults = [] )
+	public function render(array $defaults = [])
 	{
-		$defaults = wp_parse_args( $defaults, [
+		$defaults = wp_parse_args($defaults, [
 			'class' => 'regular-text',
 			'type'  => 'text',
 		]);
 
-		return sprintf( '<input %s/>%s',
-			$this->implodeAttributes( $defaults ),
+		return sprintf('<input %s/>%s',
+			$this->implodeAttributes($defaults),
 			$this->generateDescription()
 		);
 	}

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'] );