#5420 closed enhancement (fixed)
Better exit for wp-mail.php
Reported by: | lexhair | Owned by: | westi |
---|---|---|---|
Milestone: | 2.5 | Priority: | normal |
Severity: | minor | Version: | 2.3.1 |
Component: | General | Keywords: | has-patch tested |
Focuses: | Cc: |
Description
Executing wp-mail.php when the POP3 mailbox is empty results in wp_die() and a 500 error. When you run wp-mail.php from the cron tab, it logs the 500 error.
I don't believe it should return an error in this case. The script runs as it should and just doesn't find any data to manipulate like a zero row query. Perhaps a more graceful exit could be considered. Thank you.
Attachments (2)
Change History (9)
#4
@
17 years ago
A user/password failure would look just like there were no messages. New patch checks user/password status and fails with 500 error if either failed. If login is successful and there are no messages, just the 'no messages' message is displayed (no 500 error).
#5
@
17 years ago
Tested with WP 2.3.1, FreeBSD 6.2-RELEASE-p1, MySQL 4.1.20, with invalid username, invalid password, valid user/pass but no messages, valid user/pass with messages. Works correctly.
Don't return 500 error if no email