On Mon, Aug 16, 2010 at 11:10 AM, Jukka Zitting <[email protected]> wrote: > Hi, > > On Mon, Aug 16, 2010 at 10:31 AM, Jukka Zitting <[email protected]> > wrote: >> On Mon, Aug 16, 2010 at 10:12 AM, Stefan Guggisberg >> <[email protected]> wrote: >>> sorry, but i don't agree with these changes. >>> ItemState classes IMO shouldn't have any >>> knowledge of node type semantics. >> >> OK, I'll move them away from there. > > I reverted my earlier changes and committed an alternative solution in > revision 985833. The relevant code now stays in > NodeTypeInstanceHandler (with the extra helper method I added for > setting default property values), but I removed the reference from > SessionImpl to NodeTypeInstanceHandler. We may want to reintroduce > that reference later if for example we want to make the > NodeTypeInstanceHandler pluggable, but for now I'd like to avoid > keeping extra stuff in SessionImpl.
agreed, thanks! cheers stefan > > BR, > > Jukka Zitting >
