CMB2_REST_Access - Code Metrics - Inspection of "CMB2 REST API" - WebDevStudios/CMB2 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — trunk (#541)

by Justin

created 2015-12-09 20:46 UTC

CMB2_REST_Access B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	257
Duplicated Lines	7 %

Coupling/Cohesion

Components	1
Dependencies	3

Metric	Value
wmc	45
lcom	1
cbo	3
dl	18
loc	257
rs	8.3673

11 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	universal_hooks()	0	10	1
A	register_fields()	0	18	2
A	prepare_read_write_fields()	0	12	4
A	maybe_add_read_field()	9	9	3
A	maybe_add_write_field()	9	9	3
B	get_restable_field_values()	0	21	5
C	update_restable_field_values()	0	60	12
A	is_protected_meta()	0	7	2
A	field_can_update()	0	10	4
C	get_object_data()	0	22	8

How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class

Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like CMB2_REST_Access often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use CMB2_REST_Access, and based on these observations, apply Extract Interface, too.

<?php
/**
 * Handles hooking CMB2 objects/fields into the WordPres REST API
 * which can allow fields to be read and/or updated.
 *
 * @since  2.2.0
 *
 * @category  WordPress_Plugin
 * @package   CMB2
 * @author    WebDevStudios
 * @license   GPL-2.0+
 * @link      http://webdevstudios.com
 */
class CMB2_REST_Access extends CMB2_Hookup_Base {
namespace YourVendor;

class YourClass { }

	/**
	 * @var   CMB2[] objects
	 * @since 2.2.0
	 */
	protected static $boxes;

	/**
	 * Whether metabox fields can be read via REST API
	 * @var   bool
	 * @since 2.2.0
	 */
	protected $can_read = true;

	/**
	 * Array of readable field objects.
	 * @var   CMB2_Field[]
	 * @since 2.2.0
	 */
	protected static $read_fields = array();

	/**
	 * Array of writeable field objects.
	 * @var   CMB2_Field[]
	 * @since 2.2.0
	 */
	protected static $write_fields = array();

	/**
	 * Whether metabox fields can be written via REST API
	 * @var   bool
	 * @since 2.2.0
	 */
	protected $can_write = false;

	public function __construct( CMB2 $cmb ) {
		$this->cmb = $cmb;
		self::$boxes[ $cmb->cmb_id ] = $cmb;

		$show_value = $this->cmb->prop( 'show_in_rest' );
		$this->cmb->rest_read  = 'write_only' !== $show_value;

		$this->cmb->rest_write = in_array( $show_value, array( 'read_and_write', 'write_only' ), true );

	}

	public function universal_hooks() {
		$this->once( 'rest_api_init', array( __CLASS__, 'register_fields' ), 50 );

		// hook up the CMB rest endpoint class
		$this->once( 'rest_api_init', array( cmb2_rest_endpoints(), 'register_routes' ), 0 );

		$this->prepare_read_write_fields();

		add_filter( 'is_protected_meta', array( $this, 'is_protected_meta' ), 10, 3 );
	}

	public static function register_fields() {

		$types = array();
		foreach ( self::$boxes as $cmb_id => $cmb ) {
			$types = array_merge( $types, $cmb->prop( 'object_types' ) );
		}
		$types = array_unique( $types );

		register_api_field(
			$types,
			'cmb2',
			array(
				'get_callback' => array( __CLASS__, 'get_restable_field_values' ),
				'update_callback' => array( __CLASS__, 'update_restable_field_values' ),
				'schema' => null,
			)
		);
	}

	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;

			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 );
		}
	}

	protected function maybe_add_read_field( $field_id, $show_in_rest ) {

		$can_read = $this->cmb->rest_read
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			? 'write_only' !== $show_in_rest
			: in_array( $show_in_rest, array( 'read_and_write', 'read_only' ), true );

		if ( $can_read ) {
			self::$read_fields[ $this->cmb->cmb_id ][] = $field_id;
		}
	}

	protected function maybe_add_write_field( $field_id, $show_in_rest ) {

		$can_update = $this->cmb->rest_write
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			? 'read_only' !== $show_in_rest
			: in_array( $show_in_rest, array( 'read_and_write', 'write_only' ), true );

		if ( $can_update ) {
			self::$write_fields[ $this->cmb->cmb_id ][] = $field_id;
		}
	}

	/**
	 * Handler for getting custom field data.
	 * @since  2.2.0
	 * @param  array           $object   The object from the response
	 * @param  string          $field_id Name of field
	 * @param  WP_REST_Request $request  Current request
	 * @return mixed
	 */
	public static function get_restable_field_values( $object, $field_id, $request ) {

		$values = array();
		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'];

				if ( isset( $object->type ) ) {
					$field->object_type = $object->type;
				}

				$values[ $cmb_id ][ $field->id( true ) ] = $field->get_data();
			}
		}

		return $values;
	}

	/**
	 * Handler for updating custom field data.
	 * @since  2.2.0
	 * @param  mixed    $value    The value of the field
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
	 * @param  object   $object   The object from the response
	 * @param  string   $field_id Name of field
	 * @return bool|int
	 */
	public static function update_restable_field_values( $values, $object, $field_id ) {
		if ( empty( $values ) || ! is_array( $values ) || 'cmb2' !== $field_id ) {
			return;
		}

		$data = self::get_object_data( $object );
		if ( ! $data ) {
			return;
		}

		$object_id   = $data['object_id'];
		$object_type = $data['object_type'];
		$updated     = array();

		foreach ( self::$write_fields as $cmb_id => $fields ) {
			if ( ! array_key_exists( $cmb_id, $values ) ) {
				continue;
			}

			$cmb = self::$boxes[ $cmb_id ];

			$cmb->object_type( $object_id );
			$cmb->object_type( $object_type );

			$cmb->pre_process();

			foreach ( $fields as $field_id ) {
				if ( ! array_key_exists( $field_id, $values[ $cmb_id ] ) ) {
					continue;
				}

				$field = $cmb->get_field( $field_id );

				if ( 'title' == $field->type() ) {
					continue;
				}

				$field->object_id   = $object_id;
				$field->object_type = $object_type;

				if ( 'group' == $field->type() ) {
					$fields = $field->fields();
					if ( empty( $fields ) ) {
						continue;
					}

					$cmb->data_to_save[ $field_id ] = $values[ $cmb_id ][ $field_id ];
					$updated[ $cmb_id ][ $field_id ] = $cmb->save_group_field( $field );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

				} else {
					$updated[ $cmb_id ][ $field_id ] = $field->save_field( $values[ $cmb_id ][ $field_id ] );
				}

			}

			$cmb->after_save();
		}

		return $updated;
	}

	/**
	 * Filter whether a meta key is protected.
	 * @since 2.2.0
	 * @param bool   $protected Whether the key is protected. Default false.
	 * @param string $meta_key  Meta key.
	 * @param string $meta_type Meta type.
	 */
	public function is_protected_meta( $protected, $meta_key, $meta_type ) {

		if ( $this->field_can_update( $meta_key ) ) {
			return false;
		}

		return $protected;
	}

	public function field_can_update( $field_id ) {

		$field        = $this->cmb->get_field( $field_id );
		$show_in_rest = $field ? $field->args( 'show_in_rest' ) : 'no';
		$can_update   = $this->cmb->rest_write
<?php

/**
 * @property int $x
 * @property int $y
 * @property string $text
 */
class MyLabel
{
    private $properties;

    private $allowedProperties = array('x', 'y', 'text');

    public function __get($name)
    {
        if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) {
            return $properties[$name];
        } else {
            return null;
        }
    }

    public function __set($name, $value)
    {
        if (in_array($name, $this->allowedProperties)) {
            $properties[$name] = $value;
        } else {
            throw new \LogicException("Property $name is not defined.");
        }
    }

}
			? 'read_only' !== $show_in_rest
			: in_array( $show_in_rest, array( 'read_and_write', 'write_only' ), true );

		return $can_update ? $field : false;
	}

	protected static function get_object_data( $object ) {
		$object_id = 0;
		if ( isset( $object->ID ) ) {
			$object_id   = intval( $object->ID );
			$object_type = isset( $object->user_login ) ? 'user' : 'post';
		} 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'] );
			$object_type = 'term';
		} elseif ( isset( $object->term_id ) ) {
			$object_id   = intval( $object->term_id );
			$object_type = 'term';
		}

		if ( empty( $object_id ) ) {
			return false;
		}

		return compact( 'object_id', 'object_type' );
	}

}


