My only concern about this change is that this code first gets the existing "content" value, which may be a document or may be a binary. ModeShape never uses 'binary' here, but Schematic was designed to allow storing either under "content". Now, if we want to keep that behavior, then line 266 should be `entryDocument.get(FieldName.CONTENT)` (rather than assuming it's a Document instance).