[kepler-dev] [Bug 4319] - LibraryManager

[bugzilla-daemon at ecoinformatics.org]

http://bugzilla.ecoinformatics.org/show_bug.cgi?id=4319





------- Comment #2 from aschultz at nceas.ucsb.edu  2009-08-18 12:22 -------
The bug here was just poor API design (or more likely an unfinished API).  The
library was dependent solely on the library index for modification which meant
that to change the library one needed to have intimate knowledge of how the
index worked and had to include index specific code in their functions to get
the library to do anything.  A refactoring into multiple classes and the
inclusion of methods in the LibraryManager for modification of the library that
instead manages the index after the fact (instead of requiring the developer to
do it before hand) is the main goal of this enhancement.  To see this compare
the two versions of the addKAR method in LibraryManager.  One takes only a file
and updates the index as it goes, whereas the other requires a pre-built
hierarchical LibraryIndexItem to be passed to it.  Separation of the code also
gives the opportunity for substituting other mechanisms for managing the index
(such as using sql or java serialization instead of xml).