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

queue() method in Message.php seems incomplete #35

Open
wirepaq opened this issue Mar 17, 2017 · 3 comments
Open

queue() method in Message.php seems incomplete #35

wirepaq opened this issue Mar 17, 2017 · 3 comments

Comments

@wirepaq
Copy link

wirepaq commented Mar 17, 2017

Hi,

I just encountered a problem where queue was saving just the subject in mailqueue table. Upon inspection and comparing to old version it seems queue() function lacks the code that set from/to, attachements etc. Is this case of committing wrong code?

Thanks

@nterms
Copy link
Owner

nterms commented Mar 17, 2017

Hi @wirepaq,

Now the message is serialized as a whole and saved to the table in one field. Please try re-installing the extension.

@wirepaq
Copy link
Author

wirepaq commented Mar 20, 2017

I think I got the problem, the
https://github.com/nterms/yii2-mailqueue/blob/master/migrations/m170217_124201_drop_obsolete_columns_from_mail_queue_table

misses .php extensions. So obsolete columns were not deleted as expected.

@nterms
Copy link
Owner

nterms commented Mar 20, 2017

Renamed the file and made a release. Didn't test though.

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

2 participants