Completed
Pull Request — trunk (#541)
by Justin
12:21 queued 08:58
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();
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
 	 * @return mixed             Value of field argument
140 140
 	 */
141 141
 	public function __call( $name, $arguments ) {
142
-		$key = isset( $arguments[0] ) ? $arguments[0] : false;
142
+		$key = isset( $arguments[ 0 ] ) ? $arguments[ 0 ] : false;
143 143
 		return $this->args( $name, $key );
144 144
 	}
145 145
 
@@ -210,9 +210,9 @@  discard block
 block discarded – undo
210 210
 	 */
211 211
 	public function get_data( $field_id = '', $args = array() ) {
212 212
 		if ( $field_id ) {
213
-			$args['field_id'] = $field_id;
213
+			$args[ 'field_id' ] = $field_id;
214 214
 		} else if ( $this->group ) {
215
-			$args['field_id'] = $this->group->id();
215
+			$args[ 'field_id' ] = $this->group->id();
216 216
 		}
217 217
 
218 218
 		$a = $this->data_args( $args );
@@ -253,13 +253,13 @@  discard block
 block discarded – undo
253 253
 		 *
254 254
 		 * @since 2.0.0
255 255
 		 */
256
-		$data = apply_filters( "cmb2_override_{$a['field_id']}_meta_value", $data, $this->object_id, $a, $this );
256
+		$data = apply_filters( "cmb2_override_{$a[ 'field_id' ]}_meta_value", $data, $this->object_id, $a, $this );
257 257
 
258 258
 		// If no override, get value normally
259 259
 		if ( 'cmb2_field_no_override_val' === $data ) {
260
-			$data = 'options-page' === $a['type']
261
-				? cmb2_options( $a['id'] )->get( $a['field_id'] )
262
-				: get_metadata( $a['type'], $a['id'], $a['field_id'], ( $a['single'] || $a['repeat'] ) );
260
+			$data = 'options-page' === $a[ 'type' ]
261
+				? cmb2_options( $a[ 'id' ] )->get( $a[ 'field_id' ] )
262
+				: get_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], ( $a[ 'single' ] || $a[ 'repeat' ] ) );
263 263
 		}
264 264
 
