WebDevStudios / CMB2

includes/CMB2_Base.php

Doc Comments +1 added lines, -2 removed lines

@@ -159,7 +159,7 @@ 
 	 * Get object property and optionally set a fallback
 	 * @since  2.0.0
 	 * @param  string $property Metabox config property to retrieve
-	 * @param  mixed  $fallback Fallback value to set if no value found
+	 * @param  boolean|null  $fallback Fallback value to set if no value found
 	 * @return mixed            Metabox config property value or false
 	 */
 	public function prop( $property, $fallback = null ) {
@@ -464,7 +464,6 @@ 
 	 * Allows overloading the object with methods... Whooaaa oooh it's magic, y'knoooow.
 	 * @since 1.0.0
 	 * @param string $method Non-existent method.
-	 * @param array  $arguments All arguments passed to the method
 	 */
 	public function __call( $method, $args ) {
 		$object_class = strtolower( get_class( $this ) );

Indentation +4 added lines, -4 removed lines

@@ -315,10 +315,10 @@
 	 * which is callable. If so, it registers the callback, and if not,
 	 * converts the maybe-modified $val to a boolean for return.
 	 *
- 	 * The registered handlers will have a parameter name which matches the filter, except:
- 	 * - The 'cmb2_api' prefix will be removed
- 	 * - A '_cb' suffix will be added (to stay inline with other '*_cb' parameters).
- 	 *
+	 * The registered handlers will have a parameter name which matches the filter, except:
+	 * - The 'cmb2_api' prefix will be removed
+	 * - A '_cb' suffix will be added (to stay inline with other '*_cb' parameters).
+	 *
 	 * @since  2.2.3
 	 *
 	 * @param  string $hook_name     The hook name.

Spacing +4 added lines, -4 removed lines

@@ -358,7 +358,7 @@ 
 		}
 
 		// Cast to bool.
-		return !! $val;
+		return ! ! $val;
 	}
 
 	/**
@@ -386,11 +386,11 @@ 
 			switch ( $message ) {
 
 				case self::DEPRECATED_PARAM:
-					$message = sprintf( __( 'The "%s" field parameter has been deprecated in favor of the "%s" parameter.', 'cmb2' ), $args[3], $args[4] );
+					$message = sprintf( __( 'The "%s" field parameter has been deprecated in favor of the "%s" parameter.', 'cmb2' ), $args[ 3 ], $args[ 4 ] );
 					break;
 
 				case self::DEPRECATED_CB_PARAM:
-					$message = sprintf( __( 'Using the "%s" field parameter as a callback has been deprecated in favor of the "%s" parameter.', 'cmb2' ), $args[3], $args[4] );
+					$message = sprintf( __( 'Using the "%s" field parameter as a callback has been deprecated in favor of the "%s" parameter.', 'cmb2' ), $args[ 3 ], $args[ 4 ] );
 					break;
 
 				default:
@@ -469,7 +469,7 @@ 
 	public function __call( $method, $args ) {
 		$object_class = strtolower( get_class( $this ) );
 
-		if ( ! has_filter(  "{$object_class}_inherit_{$method}" ) ) {
+		if ( ! has_filter( "{$object_class}_inherit_{$method}" ) ) {
 			throw new Exception( sprintf( esc_html__( 'Invalid %1$s method: %2$s', 'give' ), get_class( $this ), $method ) );
 		}
 

includes/rest-api/CMB2_REST_Controller.php

Doc Comments +1 added lines, -2 removed lines

@@ -198,7 +198,7 @@ 
 	 * @since  2.2.3
 	 *
 	 * @param  mixed $cb Callable function/method.
-	 * @return mixed     Results of output buffer after calling function/method.
+	 * @return string     Results of output buffer after calling function/method.
 	 */
 	public function get_cb_results( $cb ) {
 		$args = func_get_args();
@@ -214,7 +214,6 @@ 
 	 *
 	 * @since 2.2.3
 	 *
-	 * @param  mixed            $item     WordPress representation of the item.
 	 * @param  WP_REST_Request  $request  Request object.
 	 * @return WP_REST_Response $response
 	 */

includes/rest-api/CMB2_REST_Controller_Boxes.php

Spacing +10 added lines, -10 removed lines

@@ -68,7 +68,7 @@ 
 			'schema' => array( $this, 'get_item_schema' ),
 		) );
 
