Got this error which killed the page.

Apachesolr has changed this function from

apachesolr_mark_node($nid) to
apachesolr_mark_entity('node', $nid);

Patch attached.

cheers,

DT

CommentFileSizeAuthor
#14 nodequeue_apachesolr_fatal_error-1425326-14.patch1.01 KBm.stenta
#8 php-fatal-error-1425326-8.patch1.09 KBkillua99
#7 php-fatal-error-1425326-7.patch1.08 KBkillua99
#5 nodequeue-7.x-2.0-beat1-apachesolr-1425326-6028750.patch865 bytesabhishek.kumar
nodequeue-apachesolr-fix.patch546 bytesdavidwhthomas

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

This appears to be already fixed in the latest -dev version, if you check the codebase. Thanks for the patch though!

davidwhthomas’s picture

davidwhthomas commented

Ah glad to hear it.

Status: Fixed » Closed (fixed)

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

abhishek.kumar’s picture

abhishek.kumar commented
Status: Needs review » Needs work

Still getting error need some improvements in patch. While removing the nodes from nodequeue.

abhishek.kumar’s picture

abhishek.kumar commented
Status: Closed (fixed) » Needs review
StatusFileSize
new nodequeue-7.x-2.0-beat1-apachesolr-1425326-6028750.patch865 bytes

I have done some improvements on the existing patch.

killua99’s picture

killua99 commented
Version: 7.x-2.0-beta1 » 6.x-2.x-dev
Status: Needs work » Patch (to be ported)

Backporting to 6.x-2.x

killua99’s picture

killua99 commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new php-fatal-error-1425326-7.patch1.08 KB

Need review, and please a commit after this. Some one are upgrading the new apachesolr module and need support.

killua99’s picture

killua99 commented
StatusFileSize
new php-fatal-error-1425326-8.patch1.09 KB

Ou auto complete IDE make a bad joke.

module_exits is not the correct function. function_extis is.

drupalninja99’s picture

drupalninja99 commented

You are missing a ')'

if (function_exists('apachesolr_mark_node') {

amateescu’s picture

amateescu commented
Status: Needs review » Fixed

Committed a revised patch to 6.x-2.x. Sorry for the delay..

http://drupalcode.org/project/nodequeue.git/commit/b8270a8

Status: Fixed » Closed (fixed)

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

dago.aceves’s picture

dago.aceves commented
Status: Closed (fixed) » Reviewed & tested by the community

This patch (http://drupal.org/node/1425326#comment-6029228) should be accepted by the project on the 7.x branch.

apachesolr_mark_node D6 for older versions of apache solr.
apachesolr_mark_entity is for later versions apachesolr on D6 and D7.

Works for me.

kmonty’s picture

kmonty
He/Him
Location Berlin, DE + SF, CA
commented
Status: Reviewed & tested by the community » Closed (fixed)

It is in the 7.x dev branch, just not the stable release.

m.stenta’s picture

m.stenta
he/him
Primary language English
commented
StatusFileSize
new nodequeue_apachesolr_fatal_error-1425326-14.patch1.01 KB

Just made a quick patch file for the D6 branch for anyone that needs it (for example if you're patching via Drush Make). This is not meant to affect this issue... it's just a file for anyone who needs it quickly.

joshuautley’s picture

joshuautley commented

#5 for D7 with apache solr worked for me. Thank you!

technikh’s picture

technikh commented
Issue summary: View changes

#5 worked for me too. Thank you.

dharizza’s picture

dharizza commented

I have the module with version 7.x-2.0-beta1 and there is still the issue but #5 worked for me in Drupal 7 with Apache Solr.
Thanks.

henrikakselsen’s picture

henrikakselsen commented
Version: 6.x-2.x-dev » 7.x-2.0-beta1
Status: Closed (fixed) » Reviewed & tested by the community

Same issue for me, but #5 works and seems like the right solution.

Sneakyvv’s picture

Sneakyvv commented
Status: Reviewed & tested by the community » Closed (fixed)

This seems to have been committed for the 7.x branch as well.