Re: Callbacks tab in CM

From: Michael Gentry (mgentr..asslight.net)
Date: Fri Mar 20 2009 - 10:55:20 EDT

  • Next message: Kevin Menard: "Re: [GSoC 2009] Apache is officially participating in Google Summer of Code 2009"

    Could it not be ordered then?

    On Fri, Mar 20, 2009 at 10:50 AM, Andrus Adamchik
    <andru..bjectstyle.org> wrote:
    > The ordering within each callback type is significant. It will be obscured
    > if we display all types together. Otherwise I agree - it is confusing.
    >
    > Andrus
    >
    > On Mar 20, 2009, at 4:42 PM, Michael Gentry wrote:
    >
    >> I haven't really looked into the callbacks before, so I am curious and
    >> taking a peek.  In Cayenne Modeler there is a "Callbacks" tab when you
    >> select the ObjEntity.  (These comments will most likely apply to the
    >> "Listeners" tab for the ObjEntity and DataMap as well.)  There is a
    >> Callback Type popup and then you click the Add Method button (which
    >> looks dimmed out to me, like it is disabled).  Below them is a table
    >> with a single Method column.  (Wish I could do screenshots here.)  If
    >> you add a pre-persist callback (select it then click the Add Method
    >> button) and then change the Callback Type popup to post-persist, it no
    >> longer shows the pre-persist callback method.  This seems awkward to
    >> me.
    >>
    >> I'd suggest removing the Callback Type popup from where it currently
    >> is and making the table below have two columns (Method, Callback
    >> Type).  Clicking the Add Method button would add a new row to the
    >> table and the Callback Type column would contain a popup or pulldown
    >> that the user could select.  This would allow the user to see all of
    >> their callbacks at the same time instead of hiding them.  It could
    >> default to pre-persist (along with the default onPrePersist method
    >> name).  If they change the Callback Type and haven't edited the method
    >> name (it is still at the default), then change the method name to the
    >> default for the selected Callback Type.  If they have changed it,
    >> don't alter the method name.
    >>
    >> Thoughts?
    >>
    >> Thanks!
    >>
    >> mrg
    >>
    >
    >



    This archive was generated by hypermail 2.0.0 : Fri Mar 20 2009 - 10:55:51 EDT