Skip to content

Fix for flushQueue db fetchAll#1

Open
jchen-privco wants to merge 2 commits intolibreworks:masterfrom
privco:jchen/hotfix-spool-flush
Open

Fix for flushQueue db fetchAll#1
jchen-privco wants to merge 2 commits intolibreworks:masterfrom
privco:jchen/hotfix-spool-flush

Conversation

@jchen-privco
Copy link
Copy Markdown

While testing this in development of the email utility feature I discovered the original fork's flushQueue did not fetch the serialized emails from the database as intended. Using PDO::FETCH_COLUMN resulted in a unserialize failure.

@doublecompile
Copy link
Copy Markdown
Member

Hi, @jchen-privco. Thanks for the pull request! If you could please ensure the unit tests pass, I can accept this. Currently, all of the Travis CI builds fail.

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

Successfully merging this pull request may close these issues.

2 participants