WordPress.org
Get WordPress

Make WordPress Core

Opened 18 years ago

Closed 18 years ago

#5752 closed defect (bug) (duplicate)

PHP Notice Cleanup

Reported by: viper007bond's profile Viper007Bond Owned by:
Milestone: Priority: low
Severity: normal Version: 2.5
Component: General Keywords:
Focuses: Cc:

Description

A generic ticket for cleaning up the long, long list of PHP notices that pop up when WP_DEBUG is set to TRUE.

Will attach patches as I go.

Attachments (1)

notice-cleanup-1.patch​ (38.7 KB) - added by Viper007Bond 18 years ago.
Remove all noticed that I could find

Download all attachments as: .zip

Change History (6)

#1 @Viper007Bond
18 years ago

  • Keywords needs-testing 2nd-opinion added

@Viper007Bond
18 years ago

Remove all noticed that I could find

#2 @Viper007Bond
18 years ago

Screw multiple confusing patches. There's one big fat patch removing all PHP notices that I could find in both the main blog and the admin area.

It required a few actual minor code tweaks here and there, so please, someone read over my code to make sure I didn't mess up any logic or anything. :)

*passes out*

#3 @Viper007Bond
18 years ago

  • Priority changed from normal to low

Oh, I nearly forgot. I didn't touch /wp-admin/dashboard.php as it's a work in progress and things are VERY likely to change I assume code wise, so...

Also, this was done on my local machine which is running PHP 5.2.5, the latest as of this post.

#4 @ryan
18 years ago

How about having get_link_to_edit(), get_category_to_edit(), get_post_to_edit(), etc. fill in unset variables with empty strings so that we don't have to clutter the templates if empty checks?

#5 @anonymized_218323
18 years ago

  • Keywords has-patch needs-testing 2nd-opinion removed
  • Milestone 2.9 deleted
  • Resolution set to duplicate
  • Status changed from new to closed

Duplicate of #6669, many issues where fixed in that ticket. See #7509 for updated patch.

Note: See TracTickets for help on using tickets.