hi ,
i ported this module to drupal 7.

CommentFileSizeAuthor
#3 comment_count_notify_7.patch2.72 KBnit3ch
comment_count_notify_7.x.patch2.5 KBnit3ch

Comments

fotuzlab’s picture

fotuzlab commented

Thanx nit3ch for the patch.

I tested the module, but sadly it doesn't work for me.
A couple of observations:

  • The sql query needs to be modified. In D6, comments were stored in {comments} table while in D7 they are stored in {comment} table.
  • db_fetch_object is depreciated in D7, we need to replace it by ->fetchObject()

I'll follow with a patch asa I get time. In case you want to redo the above patch ... I'd be glad :)

nit3ch’s picture

nit3ch commented
Status: Needs review » Needs work

Thanks for the review , i will soon submit a new patch rectifying these issues.

nit3ch’s picture

nit3ch commented
Status: Needs work » Needs review
StatusFileSize
new comment_count_notify_7.patch2.72 KB

hi ,
here is the new patch with the fixes .

fotuzlab’s picture

fotuzlab commented

Thanx nit3ch. I'll review it in a while.

fotuzlab’s picture

fotuzlab commented
Status: Needs review » Closed (fixed)

I tried the patch. There were some minor bugs and couple of notices. Corrected and ported.
Thanx for your contribution.