Fix row index

[felipecruz91]

@benja-M-1 reported the following issue:

I import a volume from a fake repo: aaa/aaa:aaa
I delete the created volume
I clone a volume
-> the cloned volume name changes, I have two volumes with the same name that include the -cloned suffix

Screen.Recording.2022-09-23.at.17.23.28.mov

The row index was not being calculated correctly, causing two rows to share the same "unique" id:

[github-actions]

Docker image tag(s) pushed:

docker/volumes-backup-extension:pr-86

Labels added to images:

org.opencontainers.image.title=volumes-backup-extension
org.opencontainers.image.description=Back up, clone, restore, and share Docker volumes effortlessly.
org.opencontainers.image.url=https://github.com/docker/volumes-backup-extension
org.opencontainers.image.source=https://github.com/docker/volumes-backup-extension
org.opencontainers.image.version=pr-86
org.opencontainers.image.created=2022-09-23T15:50:08.205Z
org.opencontainers.image.revision=648ce0c84f63b9e483c9b12da39d76cbee2b859d
org.opencontainers.image.licenses=Apache-2.0

[benja-M-1]

isn't it similar to

Suggested change

	const highestId = rows.slice(-1)[0].id;
	const highestId = rows.pop()?.id;

I have added a ? because if rows is empty I think pop returns undefined

[felipecruz91]

Wouldn't pop() eliminate the row from the array? I just want to get the highest id from the rows and don't eliminate any items from the array.

[benja-M-1]

Oh yes, you are right!
Initially I was looking for an alternative to the [0] to avoid any potential error. But accessing an index that does not exist returns undefined. so I guess what could be missing here is only the ?