Closed (fixed)
Project:
Subscriptions
Version:
5.x-1.2
Component:
User interface
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Feb 2007 at 19:20 UTC
Updated:
20 Feb 2007 at 23:18 UTC
After submitting a comment, a page looking like the following is returned.
Array ( [body] => Greetings, someone. A content type to which you have subscribed has been updated. To view the thread, navigate to http://example.org/node/ -- This is an automatic message from Example. To manage your subscriptions, browse to http://example.org/user/4/subscriptions [subject] => [Example] content type subscription update for someone : )
Other than seeing that strange page, everything seems fine (the comment is posted when you look at the node and subscriptions go out).
Maybe lines 463 & 464 (of version 1.62.2.14), part of the subscriptions_mailvars function, are to blame? Maybe it's just accidentally left-in debugging code?
print "<pre>"; print_r($mail);
The lines were introduced in subscriptions.module version 1.62.2.12.
I'm a noob, so not sure. I commented those two lines out of my installation, we'll see how it goes.
I'm excited about what this module can do. Thanks to everyone working on it!
Comments
Comment #1
i10neorg commentedOops. Seems I needed to comment out the following
exit()line as well. (I'm learning!)Comment #2
arthurf commentedUgh! I left some debug code in the version I submitted. Please upgrade to 5.1.3!
Comment #3
i10neorg commentedBut there's no supporting files (e.g. subscriptions.install) with 5.1.3 ?
Comment #4
huayen commentedI installed v5.x-1.3 and still have same problem.
Also, this module inteferes with some modules (such as taxonomy context module), having database error warning. If I uninstall subscription module, those modules work fine.
Comment #5
mindless commentedhm, the 1.62.2.15 commit seems to revert some earlier changes for 5.x and doesn't address the debug output..
Comment #6
arthurf commentedLesson learned- don't make releases when you haven't slept. 5.1.4 fixes this
Comment #7
i10neorg commentedLooks good, thanks.
And authurf, get some sleep! :^)
Comment #8
mindless commentedthe same cvs version is tagged for 1.4 that was used for 1.3... ?
Comment #9
(not verified) commented