Add posibility to control which node bundles are being monitored. Replace UNIX_TIMESTAM() with drupal's REQUEST_TIME so that it works on postgreSQL

CommentFileSizeAuthor
#3 2125271-most_viewed-3.patch5.24 KBAnonymous (not verified)
20131101-most_viewed-add_bundles_option_and_cleanup.patch3.94 KBAnonymous (not verified)

Comments

samvel’s picture

samvel commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 20131101-most_viewed-add_bundles_option_and_cleanup.patch, failed testing.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new 2125271-most_viewed-3.patch5.24 KB

Updated patch

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
samvel’s picture

samvel commented
Status: Needs review » Fixed

Hi Schneider,

I checked your patch. It looks good for me.

Thank you for your contribution.

Comitted.

Status: Fixed » Closed (fixed)

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