Replaced problem anonymous function.

[philcali]

It was recently brought to our attention that a particular anonymous function was causing troubles in quickmail::list_entries. This code aims to fix that problem by replacing the problem code with tried and true PHP.

@adamzap, please review.

[rrusso]

Good catch.

On Oct 24, 2011, at 7:45 AM, "Philip Cali" reply@reply.github.com wrote:

It was recently brought to our attention that a particular anonymous function was causing troubles in quickmail::list_entries. This code aims to fix that problem by replacing the problem code with tried and true PHP.

@adamzap, please review.

You can merge this Pull Request by running:

git pull https://github.com/lsuits/quickmail remove_anonymous_func

Or you can view, comment on it, or merge it online at:

#8

-- Commit Summary --

• Removed anonymous function, other minor refactorings.

-- File Changes --

M email.php (7)
M lib.php (19)

-- Patch Links --

https://github.com/lsuits/quickmail/pull/8.patch
https://github.com/lsuits/quickmail/pull/8.diff

Reply to this email directly or view it on GitHub:
#8

[adamzap]

👍

Are the changes in email.php related?

[philcali]

Minor refactoring. It was showing the notifyproblem coloring for draft saves.

[philcali]

@rrusso, can you test this change when you get a chance. It should work the same, just more compatible for all versions of PHP.

[adamzap]

@rrusso, have you tested this?

[rrusso]

Yes. Please merge and close.