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

Opened 7 years ago

Last modified 7 years ago

Another ajaxcat tweak re: shutdown functions.

Reported by: mdawaffe Owned by: markjaquith
Priority: normal Milestone:
Component: Administration Version: 1.6
Severity: normal Keywords: ajaxcat shutdown register_shutdown_function bg|has-patch bg|dev-feedback
Cc:

Description

If a plugin registers a shutdown function that either intentionally sends output (why?) or is broken and so sends error messages, that information is also sent to the ajaxcat code.

The included patch makes the JS more robust via parseInt, and, on the server side, registers an exit() calling function to WordPress' shutdown action.

Attachments

dynamo5.diff Download (1.1 KB) - added by mdawaffe 7 years ago.
nip output in the bud

Change History

comment:1   mdawaffe — 7 years ago

  • Keywords bg|has-patch added

mdawaffe — 7 years ago

nip output in the bud

comment:2   markjaquith — 7 years ago

  • Keywords bg|dev-feedback added
  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned

This stuff is over my head, so I'm just bumping up to dev-feedback so Matt can take a look at it.

comment:3   mdawaffe — 7 years ago

Just a note that the patch on #1573 should be applied first, IIRC.

comment:4   mdawaffe — 7 years ago

  • Status changed from assigned to closed
  • Resolution set to duplicate

Fix moved to #1605

Note: See TracTickets for help on using tickets.