Re: Suggestion for lookupObjEntity()

From: Andrus (andru..bjectstyle.org)
Date: Sat Nov 16 2002 - 13:37:37 EST

  • Next message: Holger Hoffstätte: "Re: DataContext.registerNewObject() vs. entityResolver"

    At 05:18 PM 11/16/2002 +0100, Holger Hoffstätte wrote:

    >I just accidentally passed an instance of DataObject into lookupEntity()
    >(instead of the Class) and of course got a null back. Any reason why we
    >shouldn't handle this case as well? A simple
    >
    > if (object instanceof DataObject) {
    > object = object.getClass();
    > }
    >
    >could help and make it more generic without disturbing the rest of the
    >method.

    I think this wouldn't hurt. I have no objections to this.

    >Another related suggestion: maybe we should only store either the class or
    >the class names in the cache? I think this will keep it smaller and
    >simpler.

    We may in the future after the deprecated methods that use object entity
    name are removed. I think we should keep it for now.

    Andrus

    This archive was generated by hypermail 2.0.0 : Sat Nov 16 2002 - 13:36:40 EST