recognize icon indicating copy to clipboard operation
recognize copied to clipboard

Uninstalling the app does not cleanup database

Open storm1er opened this issue 2 years ago • 4 comments

Which version of recognize are you using?

3.3.4

Enabled Modes

Object recognition, Face recognition, Video recognition, Music recognition

TensorFlow mode

WASM mode

Which Nextcloud version do you have installed?

25.0.2

Which Operating system do you have installed?

Debian 11

Which Docker container are you using to run Nextcloud? (if applicable)

v23.0.1

How much RAM does your server have?

2GB

What processor Architecture does your CPU have?

ARM64

Describe the Bug

Uninstalling and removing recognize app does not remove any tables from recognize, neither cleanup their content

Expected Behavior

oc_recognize_* tables should be removed on uninstall or removal of the app

To Reproduce

Install then uninstall the app, check table list

Debug log

No response

storm1er avatar Jan 09 '23 08:01 storm1er

oc_recognize_* tables should be removed on uninstall or removal of the app

There is no uninstall routine in Nextcloud, sadly. There is only a disable routine which I wouldn't want to trigger data removal.

marcelklehr avatar Jan 09 '23 12:01 marcelklehr

See https://github.com/nextcloud/server/issues/5539 for tracking of the upstream issue.

kc9jud avatar Jan 10 '23 20:01 kc9jud

@marcelklehr

There is only a disable routine which I wouldn't want to trigger data removal.

I understand why you wouldn't want it, but:

  • There's a need for it (like any other app, but more here due to the nature of the app)
  • It could have helped people in cases where atomic reset doesn't work (like duplicates and more)

Since the upstream issue has been created in 2017, waiting for it doesn't seem to be a good idea

storm1er avatar Jan 11 '23 08:01 storm1er

This is actually quite handy in my case, as that means I can uninstall and reinstall the app without losing data: https://github.com/nextcloud/recognize/issues/628#issuecomment-1384241651

MayeulC avatar Jan 16 '23 16:01 MayeulC