In apachesolr.index.inc in function apachesolr_index_node_solr_document() there is duplicated code and comments for indexing extra data, plus we are missing a variable_get() call to find excluded nodeapi implementations.

CommentFileSizeAuthor
#4 broken_logic-1621142-4.patch12.91 KBkillua99
#1 1621142-1.patch2.76 KBpwolanin

Comments

pwolanin’s picture

Status: Active » Needs review
StatusFileSize
new2.76 KB

untested patch

nick_vh’s picture

Status: Needs review » Fixed

Looks good to me, committing to D7. Gave credit where credit was due :-)

nick_vh’s picture

Version: 7.x-1.x-dev » 6.x-3.x-dev
Status: Fixed » Patch (to be ported)
nick_vh’s picture

Woa, can't you split them in separate issues. Will be easier to review :-)

Thanks for all of your effort!!

killua99’s picture

Yes, I can, but I did a mistake, I did not split the code issue by issue.

So the lines is not 100% percent accurate with the last pull I did :/

pwolanin’s picture

I think the external File Field schema change was committed together by mistake - we should just sync the conf in another issue.

killua99’s picture

I did a git pull origin 6.x-3.x after each diff I did.

I need to learn split the code for each issue :/

nick_vh’s picture

Status: Needs review » Fixed

committed pwolanins patch to D6. killua, thanks for the effort anyhow. I think you'll do better next time ;-)

nick_vh’s picture

Status: Fixed » Closed (fixed)