Make WordPress Core

Opened 12 years ago

Closed 11 years ago

#2132 closed defect (bug) (wontfix)

E_ALL Errors in classes.php

Reported by: niceguyeddie Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.0
Component: Optimization Keywords: E_ALL
Focuses: Cc:


This should fix all E_ALL errors in wp-includes/classes.php

Attachments (1)

patch.txt (9.4 KB) - added by niceguyeddie 12 years ago.

Download all attachments as: .zip

Change History (6)

12 years ago

#1 @davidhouse
12 years ago

  • Keywords bg|has-patch added

#2 @masquerade
12 years ago

This patch will surely break things, even looking at the first few lines beginning with a +. You're now assigning the return value of isset? Not a good idea. Fix your patch.

#3 @masquerade
12 years ago

  • Keywords bg|has-patch removed

#4 @rob1n
11 years ago

Yeah, the patch is pretty broken. Do we really care about running silent in E_ALL? It's not recommended to run a script in E_ALL in a production environment...

#5 @foolswisdom
11 years ago

  • Resolution set to wontfix
  • Status changed from new to closed

Closing WONTFIX for now. Possibly, consider more conservative, more tested new tickets with patches.

Note: See TracTickets for help on using tickets.