Skip to content
This repository has been archived by the owner on Apr 8, 2023. It is now read-only.

Non admins can not delete autosaved document #766

Open
Paula-Kli opened this issue Mar 13, 2021 · 3 comments
Open

Non admins can not delete autosaved document #766

Paula-Kli opened this issue Mar 13, 2021 · 3 comments

Comments

@Paula-Kli
Copy link
Contributor

When not saving a new document and creating a new one, one gets a hint to maybe reload the unsaved document. Being not a admin it is not possible to delete it afterwards

@frcroth
Copy link
Contributor

frcroth commented Mar 13, 2021

When you create a document and do not save it, you get the same behavior. It seems that only on save the author gets the required permissions.

@frcroth
Copy link
Contributor

frcroth commented Mar 13, 2021

It seems that at creation, the author is not saved into the database (information pages table). This makes it hard to check who the author is on deletion to allow the deletion to proceed.

@frcroth
Copy link
Contributor

frcroth commented Mar 13, 2021

It seems the user id is only saved in revisions, and on creation there is no revision yet, thus no user is saved. Naively this could be handled in two ways:

  • Add an original author to documents. This would require a migration
  • Allow deletion of documents without revisions to everyone (do not check permissions)
  • Save the document and then delete it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

2 participants