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

Files: 
CommentFileSizeAuthor
#14 nodequeue_apachesolr_fatal_error-1425326-14.patch1.01 KBm.stenta
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]
#8 php-fatal-error-1425326-8.patch1.09 KBkillua99
Test request sent.
[ View ]
#7 php-fatal-error-1425326-7.patch1.08 KBkillua99
Test request sent.
[ View ]
#5 nodequeue-7.x-2.0-beat1-apachesolr-1425326-6028750.patch865 bytesabhishek.kumar
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]
nodequeue-apachesolr-fix.patch546 bytesdavidwhthomas
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

Comments

DamienMcKenna’s picture

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

Ah glad to hear it.

Status:Fixed» Closed (fixed)

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

abhishek.kumar’s picture

Status:Needs review» Needs work

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

abhishek.kumar’s picture

Status:Closed (fixed)» Needs review
StatusFileSize
new865 bytes
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

I have done some improvements on the existing patch.

killua99’s picture

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

Status:Patch (to be ported)» Needs review
StatusFileSize
new1.08 KB
Test request sent.
[ View ]

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

killua99’s picture

StatusFileSize
new1.09 KB
Test request sent.
[ View ]

Ou auto complete IDE make a bad joke.

module_exits is not the correct function. function_extis is.

drupalninja99’s picture

You are missing a ')'

if (function_exists('apachesolr_mark_node') {

amateescu’s picture

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

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

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

StatusFileSize
new1.01 KB
PASSED: [[SimpleTest]]: [MySQL] 0 pass(es).
[ View ]

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

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

TechNikh’s picture

Issue summary:View changes

#5 worked for me too. Thank you.

dharizza’s picture

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

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

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

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