Completed
Pull Request — master (#1356)
by Stephanie
01:05
created
classes/controllers/FrmOverlayController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
 	private function update_next_execution_time() {
86 86
 		$config_option_name = $this->config['config-option-name'];
87 87
 
88
-		$this->options_data[ $config_option_name ] = $this->get_next_execution();
88
+		$this->options_data[$config_option_name] = $this->get_next_execution();
89 89
 		$this->update_options_data();
90 90
 	}
91 91
 
@@ -104,11 +104,11 @@  discard block
 block discarded – undo
104 104
 	 * @return boolean
105 105
 	 */
106 106
 	private function is_time_to_execute() {
107
-		if ( ! isset( $this->options_data[ $this->config['config-option-name'] ] ) ) {
107
+		if ( ! isset( $this->options_data[$this->config['config-option-name']] ) ) {
108 108
 			return true;
109 109
 		}
110 110
 
111
-		$options = $this->options_data[ $this->config['config-option-name'] ];
111
+		$options = $this->options_data[$this->config['config-option-name']];
112 112
 
113 113
 		return ! ( isset( $options['timestamp'] ) && (int) $options['timestamp'] > $this->get_time() );
114 114
 	}
Please login to merge, or discard this patch.
stripe/controllers/FrmTransLiteActionsController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -391,8 +391,8 @@
 block discarded – undo
391 391
 			return $values;
392 392
 		}
393 393
 
394
-		if ( is_array( $previous_entry->metas ) && isset( $previous_entry->metas[ $field->id ] ) ) {
395
-			$values['value'] = $previous_entry->metas[ $field->id ];
394
+		if ( is_array( $previous_entry->metas ) && isset( $previous_entry->metas[$field->id] ) ) {
395
+			$values['value'] = $previous_entry->metas[$field->id];
396 396
 		}
397 397
 
398 398
 		$frm_vars['trans_filled'] = true;
Please login to merge, or discard this patch.
stripe/models/FrmStrpLiteAuth.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
 			return $html;
40 40
 		}
41 41
 
42
-		$atts    = array(
42
+		$atts = array(
43 43
 			'fields' => FrmFieldsHelper::get_form_fields( $form_id ),
44 44
 			'entry'  => $details['entry'],
45 45
 		);
@@ -161,7 +161,7 @@  discard block
 block discarded – undo
161 161
 		$atts['form']        = FrmForm::getOne( $atts['entry']->form_id );
162 162
 		$atts['entry_id']    = $atts['entry']->id;
163 163
 		$opt                 = 'success_action';
164
-		$atts['conf_method'] = ! empty( $atts['form']->options[ $opt ] ) ? $atts['form']->options[ $opt ] : 'message';
164
+		$atts['conf_method'] = ! empty( $atts['form']->options[$opt] ) ? $atts['form']->options[$opt] : 'message';
165 165
 	}
166 166
 
167 167
 	/**
@@ -238,10 +238,10 @@  discard block
 block discarded – undo
238 238
 	 */
239 239
 	public static function get_payment_intents( $name ) {
240 240
 		// phpcs:ignore WordPress.Security.NonceVerification.Missing
241
-		if ( ! isset( $_POST[ $name ] ) ) {
241
+		if ( ! isset( $_POST[$name] ) ) {
242 242
 			return array();
243 243
 		}
244
-		$intents = $_POST[ $name ]; // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
244
+		$intents = $_POST[$name]; // phpcs:ignore WordPress.Security.ValidatedSanitizedInput.MissingUnslash, WordPress.Security.ValidatedSanitizedInput.InputNotSanitized, WordPress.Security.NonceVerification.Missing
245 245
 		FrmAppHelper::sanitize_value( 'sanitize_text_field', $intents );
246 246
 		return $intents;
247 247
 	}
@@ -268,7 +268,7 @@  discard block
 block discarded – undo
268 268
 		self::format_form_data( $form );
269 269
 
270 270
 		$form_id = absint( $form['form_id'] );
271
-		$intents = isset( $form[ 'frmintent' . $form_id ] ) ? $form[ 'frmintent' . $form_id ] : array();
271
+		$intents = isset( $form['frmintent' . $form_id] ) ? $form['frmintent' . $form_id] : array();
272 272
 
273 273
 		if ( empty( $intents ) ) {
274 274
 			wp_die();
@@ -278,8 +278,8 @@  discard block
 block discarded – undo
278 278
 			$intents = array( $intents );
279 279
 		} else {
280 280
 			foreach ( $intents as $k => $intent ) {
281
-				if ( is_array( $intent ) && isset( $intent[ $k ] ) ) {
282
-					$intents[ $k ] = $intent[ $k ];
281
+				if ( is_array( $intent ) && isset( $intent[$k] ) ) {
282
+					$intents[$k] = $intent[$k];
283 283
 				}
284 284
 			}
285 285
 		}
@@ -327,12 +327,12 @@  discard block
 block discarded – undo
327 327
 				continue;
328 328
 			}
329 329
 
330
-			$saved     = FrmStrpLiteAppHelper::call_stripe_helper_class( 'get_intent', $intent_id );
330
+			$saved = FrmStrpLiteAppHelper::call_stripe_helper_class( 'get_intent', $intent_id );
331 331
 			foreach ( $actions as $action ) {
332 332
 				if ( $saved->metadata->action != $action->ID ) {
333 333
 					continue;
334 334
 				}
335
-				$intents[ $k ] = array(
335
+				$intents[$k] = array(
336 336
 					'id'     => $intent,
337 337
 					'action' => $action->ID,
338 338
 				);
@@ -375,7 +375,7 @@  discard block
 block discarded – undo
375 375
 			if ( $k === 'item_meta' ) {
376 376
 				foreach ( $v as $f => $value ) {
377 377
 					FrmAppHelper::sanitize_value( 'wp_kses_post', $value );
378
-					$entry->metas[ absint( $f ) ] = $value;
378
+					$entry->metas[absint( $f )] = $value;
379 379
 				}
380 380
 			} else {
381 381
 				FrmAppHelper::sanitize_value( 'wp_kses_post', $v );
@@ -399,14 +399,14 @@  discard block
 block discarded – undo
399 399
 
400 400
 		foreach ( $form as $input ) {
401 401
 			$key = $input['name'];
402
-			if ( isset( $formatted[ $key ] ) ) {
403
-				if ( is_array( $formatted[ $key ] ) ) {
404
-					$formatted[ $key ][] = $input['value'];
402
+			if ( isset( $formatted[$key] ) ) {
403
+				if ( is_array( $formatted[$key] ) ) {
404
+					$formatted[$key][] = $input['value'];
405 405
 				} else {
406
-					$formatted[ $key ] = array( $formatted[ $key ], $input['value'] );
406
+					$formatted[$key] = array( $formatted[$key], $input['value'] );
407 407
 				}
408 408
 			} else {
409
-				$formatted[ $key ] = $input['value'];
409
+				$formatted[$key] = $input['value'];
410 410
 			}
411 411
 		}
412 412
 
@@ -542,7 +542,7 @@  discard block
 block discarded – undo
542 542
 
543 543
 		foreach ( $actions as $k => $action ) {
544 544
 			$amount = self::get_amount_before_submit( compact( 'action', 'form' ) );
545
-			$actions[ $k ]->post_content['amount'] = $amount;
545
+			$actions[$k]->post_content['amount'] = $amount;
546 546
 		}
547 547
 	}
548 548
 
Please login to merge, or discard this patch.
stripe/helpers/FrmStrpLiteUrlParamHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -23,10 +23,10 @@  discard block
 block discarded – undo
23 23
 	 * @return array|false
24 24
 	 */
25 25
 	public static function get_details_for_form( $form_id ) {
26
-		if ( ! isset( self::$details_by_form_id[ $form_id ] ) ) {
26
+		if ( ! isset( self::$details_by_form_id[$form_id] ) ) {
27 27
 			self::set_details_for_form( (int) $form_id );
28 28
 		}
29
-		return isset( self::$details_by_form_id[ $form_id ] ) ? self::$details_by_form_id[ $form_id ] : false;
29
+		return isset( self::$details_by_form_id[$form_id] ) ? self::$details_by_form_id[$form_id] : false;
30 30
 	}
31 31
 
32 32
 	/**
@@ -87,7 +87,7 @@  discard block
 block discarded – undo
87 87
 			return;
88 88
 		}
89 89
 
90
-		self::$details_by_form_id[ $form_id ] = array(
90
+		self::$details_by_form_id[$form_id] = array(
91 91
 			'entry'   => $entry,
92 92
 			'intent'  => $intent,
93 93
 			'payment' => $payment,
Please login to merge, or discard this patch.
classes/controllers/FrmAppController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -388,7 +388,7 @@
 block discarded – undo
388 388
 	 * @return void
389 389
 	 */
390 390
 	public static function api_email_form( $form_key, $title, $description ) {
391
-		$user        = wp_get_current_user();
391
+		$user = wp_get_current_user();
392 392
 		$args = array(
393 393
 			'api_url'     => 'https://sandbox.formidableforms.com/api/wp-json/frm/v2/forms/' . $form_key . '?return=html&exclude_script=jquery&exclude_style=formidable-css',
394 394
 			'title'       => $title,
Please login to merge, or discard this patch.
classes/helpers/FrmEntriesListHelper.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 
71 71
 		if ( strpos( $orderby, 'meta' ) !== false ) {
72 72
 			$order_field_type = FrmField::get_type( str_replace( 'meta_', '', $orderby ) );
73
-			$orderby          .= in_array( $order_field_type, array( 'number', 'scale', 'star' ) ) ? '+0' : '';
73
+			$orderby .= in_array( $order_field_type, array( 'number', 'scale', 'star' ) ) ? '+0' : '';
74 74
 		}
75 75
 
76 76
 		$order = self::get_param(
@@ -116,7 +116,7 @@  discard block
 block discarded – undo
116 116
 			$form_ids              = $this->get_form_ids( $form_id );
117 117
 			$s_query['it.form_id'] = count( $form_ids ) > 1 ? $form_ids : $form_ids[0];
118 118
 		} else {
119
-			$s_query[]          = array(
119
+			$s_query[] = array(
120 120
 				'or'               => 1,
121 121
 				'parent_form_id'   => null,
122 122
 				'parent_form_id <' => 1,
@@ -265,7 +265,7 @@  discard block
 block discarded – undo
265 265
 
266 266
 		$r = "<tr id='item-action-{$item->id}'$style>";
267 267
 
268
-		list( $columns, $hidden, , $primary ) = $this->get_column_info();
268
+		list( $columns, $hidden,, $primary ) = $this->get_column_info();
269 269
 		$action_col                           = false;
270 270
 		$action_columns                       = $this->get_action_columns();
271 271
 
Please login to merge, or discard this patch.
classes/models/FrmDb.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 		}
65 65
 
66 66
 		foreach ( $args as $key => $value ) {
67
-			$where          .= empty( $where ) ? $base_where : $condition;
67
+			$where .= empty( $where ) ? $base_where : $condition;
68 68
 			$array_inc_null = ( ! is_numeric( $key ) && is_array( $value ) && in_array( null, $value ) );
69 69
 			if ( is_numeric( $key ) || $array_inc_null ) {
70 70
 				$where        .= ' ( ';
@@ -113,12 +113,12 @@  discard block
 block discarded – undo
113 113
 						$where .= ' OR ';
114 114
 					}
115 115
 					$start    = false;
116
-					$where    .= $key . ' %s';
116
+					$where .= $key . ' %s';
117 117
 					$values[] = '%' . self::esc_like( $v ) . '%';
118 118
 				}
119 119
 				$where .= ')';
120 120
 			} elseif ( ! empty( $value ) ) {
121
-				$where  .= ' in (' . self::prepare_array_values( $value, '%s' ) . ')';
121
+				$where .= ' in (' . self::prepare_array_values( $value, '%s' ) . ')';
122 122
 				$values = array_merge( $values, $value );
123 123
 			}
124 124
 		} elseif ( strpos( $lowercase_key, 'like' ) !== false ) {
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
 				$where .= 'like';
139 139
 			}
140 140
 
141
-			$where    .= ' %s';
141
+			$where .= ' %s';
142 142
 			$values[] = $start . self::esc_like( $value ) . $end;
143 143
 
144 144
 		} elseif ( $value === null ) {
@@ -306,8 +306,8 @@  discard block
 block discarded – undo
306 306
 		);
307 307
 
308 308
 		$where_is = strtolower( $where_is );
309
-		if ( isset( $switch_to[ $where_is ] ) ) {
310
-			return ' ' . $switch_to[ $where_is ];
309
+		if ( isset( $switch_to[$where_is] ) ) {
310
+			return ' ' . $switch_to[$where_is];
311 311
 		}
312 312
 
313 313
 		// > and < need a little more work since we don't want them switched to >= and <=
@@ -371,13 +371,13 @@  discard block
 block discarded – undo
371 371
 		$temp_args = $args;
372 372
 		foreach ( $temp_args as $k => $v ) {
373 373
 			if ( $v == '' ) {
374
-				unset( $args[ $k ] );
374
+				unset( $args[$k] );
375 375
 				continue;
376 376
 			}
377 377
 
378 378
 			$db_name = strtoupper( str_replace( '_', ' ', $k ) );
379 379
 			if ( strpos( $v, $db_name ) === false ) {
380
-				$args[ $k ] = $db_name . ' ' . $v;
380
+				$args[$k] = $db_name . ' ' . $v;
381 381
 			}
382 382
 		}
383 383
 
@@ -451,13 +451,13 @@  discard block
 block discarded – undo
451 451
 	private static function esc_query_args( &$args ) {
452 452
 		foreach ( $args as $param => $value ) {
453 453
 			if ( $param == 'order_by' ) {
454
-				$args[ $param ] = self::esc_order( $value );
454
+				$args[$param] = self::esc_order( $value );
455 455
 			} elseif ( $param == 'limit' ) {
456
-				$args[ $param ] = self::esc_limit( $value );
456
+				$args[$param] = self::esc_limit( $value );
457 457
 			}
458 458
 
459
-			if ( $args[ $param ] == '' ) {
460
-				unset( $args[ $param ] );
459
+			if ( $args[$param] == '' ) {
460
+				unset( $args[$param] );
461 461
 			}
462 462
 		}
463 463
 	}
@@ -540,7 +540,7 @@  discard block
 block discarded – undo
540 540
 		$limit = explode( ',', trim( $limit ) );
541 541
 		foreach ( $limit as $k => $l ) {
542 542
 			if ( is_numeric( $l ) ) {
543
-				$limit[ $k ] = $l;
543
+				$limit[$k] = $l;
544 544
 			}
545 545
 		}
546 546
 
@@ -691,7 +691,7 @@  discard block
 block discarded – undo
691 691
 	 */
692 692
 	public static function add_key_to_group_cache( $key, $group ) {
693 693
 		$cached         = self::get_group_cached_keys( $group );
694
-		$cached[ $key ] = $key;
694
+		$cached[$key] = $key;
695 695
 		wp_cache_set( 'cached_keys', $cached, $group, 300 );
696 696
 	}
697 697
 
Please login to merge, or discard this patch.
classes/helpers/FrmCurrencyHelper.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -16,10 +16,10 @@
 block discarded – undo
16 16
 	public static function get_currency( $currency ) {
17 17
 		$currency   = strtoupper( $currency );
18 18
 		$currencies = self::get_currencies();
19
-		if ( isset( $currencies[ $currency ] ) ) {
20
-			$currency = $currencies[ $currency ];
21
-		} elseif ( isset( $currencies[ strtolower( $currency ) ] ) ) {
22
-			$currency = $currencies[ strtolower( $currency ) ];
19
+		if ( isset( $currencies[$currency] ) ) {
20
+			$currency = $currencies[$currency];
21
+		} elseif ( isset( $currencies[strtolower( $currency )] ) ) {
22
+			$currency = $currencies[strtolower( $currency )];
23 23
 		} else {
24 24
 			$currency = $currencies['USD'];
25 25
 		}
Please login to merge, or discard this patch.
classes/models/FrmInbox.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -105,7 +105,7 @@  discard block
 block discarded – undo
105 105
 			return;
106 106
 		}
107 107
 
108
-		if ( isset( self::$messages[ $message['key'] ] ) && ! isset( $message['force'] ) ) {
108
+		if ( isset( self::$messages[$message['key']] ) && ! isset( $message['force'] ) ) {
109 109
 			// Don't replace messages unless required.
110 110
 			return;
111 111
 		}
@@ -114,13 +114,13 @@  discard block
 block discarded – undo
114 114
 			return;
115 115
 		}
116 116
 
117
-		if ( isset( self::$messages[ $message['key'] ] ) ) {
117
+		if ( isset( self::$messages[$message['key']] ) ) {
118 118
 			// Move up and mark as new.
119
-			unset( self::$messages[ $message['key'] ] );
119
+			unset( self::$messages[$message['key']] );
120 120
 		}
121 121
 
122 122
 		$this->fill_message( $message );
123
-		self::$messages[ $message['key'] ] = $message;
123
+		self::$messages[$message['key']] = $message;
124 124
 
125 125
 		$this->update_list();
126 126
 
@@ -156,11 +156,11 @@  discard block
 block discarded – undo
156 156
 	private function clean_messages() {
157 157
 		$removed = false;
158 158
 		foreach ( self::$messages as $t => $message ) {
159
-			$read      = isset( $message['read'] ) && ! empty( $message['read'] ) && isset( $message['read'][ get_current_user_id() ] ) && $message['read'][ get_current_user_id() ] < strtotime( '-1 month' );
160
-			$dismissed = isset( $message['dismissed'] ) && ! empty( $message['dismissed'] ) && isset( $message['dismissed'][ get_current_user_id() ] ) && $message['dismissed'][ get_current_user_id() ] < strtotime( '-1 week' );
159
+			$read      = isset( $message['read'] ) && ! empty( $message['read'] ) && isset( $message['read'][get_current_user_id()] ) && $message['read'][get_current_user_id()] < strtotime( '-1 month' );
160
+			$dismissed = isset( $message['dismissed'] ) && ! empty( $message['dismissed'] ) && isset( $message['dismissed'][get_current_user_id()] ) && $message['dismissed'][get_current_user_id()] < strtotime( '-1 week' );
161 161
 			$expired   = $this->is_expired( $message );
162 162
 			if ( $read || $expired || $dismissed ) {
163
-				unset( self::$messages[ $t ] );
163
+				unset( self::$messages[$t] );
164 164
 				$removed = true;
165 165
 			}
166 166
 		}
@@ -176,11 +176,11 @@  discard block
 block discarded – undo
176 176
 	public function filter_messages( &$messages, $type = 'unread' ) {
177 177
 		$user_id = get_current_user_id();
178 178
 		foreach ( $messages as $k => $message ) {
179
-			$dismissed = isset( $message['dismissed'] ) && isset( $message['dismissed'][ $user_id ] );
179
+			$dismissed = isset( $message['dismissed'] ) && isset( $message['dismissed'][$user_id] );
180 180
 			if ( empty( $k ) || $this->is_expired( $message ) || ( $dismissed && 'dismissed' !== $type ) || ( 'dismissed' === $type && ! $dismissed ) ) {
181
-				unset( $messages[ $k ] );
181
+				unset( $messages[$k] );
182 182
 			} elseif ( ! $this->is_for_user( $message ) ) {
183
-				unset( $messages[ $k ] );
183
+				unset( $messages[$k] );
184 184
 			}
185 185
 		}
186 186
 		$messages = apply_filters( 'frm_filter_inbox', $messages );
@@ -226,14 +226,14 @@  discard block
 block discarded – undo
226 226
 	 * @return void
227 227
 	 */
228 228
 	public function mark_read( $key ) {
229
-		if ( ! $key || ! isset( self::$messages[ $key ] ) ) {
229
+		if ( ! $key || ! isset( self::$messages[$key] ) ) {
230 230
 			return;
231 231
 		}
232 232
 
233
-		if ( ! isset( self::$messages[ $key ]['read'] ) ) {
234
-			self::$messages[ $key ]['read'] = array();
233
+		if ( ! isset( self::$messages[$key]['read'] ) ) {
234
+			self::$messages[$key]['read'] = array();
235 235
 		}
236
-		self::$messages[ $key ]['read'][ get_current_user_id() ] = time();
236
+		self::$messages[$key]['read'][get_current_user_id()] = time();
237 237
 
238 238
 		$this->update_list();
239 239
 	}
@@ -246,9 +246,9 @@  discard block
 block discarded – undo
246 246
 	 * @return void
247 247
 	 */
248 248
 	public function mark_unread( $key ) {
249
-		$is_read = isset( self::$messages[ $key ] ) && isset( self::$messages[ $key ]['read'] ) && isset( self::$messages[ $key ]['read'][ get_current_user_id() ] );
249
+		$is_read = isset( self::$messages[$key] ) && isset( self::$messages[$key]['read'] ) && isset( self::$messages[$key]['read'][get_current_user_id()] );
250 250
 		if ( $is_read ) {
251
-			unset( self::$messages[ $key ]['read'][ get_current_user_id() ] );
251
+			unset( self::$messages[$key]['read'][get_current_user_id()] );
252 252
 			$this->update_list();
253 253
 		}
254 254
 	}
@@ -264,14 +264,14 @@  discard block
 block discarded – undo
264 264
 			return;
265 265
 		}
266 266
 
267
-		if ( ! isset( self::$messages[ $key ] ) ) {
267
+		if ( ! isset( self::$messages[$key] ) ) {
268 268
 			return;
269 269
 		}
270 270
 
271
-		if ( ! isset( self::$messages[ $key ]['dismissed'] ) ) {
272
-			self::$messages[ $key ]['dismissed'] = array();
271
+		if ( ! isset( self::$messages[$key]['dismissed'] ) ) {
272
+			self::$messages[$key]['dismissed'] = array();
273 273
 		}
274
-		self::$messages[ $key ]['dismissed'][ get_current_user_id() ] = time();
274
+		self::$messages[$key]['dismissed'][get_current_user_id()] = time();
275 275
 
276 276
 		$this->update_list();
277 277
 	}
@@ -285,11 +285,11 @@  discard block
 block discarded – undo
285 285
 		$user_id = get_current_user_id();
286 286
 		foreach ( self::$messages as $key => $message ) {
287 287
 			if ( ! isset( $message['dismissed'] ) ) {
288
-				self::$messages[ $key ]['dismissed'] = array();
288
+				self::$messages[$key]['dismissed'] = array();
289 289
 			}
290 290
 
291
-			if ( ! isset( $message['dismissed'][ $user_id ] ) ) {
292
-				self::$messages[ $key ]['dismissed'][ $user_id ] = time();
291
+			if ( ! isset( $message['dismissed'][$user_id] ) ) {
292
+				self::$messages[$key]['dismissed'][$user_id] = time();
293 293
 			}
294 294
 		}
295 295
 		$this->update_list();
@@ -299,8 +299,8 @@  discard block
 block discarded – undo
299 299
 		$messages = $this->get_messages( 'filter' );
300 300
 		$user_id  = get_current_user_id();
301 301
 		foreach ( $messages as $t => $message ) {
302
-			if ( isset( $message['read'] ) && isset( $message['read'][ $user_id ] ) ) {
303
-				unset( $messages[ $t ] );
302
+			if ( isset( $message['read'] ) && isset( $message['read'][$user_id] ) ) {
303
+				unset( $messages[$t] );
304 304
 			}
305 305
 		}
306 306
 		return $messages;
@@ -328,8 +328,8 @@  discard block
 block discarded – undo
328 328
 	 * @return void
329 329
 	 */
330 330
 	public function remove( $key ) {
331
-		if ( isset( self::$messages[ $key ] ) ) {
332
-			unset( self::$messages[ $key ] );
331
+		if ( isset( self::$messages[$key] ) ) {
332
+			unset( self::$messages[$key] );
333 333
 			$this->update_list();
334 334
 		}
335 335
 	}
Please login to merge, or discard this patch.