Missing argument 3 for fivestar_nodeapi() in /var/www/html/boek/sites/all/modules/contrib/fivestar/fivestar.module on line 914.
Missing argument 4 for fivestar_nodeapi() in /var/www/html/boek/sites/all/modules/contrib/fivestar/fivestar.module on line 913.

Attached patch fixes the noticeb by adding $page & $teaser variable

I wasn't sure if I should add this to the fivestar queue or not, if it belongs there, you can move it of course.

CommentFileSizeAuthor
#2 apachesolr_index.patch780 bytesrobertdouglass
apachesolr_index_nodeapi_notice.patch676 bytesswentel

Comments

swentel’s picture

swentel commented

Small note, these errors also pop up when path (core module) and pathauto module are enabled simply because their implementation of hook_nodeapi isn't like it should be, so I guess this patch really should go in - at least, we will in our project at this moment ;)

robertdouglass’s picture

robertdouglass commented
Version: 6.x-2.x-dev » 6.x-1.x-dev
StatusFileSize
new apachesolr_index.patch780 bytes

Committing the attached version. Your patch is a couple of commits behind. Thanks!

robertdouglass’s picture

robertdouglass commented
Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Fixed

Committed to DRUPAL-6--1 and DRUPAL-5--2 as well.

Status: Fixed » Closed (fixed)

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