Problem with optimistic locking wiki page "Example functionality" section.

[coblej]

The “Example functionality” section of this wiki page — https://github.com/samvera/valkyrie/wiki/Optimistic-Locking — doesn't demonstrate what it is intended to, since the initial resource is only instantiated, not persisted. Hence the two 'change_set_persister.save' lines in the body of the example don’t create a StaleObjectError . Rather they create (persist) two different instances of the resource.

I'll provide a proposed alternative code example in a comment.

[coblej]

Actually, I think that all that's needed is to add this line resource = Valkyrie.config.metadata_adapter.persister.save(resource: resource) after the existing resource = MyLockingResource.new(title: "test title 1") line, so that the first block looks like this ...

# Create a resource
resource = MyLockingResource.new(title: "test title 1")
resource = Valkyrie.config.metadata_adapter.persister.save(resource: resource)

[tpendragon]

I applied those changes. Thanks @coblej !