#689: BibAuthorID: improve indexing gateway
--------------------------+--------------------
  Reporter:  simko        |      Owner:  scarli
      Type:  defect       |     Status:  closed
  Priority:  blocker      |  Milestone:  v1.0
 Component:  BibAuthorID  |    Version:
Resolution:  fixed        |   Keywords:
--------------------------+--------------------

Comment (by cdickins):

 Replying to [comment:7 Tibor Simko <tibor.simko@…>]:
 > In [ea93c779623891a5d6cbfcaa1a95cc3ce045858f]:
 > {{{
 > #!CommitTicketReference repository=""
 revision="ea93c779623891a5d6cbfcaa1a95cc3ce045858f"
 > BibIndex: call BibAuthorID for modified person IDs
 >
 > * When indexing author indexes, in addition to looking at record
 modification
 >   dates, consult also BibAuthorID tables to see whether there have been
 some
 >   claims or other canonical author ID changes, in order to index
 properly
 >   canonical author IDs.  (closes #689)
 >
 > * Amends BibAuthorID's get_recids_affected_since() in order to return
 list of
 >   integers, not list of strings representing integers.
 > }}}

 Tibor, in this commit, in bibindex_engine.py lines 1064ff you created a
 separate "alist" of recIDs which you then add with self.add_recIDs().
 Since I need to add similar code for authority control, would it be o.k.
 if I refactor both your and my code to create a single "alist" of unique
 recIDs before passing the whole thing to self.add_recIDs() ?

-- 
Ticket URL: <http://invenio-software.org/ticket/689#comment:8>
Invenio <http://invenio-software.org>

Reply via email to