#2 delete_document() bug fix

closed
None
5
2002-09-01
2002-05-02
No

This patch is to fix a bug associated with the
delete_document() method on 'String' frontends.
More detail on this bug could be found here:
https://sourceforge.net/mailarchive/forum.php?forum_id=6211.

Fortunately, due to the way FullTextSearch was
designed, I only had to patch the String.pm module to
fix the bug. Column is a 'backend' type, therefore,
delete_document() would first go through String
frontend object. Inside String I simply convert
document name (string) to an internal document id
(numerical) and pass it on to the delete_document()
method of the super class (FullTextSearch in case of
string). In turn, the super class simply invokes
current backend object's delete_document() method (now
passing numerical document id instead of a 'string').

This patch should work with existing test cases. I
also tested it extensively on my own.

Discussion

  • Vladimir Bogdanov

    • assigned_to: nobody --> tjmather
     
  • Thomas J. Mather

    • status: open --> closed
     

Log in to post a comment.