protege-client icon indicating copy to clipboard operation
protege-client copied to clipboard

Getting the latest changes shouldn't always anchor from R0

Open johardi opened this issue 9 years ago • 3 comments

Currently, this is hard-coded in this line:

ChangeHistory latestChanges = getLatestChanges(sdoc, DocumentRevision.START_REVISION);

The anchoring revision should come from the latest revision (i.e., head revision) of the local ontology copy, such that (as an illustration):

ChangeHistory latestChanges = getLatestChanges(sdoc, localCopy.getHeadRevision());

johardi avatar Sep 30 '16 17:09 johardi

this call is made after you load the snapshot, which only happens on an open. Pretty sure you want this to start from R0

bdionne avatar Sep 30 '16 18:09 bdionne

It will work correctly for the first snapshot because the snapshot captures the very beginning of the ontology. But it will no longer be valid for the subsequent snapshots which may come from a particular revision.

johardi avatar Sep 30 '16 20:09 johardi

I think it will actually, when we create new snapshots, the server history goes away, so this initial load will always use R0.

What might be different is when we start to persist local history. Then the server pull will be with respect to the local head

bdionne avatar Sep 30 '16 21:09 bdionne