WordPress.org
Get WordPress

Make WordPress Core

Opened 19 years ago

Closed 18 years ago

#2132 closed defect (bug) (wontfix)

E_ALL Errors in classes.php

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

Description

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

Attachments (1)

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

Download all attachments as: .zip

Change History (6)

@niceguyeddie
19 years ago

#1 @davidhouse
19 years ago

  • Keywords bg|has-patch added

#2 @masquerade
19 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
19 years ago

  • Keywords bg|has-patch removed

#4 @rob1n
18 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
18 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.