VersioningConsumer and in_archive flag

[kosarko]

https://github.com/dataquest-dev/DSpace/blob/3a5766a5d2c28ad312d6658b9eb938f740677613/dspace-api/src/main/java/org/dspace/versioning/VersioningConsumer.java#L134 switches the in_archive flag on the old version to false

this feels like a soft delete (well soft withdraw)

it's not being fully withdrawn, just having the item archive flag set to false. It means it won't show up in OAI feeds (by the default filter), or get processed for browse and authority indexing, exported by export methods, included in content reports, etc. If the new version is deleted the older one will then be set back to archived=true

Which is not how we've imagined versioning so far...

[kuchtiak-ufal]

Pull request: #1258

[kuchtiak-ufal]

Hi Ondrej (@kosarko)

Should we create some clarin-dspace.cfg property that will override the default behaviour on this, like:
versioning.unarchive.previous.version = false

then check this property in VersioningConsumer ?

[kuchtiak-ufal]

I've added configuration property:
versioning.unarchive.previous.version = false
to clarin-dspace.cfg

that will override the default behaviour, where previous version is unarchived.