Closed (fixed)
Project:
Freelinking
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 May 2009 at 20:36 UTC
Updated:
8 Dec 2009 at 06:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
Grayside commentedI believe Merlin meant nodeapi $op = 'update index', as I don't see a 'search' op in the nodeapi documentation.
Comment #2
Grayside commentedOkay, this patch seems to provide the needed functionality.
For every node being indexed for search, the node input format is checked. If that format includes the freelinking filter, it processes the node's body through the freelinking filter. The results of that processing are then extracted and added to the search index.
There is a minor problem with this approach- because only the freelinking filter is applied to the text, any other filters in the node's input format are ignored.
Comment #3
Grayside commentedForgot this. Sorry for the triple post.
Comment #4
douggreen commentedThis had the right idea, but wasn't matching the filter properly and needed to replace the path alias with the node/nid. See attached patch.
Comment #5
GregoryHeller commentedI've tested @douggreen's patch 452828 on my client's site and it appears to be working.
Comment #6
GregoryHeller commentedupon further testing i think the patch has only solved the problem when CamelCase freelinking is used but not when [[Double Bracket]] free linking is used.
Comment #7
douggreen commentedI think that the issue is not with the [[Wiki Link]] style perse, but rather with having spaces in the title, that get html encoded. The updated patch should fix this.
Comment #8
eafarris commentedThis patch is not for version 3.
Comment #9
eafarris commentedNote that version 3 works "out of the box," since it doesn't use the freelinking/ namespace, but generates search.module-friendly links directly to node/(nid).
Comment #10
Grayside commentedMy initial patch was for Freelinking 3, with some refinement it should be fine.
Comment #11
GregoryHeller commented@douggreen's patch in #7 works for me. I can now see backlinks info for [[Double Bracket]] style wiki pages with spaces in their names.
Comment #12
Grayside commentedComment #13
Grayside commentedPlease test the -dev version so the change can be released as part of Freelinking 1.9.