Completed
Pull Request — trunk (#541)
by Justin
22:39 queued 06:07
created
includes/CMB2_Field.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -735,7 +735,7 @@  discard block
 block discarded – undo
735 735
 	 * Return a formatted timestamp for a field
736 736
 	 * @since  2.0.0
737 737
 	 * @param  string $format     Either date_format or time_format
738
-	 * @param  string $meta_value Optional meta value to check
738
+	 * @param  integer $meta_value Optional meta value to check
739 739
 	 * @return string             Formatted date
740 740
 	 */
741 741
 	public function get_timestamp_format( $format = 'date_format', $meta_value = 0 ) {
@@ -1007,7 +1007,7 @@  discard block
 block discarded – undo
1007 1007
 	 * Retrieve options args. Calls options_cb if it exists.
1008 1008
 	 * @since  2.0.0
1009 1009
 	 * @param  string  $key Specific option to retrieve
1010
-	 * @return array        Array of options
1010
+	 * @return string        Array of options
1011 1011
 	 */
1012 1012
 	public function options( $key = '' ) {
1013 1013
 		if ( ! empty( $this->field_options ) ) {
Please login to merge, or discard this patch.
Spacing   +78 added lines, -78 removed lines patch added patch discarded remove patch
@@ -110,21 +110,21 @@  discard block
 block discarded – undo
110 110
 	 */
111 111
 	public function __construct( $args ) {
112 112
 
113
-		if ( ! empty( $args['group_field'] ) ) {
114
-			$this->group       = $args['group_field'];
113
+		if ( ! empty( $args[ 'group_field' ] ) ) {
114
+			$this->group       = $args[ 'group_field' ];
115 115
 			$this->object_id   = $this->group->object_id;
116 116
 			$this->object_type = $this->group->object_type;
117 117
 			$this->cmb_id      = $this->group->cmb_id;
118 118
 		} else {
119
-			$this->object_id   = isset( $args['object_id'] ) && '_' !== $args['object_id'] ? $args['object_id'] : 0;
120
-			$this->object_type = isset( $args['object_type'] ) ? $args['object_type'] : 'post';
119
+			$this->object_id   = isset( $args[ 'object_id' ] ) && '_' !== $args[ 'object_id' ] ? $args[ 'object_id' ] : 0;
120
+			$this->object_type = isset( $args[ 'object_type' ] ) ? $args[ 'object_type' ] : 'post';
121 121
 
122
-			if ( isset( $args['cmb_id'] ) ) {
123
-				$this->cmb_id = $args['cmb_id'];
122
+			if ( isset( $args[ 'cmb_id' ] ) ) {
123
+				$this->cmb_id = $args[ 'cmb_id' ];
124 124
 			}
125 125
 		}
126 126
 
127
-		$this->args = $this->_set_field_defaults( $args['field_args'], $args );
127
+		$this->args = $this->_set_field_defaults( $args[ 'field_args' ], $args );
128 128
 
129 129
 		if ( $this->object_id ) {
130 130
 			$this->value = $this->get_data();
@@ -143,7 +143,7 @@  discard block
 block discarded – undo
143 143
 			return call_user_func_array( array( $this, 'get_string' ), $arguments );
144 144
 		}
145 145
 
146
-		$key = isset( $arguments[0] ) ? $arguments[0] : false;
146
+		$key = isset( $arguments[ 0 ] ) ? $arguments[ 0 ] : false;
147 147
 		return $this->args( $name, $key );
148 148
 	}
149 149
 
@@ -214,9 +214,9 @@  discard block
 block discarded – undo
214 214
 	 */
215 215
 	public function get_data( $field_id = '', $args = array() ) {
216 216
 		if ( $field_id ) {
217
-			$args['field_id'] = $field_id;
217
+			$args[ 'field_id' ] = $field_id;
218 218
 		} else if ( $this->group ) {
219
-			$args['field_id'] = $this->group->id();
219
+			$args[ 'field_id' ] = $this->group->id();
220 220
 		}
221 221
 
222 222
 		$a = $this->data_args( $args );
@@ -257,13 +257,13 @@  discard block
 block discarded – undo
257 257
 		 *
258 258
 		 * @since 2.0.0
259 259
 		 */
260
-		$data = apply_filters( "cmb2_override_{$a['field_id']}_meta_value", $data, $this->object_id, $a, $this );
260
+		$data = apply_filters( "cmb2_override_{$a[ 'field_id' ]}_meta_value", $data, $this->object_id, $a, $this );
261 261
 
262 262
 		// If no override, get value normally
263 263
 		if ( 'cmb2_field_no_override_val' === $data ) {
264
-			$data = 'options-page' === $a['type']
265
-				? cmb2_options( $a['id'] )->get( $a['field_id'] )
266
-				: get_metadata( $a['type'], $a['id'], $a['field_id'], ( $a['single'] || $a['repeat'] ) );
264
+			$data = 'options-page' === $a[ 'type' ]
265
+				? cmb2_options( $a[ 'id' ] )->get( $a[ 'field_id' ] )
266
+				: get_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], ( $a[ 'single' ] || $a[ 'repeat' ] ) );
267 267
 		}
268 268
 
269 269
 		if ( $this->group ) {
@@ -285,7 +285,7 @@  discard block
 block discarded – undo
285 285
 	public function update_data( $new_value, $single = true ) {
286 286
 		$a = $this->data_args( array( 'single' => $single ) );
287 287
 
288
-		$a['value'] = $a['repeat'] ? array_values( $new_value ) : $new_value;
288
+		$a[ 'value' ] = $a[ 'repeat' ] ? array_values( $new_value ) : $new_value;
289 289
 
290 290
 		/**
291 291
 		 * Filter whether to override saving of meta value.
@@ -321,7 +321,7 @@  discard block
 block discarded – undo
321 321
 		 *
322 322
 		 * @since 2.0.0
323 323
 		 */
324
-		$override = apply_filters( "cmb2_override_{$a['field_id']}_meta_save", $override, $a, $this->args(), $this );
324
+		$override = apply_filters( "cmb2_override_{$a[ 'field_id' ]}_meta_save", $override, $a, $this->args(), $this );
325 325
 
326 326
 		// If override, return that
327 327
 		if ( null !== $override ) {
@@ -329,22 +329,22 @@  discard block
 block discarded – undo
329 329
 		}
330 330
 
331 331
 		// Options page handling (or temp data store)
332
-		if ( 'options-page' === $a['type'] || empty( $a['id'] ) ) {
333
-			return cmb2_options( $a['id'] )->update( $a['field_id'], $a['value'], false, $a['single'] );
332
+		if ( 'options-page' === $a[ 'type' ] || empty( $a[ 'id' ] ) ) {
333
+			return cmb2_options( $a[ 'id' ] )->update( $a[ 'field_id' ], $a[ 'value' ], false, $a[ 'single' ] );
334 334
 		}
335 335
 
336 336
 		// Add metadata if not single
337
-		if ( ! $a['single'] ) {
338
-			return add_metadata( $a['type'], $a['id'], $a['field_id'], $a['value'], false );
337
+		if ( ! $a[ 'single' ] ) {
338
+			return add_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $a[ 'value' ], false );
339 339
 		}
340 340
 
341 341
 		// Delete meta if we have an empty array
342
-		if ( is_array( $a['value'] ) && empty( $a['value'] ) ) {
343
-			return delete_metadata( $a['type'], $a['id'], $a['field_id'], $this->value );
342
+		if ( is_array( $a[ 'value' ] ) && empty( $a[ 'value' ] ) ) {
343
+			return delete_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $this->value );
344 344
 		}
345 345
 
346 346
 		// Update metadata
347
-		return update_metadata( $a['type'], $a['id'], $a['field_id'], $a['value'] );
347
+		return update_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $a[ 'value' ] );
348 348
 	}
349 349
 
350 350
 	/**
@@ -394,19 +394,19 @@  discard block
 block discarded – undo
394 394
 		 * @param array $field_args All field arguments
395 395
 		 * @param CMB2_Field object $field This field object
396 396
 		 */
397
-		$override = apply_filters( "cmb2_override_{$a['field_id']}_meta_remove", $override, $a, $this->args(), $this );
397
+		$override = apply_filters( "cmb2_override_{$a[ 'field_id' ]}_meta_remove", $override, $a, $this->args(), $this );
398 398
 
399 399
 		// If no override, remove as usual
400 400
 		if ( null !== $override ) {
401 401
 			return $override;
402 402
 		}
403 403
 		// Option page handling
404
-		elseif ( 'options-page' === $a['type'] || empty( $a['id'] ) ) {
405
-			return cmb2_options( $a['id'] )->remove( $a['field_id'] );
404
+		elseif ( 'options-page' === $a[ 'type' ] || empty( $a[ 'id' ] ) ) {
405
+			return cmb2_options( $a[ 'id' ] )->remove( $a[ 'field_id' ] );
406 406
 		}
407 407
 
408 408
 		// Remove metadata
409
-		return delete_metadata( $a['type'], $a['id'], $a['field_id'], $old );
409
+		return delete_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $old );
410 410
 	}
411 411
 
412 412
 	/**
@@ -858,7 +858,7 @@  discard block
 block discarded – undo
858 858
 
859 859
 		foreach ( $conditional_classes as $class => $condition ) {
860 860
 			if ( $condition ) {
861
-				$classes[] = $class;
861
+				$classes[ ] = $class;
862 862
 			}
863 863
 		}
864 864
 
@@ -869,7 +869,7 @@  discard block
 block discarded – undo
869 869
 		}
870 870
 
871 871
 		if ( $added_classes ) {
872
-			$classes[] = esc_attr( $added_classes );
872
+			$classes[ ] = esc_attr( $added_classes );
873 873
 		}
874 874
 
875 875
 		/**
@@ -969,10 +969,10 @@  discard block
 block discarded – undo
969 969
 	public function get_string( $text_key, $fallback ) {
970 970
 		// If null, populate with our field strings values.
971 971
 		if ( null === $this->strings ) {
972
-			$this->strings = (array) $this->args['text'];
972
+			$this->strings = (array) $this->args[ 'text' ];
973 973
 
974
-			if ( is_callable( $this->args['text_cb'] ) ) {
975
-				$strings = call_user_func( $this->args['text_cb'], $this );
974
+			if ( is_callable( $this->args[ 'text_cb' ] ) ) {
975
+				$strings = call_user_func( $this->args[ 'text_cb' ], $this );
976 976
 
977 977
 				if ( $strings && is_array( $strings ) ) {
978 978
 					$this->strings += $strings;
@@ -1006,10 +1006,10 @@  discard block
 block discarded – undo
1006 1006
 			return $this->field_options;
1007 1007
 		}
1008 1008
 
1009
-		$this->field_options = (array) $this->args['options'];
1009
+		$this->field_options = (array) $this->args[ 'options' ];
1010 1010
 
1011
-		if ( is_callable( $this->args['options_cb'] ) ) {
1012
-			$options = call_user_func( $this->args['options_cb'], $this );
1011
+		if ( is_callable( $this->args[ 'options_cb' ] ) ) {
1012
+			$options = call_user_func( $this->args[ 'options_cb' ], $this );
1013 1013
 
1014 1014
 			if ( $options && is_array( $options ) ) {
1015 1015
 				$this->field_options = $options + $this->field_options;
@@ -1030,7 +1030,7 @@  discard block
 block discarded – undo
1030 1030
 	 */
1031 1031
 	public function add_js_dependencies( $dependencies = array() ) {
1032 1032
 		foreach ( (array) $dependencies as $dependency ) {
1033
-			$this->args['js_dependencies'][ $dependency ] = $dependency;
1033
+			$this->args[ 'js_dependencies' ][ $dependency ] = $dependency;
1034 1034
 		}
1035 1035
 
1036 1036
 		CMB2_JS::add_dependencies( $dependencies );
@@ -1044,17 +1044,17 @@  discard block
 block discarded – undo
1044 1044
 	 * @return mixed  Default field value
1045 1045
 	 */
1046 1046
 	public function get_default() {
1047
-		if ( null !== $this->args['default'] ) {
1048
-			return $this->args['default'];
1047
+		if ( null !== $this->args[ 'default' ] ) {
1048
+			return $this->args[ 'default' ];
1049 1049
 		}
1050 1050
 
1051
-		$param = is_callable( $this->args['default_cb'] ) ? 'default_cb' : 'default';
1051
+		$param = is_callable( $this->args[ 'default_cb' ] ) ? 'default_cb' : 'default';
1052 1052
 		$default = $this->get_param_callback_result( $param );
1053 1053
 
1054 1054
 		// Allow a filter override of the default value
1055
-		$this->args['default'] = apply_filters( 'cmb2_default_filter', $default, $this );
1055
+		$this->args[ 'default' ] = apply_filters( 'cmb2_default_filter', $default, $this );
1056 1056
 
1057
-		return $this->args['default'];
1057
+		return $this->args[ 'default' ];
1058 1058
 	}
1059 1059
 
1060 1060
 	/**
@@ -1084,18 +1084,18 @@  discard block
 block discarded – undo
1084 1084
 			'classes_cb'        => '',
1085 1085
 			'select_all_button' => true,
1086 1086
 			'multiple'          => false,
1087
-			'repeatable'        => isset( $args['type'] ) && 'group' == $args['type'],
1087
+			'repeatable'        => isset( $args[ 'type' ] ) && 'group' == $args[ 'type' ],
1088 1088
 			'inline'            => false,
1089 1089
 			'on_front'          => true,
1090 1090
 			'show_names'        => true,
1091 1091
 			'save_field'        => true, // Will not save if false
1092 1092
 			'date_format'       => 'm\/d\/Y',
1093 1093
 			'time_format'       => 'h:i A',
1094
-			'description'       => isset( $args['desc'] ) ? $args['desc'] : '',
1095
-			'preview_size'      => 'file' == $args['type'] ? array( 350, 350 ) : array( 50, 50 ),
1094
+			'description'       => isset( $args[ 'desc' ] ) ? $args[ 'desc' ] : '',
1095
+			'preview_size'      => 'file' == $args[ 'type' ] ? array( 350, 350 ) : array( 50, 50 ),
1096 1096
 			'render_row_cb'     => array( $this, 'render_field_callback' ),
1097 1097
 			'display_cb'        => array( $this, 'display_value_callback' ),
1098
-			'label_cb'          => 'title' != $args['type'] ? array( $this, 'label' ) : '',
1098
+			'label_cb'          => 'title' != $args[ 'type' ] ? array( $this, 'label' ) : '',
1099 1099
 			'column'            => false,
1100 1100
 			'js_dependencies'   => array(),
1101 1101
 		) );
@@ -1109,44 +1109,44 @@  discard block
 block discarded – undo
1109 1109
 		 */
1110 1110
 		$args = $this->convert_deprecated_params( $args );
1111 1111
 
1112
-		$args['repeatable'] = $args['repeatable'] && ! $this->repeatable_exception( $args['type'] );
1113
-		$args['inline']     = $args['inline'] || false !== stripos( $args['type'], '_inline' );
1112
+		$args[ 'repeatable' ] = $args[ 'repeatable' ] && ! $this->repeatable_exception( $args[ 'type' ] );
1113
+		$args[ 'inline' ]     = $args[ 'inline' ] || false !== stripos( $args[ 'type' ], '_inline' );
1114 1114
 
1115
-		$args['options']    = 'group' == $args['type'] ? wp_parse_args( $args['options'], array(
1115
+		$args[ 'options' ]    = 'group' == $args[ 'type' ] ? wp_parse_args( $args[ 'options' ], array(
1116 1116
 			'add_button'    => esc_html__( 'Add Group', 'cmb2' ),
1117 1117
 			'remove_button' => esc_html__( 'Remove Group', 'cmb2' ),
1118
-		) ) : $args['options'];
1118
+		) ) : $args[ 'options' ];
1119 1119
 
1120
-		$args['_id']        = $args['id'];
1121
-		$args['_name']      = $args['id'];
1120
+		$args[ '_id' ]        = $args[ 'id' ];
1121
+		$args[ '_name' ]      = $args[ 'id' ];
1122 1122
 
1123 1123
 		if ( $this->group ) {
1124 1124
 
1125
-			$args['id']    = $this->group->args( 'id' ) . '_' . $this->group->index . '_' . $args['id'];
1126
-			$args['_name'] = $this->group->args( 'id' ) . '[' . $this->group->index . '][' . $args['_name'] . ']';
1125
+			$args[ 'id' ]    = $this->group->args( 'id' ) . '_' . $this->group->index . '_' . $args[ 'id' ];
1126
+			$args[ '_name' ] = $this->group->args( 'id' ) . '[' . $this->group->index . '][' . $args[ '_name' ] . ']';
1127 1127
 		}
1128 1128
 
1129
-		if ( 'wysiwyg' == $args['type'] ) {
1130
-			$args['id'] = strtolower( str_ireplace( '-', '_', $args['id'] ) );
1131
-			$args['options']['textarea_name'] = $args['_name'];
1129
+		if ( 'wysiwyg' == $args[ 'type' ] ) {
1130
+			$args[ 'id' ] = strtolower( str_ireplace( '-', '_', $args[ 'id' ] ) );
1131
+			$args[ 'options' ][ 'textarea_name' ] = $args[ '_name' ];
1132 1132
 		}
1133 1133
 
1134 1134
 		$option_types = apply_filters( 'cmb2_all_or_nothing_types', array( 'select', 'radio', 'radio_inline', 'taxonomy_select', 'taxonomy_radio', 'taxonomy_radio_inline' ), $this );
1135 1135
 
1136
-		if ( in_array( $args['type'], $option_types, true ) ) {
1136
+		if ( in_array( $args[ 'type' ], $option_types, true ) ) {
1137 1137
 
1138
-			$args['show_option_none'] = isset( $args['show_option_none'] ) ? $args['show_option_none'] : null;
1139
-			$args['show_option_none'] = true === $args['show_option_none'] ? esc_html__( 'None', 'cmb2' ) : $args['show_option_none'];
1138
+			$args[ 'show_option_none' ] = isset( $args[ 'show_option_none' ] ) ? $args[ 'show_option_none' ] : null;
1139
+			$args[ 'show_option_none' ] = true === $args[ 'show_option_none' ] ? esc_html__( 'None', 'cmb2' ) : $args[ 'show_option_none' ];
1140 1140
 
1141
-			if ( null === $args['show_option_none'] ) {
1142
-				$off_by_default = in_array( $args['type'], array( 'select', 'radio', 'radio_inline' ), true );
1143
-				$args['show_option_none'] = $off_by_default ? false : esc_html__( 'None', 'cmb2' );
1141
+			if ( null === $args[ 'show_option_none' ] ) {
1142
+				$off_by_default = in_array( $args[ 'type' ], array( 'select', 'radio', 'radio_inline' ), true );
1143
+				$args[ 'show_option_none' ] = $off_by_default ? false : esc_html__( 'None', 'cmb2' );
1144 1144
 			}
1145 1145
 
1146 1146
 		}
1147 1147
 
1148
-		$args['has_supporting_data'] = in_array(
1149
-			$args['type'],
1148
+		$args[ 'has_supporting_data' ] = in_array(
1149
+			$args[ 'type' ],
1150 1150
 			array(
1151 1151
 				// CMB2_Sanitize::_save_file_id_value()/CMB2_Sanitize::_get_group_file_value_array()
1152 1152
 				'file',
@@ -1169,10 +1169,10 @@  discard block
 block discarded – undo
1169 1169
 	protected function get_default_args( $field_args, $field_group = null ) {
1170 1170
 		$args = parent::get_default_args( array(), $this->group );
1171 1171
 
1172
-		if ( isset( $field_args['field_args'] ) ) {
1172
+		if ( isset( $field_args[ 'field_args' ] ) ) {
1173 1173
 			$args = wp_parse_args( $field_args, $args );
1174 1174
 		} else {
1175
-			$args['field_args'] = wp_parse_args( $field_args, $this->args );
1175
+			$args[ 'field_args' ] = wp_parse_args( $field_args, $this->args );
1176 1176
 		}
1177 1177
 
1178 1178
 		return $args;
@@ -1215,43 +1215,43 @@  discard block
 block discarded – undo
1215 1215
 	 */
1216 1216
 	protected function convert_deprecated_params( $args ) {
1217 1217
 
1218
-		if ( isset( $args['row_classes'] ) ) {
1218
+		if ( isset( $args[ 'row_classes' ] ) ) {
1219 1219
 
1220 1220
 			$this->deprecated_param( __CLASS__ . '::__construct()', '2.2.3', self::DEPRECATED_PARAM, 'row_classes', 'classes' );
1221 1221
 
1222 1222
 			// row_classes param could be a callback
1223
-			if ( is_callable( $args['row_classes'] ) ) {
1223
+			if ( is_callable( $args[ 'row_classes' ] ) ) {
1224 1224
 
1225 1225
 				$this->deprecated_param( __CLASS__ . '::__construct()', '2.2.3', self::DEPRECATED_CB_PARAM, 'row_classes', 'classes_cb' );
1226 1226
 
1227
-				$args['classes_cb'] = $args['row_classes'];
1228
-				$args['classes'] = null;
1227
+				$args[ 'classes_cb' ] = $args[ 'row_classes' ];
1228
+				$args[ 'classes' ] = null;
1229 1229
 			} else {
1230 1230
 
1231 1231
 
1232
-				$args['classes'] = $args['row_classes'];
1232
+				$args[ 'classes' ] = $args[ 'row_classes' ];
1233 1233
 			}
1234 1234
 
1235
-			unset( $args['row_classes'] );
1235
+			unset( $args[ 'row_classes' ] );
1236 1236
 		}
1237 1237
 
1238 1238
 
1239 1239
 		// default param can be passed a callback as well
1240
-		if ( is_callable( $args['default'] ) ) {
1240
+		if ( is_callable( $args[ 'default' ] ) ) {
1241 1241
 
1242 1242
 			$this->deprecated_param( __CLASS__ . '::__construct()', '2.2.3', self::DEPRECATED_CB_PARAM, 'default', 'default_cb' );
1243 1243
 
1244
-			$args['default_cb'] = $args['default'];
1245
-			$args['default'] = null;
1244
+			$args[ 'default_cb' ] = $args[ 'default' ];
1245
+			$args[ 'default' ] = null;
1246 1246
 		}
1247 1247
 
1248 1248
 		// options param can be passed a callback as well
1249
-		if ( is_callable( $args['options'] ) ) {
1249
+		if ( is_callable( $args[ 'options' ] ) ) {
1250 1250
 
1251 1251
 			$this->deprecated_param( __CLASS__ . '::__construct()', '2.2.3', self::DEPRECATED_CB_PARAM, 'options', 'options_cb' );
1252 1252
 
1253
-			$args['options_cb'] = $args['options'];
1254
-			$args['options'] = array();
1253
+			$args[ 'options_cb' ] = $args[ 'options' ];
1254
+			$args[ 'options' ] = array();
1255 1255
 		}
1256 1256
 
1257 1257
 		return $args;
Please login to merge, or discard this patch.
includes/types/CMB2_Type_Taxonomy_Select.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -15,7 +15,7 @@
 block discarded – undo
15 15
 	public function render() {
16 16
 		$names = $this->get_object_terms();
17 17
 
18
-		$saved_term  = is_wp_error( $names ) || empty( $names ) ? $this->field->get_default() : $names[key( $names )]->slug;
18
+		$saved_term  = is_wp_error( $names ) || empty( $names ) ? $this->field->get_default() : $names[ key( $names ) ]->slug;
19 19
 		$terms       = $this->get_terms();
20 20
 		$options     = '';
21 21
 		$option_none = $this->field->args( 'show_option_none' );
Please login to merge, or discard this patch.
includes/CMB2_Field_Display.php 2 patches
Doc Comments   -2 removed lines patch added patch discarded remove patch
@@ -113,8 +113,6 @@
 block discarded – undo
113 113
 	 * Catchall method if field's 'display_cb' is NOT defined, or field type does
114 114
 	 * not have a corresponding display method
115 115
 	 * @since 2.2.2
116
-	 * @param  string $method    Non-existent method name
117
-	 * @param  array  $arguments All arguments passed to the method
118 116
 	 */
119 117
 	public function display() {
120 118
 		// If repeatable
Please login to merge, or discard this patch.
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 			if ( is_array( $this->field->value ) ) {
125 125
 
126 126
 				// Then loop and output.
127
-				echo '<ul class="cmb2-'. str_replace( '_', '-', $this->field->type() ) .'">';
127
+				echo '<ul class="cmb2-' . str_replace( '_', '-', $this->field->type() ) . '">';
128 128
 				foreach ( $this->field->value as $val ) {
129 129
 					$this->value = $val;
130 130
 					echo '<li>', $this->_display(), '</li>';
@@ -197,8 +197,8 @@  discard block
 block discarded – undo
197 197
 		$options = $this->field->options();
198 198
 
199 199
 		$fallback = $this->field->args( 'show_option_none' );
200
-		if ( ! $fallback && isset( $options[''] ) ) {
201
-			$fallback = $options[''];
200
+		if ( ! $fallback && isset( $options[ '' ] ) ) {
201
+			$fallback = $options[ '' ];
202 202
 		}
203 203
 		if ( ! $this->value && $fallback ) {
204 204
 			echo $fallback;
@@ -225,9 +225,9 @@  discard block
 block discarded – undo
225 225
 		$output = array();
226 226
 		foreach ( $this->value as $val ) {
227 227
 			if ( isset( $options[ $val ] ) ) {
228
-				$output[] = $options[ $val ];
228
+				$output[ ] = $options[ $val ];
229 229
 			} else {
230
-				$output[] = esc_attr( $val );
230
+				$output[ ] = esc_attr( $val );
231 231
 			}
232 232
 		}
233 233
 
@@ -251,7 +251,7 @@  discard block
 block discarded – undo
251 251
 	 * @since 2.2.2
252 252
 	 */
253 253
 	protected function _display() {
254
-		echo '<xmp class="cmb2-code">'. print_r( $this->value, true ) .'</xmp>';
254
+		echo '<xmp class="cmb2-code">' . print_r( $this->value, true ) . '</xmp>';
255 255
 	}
256 256
 }
257 257
 
@@ -293,7 +293,7 @@  discard block
 block discarded – undo
293 293
 		if ( $datetime && $datetime instanceof DateTime ) {
294 294
 			$tz       = $datetime->getTimezone();
295 295
 			$tzstring = $tz->getName();
296
-			$this->value    = $datetime->getTimestamp();
296
+			$this->value = $datetime->getTimestamp();
297 297
 		}
298 298
 
299 299
 		$date = $this->field->get_timestamp_format( 'date_format', $this->value );
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
 		if ( is_wp_error( $terms ) || empty( $terms ) && ( $default = $this->field->get_default() ) ) {
318 318
 			$term = get_term_by( 'slug', $default, $taxonomy );
319 319
 		} elseif ( ! empty( $terms ) ) {
320
-			$term = $terms[key( $terms )];
320
+			$term = $terms[ key( $terms ) ];
321 321
 		}
322 322
 
323 323
 		if ( $term ) {
@@ -341,10 +341,10 @@  discard block
 block discarded – undo
341 341
 			$terms = array();
342 342
 			if ( is_array( $default ) ) {
343 343
 				foreach ( $default as $slug ) {
344
-					$terms[] = get_term_by( 'slug', $slug, $taxonomy );
344
+					$terms[ ] = get_term_by( 'slug', $slug, $taxonomy );
345 345
 				}
346 346
 			} else {
347
-				$terms[] = get_term_by( 'slug', $default, $taxonomy );
347
+				$terms[ ] = get_term_by( 'slug', $default, $taxonomy );
348 348
 			}
349 349
 		}
350 350
 
@@ -353,7 +353,7 @@  discard block
 block discarded – undo
353 353
 			$links = array();
354 354
 			foreach ( $terms as $term ) {
355 355
 				$link = get_edit_term_link( $term->term_id, $taxonomy );
356
-				$links[] = '<a href="'. esc_url( $link ) .'">'. esc_html( $term->name ) .'</a>';
356
+				$links[ ] = '<a href="' . esc_url( $link ) . '">' . esc_html( $term->name ) . '</a>';
357 357
 			}
358 358
 			// Then loop and output.
359 359
 			echo '<div class="cmb2-taxonomy-terms-', esc_attr( $taxonomy ), '">';
@@ -396,7 +396,7 @@  discard block
 block discarded – undo
396 396
 			if ( $id ) {
397 397
 				$image = wp_get_attachment_image( $id, $img_size, null, array( 'class' => 'cmb-image-display' ) );
398 398
 			} else {
399
-				$size = is_array( $img_size ) ? $img_size[0] : 200;
399
+				$size = is_array( $img_size ) ? $img_size[ 0 ] : 200;
400 400
 				$image = '<img class="cmb-image-display" style="max-width: ' . absint( $size ) . 'px; width: 100%; height: auto;" src="' . $url_value . '" alt="" />';
401 401
 			}
402 402
 
Please login to merge, or discard this patch.
includes/CMB2_Hookup_Base.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
 			return;
55 55
 		}
56 56
 
57
-		$hooks_completed[] = $key;
57
+		$hooks_completed[ ] = $key;
58 58
 		add_filter( $action, $hook, $priority, $accepted_args );
59 59
 	}
60 60
 
Please login to merge, or discard this patch.
includes/CMB2_JS.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -56,23 +56,23 @@
 block discarded – undo
56 56
 		$min = $debug ? '' : '.min';
57 57
 
58 58
 		// if colorpicker
59
-		if ( ! is_admin() && isset( $dependencies['wp-color-picker'] ) ) {
59
+		if ( ! is_admin() && isset( $dependencies[ 'wp-color-picker' ] ) ) {
60 60
 			self::colorpicker_frontend();
61 61
 		}
62 62
 
63 63
 		// if file/file_list
64
-		if ( isset( $dependencies['media-editor'] ) ) {
64
+		if ( isset( $dependencies[ 'media-editor' ] ) ) {
65 65
 			wp_enqueue_media();
66 66
 		}
67 67
 
68 68
 		// if timepicker
69
-		if ( isset( $dependencies['jquery-ui-datetimepicker'] ) ) {
69
+		if ( isset( $dependencies[ 'jquery-ui-datetimepicker' ] ) ) {
70 70
 			wp_register_script( 'jquery-ui-datetimepicker', cmb2_utils()->url( 'js/jquery-ui-timepicker-addon.min.js' ), array( 'jquery-ui-slider' ), CMB2_VERSION );
71 71
 		}
72 72
 
73 73
 		// if cmb2-wysiwyg
74
-		$enqueue_wysiwyg = isset( $dependencies['cmb2-wysiwyg'] ) && $debug;
75
-		unset( $dependencies['cmb2-wysiwyg'] );
74
+		$enqueue_wysiwyg = isset( $dependencies[ 'cmb2-wysiwyg' ] ) && $debug;
75
+		unset( $dependencies[ 'cmb2-wysiwyg' ] );
76 76
 
77 77
 		// Enqueue cmb JS
78 78
 		wp_enqueue_script( self::$handle, cmb2_utils()->url( "js/cmb2{$min}.js" ), $dependencies, CMB2_VERSION, true );
Please login to merge, or discard this patch.
includes/types/CMB2_Type_Title.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -28,10 +28,10 @@
 block discarded – undo
28 28
 		return $this->rendered(
29 29
 			sprintf(
30 30
 				'<%1$s %2$s>%3$s</%1$s>%4$s',
31
-				$a['tag'],
31
+				$a[ 'tag' ],
32 32
 				$this->concat_attrs( $a, array( 'tag', 'name', 'desc' ) ),
33
-				$a['name'],
34
-				$a['desc']
33
+				$a[ 'name' ],
34
+				$a[ 'desc' ]
35 35
 			)
36 36
 		);
37 37
 	}
Please login to merge, or discard this patch.
includes/CMB2_Utils.php 2 patches
Spacing   +21 added lines, -22 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
 		$dir = wp_upload_dir();
38 38
 
39 39
 		// Is URL in uploads directory?
40
-		if ( false === strpos( $img_url, $dir['baseurl'] . '/' ) ) {
40
+		if ( false === strpos( $img_url, $dir[ 'baseurl' ] . '/' ) ) {
41 41
 			return false;
42 42
 		}
43 43
 
@@ -62,8 +62,8 @@  discard block
 block discarded – undo
62 62
 
63 63
 			foreach ( $query->posts as $post_id ) {
64 64
 				$meta = wp_get_attachment_metadata( $post_id );
65
-				$original_file       = basename( $meta['file'] );
66
-				$cropped_image_files = isset( $meta['sizes'] ) ? wp_list_pluck( $meta['sizes'], 'file' ) : array();
65
+				$original_file       = basename( $meta[ 'file' ] );
66
+				$cropped_image_files = isset( $meta[ 'sizes' ] ) ? wp_list_pluck( $meta[ 'sizes' ], 'file' ) : array();
67 67
 				if ( $original_file === $file || in_array( $file, $cropped_image_files ) ) {
68 68
 					$attachment_id = $post_id;
69 69
 					break;
@@ -147,8 +147,7 @@  discard block
 block discarded – undo
147 147
 		}
148 148
 
149 149
 		return self::is_valid_time_stamp( $string )
150
-			? (int) $string :
151
-			strtotime( (string) $string );
150
+			? (int) $string : strtotime( (string) $string );
152 151
 	}
153 152
 
154 153
 	/**
@@ -179,7 +178,7 @@  discard block
 block discarded – undo
179 178
 	 * @param  mixed $value Value to check
180 179
 	 * @return bool         True or false
181 180
 	 */
182
-	public static function notempty( $value ){
181
+	public static function notempty( $value ) {
183 182
 		return null !== $value && '' !== $value && false !== $value;
184 183
 	}
185 184
 
@@ -337,24 +336,24 @@  discard block
 block discarded – undo
337 336
 
338 337
 		// order is relevant here, since the replacement will be done sequentially.
339 338
 		$supported_options = array(
340
-			'd' => 'dd',  // Day, leading 0
341
-			'j' => 'd',   // Day, no 0
342
-			'z' => 'o',   // Day of the year, no leading zeroes,
339
+			'd' => 'dd', // Day, leading 0
340
+			'j' => 'd', // Day, no 0
341
+			'z' => 'o', // Day of the year, no leading zeroes,
343 342
 			// 'D' => 'D',   // Day name short, not sure how it'll work with translations
344 343
 			// 'l' => 'DD',  // Day name full, idem before
345
-			'm' => 'mm',  // Month of the year, leading 0
346
-			'n' => 'm',   // Month of the year, no leading 0
344
+			'm' => 'mm', // Month of the year, leading 0
345
+			'n' => 'm', // Month of the year, no leading 0
347 346
 			// 'M' => 'M',   // Month, Short name
348 347
 			// 'F' => 'MM',  // Month, full name,
349
-			'y' => 'y',   // Year, two digit
350
-			'Y' => 'yy',  // Year, full
351
-			'H' => 'HH',  // Hour with leading 0 (24 hour)
352
-			'G' => 'H',   // Hour with no leading 0 (24 hour)
353
-			'h' => 'hh',  // Hour with leading 0 (12 hour)
354
-			'g' => 'h',   // Hour with no leading 0 (12 hour),
355
-			'i' => 'mm',  // Minute with leading 0,
356
-			's' => 'ss',  // Second with leading 0,
357
-			'a' => 'tt',  // am/pm
348
+			'y' => 'y', // Year, two digit
349
+			'Y' => 'yy', // Year, full
350
+			'H' => 'HH', // Hour with leading 0 (24 hour)
351
+			'G' => 'H', // Hour with no leading 0 (24 hour)
352
+			'h' => 'hh', // Hour with leading 0 (12 hour)
353
+			'g' => 'h', // Hour with no leading 0 (12 hour),
354
+			'i' => 'mm', // Minute with leading 0,
355
+			's' => 'ss', // Second with leading 0,
356
+			'a' => 'tt', // am/pm
358 357
 			'A' => 'TT'   // AM/PM
359 358
 		);
360 359
 
@@ -375,7 +374,7 @@  discard block
 block discarded – undo
375 374
 	 * @return string Modified value
376 375
 	 */
377 376
 	public static function wrap_escaped_chars( $value ) {
378
-		return "&#39;" . str_replace( '\\', '', $value[0] ) . "&#39;";
377
+		return "&#39;" . str_replace( '\\', '', $value[ 0 ] ) . "&#39;";
379 378
 	}
380 379
 
381 380
 	/**
@@ -435,7 +434,7 @@  discard block
 block discarded – undo
435 434
 	 * @return string               String of attributes for form element.
436 435
 	 */
437 436
 	public static function concat_attrs( $attrs, $attr_exclude = array() ) {
438
-		$attr_exclude[] = 'rendered';
437
+		$attr_exclude[ ] = 'rendered';
439 438
 		$attributes = '';
440 439
 		foreach ( $attrs as $attr => $val ) {
441 440
 			$excluded = in_array( $attr, (array) $attr_exclude, true );
Please login to merge, or discard this patch.
Braces   +3 added lines, -2 removed lines patch added patch discarded remove patch
@@ -122,7 +122,8 @@  discard block
 block discarded – undo
122 122
 			$tzstring = '';
123 123
 		}
124 124
 
125
-		if ( empty( $tzstring ) ) { // Create a UTC+- zone if no timezone string exists
125
+		if ( empty( $tzstring ) ) {
126
+// Create a UTC+- zone if no timezone string exists
126 127
 			if ( 0 == $current_offset ) {
127 128
 				$tzstring = 'UTC+0';
128 129
 			} elseif ( $current_offset < 0 ) {
@@ -179,7 +180,7 @@  discard block
 block discarded – undo
179 180
 	 * @param  mixed $value Value to check
180 181
 	 * @return bool         True or false
181 182
 	 */
182
-	public static function notempty( $value ){
183
+	public static function notempty( $value ) {
183 184
 		return null !== $value && '' !== $value && false !== $value;
184 185
 	}
185 186
 
Please login to merge, or discard this patch.
includes/types/CMB2_Type_File_Base.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -55,11 +55,11 @@  discard block
 block discarded – undo
55 55
 	 */
56 56
 	public function img_status_output( $args ) {
57 57
 		return sprintf( '<%1$s class="img-status">%2$s<p class="cmb2-remove-wrapper"><a href="#" class="cmb2-remove-file-button"%3$s>%4$s</a></p>%5$s</%1$s>',
58
-			$args['tag'],
59
-			$args['image'],
60
-			isset( $args['cached_id'] ) ? ' rel="' . $args['cached_id'] . '"' : '',
58
+			$args[ 'tag' ],
59
+			$args[ 'image' ],
60
+			isset( $args[ 'cached_id' ] ) ? ' rel="' . $args[ 'cached_id' ] . '"' : '',
61 61
 			esc_html( $this->_text( 'remove_image_text', esc_html__( 'Remove Image', 'cmb2' ) ) ),
62
-			isset( $args['id_input'] ) ? $args['id_input'] : ''
62
+			isset( $args[ 'id_input' ] ) ? $args[ 'id_input' ] : ''
63 63
 		);
64 64
 	}
65 65
 
@@ -71,14 +71,14 @@  discard block
 block discarded – undo
71 71
 	 */
72 72
 	public function file_status_output( $args ) {
73 73
 		return sprintf( '<%1$s class="file-status"><span>%2$s <strong>%3$s</strong></span>&nbsp;&nbsp; (<a href="%4$s" target="_blank" rel="external">%5$s</a> / <a href="#" class="cmb2-remove-file-button"%6$s>%7$s</a>)%8$s</%1$s>',
74
-			$args['tag'],
74
+			$args[ 'tag' ],
75 75
 			esc_html( $this->_text( 'file_text', esc_html__( 'File:', 'cmb2' ) ) ),
76
-			CMB2_Utils::get_file_name_from_path( $args['value'] ),
77
-			$args['value'],
76
+			CMB2_Utils::get_file_name_from_path( $args[ 'value' ] ),
77
+			$args[ 'value' ],
78 78
 			esc_html( $this->_text( 'file_download_text', esc_html__( 'Download', 'cmb2' ) ) ),
79
-			isset( $args['cached_id'] ) ? ' rel="' . $args['cached_id'] . '"' : '',
79
+			isset( $args[ 'cached_id' ] ) ? ' rel="' . $args[ 'cached_id' ] . '"' : '',
80 80
 			esc_html( $this->_text( 'remove_text', esc_html__( 'Remove', 'cmb2' ) ) ),
81
-			isset( $args['id_input'] ) ? $args['id_input'] : ''
81
+			isset( $args[ 'id_input' ] ) ? $args[ 'id_input' ] : ''
82 82
 		);
83 83
 	}
84 84
 
Please login to merge, or discard this patch.
includes/CMB2_Types.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -170,11 +170,11 @@  discard block
 block discarded – undo
170 170
 
171 171
 		$message = array( sprintf( esc_html__( 'Custom field types require a Type object instantiation to use this method. This method was called by the \'%s\' field type.' ), $this->field->type() ) );
172 172
 
173
-		$message[] = is_object( $this->type )
173
+		$message[ ] = is_object( $this->type )
174 174
 			? esc_html__( 'That field type may not work as expected.', 'cmb2' )
175 175
 			: esc_html__( 'That field type will not work as expected.', 'cmb2' );
176 176
 
177
-		$message[] = esc_html__( 'For more information about this change see: https://github.com/mustardBees/cmb-field-select2/pull/34w', 'cmb2' );
177
+		$message[ ] = esc_html__( 'For more information about this change see: https://github.com/mustardBees/cmb-field-select2/pull/34w', 'cmb2' );
178 178
 
179 179
 		_doing_it_wrong( __CLASS__ . '::' . $method, implode( ' ', $message ), '2.2.2' );
180 180
 	}
@@ -394,8 +394,8 @@  discard block
 block discarded – undo
394 394
 			'class' => 'cmb2-hidden',
395 395
 		);
396 396
 		if ( $this->field->group ) {
397
-			$args['data-groupid'] = $this->field->group->id();
398
-			$args['data-iterator'] = $this->iterator;
397
+			$args[ 'data-groupid' ] = $this->field->group->id();
398
+			$args[ 'data-iterator' ] = $this->iterator;
399 399
 		}
400 400
 		return $this->input( $args );
401 401
 	}
Please login to merge, or discard this patch.