Opened 16 months ago

Closed 15 months ago

Last modified 13 months ago

#20051 closed defect (bug) (fixed)

Kill debug_fwrite(), log_app(), logIO()

Reported by: nacin Owned by:
Priority: normal Milestone: 3.4
Component: General Version:
Severity: normal Keywords: has-patch
Cc:

Description

Three super-old logging functions for enclosures, APP, and XML-RPC, respectively. Let's nuke them.

Attachments (1)

20051.diff​ (22.5 KB) - added by nacin 16 months ago.

Download all attachments as: .zip

Change History (9)

nacin — 16 months ago

comment:1   nacin — 16 months ago

  • Keywords has-patch added

comment:2   kurtpayne — 16 months ago

+1. Happy to see these go. What is best practice moving forward? error_log() ?

comment:3   duck_ — 15 months ago

Agreed. Should debug_fwrite() check the $debug global like log_IO and log_app do with $app_logging and $xmlrpc_logging? Maybe even switch to WP_DEBUG instead.

comment:4 follow-up: ↓ 5   kawauso — 15 months ago

s/erorr_log/error_log on line 825 of the patch / line 76 of patched xmlrpc.php

comment:5 in reply to: ↑ 4   nacin — 15 months ago

Replying to kawauso:

s/erorr_log/error_log on line 825 of the patch / line 76 of patched xmlrpc.php

Thanks. Caught that locally, haven't posted a patch yet.

comment:6   nacin — 15 months ago

In [19935]:

Deprecate ancient "debugging" tools.

  • logIO() and the global $xmlrpc_logging in XML-RPC.
  • log_app() and the global $app_logging in APP.
  • debug_fwrite(), debug_fopen(), debug_fclose(), and $debug.

see #20051.

comment:7   nacin — 15 months ago

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

comment:8   ocean90 — 13 months ago

Typo in r19935, see #20670.

Note: See TracTickets for help on using tickets.