[Biojava-l] Bug fix for biojava class RichObjectFactory.java in current maven based project

[Andreas Prlic]

Hi Deepak,

thanks for spotting this. This factory method should clearly be moved to the
biosql module and not be part of the core.  Anybody who has a deeper
knowledge of the biosql code: Where is the best place in the biosql module
to move this to?

A work around the compile problem would be to use reflection to mask the
calls to the methods in the other module, but it feels like a hack...

Andreas

On Mon, Mar 8, 2010 at 1:11 PM, Deepak Sheoran <sheoran143 at gmail.com> wrote:

> Hi
> I was making a local version of current maven project on my machine so that
> i can fix some reference related bugs in biojava. But when I build the local
> version and tried to use it. I got an error on method
> RichObjectFactory.connectToBioSql(Object session) of current version of
> bio-java live. when I had a look on it I saw a comment on it
>
>    "// commenting out for the moment, since it prevents core from
> compiling.
>    // TODO: move to BioSql module"
>
> then I uncommitted the code and add these import statements to
> RichObjectFactory.java and the problem is fixed :
>
> import org.biojavax.bio.db.biosql.BioSQLCrossReferenceResolver;
> import org.biojavax.bio.db.biosql.BioSQLRichObjectBuilder;
> import org.biojavax.bio.db.biosql.BioSQLRichSequenceHandler;
>
> After this I tried compiling bioSql module it went successfully and also
> when I compiled Core module it went successfully too.I don't if this is the
> only reason then please uncomment these line in main svn version since i
> don't how to do it.
>
> Thanks
> Deepak Sheoran
>
>
> _______________________________________________
> Biojava-l mailing list  -  Biojava-l at lists.open-bio.org
> http://lists.open-bio.org/mailman/listinfo/biojava-l
>