Make WordPress Core

Opened 12 years ago

Closed 11 years ago

Last modified 5 years ago

#22228 closed enhancement (fixed)

Remove one-time variable from add_settings_error

Reported by: niallkennedy's profile niallkennedy Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.0 Priority: normal
Severity: trivial Version: 3.0
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

Append an array to $wp_settings_errors without assigning it to a variable in the previous line.

Before:
$var = array( 'type' => $type ... );
$wp_settings_errors[] = $var;

After:
$wp_settings_errors[] = array( 'type' => $type ... );

Attachments (2)

template.diff (653 bytes) - added by niallkennedy 12 years ago.
append array directly to $wp_settings_errors
22228.diff (700 bytes) - added by obenland 11 years ago.
Refreshed patch

Download all attachments as: .zip

Change History (6)

@niallkennedy
12 years ago

append array directly to $wp_settings_errors

@obenland
11 years ago

Refreshed patch

#1 @obenland
11 years ago

  • Keywords has-patch added
  • Version set to 3.0

#2 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 4.0

#3 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 28681:

Remove one-time variable from add_settings_error().

props niallkennedy.
fixes #22228.

This ticket was mentioned in Slack in #core-editor by zieladam. View the logs.


5 years ago

Note: See TracTickets for help on using tickets.