-		$args['_rendered'] = array(
+		$args[ '_rendered' ] = array(
 			'description' => __( 'Includes the fully rendered attributes, \'form_open\', \'form_close\', as well as the enqueued \'js_dependencies\' script handles, and \'css_dependencies\' stylesheet handles.', 'cmb2' ),
 		);
 
@@ -132,9 +132,9 @@ 
 				// And make sure current user can view this box.
 				&& $this->get_item_permissions_check_filter( $this->request )
 			) {
-				$boxes_data[] = $this->server->response_to_data(
+				$boxes_data[ ] = $this->server->response_to_data(
 					$this->get_rest_box(),
-					isset( $this->request['_embed'] )
+					isset( $this->request[ '_embed' ] )
 				);
 			}
 		}
@@ -210,9 +210,9 @@ 
 
 		$boxes_data = $cmb->meta_box;
 
-		if ( isset( $this->request['_rendered'] ) && $this->namespace_base !== ltrim( CMB2_REST_Controller::get_intial_route(), '/' ) ) {
-			$boxes_data['form_open'] = $this->get_cb_results( array( $cmb, 'render_form_open' ) );
-			$boxes_data['form_close'] = $this->get_cb_results( array( $cmb, 'render_form_close' ) );
+		if ( isset( $this->request[ '_rendered' ] ) && $this->namespace_base !== ltrim( CMB2_REST_Controller::get_intial_route(), '/' ) ) {
+			$boxes_data[ 'form_open' ] = $this->get_cb_results( array( $cmb, 'render_form_open' ) );
+			$boxes_data[ 'form_close' ] = $this->get_cb_results( array( $cmb, 'render_form_close' ) );
 
 			global $wp_scripts, $wp_styles;
 			$before_css = $wp_styles->queue;
@@ -220,15 +220,15 @@ 
 
 			CMB2_JS::enqueue();
 
-			$boxes_data['js_dependencies'] = array_values( array_diff( $wp_scripts->queue, $before_js ) );
-			$boxes_data['css_dependencies'] = array_values( array_diff( $wp_styles->queue, $before_css ) );
+			$boxes_data[ 'js_dependencies' ] = array_values( array_diff( $wp_scripts->queue, $before_js ) );
+			$boxes_data[ 'css_dependencies' ] = array_values( array_diff( $wp_styles->queue, $before_css ) );
 		}
 
 		// TODO: look into 'embed' parameter.
 		// http://demo.wp-api.org/wp-json/wp/v2/posts?_embed
-		unset( $boxes_data['fields'] );
+		unset( $boxes_data[ 'fields' ] );
 		// Handle callable properties.
-		unset( $boxes_data['show_on_cb'] );
+		unset( $boxes_data[ 'show_on_cb' ] );
 
 		$response = rest_ensure_response( $boxes_data );
 

includes/rest-api/CMB2_REST_Controller_Fields.php

Spacing +17 added lines, -17 removed lines

@@ -49,8 +49,8 @@ 
 		) );
 
 		$delete_args = $args;
