Item1237: Finish porting

pencil
Priority: Urgent
Current State: Closed
Released In: 1.1.0
Target Release: minor
Applies To: Extension
Component: NativeSearchContrib
Branches:
Reported By: CrawfordCurrie
Waiting For:
Last Change By: CrawfordCurrie
Still mentions (tm)wiki

Doesn't work with result sets


Ported, but not confident that Sven has actually finished work on the callers, so not releasing yet. There are a couple of big SMELLs - why I have to duplicate all the code from Forking, why the iterator has to be reset - but i'm not going to touch those until I've heard from Sven.

-- CrawfordCurrie - 17 Apr 2010

work has finished for 1.1 - more will happen in 2.0 - sorry, i meant to port nativesearch, and will be doing mongo and kino in the next weeks :/

the iterator has to be reset because there is no guarantee that it is at the beginning of the container, and I'm not willing to assume in the code that the containers will always be created where they are atm. This is one of those major examples why iterators should not be containers - but I am working with what's there.

wrt duplicating code from forking, y - i didn't go further because we have hit the 1.1 freeze date - things are better, but not complete.

result sets are mostly a 2.0 feature - always have been - though I had to start on them to get paging on multi-webs.

-- SvenDowideit - 18 Apr 2010

OK, no worries. I'm not going to do any more on this right now myself.

-- CrawfordCurrie - 18 Apr 2010
Topic revision: r11 - 18 Apr 2010, CrawfordCurrie
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy