WebDevStudios / CMB2

includes/CMB2_Utils.php

Spacing +19 added lines, -20 removed lines

@@ -30,7 +30,7 @@ 
 		$dir = wp_upload_dir();
 
 		// Is URL in uploads directory?
-		if ( false === strpos( $img_url, $dir['baseurl'] . '/' ) ) {
+		if ( false === strpos( $img_url, $dir[ 'baseurl' ] . '/' ) ) {
 			return false;
 		}
 
@@ -55,8 +55,8 @@ 
 
 			foreach ( $query->posts as $post_id ) {
 				$meta = wp_get_attachment_metadata( $post_id );
-				$original_file       = basename( $meta['file'] );
-				$cropped_image_files = isset( $meta['sizes'] ) ? wp_list_pluck( $meta['sizes'], 'file' ) : array();
+				$original_file       = basename( $meta[ 'file' ] );
+				$cropped_image_files = isset( $meta[ 'sizes' ] ) ? wp_list_pluck( $meta[ 'sizes' ], 'file' ) : array();
 				if ( $original_file === $file || in_array( $file, $cropped_image_files ) ) {
 					$attachment_id = $post_id;
 					break;
@@ -135,8 +135,7 @@ 
 		}
 
 		return $this->is_valid_time_stamp( $string )
-			? (int) $string :
-			strtotime( (string) $string );
+			? (int) $string : strtotime( (string) $string );
 	}
 
 	/**
@@ -241,24 +240,24 @@ 
 
 		// order is relevant here, since the replacement will be done sequentially.
 		$supported_options = array(
-			'd' => 'dd',  // Day, leading 0
-			'j' => 'd',   // Day, no 0
-			'z' => 'o',   // Day of the year, no leading zeroes,
+			'd' => 'dd', // Day, leading 0
+			'j' => 'd', // Day, no 0
+			'z' => 'o', // Day of the year, no leading zeroes,
 			// 'D' => 'D',   // Day name short, not sure how it'll work with translations
 			// 'l' => 'DD',  // Day name full, idem before
-			'm' => 'mm',  // Month of the year, leading 0
-			'n' => 'm',   // Month of the year, no leading 0
+			'm' => 'mm', // Month of the year, leading 0
+			'n' => 'm', // Month of the year, no leading 0
 			// 'M' => 'M',   // Month, Short name
 			// 'F' => 'MM',  // Month, full name,
-			'y' => 'y',   // Year, two digit
-			'Y' => 'yy',  // Year, full
-			'H' => 'HH',  // Hour with leading 0 (24 hour)
-			'G' => 'H',   // Hour with no leading 0 (24 hour)
-			'h' => 'hh',  // Hour with leading 0 (12 hour)
-			'g' => 'h',   // Hour with no leading 0 (12 hour),
-			'i' => 'mm',  // Minute with leading 0,
-			's' => 'ss',  // Second with leading 0,
-			'a' => 'tt',  // am/pm
+			'y' => 'y', // Year, two digit
+			'Y' => 'yy', // Year, full
+			'H' => 'HH', // Hour with leading 0 (24 hour)
+			'G' => 'H', // Hour with no leading 0 (24 hour)
+			'h' => 'hh', // Hour with leading 0 (12 hour)
+			'g' => 'h', // Hour with no leading 0 (12 hour),
+			'i' => 'mm', // Minute with leading 0,
+			's' => 'ss', // Second with leading 0,
+			'a' => 'tt', // am/pm
 			'A' => 'TT'   // AM/PM
 		);
 
@@ -279,7 +278,7 @@ 
 	 * @return string Modified value
 	 */
 	public function wrap_escaped_chars( $value ) {
-		return "'" . str_replace( '\\', '', $value[0] ) . "'";
+		return "'" . str_replace( '\\', '', $value[ 0 ] ) . "'";
 	}
 
 	/**

Braces +2 added lines, -1 removed lines

@@ -110,7 +110,8 @@
 		$current_offset = get_option( 'gmt_offset' );
 		$tzstring       = get_option( 'timezone_string' );
 
-		if ( empty( $tzstring ) ) { // Create a UTC+- zone if no timezone string exists
+		if ( empty( $tzstring ) ) {
+// Create a UTC+- zone if no timezone string exists
 			if ( 0 == $current_offset ) {
 				$tzstring = 'UTC+0';
 			} elseif ( $current_offset < 0 ) {

includes/helper-functions.php

Spacing +19 added lines, -19 removed lines

@@ -178,7 +178,7 @@ 
 		$cmb = CMB2_Boxes::get( $meta_box );
 	} else {
 		// See if we already have an instance of this metabox
-		$cmb = CMB2_Boxes::get( $meta_box['id'] );
+		$cmb = CMB2_Boxes::get( $meta_box[ 'id' ] );
 		// If not, we'll initate a new metabox
 		$cmb = $cmb ? $cmb : new CMB2( $meta_box, $object_id );
 	}
@@ -254,7 +254,7 @@ 
 	) );
 
 	// Set object type explicitly (rather than trying to guess from context)
-	$cmb->object_type( $args['object_type'] );
+	$cmb->object_type( $args[ 'object_type' ] );
 
 	// Save the metabox if it's been submitted
 	// check permissions
@@ -262,32 +262,32 @@ 
 	if (
 		$cmb->prop( 'save_fields' )
 		// check nonce
-		&& isset( $_POST['submit-cmb'], $_POST['object_id'], $_POST[ $cmb->nonce() ] )
+		&& isset( $_POST[ 'submit-cmb' ], $_POST[ 'object_id' ], $_POST[ $cmb->nonce() ] )
 		&& wp_verify_nonce( $_POST[ $cmb->nonce() ], $cmb->nonce() )
-		&& $object_id && $_POST['object_id'] == $object_id
+		&& $object_id && $_POST[ 'object_id' ] == $object_id
 	) {
 		$cmb->save_fields( $object_id, $cmb->object_type(), $_POST );
 	}
 
 	// Enqueue JS/CSS
-	if ( $args['cmb_styles'] ) {
+	if ( $args[ 'cmb_styles' ] ) {
 		CMB2_hookup::enqueue_cmb_css();
 	}
 
-	if ( $args['enqueue_js'] ) {
+	if ( $args[ 'enqueue_js' ] ) {
 		CMB2_hookup::enqueue_cmb_js();
 	}
 
-	$form_format = apply_filters( 'cmb2_get_metabox_form_format', $args['form_format'], $object_id, $cmb );
+	$form_format = apply_filters( 'cmb2_get_metabox_form_format', $args[ 'form_format' ], $object_id, $cmb );
 
 	$format_parts = explode( '%3$s', $form_format );
 
 	// Show cmb form
-	printf( $format_parts[0], $cmb->cmb_id, $object_id );
+	printf( $format_parts[ 0 ], $cmb->cmb_id, $object_id );
 	$cmb->show_form();
 
-	if ( isset( $format_parts[1] ) && $format_parts[1] ) {
-		printf( str_ireplace( '%4$s', '%1$s', $format_parts[1] ), $args['save_button'] );
+	if ( isset( $format_parts[ 1 ] ) && $format_parts[ 1 ] ) {
+		printf( str_ireplace( '%4$s', '%1$s', $format_parts[ 1 ] ), $args[ 'save_button' ] );
 	}
 
 }
@@ -300,7 +300,7 @@ 
  * @param  array   $args      Optional arguments array
  */
 function cmb2_metabox_form( $meta_box, $object_id = 0, $args = array() ) {
-	if ( ! isset( $args['echo'] ) || $args['echo'] ) {
+	if ( ! isset( $args[ 'echo' ] ) || $args[ 'echo' ] ) {
 		cmb2_print_metabox_form( $meta_box, $object_id, $args );
 	} else {
 		return cmb2_get_metabox_form( $meta_box, $object_id, $args );
@@ -322,7 +322,7 @@ 
 
 		$schedule_format = str_replace(
 			array( 'M', 'Y', 'm', 'd', 'H', 'i', 'a' ),
-			array('%b', '%Y', '%m', '%d', '%H', '%M', '%p' ),
+			array( '%b', '%Y', '%m', '%d', '%H', '%M', '%p' ),
 			$date_format
 		);
 
@@ -339,14 +339,14 @@ 
 			 * 4 or 2 characters, as needed
 			 */
 			'%04d-%02d-%02d %02d:%02d:%02d',
-			$parsed_time['tm_year'] + 1900,  // This will be "111", so we need to add 1900.
-			$parsed_time['tm_mon'] + 1,      // This will be the month minus one, so we add one.
-			$parsed_time['tm_mday'],
-			$parsed_time['tm_hour'],
-			$parsed_time['tm_min'],
-			$parsed_time['tm_sec']
+			$parsed_time[ 'tm_year' ] + 1900, // This will be "111", so we need to add 1900.
+			$parsed_time[ 'tm_mon' ] + 1, // This will be the month minus one, so we add one.
+			$parsed_time[ 'tm_mday' ],
+			$parsed_time[ 'tm_hour' ],
+			$parsed_time[ 'tm_min' ],
+			$parsed_time[ 'tm_sec' ]
 		);
 
-		return new DateTime($ymd);
+		return new DateTime( $ymd );
 	}
 }

includes/CMB2_Boxes.php

Doc Comments +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 	 * Retrieve all CMB2 instances that have the specified property set.
 	 * @since  2.2.0
 	 * @param  string $property Property name.
-	 * @param  mixed  $ignore   The value to ignore.
+	 * @param  boolean  $ignore   The value to ignore.
 	 * @return CMB2[]           Array of matching cmb2 instances.
 	 */
 	public static function get_by_property( $property, $ignore = null ) {

includes/CMB2_REST_Access.php

Doc Comments +4 added lines, -1 removed lines

@@ -152,7 +152,7 @@ 
 	/**
 	 * Handler for updating custom field data.
 	 * @since  2.2.0
-	 * @param  mixed    $value    The value of the field
+	 * @param  mixed    $values    The value of the field
 	 * @param  object   $object   The object from the response
 	 * @param  string   $field_id Name of field
 	 * @return bool|int
@@ -233,6 +233,9 @@ 
 		return $protected;
 	}
 
+	/**
+	 * @param string $field_id
+	 */
 	public function field_can_update( $field_id ) {
 
 		$field        = $this->cmb->get_field( $field_id );

Spacing +11 added lines, -11 removed lines

@@ -88,14 +88,14 @@ 
 
 	protected function prepare_read_write_fields() {
 		foreach ( $this->cmb->prop( 'fields' ) as $field ) {
-			$show_in_rest = isset( $field['show_in_rest'] ) ? $field['show_in_rest'] : null;
+			$show_in_rest = isset( $field[ 'show_in_rest' ] ) ? $field[ 'show_in_rest' ] : null;
 
 			if ( false === $show_in_rest ) {
 				continue;
 			}
 
-			$this->maybe_add_read_field( $field['id'], $show_in_rest );
-			$this->maybe_add_write_field( $field['id'], $show_in_rest );
+			$this->maybe_add_read_field( $field[ 'id' ], $show_in_rest );
+			$this->maybe_add_write_field( $field[ 'id' ], $show_in_rest );
 		}
 	}
 
@@ -105,7 +105,7 @@ 
 			: in_array( $show_in_rest, array( 'read_and_write', 'read_only' ), true );
 
 		if ( $can_read ) {
-			self::$read_fields[ $this->cmb->cmb_id ][] = $field_id;
+			self::$read_fields[ $this->cmb->cmb_id ][ ] = $field_id;
 		}
 	}
 
@@ -115,7 +115,7 @@ 
 			: in_array( $show_in_rest, array( 'read_and_write', 'write_only' ), true );
 
 		if ( $can_update ) {
-			self::$write_fields[ $this->cmb->cmb_id ][] = $field_id;
+			self::$write_fields[ $this->cmb->cmb_id ][ ] = $field_id;
 		}
 	}
 
@@ -129,14 +129,14 @@ 
 	 */
 	public static function get_restable_field_values( $object, $field_id, $request ) {
 		$values = array();
-		if ( ! isset( $object['id'] ) ) {
+		if ( ! isset( $object[ 'id' ] ) ) {
 			return;
 		}
 
 		foreach ( self::$read_fields as $cmb_id => $fields ) {
 			foreach ( $fields as $field_id ) {
 				$field = self::$boxes[ $cmb_id ]->get_field( $field_id );
-				$field->object_id = $object['id'];
+				$field->object_id = $object[ 'id' ];
 
 				if ( isset( $object->type ) ) {
 					$field->object_type = $object->type;
@@ -167,8 +167,8 @@ 
 			return;
 		}
 
-		$object_id   = $data['object_id'];
-		$object_type = $data['object_type'];
+		$object_id   = $data[ 'object_id' ];
+		$object_type = $data[ 'object_type' ];
 		$updated     = array();
 
 		foreach ( self::$write_fields as $cmb_id => $fields ) {
@@ -252,8 +252,8 @@ 
 		} elseif ( isset( $object->comment_ID ) ) {
 			$object_id   = intval( $object->comment_ID );
 			$object_type = 'comment';
-		} elseif ( is_array( $object ) && isset( $object['term_id'] ) ) {
-			$object_id   = intval( $object['term_id'] );
+		} elseif ( is_array( $object ) && isset( $object[ 'term_id' ] ) ) {
+			$object_id   = intval( $object[ 'term_id' ] );
 			$object_type = 'term';
 		} elseif ( isset( $object->term_id ) ) {
 			$object_id   = intval( $object->term_id );

includes/CMB2_Hookup_Base.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
 			return;
 		}
 
-		self::$hooks_completed[] = $key;
+		self::$hooks_completed[ ] = $key;
 		add_filter( $action, $hook, $priority, $accepted_args );
 	}
 

includes/CMB2_REST_Endpoints.php

Doc Comments -1 removed lines

@@ -344,7 +344,6 @@
 	 *
 	 * @since 2.2.0
 	 *
-	 * @param  mixed $data
 	 * @return array $data
 	 */
 	public function prepare_item( $post ) {

Spacing +9 added lines, -9 removed lines

@@ -156,12 +156,12 @@ 
 		if ( $cmb_id && ( $cmb = cmb2_get_metabox( $cmb_id ) ) ) {
 			$fields = array();
 			foreach ( $cmb->prop( 'fields', array() ) as $field ) {
-				$field = $this->get_rest_field( $cmb, $field['id'] );
+				$field = $this->get_rest_field( $cmb, $field[ 'id' ] );
 
 				if ( ! is_wp_error( $field ) ) {
-					$fields[ $field['id'] ] = $field;
+					$fields[ $field[ 'id' ] ] = $field;
 				} else {
-					$fields[ $field['id'] ] = array( 'error' => $field->get_error_message() );
+					$fields[ $field[ 'id' ] ] = array( 'error' => $field->get_error_message() );
 				}
 			}
 
@@ -183,7 +183,7 @@ 
 		// TODO: handle callable properties
 		$boxes_data = $cmb->meta_box;
 
-		unset( $boxes_data['fields'] );
+		unset( $boxes_data[ 'fields' ] );
 
 		$base = $this->namespace . '/boxes/' . $cmb->cmb_id;
 		$boxbase = $base . '/' . $cmb->cmb_id;
@@ -201,7 +201,7 @@ 
 			),
 		) );
 
-		$boxes_data['_links'] = $response->get_links();
+		$boxes_data[ '_links' ] = $response->get_links();
 
 		return $boxes_data;
 	}
@@ -273,7 +273,7 @@ 
 			if ( 'render_row_cb' === $key ) {
 				$value = '';
 				// TODO: Use request get object
-				if ( isset( $_GET['rendered'] ) && ( $cb = $field->maybe_callback( 'render_row_cb' ) ) ) {
+				if ( isset( $_GET[ 'rendered' ] ) && ( $cb = $field->maybe_callback( 'render_row_cb' ) ) ) {
 					// Ok, callback is good, let's run it.
 					ob_start();
 					call_user_func( $cb, $field->args(), $this );
@@ -296,12 +296,12 @@ 
 				$field_data[ $key ] = __( 'Value Error', 'cmb2' );
 			}
 		}
-		$field_data['value'] = $field->get_data();
+		$field_data[ 'value' ] = $field->get_data();
 
 		$base = $this->namespace . '/boxes/' . $cmb->cmb_id;
 
 		// Entity meta
-		$field_data['_links'] = array(
+		$field_data[ '_links' ] = array(
 			'self' => array(
 				'href' => rest_url( trailingslashit( $base ) . 'fields/' . $field->_id() ),
 			),
@@ -362,7 +362,7 @@ 
 	 */
 	public function prepare_item_for_response( $data, $request ) {
 
-		$context = ! empty( $this->request['context'] ) ? $this->request['context'] : 'view';
+		$context = ! empty( $this->request[ 'context' ] ) ? $this->request[ 'context' ] : 'view';
 		$data = $this->filter_response_by_context( $data, $context );
 
 		/**