Ticket #2914 (closed enhancement: fixed)

Opened 2 years ago

Last modified 2 years ago

Replace die('s with wp_die('s

Reported by: filosofo Assigned to: filosofo
Priority: normal Milestone: 2.1
Component: Administration Version: 2.1
Severity: normal Keywords: bg|has-patch
Cc:

Description

Latest revision; excluded ajax and wp-blog-header, as well as a few empty die()'s.

Attachments

wp_dies.diff (41.3 kB) - added by filosofo on 07/05/06 21:21:52.

Change History

07/05/06 21:21:52 changed by filosofo

  • attachment wp_dies.diff added.

07/05/06 22:00:10 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

(In [3985]) Use wp_die(). Props filosofo. fixes #2914

07/05/06 22:02:06 changed by ryan

I had to make a few changes. Don't wp_die after a redirect. Don't wp_die when wp-config cannot be loaded. Don't wp_die when an importer is using die as a means of sending non error output. A good general rule is to only wp_die when there is a die with a translatable string in it.

I remove a few wp_dies from wp-login in the reset/retrieve password areas. We could add these back if we integrate the echos that come before the die into the die.