Ticket #5752 (closed defect (bug): duplicate)

Opened 1 year ago

Last modified 5 months ago

PHP Notice Cleanup

Reported by: Viper007Bond Assigned to: anonymous
Priority: low Milestone:
Component: General Version: 2.5
Severity: normal Keywords:
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

notice-cleanup-1.patch (38.7 kB) - added by Viper007Bond on 02/02/08 16:00:58.
Remove all noticed that I could find

Change History

02/02/08 15:38:44 changed by Viper007Bond

  • keywords changed from has-patch to has-patch needs-testing 2nd-opinion.

02/02/08 16:00:58 changed by Viper007Bond

  • attachment notice-cleanup-1.patch added.

Remove all noticed that I could find

02/02/08 16:02:20 changed by Viper007Bond

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*

02/02/08 16:07:52 changed by Viper007Bond

  • 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.

02/02/08 18:45:26 changed by ryan

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?

08/13/08 17:21:49 changed by santosj

  • keywords deleted.
  • status changed from new to closed.
  • resolution set to duplicate.
  • milestone deleted.

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