265 265
 		if ( $this->group ) {
@@ -281,7 +281,7 @@  discard block
 block discarded – undo
281 281
 	public function update_data( $new_value, $single = true ) {
282 282
 		$a = $this->data_args( array( 'single' => $single ) );
283 283
 
284
-		$a['value'] = $a['repeat'] ? array_values( $new_value ) : $new_value;
284
+		$a[ 'value' ] = $a[ 'repeat' ] ? array_values( $new_value ) : $new_value;
285 285
 
286 286
 		/**
287 287
 		 * Filter whether to override saving of meta value.
@@ -317,7 +317,7 @@  discard block
 block discarded – undo
317 317
 		 *
318 318
 		 * @since 2.0.0
319 319
 		 */
320
-		$override = apply_filters( "cmb2_override_{$a['field_id']}_meta_save", $override, $a, $this->args(), $this );
320
+		$override = apply_filters( "cmb2_override_{$a[ 'field_id' ]}_meta_save", $override, $a, $this->args(), $this );
321 321
 
322 322
 		// If override, return that
323 323
 		if ( null !== $override ) {
@@ -325,22 +325,22 @@  discard block
 block discarded – undo
325 325
 		}
326 326
 
327 327
 		// Options page handling (or temp data store)
328
-		if ( 'options-page' === $a['type'] || empty( $a['id'] ) ) {
329
-			return cmb2_options( $a['id'] )->update( $a['field_id'], $a['value'], false, $a['single'] );
328
+		if ( 'options-page' === $a[ 'type' ] || empty( $a[ 'id' ] ) ) {
329
+			return cmb2_options( $a[ 'id' ] )->update( $a[ 'field_id' ], $a[ 'value' ], false, $a[ 'single' ] );
330 330
 		}
331 331
 
332 332
 		// Add metadata if not single
333
-		if ( ! $a['single'] ) {
334
-			return add_metadata( $a['type'], $a['id'], $a['field_id'], $a['value'], false );
333
+		if ( ! $a[ 'single' ] ) {
334
+			return add_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $a[ 'value' ], false );
335 335
 		}
336 336
 
337 337
 		// Delete meta if we have an empty array
338
-		if ( is_array( $a['value'] ) && empty( $a['value'] ) ) {
339
-			return delete_metadata( $a['type'], $a['id'], $a['field_id'], $this->value );
338
+		if ( is_array( $a[ 'value' ] ) && empty( $a[ 'value' ] ) ) {
339
+			return delete_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $this->value );
340 340
 		}
341 341
 
342 342
 		// Update metadata
343
-		return update_metadata( $a['type'], $a['id'], $a['field_id'], $a['value'] );
343
+		return update_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $a[ 'value' ] );
344 344
 	}
345 345
 
346 346
 	/**
@@ -390,19 +390,19 @@  discard block
 block discarded – undo
390 390
 		 * @param array $field_args All field arguments
391 391
 		 * @param CMB2_Field object $field This field object
392 392
 		 */
393
-		$override = apply_filters( "cmb2_override_{$a['field_id']}_meta_remove", $override, $a, $this->args(), $this );
393
+		$override = apply_filters( "cmb2_override_{$a[ 'field_id' ]}_meta_remove", $override, $a, $this->args(), $this );
394 394
 
395 395
 		// If no override, remove as usual
396 396
 		if ( null !== $override ) {
397 397
 			return $override;
398 398
 		}
399 399
 		// Option page handling
400
-		elseif ( 'options-page' === $a['type'] || empty( $a['id'] ) ) {
401
-			return cmb2_options( $a['id'] )->remove( $a['field_id'] );
400
+		elseif ( 'options-page' === $a[ 'type' ] || empty( $a[ 'id' ] ) ) {
401
+			return cmb2_options( $a[ 'id' ] )->remove( $a[ 'field_id' ] );
402 402
 		}
403 403
 
404 404
 		// Remove metadata
405
-		return delete_metadata( $a['type'], $a['id'], $a['field_id'], $old );
405
+		return delete_metadata( $a[ 'type' ], $a[ 'id' ], $a[ 'field_id' ], $old );
406 406
 	}
407 407
 
408 408
 	/**
@@ -854,7 +854,7 @@  discard block
 block discarded – undo
854 854
 
855 855
 		foreach ( $conditional_classes as $class => $condition ) {
856 856
 			if ( $condition ) {
857
-				$classes[] = $class;
857
+				$classes[ ] = $class;
858 858
 			}
859 859
 		}
860 860
 
@@ -865,7 +865,7 @@  discard block
 block discarded – undo
865 865
 		}
866 866
 
867 867
 		if ( $added_classes ) {
868
-			$classes[] = esc_attr( $added_classes );
868
+			$classes[ ] = esc_attr( $added_classes );
869 869
 		}
870 870
 
871 871
 		/**
@@ -965,10 +965,10 @@  discard block
 block discarded – undo
965 965
 	public function string( $text_key, $fallback ) {
966 966
 		// If null, populate with our field strings values.
967 967
 		if ( null === $this->strings ) {
968
-			$this->strings = (array) $this->args['text'];
968
+			$this->strings = (array) $this->args[ 'text' ];
969 969
 
970
-			if ( is_callable( $this->args['text_cb'] ) ) {
971
-				$strings = call_user_func( $this->args['text_cb'], $this );
970
+			if ( is_callable( $this->args[ 'text_cb' ] ) ) {
971
+				$strings = call_user_func( $this->args[ 'text_cb' ], $this );
972 972
 
973 973
 				if ( $strings && is_array( $strings ) ) {
974 974
 					$this->strings += $strings;
@@ -1002,10 +1002,10 @@  discard block
 block discarded – undo
1002 1002
 			return $this->field_options;
1003 1003
 		}
1004 1004
 
1005
-		$this->field_options = (array) $this->args['options'];
1005
+		$this->field_options = (array) $this->args[ 'options' ];
1006 1006
 
1007
-		if ( is_callable( $this->args['options_cb'] ) ) {
1008
-			$options = call_user_func( $this->args['options_cb'], $this );
1007
+		if ( is_callable( $this->args[ 'options_cb' ] ) ) {
1008
+			$options = call_user_func( $this->args[ 'options_cb' ], $this );
1009 1009
 
1010 1010
 			if ( $options && is_array( $options ) ) {
1011 1011
 				$this->field_options = $options + $this->field_options;
@@ -1026,7 +1026,7 @@  discard block
 block discarded – undo
1026 1026
 	 */
1027 1027
 	public function add_js_dependencies( $dependencies = array() ) {
1028 1028
 		foreach ( (array) $dependencies as $dependency ) {
1029
-			$this->args['js_dependencies'][ $dependency ] = $dependency;
1029
+			$this->args[ 'js_dependencies' ][ $dependency ] = $dependency;
1030 1030
 		}
1031 1031
 
1032 1032
 		CMB2_JS::add_dependencies( $dependencies );
@@ -1040,17 +1040,17 @@  discard block
 block discarded – undo
1040 1040
 	 * @return mixed  Default field value
1041 1041
 	 */
1042 1042
 	public function get_default() {
1043
-		if ( null !== $this->args['default'] ) {
1044
-			return $this->args['default'];
1043
+		if ( null !== $this->args[ 'default' ] ) {
1044
+			return $this->args[ 'default' ];
1045 1045
 		}
1046 1046
 
1047
-		$param = is_callable( $this->args['default_cb'] ) ? 'default_cb' : 'default';
1047
+		$param = is_callable( $this->args[ 'default_cb' ] ) ? 'default_cb' : 'default';
1048 1048
 		$default = $this->get_param_callback_result( $param );
1049 1049
 
1050 1050
 		// Allow a filter override of the default value
1051
-		$this->args['default'] = apply_filters( 'cmb2_default_filter', $default, $this );
1051
+		$this->args[ 'default' ] = apply_filters( 'cmb2_default_filter', $default, $this );
1052 1052
 
1053
-		return $this->args['default'];
1053
+		return $this->args[ 'default' ];
1054 1054
 	}
1055 1055
 
1056 1056
 	/**
@@ -1088,18 +1088,18 @@  discard block
 block discarded – undo
1088 1088
 			'classes_cb'        => '',
1089 1089
 			'select_all_button' => true,
1090 1090
 			'multiple'          => false,
1091
-			'repeatable'        => isset( $args['type'] ) && 'group' == $args['type'],
1091
+			'repeatable'        => isset( $args[ 'type' ] ) && 'group' == $args[ 'type' ],
1092 1092
 			'inline'            => false,
1093 1093
 			'on_front'          => true,
1094 1094
 			'show_names'        => true,
1095 1095
 			'save_field'        => true, // Will not save if false
1096 1096
 			'date_format'       => 'm\/d\/Y',
1097 1097
 			'time_format'       => 'h:i A',
1098
-			'description'       => isset( $args['desc'] ) ? $args['desc'] : '',
1099
-			'preview_size'      => 'file' == $args['type'] ? array( 350, 350 ) : array( 50, 50 ),
1098
+			'description'       => isset( $args[ 'desc' ] ) ? $args[ 'desc' ] : '',
1099
+			'preview_size'      => 'file' == $args[ 'type' ] ? array( 350, 350 ) : array( 50, 50 ),
1100 1100
 			'render_row_cb'     => array( $this, 'render_field_callback' ),
1101 1101
 			'display_cb'        => array( $this, 'display_value_callback' ),
1102
-			'label_cb'          => 'title' != $args['type'] ? array( $this, 'label' ) : '',
1102
+			'label_cb'          => 'title' != $args[ 'type' ] ? array( $this, 'label' ) : '',
1103 1103
 			'column'            => false,
1104 1104
 			'js_dependencies'   => array(),
1105 1105
 		) );
@@ -1108,73 +1108,73 @@  discard block
 block discarded – undo
1108 1108
 		 * Deprecated usage.
1109 1109
 		 */
1110 1110
 
1111
-		if ( isset( $args['row_classes'] ) ) {
1111
+		if ( isset( $args[ 'row_classes' ] ) ) {
1112 1112
 
1113 1113
 			// row_classes param could be a callback
1114
-			if ( is_callable( $args['row_classes'] ) ) {
1115
-				$args['classes_cb'] = $args['row_classes'];
1116
-				$args['classes'] = null;
1114
+			if ( is_callable( $args[ 'row_classes' ] ) ) {
1115
+				$args[ 'classes_cb' ] = $args[ 'row_classes' ];
1116
+				$args[ 'classes' ] = null;
1117 1117
 			} else {
1118
-				$args['classes'] = $args['row_classes'];
1118
+				$args[ 'classes' ] = $args[ 'row_classes' ];
1119 1119
 			}
1120 1120
 
1121
-			unset( $args['row_classes'] );
1121
+			unset( $args[ 'row_classes' ] );
1122 1122
 		}
1123 1123
 
1124 1124
 		// default param can be passed a callback as well
1125
-		if ( is_callable( $args['default'] ) ) {
1126
-			$args['default_cb'] = $args['default'];
1127
-			$args['default'] = null;
1125
+		if ( is_callable( $args[ 'default' ] ) ) {
1126
+			$args[ 'default_cb' ] = $args[ 'default' ];
1127
+			$args[ 'default' ] = null;
1128 1128
 		}
1129 1129
 
1130 1130
 		/*
1131 1131
 		 * END deprecated usage.
1132 1132
 		 */
1133 1133
 
1134
-		$args['repeatable'] = $args['repeatable'] && ! $this->repeatable_exception( $args['type'] );
1135
-		$args['inline']     = $args['inline'] || false !== stripos( $args['type'], '_inline' );
1134
+		$args[ 'repeatable' ] = $args[ 'repeatable' ] && ! $this->repeatable_exception( $args[ 'type' ] );
1135
+		$args[ 'inline' ]     = $args[ 'inline' ] || false !== stripos( $args[ 'type' ], '_inline' );
1136 1136
 
1137 1137
 		// options param can be passed a callback as well
1138
-		if ( is_callable( $args['options'] ) ) {
1139
-			$args['options_cb'] = $args['options'];
1140
-			$args['options'] = array();
1138
+		if ( is_callable( $args[ 'options' ] ) ) {
1139
+			$args[ 'options_cb' ] = $args[ 'options' ];
1140
+			$args[ 'options' ] = array();
1141 1141
 		}
1142 1142
 
1143
-		$args['options']    = 'group' == $args['type'] ? wp_parse_args( $args['options'], array(
1143
+		$args[ 'options' ] = 'group' == $args[ 'type' ] ? wp_parse_args( $args[ 'options' ], array(
1144 1144
 			'add_button'    => esc_html__( 'Add Group', 'cmb2' ),
1145 1145
 			'remove_button' => esc_html__( 'Remove Group', 'cmb2' ),
1146
-		) ) : $args['options'];
1146
+		) ) : $args[ 'options' ];
1147 1147
 
1148
-		$args['_id']        = $args['id'];
1149
-		$args['_name']      = $args['id'];
1148
+		$args[ '_id' ]        = $args[ 'id' ];
1149
+		$args[ '_name' ]      = $args[ 'id' ];
1150 1150
 
1151 1151
 		if ( $this->group ) {
1152 1152
 
1153
-			$args['id']    = $this->group->args( 'id' ) . '_' . $this->group->index . '_' . $args['id'];
1154
-			$args['_name'] = $this->group->args( 'id' ) . '[' . $this->group->index . '][' . $args['_name'] . ']';
1153
+			$args[ 'id' ]    = $this->group->args( 'id' ) . '_' . $this->group->index . '_' . $args[ 'id' ];
1154
+			$args[ '_name' ] = $this->group->args( 'id' ) . '[' . $this->group->index . '][' . $args[ '_name' ] . ']';
1155 1155
 		}
1156 1156
 
1157
-		if ( 'wysiwyg' == $args['type'] ) {
1158
-			$args['id'] = strtolower( str_ireplace( '-', '_', $args['id'] ) );
1159
-			$args['options']['textarea_name'] = $args['_name'];
1157
+		if ( 'wysiwyg' == $args[ 'type' ] ) {
1158
+			$args[ 'id' ] = strtolower( str_ireplace( '-', '_', $args[ 'id' ] ) );
1159
+			$args[ 'options' ][ 'textarea_name' ] = $args[ '_name' ];
1160 1160
 		}
1161 1161
 
1162 1162
 		$option_types = apply_filters( 'cmb2_all_or_nothing_types', array( 'select', 'radio', 'radio_inline', 'taxonomy_select', 'taxonomy_radio', 'taxonomy_radio_inline' ), $this );
1163 1163
 
1164
-		if ( in_array( $args['type'], $option_types, true ) ) {
1164
+		if ( in_array( $args[ 'type' ], $option_types, true ) ) {
1165 1165
 
1166
-			$args['show_option_none'] = isset( $args['show_option_none'] ) ? $args['show_option_none'] : null;
1167
-			$args['show_option_none'] = true === $args['show_option_none'] ? esc_html__( 'None', 'cmb2' ) : $args['show_option_none'];
1166
+			$args[ 'show_option_none' ] = isset( $args[ 'show_option_none' ] ) ? $args[ 'show_option_none' ] : null;
1167
+			$args[ 'show_option_none' ] = true === $args[ 'show_option_none' ] ? esc_html__( 'None', 'cmb2' ) : $args[ 'show_option_none' ];
1168 1168
 
1169
-			if ( null === $args['show_option_none'] ) {
1170
-				$off_by_default = in_array( $args['type'], array( 'select', 'radio', 'radio_inline' ), true );
1171
-				$args['show_option_none'] = $off_by_default ? false : esc_html__( 'None', 'cmb2' );
1169
+			if ( null === $args[ 'show_option_none' ] ) {
1170
+				$off_by_default = in_array( $args[ 'type' ], array( 'select', 'radio', 'radio_inline' ), true );
1171
+				$args[ 'show_option_none' ] = $off_by_default ? false : esc_html__( 'None', 'cmb2' );
1172 1172
 			}
1173 1173
 
1174 1174
 		}
1175 1175
 
1176
-		$args['has_supporting_data'] = in_array(
1177
-			$args['type'],
1176
+		$args[ 'has_supporting_data' ] = in_array(
1177
+			$args[ 'type' ],
1178 1178
 			array(
1179 1179
 				// CMB2_Sanitize::_save_file_id_value()/CMB2_Sanitize::_get_group_file_value_array()
1180 1180
 				'file',
@@ -1197,10 +1197,10 @@  discard block
 block discarded – undo
1197 1197
 	protected function get_default_args( $field_args, $field_group = null ) {
1198 1198
 		$args = parent::get_default_args( array(), $this->group );
1199 1199
 
1200
-		if ( isset( $field_args['field_args'] ) ) {
1200
+		if ( isset( $field_args[ 'field_args' ] ) ) {
1201 1201
 			$args = wp_parse_args( $field_args, $args );
1202 1202
 		} else {
1203
-			$args['field_args'] = wp_parse_args( $field_args, $this->args );
1203
+			$args[ 'field_args' ] = wp_parse_args( $field_args, $this->args );
1204 1204
 		}
1205 1205
 
1206 1206
 		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.