Broken MTThreadMail

I seem to have broken MTThreadMail when I upgraded MT.

Since I'm the only one using threaded comments and MTThreadMail, I guess that I don't really need to be in a hurry to fix it.

I thought I had fixed it, but it doesn't seem to be working (perhaps it is just broken for those entries I made before I made my latest change?).

I'll have to dig a little deeper.

Update: I've fixed this. After a few minutes of fiddling I noticed that I was missing a $comment->notify($q->param('notify')); Oops. The 'notify' flag was therefore not getting set in the database with the comment, so the code was refusing to send email (as it should). Looks like I missed this when I went through the diffs. I'll have to modify the way I'm doing my MT upgrades. (Hopefully the upgrade to MT 3.0 won't involve the exporting and re-importing of everyone's blogs).

| 4 Comments

4 Comments

You know dude, if I had a dollar for everytime I left that particular comment out....MAN. I am so ashamed of you...

Did I leave the sarcasm on?

I really hope the 3.0 upgrade isn't painful. I suspect it will be ok. I wish mt-plugins would move along a bit more quickly though... ;^)

If it is, there's always the choice of not upgrading.

... but I suppose it won't be too hard (it would be pretty easy to hack on it and make it easier to upgrade if necessary too).

Powered by Movable Type 4.34-en
Creative Commons License
This blog is licensed under a Creative Commons License.