Hi Dave,
The options hash is not used in search_each() method.
Hi Dave,
The options hash is not used in search_each() method.
On Thu, Aug 24, 2006 at 12:11:38PM +0200, Florent S. wrote:
Hi Dave,
The options hash is not used in search_each() method.
here’s a patch that fixes this and another minor bug in Index::Index:
http://pastie.caboo.se/10032
Jens
–
webit! Gesellschaft für neue Medien mbH www.webit.de
Dipl.-Wirtschaftsingenieur Jens Krämer [email protected]
Schnorrstraße 76 Tel +49 351 46766 0
D-01069 Dresden Fax +49 351 46766 66
Jens K. wrote:
On Thu, Aug 24, 2006 at 12:11:38PM +0200, Florent S. wrote:
Hi Dave,
The options hash is not used in search_each() method.
here’s a patch that fixes this and another minor bug in Index::Index:
Parked at Loopia
Thanks Jens.
I hope Dave will include it in the SVN.
PS: Could you explain the test case for the other fix ?
(ensure_writer_open)
On Thu, Aug 24, 2006 at 04:47:35PM +0200, Florent S. wrote:
Thanks Jens.
I hope Dave will include it in the SVN.PS: Could you explain the test case for the other fix ?
(ensure_writer_open)
It occured to me with acts_as_ferret, where we make use of the :key
option for Index::Index, that the writer was not open (@writer was nil)
in that if statement. adding the additional ensure_writer_open statement
fixed this.
I wasn’t able to reproduce that with just Ferret and irb, right now, but
if you remove that line from index.rb and run the acts_as_ferret unit
tests from current trunk you’ll see it.
Jens
–
webit! Gesellschaft für neue Medien mbH www.webit.de
Dipl.-Wirtschaftsingenieur Jens Krämer [email protected]
Schnorrstraße 76 Tel +49 351 46766 0
D-01069 Dresden Fax +49 351 46766 66
It occured to me with acts_as_ferret, where we make use of the :key
option for Index::Index, that the writer was not open (@writer was nil)
in that if statement. adding the additional ensure_writer_open statement
fixed this.I wasn’t able to reproduce that with just Ferret and irb, right now, but
if you remove that line from index.rb and run the acts_as_ferret unit
tests from current trunk you’ll see it.
Thanks a lot.
This forum is not affiliated to the Ruby language, Ruby on Rails framework, nor any Ruby applications discussed here.
Sponsor our Newsletter | Privacy Policy | Terms of Service | Remote Ruby Jobs