Isis
  1. Isis
  2. ISIS-9

Fixes to the Eclipse templates

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.1.2-incubating
    • Fix Version/s: Website and Docs
    • Component/s: Website and Docs
    • Labels:
      None

      Description

      This was an email chain from Jul 2010 timeframe, against NOF 4.0.0. Kevin had the proposed changes, I've assigned to him.

      ~~~~~~~~~~~
      Sounds like those are the right templates then (there hasn't been any updates to em). I'll check it out, but if you reckon there's a bug and you have a fix, then go ahead and commit the changes (somewhere under trunk/distribution, if i remember right).

      Thanks,
      Dan

      Sent from my iPhone

      On 28 Jul 2010, at 11:28, "Kevin Meyer" <kevin@kmz.co.za> wrote:

      > > Hi Dan,
      > >
      >> >> Yes, the parent should do the work, and the child delegate to it. It's
      >> >> called the mutual registration pattern.
      > >
      > > That is what I understood. I wanted to confirm with you as I was seeing
      > > some funny behaviour.
      > >
      >> >> In your email you talk about the 'template' in the singular, but actually
      >> >> there are several templates available (I'm assuming you are using the ones
      >> >> I describe in my book?).
      > >
      > > I'm using the templates I fetched from the NOF
      > > "/distribution/tarball/.../ide/eclipse/templates" directory.
      > > I should compare them to the ones in your book...
      > >
      >> >> I don't think that there are any bugs in them; rob says he's been using
      >> >> them ok (though I think he trimmed them down a bit).
      > >
      > > The issue that I have with the 'noc-1m' "1:m parent" association
      > > template is that following two lines are the wrong way around?:
      > > // associate arg
      > > childElementName.setParentPropertyNameInChild(this);
      > > getChildCollectionName().add(childElementName);
      > >
      > > Since, if the child method "setParentPropertyNameInChild(this)"
      > > delegates to the parent (e.g. contains
      > > "parentObject.addToChildCollectionName(this)") then you've got
      > > an infinite recursion taking place - the child item is not yet
      > > in the parent collection, when the parent calls the child... :-o
      > >
      > > AFAIK, There is no equivalent child template ('nocmod' assumes no
      > > parent?).
      > >
      > >
      > >
      > > Regards,
      > > Kevin

        Activity

        Dan Haywood created issue -
        Dan Haywood made changes -
        Field Original Value New Value
        Fix Version/s 0.1.0-incubating [ 12315421 ]
        Dan Haywood made changes -
        Component/s isis-parent (build plugins + site) [ 12314073 ]
        Dan Haywood made changes -
        Fix Version/s 0.2.0-incubating [ 12315988 ]
        Fix Version/s 0.1.2-incubating [ 12315421 ]
        Dan Haywood made changes -
        Affects Version/s 0.3.0-incubating [ 12315989 ]
        Dan Haywood made changes -
        Fix Version/s 0.3.0-incubating [ 12315989 ]
        Fix Version/s 0.2.0-incubating [ 12315988 ]
        Affects Version/s 0.1.2-incubating [ 12315421 ]
        Affects Version/s 0.3.0-incubating [ 12315989 ]
        Dan Haywood made changes -
        Fix Version/s 0.3.1-incubating [ 12315989 ]
        Dan Haywood made changes -
        Fix Version/s WEBSITE only [ 12323881 ]
        Dan Haywood made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Kevin Meyer [ kevin-m ] Dan Haywood [ danhaywood ]
        Resolution Fixed [ 1 ]
        Dan Haywood made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Dan Haywood
            Reporter:
            Dan Haywood
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development