> I think auto-checkin should only be done if the node has been checked out as > part of the modification operation. Thus: What if auto-checkout is also enabled, too?
> -----Ursprüngliche Nachricht----- > Von: Felix Meschberger [mailto:fmesc...@gmail.com] > Gesendet: Mittwoch, 8. Dezember 2010 17:10 > An: dev@sling.apache.org > Betreff: Re: SlingPOSTServlet : Auto Checkin Nodes > > Hi, > > Am Mittwoch, den 08.12.2010, 16:19 +0100 schrieb Clemens Wyss: > > "Auto Checkin Nodes" is activated (i.e. true). Hence I would expect that > when I modify (a property) of my mixin:versionable node, a new version is > created. Unfortunately this is not the case. > > > > Looking at AbstractSlingPostOperation#run only CREATE, CHECKOUT or > CHECKIN "trigger" a potential checkin, but not MODIFY. > > Shouldn't we?: > > ... > > case MODIFY : > > response.onModified(change.getSource()); > > if ( versionableConfiguration.isAutoCheckin() ) { > > nodesToCheckin.add(change.getSource()); > > } > > break; > > ... > > maybe additionally check whether the node is really checked out? > > > > WDYT? > > I think auto-checkin should only be done if the node has been checked out as > part of the modification operation. Thus: > > -- if the node was checked-out before the op, then it must be > checked-out after and no version must be created > -- if the node was checked-in before the op, and the node was > checked out, then it should probably be checked in > > And: we must be very carefull to not create backwards compatibility issues > around this automatic checkin/checkout. > > Regards > Felix