Make WordPress Core

Opened 10 years ago

Closed 9 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 10 years ago.

Download all attachments as: .zip

Change History (6)

@niceguyeddie10 years ago

comment:1 @davidhouse10 years ago

  • Keywords bg|has-patch added

comment:2 @masquerade10 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.

comment:3 @masquerade10 years ago

  • Keywords bg|has-patch removed

comment:4 @rob1n9 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...

comment:5 @foolswisdom9 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.