Opened 18 months ago

Closed 16 months ago

#19377 closed defect (bug) (fixed)

Notice: Undefined index: settings-updated in wp-admin/includes/template.php on line 1322

Reported by: garyc40 Owned by: duck_
Priority: normal Milestone: 3.4
Component: Warnings/Notices Version: 3.2.1
Severity: minor Keywords: has-patch commit
Cc:

Description

When I use settings_errors() with the third argument set to true, I get the following PHP notice:

Notice: Undefined index: settings-updated in wp-admin/includes/template.php on line 1322

Attached a patch. The fix is straight-forward and harmless enough.

Attachments (4)

settings_errors.diff​ (734 bytes) - added by garyc40 18 months ago.
settings-errors.2.diff​ (592 bytes) - added by garyc40 18 months ago.
Previous patch was a bad one.
settings-errors.3.diff​ (1020 bytes) - added by garyc40 18 months ago.
What is wrong with me? The previous 2 patches were very very bad.
settings-errors.4.diff​ (537 bytes) - added by SergeyBiryukov 17 months ago.
Refreshed for current trunk

Download all attachments as: .zip

Change History (8)

garyc40 — 18 months ago

garyc40 — 18 months ago

Previous patch was a bad one.

garyc40 — 18 months ago

What is wrong with me? The previous 2 patches were very very bad.

comment:1   SergeyBiryukov — 18 months ago

The condition seems reversed. Shouldn't it be !empty()?

comment:2   SergeyBiryukov — 18 months ago

Didn't see settings-errors.3.diff​ before commenting :)

SergeyBiryukov — 17 months ago

Refreshed for current trunk

comment:3   SergeyBiryukov — 17 months ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.4

comment:4   duck_ — 16 months ago

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

In [19733]:

Prevent "Undefined index: settings-updated" notice in settings_errors(). Props garyc40. Fixes #19377.

Note: See TracTickets for help on using tickets.