> From: Stefan Bodewig [mailto:[EMAIL PROTECTED] > > I've just looked through this patch, Glenn has moved the part where > the subproject's properties get set from init to execute. > > Can anybody see a reason why we shouldn't move the whole code from > init over to execute? >
Stefan, The only issue would be the createProperty method. Obviously p1 must be instantiated. I don't know if there will be any effect of setting the user properties first. (I have to go and look at that code a little to know the difference). I'd also keep it as a separate method to be called from execute(). Conor
