Hi Emmanuel, More inline...
On 17 déc. 2010, at 14:38, Emmanuel Lecharny wrote: > Here is the description of a Delete operation, and how it's handled by the > AdministrativePoint handler. The parts between >>>--- and ---<<< are pseudo > code. > > Deleting an entry is somehow easier, as we don't update anything, except when > we delete a Subentry. > > +--+ > |AP| > +--+ > We can't delete an AP if it has any child. So deleting an AP is a > straightforward operation : we just remove the AP, and update the AP cache. > > >>>------------------------------------------- > check that the AP has no children > > if the AP is an AAP > then > for each role do > remove the AP(role) from the AP(role) cache > done > else > remove the AP(role) from the AP(role) cache > > remove the AP from the backend > -------------------------------------------<<< 100% agreed > +--------+ > |Subentry| > +--------+ > Removing a subentry is slightly more complex, as we will have to update the > associated AP SeqNumber, so that the entries part of the subtree can be > updated later. > > >>>------------------------------------------- > get a new seqNumber > update the parentAP(role) with the new seqNumber > > if the parentAP(role) is an IAP > then > update the parentAP(role) descendant with the new seqNumber > > remove the subentry from the backend > -------------------------------------------<<< I disagree. In the second part of the pseudo code we don't need to check if the parentAP(role) is an IAP. It also works if it's a SAP. My version of the pseudo code would be: >>>------------------------------------------- get a new seqNumber update the parentAP(role) with the new seqNumber for each parentAP IAP descendant update IAP with new seqNumber done remove the subentry from the backend -------------------------------------------<<< > > > +-----+ > |Entry| > +-----+ > > Deleting an entry is a straightforward operation. Nothing has to be done > > >>>------------------------------------------- > continue down the chain > -------------------------------------------<<< Agreed. Regards, Pierre-Arnaud > > -- > Regards, > Cordialement, > Emmanuel Lécharny > www.iktek.com >
