On Thu, Aug 24, 2006 at 09:47:04PM +0200, Raul Murciano wrote:
> Well, It seems to work simply replacing this line (203) on 
> /ferret-0.10.0/lib/ferret/index.rb:
> 
>     @writer.commit # original
>     @writer.commit if @writer # patched

I might be wrong but this doesn't look ok to me, because it prevents 
the change the index class intends to make to the index from being 
written.

I have a patch (http://pastie.caboo.se/10032) for ferret's index.rb file
that fixes this issue by adding ensure_writer_open() before the line in
question, which initializes the @writer variable if it's not already 
present.

> I'm not sure if the problem comes from acts_as_ferret or is a problem on 
> ferret itself.
> 
> It's strange that I am the only affected (saving ActiveRecord items is a 
> so-common operation that acts_as_ferret team have surely tested it 
> before releasing the new version).

Well, Ferret 0.10 is still new (and acts_as_ferret is only compatible to
it for a day or so) - so you're probably the first on the list who tried 
it out :-)

I discovered this bug when porting aaf to Ferret 0.10, and made the
patch. I should have communicated that when announcing the 0.10
compatibility, but I forgot...
I think Dave will release a fixed gem soon, so the patch won't be
necessary for a long time.


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
_______________________________________________
Ferret-talk mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/ferret-talk

Reply via email to