The latest release of the apachesolr module has deprecated the apachesolr_mark_node function and replaced it with apachesolr_mark_entity.

CommentFileSizeAuthor
#7 node-mark-deprecated-1402634-7.patch863 bytessquidhaven
#1 node-mark-deprecated-1402634-1.patch863 bytesedb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

edb’s picture

edb CreditAttribution: edb commented
Status: Active » Needs review
FileSize
node-mark-deprecated-1402634-1.patch863 bytes

Patch attached

amateescu’s picture

amateescu CreditAttribution: amateescu commented
Status: Needs review » Fixed

Looks good. Committed to 7.x-2.x and 7.x-3.x, thanks!

http://drupalcode.org/project/nodequeue.git/commit/050094b
http://drupalcode.org/project/nodequeue.git/commit/2671d24

Status: Fixed » Closed (fixed)

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

dman’s picture

dman CreditAttribution: dman commented

Any ETA on when this will find its way into a versioned release?
Patch was there 6 months ago but no stable version that contains it.

eiriksm’s picture

eiriksm
he/him
Primary language Norwegian Bokmål
Location Norway
CreditAttribution: eiriksm commented

^^+1

heylookalive’s picture

heylookalive CreditAttribution: heylookalive commented
Version: 7.x-3.x-dev » 7.x-2.0-beta1
Priority: Normal » Critical
Status: Closed (fixed) » Active

+1 for a release, last stable 2.x release is coming up to a year and a half ago, this bug means you can't use this module if you've got Apache SOLR. Marking as critical, re-opening.

squidhaven’s picture

squidhaven CreditAttribution: squidhaven commented
FileSize
node-mark-deprecated-1402634-7.patch863 bytes

Updated patch for most recent beta1

Noel Hung’s picture

Noel Hung CreditAttribution: Noel Hung commented
Issue summary: View changes

It works! Thanks a lot!

othermachines’s picture

othermachines CreditAttribution: othermachines commented
Status: Active » Fixed

The issue was fixed in -dev (see #2). Reopening a critical issue won't make a stable release come any faster. :-)

fizk’s picture

fizk CreditAttribution: fizk commented
Status: Fixed » Closed (fixed)
heylookalive’s picture

heylookalive CreditAttribution: heylookalive commented

At this point figured it was worth a stab :)

We're coming up on the third year of this issue requiring a patch if using a stable release and the module hasn't seen a stable release in also over three years with 39k people using the current stable. What happens is up to the maintainers but those numbers aren't anything to sniff at!

othermachines’s picture

othermachines CreditAttribution: othermachines commented

@heylookalive - Agreed. If you want to talk about a stable release - as many do! - please open another issue or join one of the existing related conversations. Or offer to help. There is now a concerted effort to take care of all outstanding release-blocking issues and you are welcome to join in so we can get it done faster. :-) Cheers -

fizk’s picture

fizk CreditAttribution: fizk commented

@heylookalive As othermachines mentioned, we've started working on getting a release ready. We can discuss anything related to the release here: #1282434: Roll new 7.x release

heylookalive’s picture

heylookalive CreditAttribution: heylookalive commented

Ha looks like I did! Good to hear on the release.
https://www.drupal.org/node/1282434#comment-8404003