Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mailWipe() will always delete. #3

Open
Hobadee opened this issue May 23, 2014 · 0 comments
Open

mailWipe() will always delete. #3

Hobadee opened this issue May 23, 2014 · 0 comments

Comments

@Hobadee
Copy link

Hobadee commented May 23, 2014

I haven't tested this, so something may be slipping by me here, but...
In line 222:
if ( ($_ENV['wipe']) && (imap_delete($this->_c,$i)) ) return imap_expunge($this->_c);

The if() statement will run imap_delete() as part of it's evaluation, resulting in the item being marked for deletion regardless of $_ENV['wipe']. The statement should be:
if ($_ENV['wipe']) {
imap_delete($this->_c,$i);
return imap_expunge($this->_c);
}

If imap_expunge() is dependent on imap_delete() completing successfully, there should be a separate evaluation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant