After updating to version 5.x-1.3, receipt these messages in log, whenever somebody enters a new commentary

Missing argument 3 for subscriptions_comment_page() in line 633 of the /var/www/html/modules/subscriptions/subscriptions.module file.

some idea that it can be

Thanks

Comments

manal’s picture

manal commented
Version: 5.x-1.3 » 5.x-1.4

Confirmed

this function is called with a missing argument at line 389, from within the function subscriptions_mailvars:
$page = subscriptions_comment_page($ssid, $nid);

where it should be called with 3 arguments:
function subscriptions_comment_page($comment, $cid, $nid) {
...
}

promatow’s picture

promatow commented

Same problem, both on cron and on-action event.

arthurf’s picture

arthurf commented
Status: Active » Fixed

this is now fixed in cvs

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
droopy75’s picture

droopy75 commented

I got the same error, in the version of subscriptions-5.x-1.4.tar.gz I downloaded today (Feb 25th 2007). I'm glad it's "fixed in CVS", but when is this likely to make it into the full release?

Alternatively, just let me know the precise edit and I'll edit my own instance of line 389...

mindless’s picture

mindless commented

hopefully within a week of a bugfix release.

mindless’s picture

mindless commented

function subscriptions_comment_page($comment, $cid, $nid = null) {

find the line like that.. adding the " = null" after $nid is the fix.