I get this error msg after posting a comment:

": Object of class stdClass could not be converted to string in /var/www/vhosts/thewatt.com/httpdocs/sites/all/modules/comment_notify/comment_notify.module on line 272."

The comment still gets posted. I guess I shouldn't be using dev.

CommentFileSizeAuthor
#5 comment_notify-object_to_string_error-229574-5.patch981 bytesgreggles

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Postponed (maintainer needs more info)

I believe this was fixed in 5.x-1.4.

Which version is this? Please either use a 5.x-1.4 or include the $id: line from the top of the comment_notify.module when reporting the issue.

benkenney’s picture

benkenney commented

It's for Drupal 5.6, comment_notify v 5.x-1.x-dev (March 2008). I'm now using 5.x-1.4 and I don't get this error anymore.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

Great to hear that it's fixed now.

romand’s picture

romand commented
Version: 5.x-1.x-dev » 5.x-1.5
Status: Closed (fixed) » Active

Active on comment_notify-5.x-1.5, Drupal 5.7:
___
recoverable fatal error: Object of class stdClass could not be converted to string in /home/uadromco/public_html/sites/all/modules/comment_notify/comment_notify.module on line 279.
___

Update: comment_notify-5.x-1.4 work fine.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs review
StatusFileSize
new comment_notify-object_to_string_error-229574-5.patch981 bytes

This was a slightly different bug (I think) but quite similar.

Can you test the attached patch? I think it was a typo in the fix I made for #219677: Missing use of db_prefix

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Just for the record, it was an accidental commit during fixing that other issue. Whoops.

romand’s picture

romand commented
Status: Needs review » Reviewed & tested by the community

patch #5 from @greggles fix problem.
Thank you!

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Reviewed & tested by the community » Fixed

Great - fixed and I'll roll a new release.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.