1		<?php
2		/**
3		* Handles hooking CMB2 objects/fields into the WordPres REST API
4		* which can allow fields to be read and/or updated.
5		*
6		* @since 2.2.0
7		*
8		* @category WordPress_Plugin
9		* @package CMB2
10		* @author WebDevStudios
11		* @license GPL-2.0+
12		* @link http://webdevstudios.com
13		*/
14		class CMB2_REST_Access extends CMB2_Hookup_Base {
		0 ignored issues – show Coding Style Compatibility introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
15
16		/**
17		* @var CMB2[] objects
18		* @since 2.2.0
19		*/
20		protected static $boxes;
21
22		/**
23		* Whether metabox fields can be read via REST API
24		* @var bool
25		* @since 2.2.0
26		*/
27		protected $can_read = true;
28
29		/**
30		* Array of readable field objects.
31		* @var CMB2_Field[]
32		* @since 2.2.0
33		*/
34		protected static $read_fields = array();
35
36		/**
37		* Array of writeable field objects.
38		* @var CMB2_Field[]
39		* @since 2.2.0
40		*/
41		protected static $write_fields = array();
42
43		/**
44		* Whether metabox fields can be written via REST API
45		* @var bool
46		* @since 2.2.0
47		*/
48		protected $can_write = false;
49
50		public function __construct( CMB2 $cmb ) {
51		$this->cmb = $cmb;
52		self::$boxes[ $cmb->cmb_id ] = $cmb;
53
54		$show_value = $this->cmb->prop( 'show_in_rest' );
55		$this->cmb->rest_read = 'write_only' !== $show_value;
		0 ignored issues – show Bug introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The property `rest_read` does not seem to exist in `CMB2`. An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
56		$this->cmb->rest_write = in_array( $show_value, array( 'read_and_write', 'write_only' ), true );
		0 ignored issues – show Bug introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The property `rest_write` does not seem to exist in `CMB2`. An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name. If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading. Loading history...
57		}
58
59		public function universal_hooks() {
60		$this->once( 'rest_api_init', array( __CLASS__, 'register_fields' ), 50 );
61
62		// hook up the CMB rest endpoint class
63		$this->once( 'rest_api_init', array( cmb2_rest_endpoints(), 'register_routes' ), 0 );
64
65		$this->prepare_read_write_fields();
66
67		add_filter( 'is_protected_meta', array( $this, 'is_protected_meta' ), 10, 3 );
68		}
69
70		public static function register_fields() {
71
72		$types = array();
73		foreach ( self::$boxes as $cmb_id => $cmb ) {
74		$types = array_merge( $types, $cmb->prop( 'object_types' ) );
75		}
76		$types = array_unique( $types );
77
78		register_api_field(
79		$types,
80		'cmb2',
81		array(
82		'get_callback' => array( __CLASS__, 'get_restable_field_values' ),
83		'update_callback' => array( __CLASS__, 'update_restable_field_values' ),
84		'schema' => null,
85		)
86		);
87		}
88
89		protected function prepare_read_write_fields() {
90		foreach ( $this->cmb->prop( 'fields' ) as $field ) {
91		$show_in_rest = isset( $field['show_in_rest'] ) ? $field['show_in_rest'] : null;
92
93		if ( false === $show_in_rest ) {
94		continue;
95		}
96
97		$this->maybe_add_read_field( $field['id'], $show_in_rest );
98		$this->maybe_add_write_field( $field['id'], $show_in_rest );
99		}
100		}
101
102	View Code Duplication	protected function maybe_add_read_field( $field_id, $show_in_rest ) {
		0 ignored issues – show Duplication introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
103		$can_read = $this->cmb->rest_read
		0 ignored issues – show Documentation introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The property `rest_read` does not exist on `object<CMB2>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
104		? 'write_only' !== $show_in_rest
105		: in_array( $show_in_rest, array( 'read_and_write', 'read_only' ), true );
106
107		if ( $can_read ) {
108		self::$read_fields[ $this->cmb->cmb_id ][] = $field_id;
109		}
110		}
111
112	View Code Duplication	protected function maybe_add_write_field( $field_id, $show_in_rest ) {
		0 ignored issues – show Duplication introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
113		$can_update = $this->cmb->rest_write
		0 ignored issues – show Documentation introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The property `rest_write` does not exist on `object<CMB2>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
114		? 'read_only' !== $show_in_rest
115		: in_array( $show_in_rest, array( 'read_and_write', 'write_only' ), true );
116
117		if ( $can_update ) {
118		self::$write_fields[ $this->cmb->cmb_id ][] = $field_id;
119		}
120		}
121
122		/**
123		* Handler for getting custom field data.
124		* @since 2.2.0
125		* @param array $object The object from the response
126		* @param string $field_id Name of field
127		* @param WP_REST_Request $request Current request
128		* @return mixed
129		*/
130		public static function get_restable_field_values( $object, $field_id, $request ) {
		0 ignored issues – show Unused Code introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The parameter `$field_id` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The parameter `$request` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
131		$values = array();
132		if ( ! isset( $object['id'] ) ) {
133		return;
134		}
135
136		foreach ( self::$read_fields as $cmb_id => $fields ) {
137		foreach ( $fields as $field_id ) {
		0 ignored issues – show Bug introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The expression `$fields` of type `object<CMB2_Field>` is not traversable. Loading history...
138		$field = self::$boxes[ $cmb_id ]->get_field( $field_id );
139		$field->object_id = $object['id'];
140
141		if ( isset( $object->type ) ) {
142		$field->object_type = $object->type;
143		}
144
145		$values[ $cmb_id ][ $field->id( true ) ] = $field->get_data();
146		}
147		}
148
149		return $values;
150		}
151
152		/**
153		* Handler for updating custom field data.
154		* @since 2.2.0
155		* @param mixed $value The value of the field
		0 ignored issues – show Documentation introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report There is no parameter named `$value`. Did you maybe mean `$values`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
156		* @param object $object The object from the response
157		* @param string $field_id Name of field
158		* @return bool\|int
159		*/
160		public static function update_restable_field_values( $values, $object, $field_id ) {
161		if ( empty( $values ) \|\| ! is_array( $values ) \|\| 'cmb2' !== $field_id ) {
162		return;
163		}
164
165		$data = self::get_object_data( $object );
166		if ( ! $data ) {
167		return;
168		}
169
170		$object_id = $data['object_id'];
171		$object_type = $data['object_type'];
172		$updated = array();
173
174		foreach ( self::$write_fields as $cmb_id => $fields ) {
175		if ( ! array_key_exists( $cmb_id, $values ) ) {
176		continue;
177		}
178
179		$cmb = self::$boxes[ $cmb_id ];
180
181		$cmb->object_type( $object_id );
182		$cmb->object_type( $object_type );
183
184		$cmb->pre_process();
185
186		foreach ( $fields as $field_id ) {
187		if ( ! array_key_exists( $field_id, $values[ $cmb_id ] ) ) {
188		continue;
189		}
190
191		$field = $cmb->get_field( $field_id );
192
193		if ( 'title' == $field->type() ) {
194		continue;
195		}
196
197		$field->object_id = $object_id;
198		$field->object_type = $object_type;
199
200		if ( 'group' == $field->type() ) {
201		$fields = $field->fields();
202		if ( empty( $fields ) ) {
203		continue;
204		}
205
206		$cmb->data_to_save[ $field_id ] = $values[ $cmb_id ][ $field_id ];
207		$updated[ $cmb_id ][ $field_id ] = $cmb->save_group_field( $field );
		0 ignored issues – show Documentation introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report `$field` is of type `object<CMB2_Field>\|false`, but the function expects a `array`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
208
209		} else {
210		$updated[ $cmb_id ][ $field_id ] = $field->save_field( $values[ $cmb_id ][ $field_id ] );
211		}
212
213		}
214
215		$cmb->after_save();
216		}
217
218		return $updated;
219		}
220
221		/**
222		* Filter whether a meta key is protected.
223		* @since 2.2.0
224		* @param bool $protected Whether the key is protected. Default false.
225		* @param string $meta_key Meta key.
226		* @param string $meta_type Meta type.
227		*/
228		public function is_protected_meta( $protected, $meta_key, $meta_type ) {
		0 ignored issues – show Unused Code introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The parameter `$meta_type` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
229		if ( $this->field_can_update( $meta_key ) ) {
230		return false;
231		}
232
233		return $protected;
234		}
235
236		public function field_can_update( $field_id ) {
237
238		$field = $this->cmb->get_field( $field_id );
239		$show_in_rest = $field ? $field->args( 'show_in_rest' ) : 'no';
240		$can_update = $this->cmb->rest_write
		0 ignored issues – show Documentation introduced 2015-12-07 19:40 UTC by Report Bug Copy Issue Report The property `rest_write` does not exist on `object<CMB2>`. Since you implemented `__get`, maybe consider adding a @property annotation. Since your code implements the magic getter `_get`, this function will be called for any read access on an undefined variable. You can add the `@property` annotation to your class or interface to document the existence of this variable. <?php /** * @property int $x * @property int $y * @property string $text */ class MyLabel { private $properties; private $allowedProperties = array('x', 'y', 'text'); public function __get($name) { if (isset($properties[$name]) && in_array($name, $this->allowedProperties)) { return $properties[$name]; } else { return null; } } public function __set($name, $value) { if (in_array($name, $this->allowedProperties)) { $properties[$name] = $value; } else { throw new \LogicException("Property $name is not defined."); } } } If the property has read access only, you can use the @property-read annotation instead. Of course, you may also just have mistyped another name, in which case you should fix the error. See also the PhpDoc documentation for @property. Loading history...
241		? 'read_only' !== $show_in_rest
242		: in_array( $show_in_rest, array( 'read_and_write', 'write_only' ), true );
243
244		return $can_update ? $field : false;
245		}
246
247		protected static function get_object_data( $object ) {
248		$object_id = 0;
249		if ( isset( $object->ID ) ) {
250		$object_id = intval( $object->ID );
251		$object_type = isset( $object->user_login ) ? 'user' : 'post';
252		} elseif ( isset( $object->comment_ID ) ) {
253		$object_id = intval( $object->comment_ID );
254		$object_type = 'comment';
255		} elseif ( is_array( $object ) && isset( $object['term_id'] ) ) {
256		$object_id = intval( $object['term_id'] );
257		$object_type = 'term';
258		} elseif ( isset( $object->term_id ) ) {
259		$object_id = intval( $object->term_id );
260		$object_type = 'term';
261		}
262
263		if ( empty( $object_id ) ) {
264		return false;
265		}
266
267		return compact( 'object_id', 'object_type' );
268		}
269
270		}
271

WebDevStudios / CMB2