[jdom-interest] setContent NOT setting parent correctly.

[Dave Bartmess]

Actually, I was hoping that setContent would be fixed <grin> It's a
major part of my application, and it doesn't make much sense to even
have it in the API if it doesn't detach the nodes itself. Otherwise, you
can just do a removeContent() and then addAll(Collection).

On Thu, 2003-10-16 at 13:40, Bradley S. Huffman wrote:
> So we need to put a note in the javadocs that all nodes in the supplied
> collection must be detached?
> 
> Brad
> 
> "Dave Bartmess" writes:
> 
> > I'm using JDOM b10 (latest from CVS). 
> > 
> > >From looking at the javadocs, Element.setContent(Collection) SHOULD set
> > the parent of each of the passed objects in the Collection to the
> > calling instance.
> > >From the javadocs:
> > In addition, all objects in the supplied List will have their parentage
> > set to this element, but the List itself will not be "live" and further
> > removals and additions will have no effect on this elements content.
> > 
> > But, from trying the code below, this is definitely NOT the case.
> > Instead, it throws an IllegalAddException because in the calls to (in
> > sequence) Element.setContent, ContentList.clearAndSet,
> > ContentList.addAll, ContentList.add(Object), and finally
> > ContentList.add(Element), nothing resets the passed List members'
> > parent, and thus throws the exception.
> > 
> > Therefore, setContent cannot work, unless you first go into the passed
> > Collection, and one by one, set the parent to null for each object.
> > 
> > Try the simple test class below, and you'll see what I mean:
> > 
> > import org.jdom.*;
> > 
> > public class Chance_1 extends Element {
> >    public Chance_1() {
> >       super("Chance");
> >       setDefaults();
> >    }
> >    public Chance_1(Chance_1 newchance) {
> >       super("Chance");
> >       setDefaults2();
> >       this.setContent(newchance.getContent());
> >    }
> >    
> >    public void setDefaults() {
> >       for (int i = 0; i < 10; i++) {
> >          Element child = new Element("Test" + i);
> >          child.setText("This is child " + i);
> >          this.addContent(child);
> >       }
> >    }
> >    
> >    public void setDefaults2() {
> >       for (int i = 100; i < 110; i++) {
> >          Element child = new Element("Test" + i);
> >          child.setText("This is child " + i);
> >          this.addContent(child);
> >       }
> >    }
> >    
> >    public static void main(String args[]) {
> >       Chance_1 test = new Chance_1();
> >       Chance_1 test2 = new Chance_1(test);
> >    }
> > }
> > 
> > -- 
> > David A. Bartmess
> > Software Configuration Manager / Sr. Software Developer
> > eDingo Enterprises
> > http://edingo.net
> > 
> > _______________________________________________
> > To control your jdom-interest membership:
> > http://lists.denveronline.net/mailman/options/jdom-interest/youraddr@yourhost
> > .com
> _______________________________________________
> To control your jdom-interest membership:
> http://lists.denveronline.net/mailman/options/jdom-interest/youraddr@yourhost.com
-- 
David A. Bartmess
Software Configuration Manager / Sr. Software Developer
eDingo Enterprises
http://edingo.net