-		$delete_args['object_id']['required'] = true;
-		$delete_args['object_type']['required'] = true;
+		$delete_args[ 'object_id' ][ 'required' ] = true;
+		$delete_args[ 'object_type' ][ 'required' ] = true;
 
 		// Returns specific field data.
 		register_rest_route( $this->namespace, '/' . $this->rest_base . '/(?P<cmb_id>[\w-]+)/fields/(?P<field_id>[\w-]+)', array(
@@ -122,13 +122,13 @@ 
 		foreach ( $this->rest_box->cmb->prop( 'fields', array() ) as $field ) {
 
 			// Make sure this field can be read.
-			$this->field = $this->rest_box->field_can_read( $field['id'], true );
+			$this->field = $this->rest_box->field_can_read( $field[ 'id' ], true );
 
 			// And make sure current user can view this box.
 			if ( $this->field && $this->get_item_permissions_check_filter() ) {
-				$fields[ $field['id'] ] = $this->server->response_to_data(
+				$fields[ $field[ 'id' ] ] = $this->server->response_to_data(
 					$this->prepare_field_response(),
-					isset( $this->request['_embed'] )
+					isset( $this->request[ '_embed' ] )
 				);
 			}
 		}
@@ -234,7 +234,7 @@ 
 	public function update_item( $request ) {
 		$this->initiate_rest_read_box( $request, 'field_value_update' );
 
-		if ( ! $this->request['value'] ) {
+		if ( ! $this->request[ 'value' ] ) {
 			return new WP_Error( 'cmb2_rest_update_field_error', __( 'CMB2 Field value cannot be updated without the value parameter specified.', 'cmb2' ), array( 'status' => 400 ) );
 		}
 
@@ -291,9 +291,9 @@ 
 	 * @param  string $activity The modification activity (updated or deleted).
 	 * @return WP_Error|WP_REST_Response
 	 */
-	public function modify_field_value( $activity) {
+	public function modify_field_value( $activity ) {
 
-		if ( ! $this->request['object_id'] || ! $this->request['object_type'] ) {
+		if ( ! $this->request[ 'object_id' ] || ! $this->request[ 'object_type' ] ) {
 			return new WP_Error( 'cmb2_rest_modify_field_value_error', __( 'CMB2 Field value cannot be modified without the object_id and object_type parameters specified.', 'cmb2' ), array( 'status' => 400 ) );
 		}
 
@@ -310,13 +310,13 @@ 
 			return new WP_Error( 'cmb2_rest_no_field_by_id_error', __( 'No field found by that id.', 'cmb2' ), array( 'status' => 403 ) );
 		}
 
-		$this->field->args["value_{$activity}"] = (bool) 'deleted' === $activity
+		$this->field->args[ "value_{$activity}" ] = (bool) 'deleted' === $activity
 			? $this->field->remove_data()
-			: $this->field->save_field( $this->request['value'] );
+			: $this->field->save_field( $this->request[ 'value' ] );
 
 		// If options page, save the $activity options
-		if ( 'options-page' == $this->request['object_type'] ) {
-			$this->field->args["value_{$activity}"] = cmb2_options( $this->request['object_id'] )->set();
+		if ( 'options-page' == $this->request[ 'object_type' ] ) {
+			$this->field->args[ "value_{$activity}" ] = cmb2_options( $this->request[ 'object_id' ] )->set();
 		}
 
 		return $this->prepare_read_field( $this->field );
@@ -391,13 +391,13 @@ 
 				$value = $field->options();
 			} elseif ( in_array( $key, CMB2_Field::$callable_fields, true ) ) {
 
-				if ( isset( $this->request['_rendered'] ) ) {
+				if ( isset( $this->request[ '_rendered' ] ) ) {
 					$value = $key === 'render_row_cb' ? $rendered : $field->get_param_callback_result( $key );
 				} elseif ( is_array( $value ) ) {
 					// We need to rewrite callbacks as string as they will cause
 					// JSON recursion errors.
-					$class = is_string( $value[0] ) ? $value[0] : get_class( $value[0] );
-					$value = $class . '::' . $value[1];
+					$class = is_string( $value[ 0 ] ) ? $value[ 0 ] : get_class( $value[ 0 ] );
+					$value = $class . '::' . $value[ 1 ];
 				}
 			}
 
@@ -410,8 +410,8 @@ 
 			}
 		}
 
-		if ( $this->request['object_id'] && $this->request['object_type'] ) {
-			$field_data['value'] = $field->get_data();
+		if ( $this->request[ 'object_id' ] && $this->request[ 'object_type' ] ) {
+			$field_data[ 'value' ] = $field->get_data();
 		}
 
 		return $field_data;