Skip to content

[stable30] fix(versions): Check the class of the file's storage instead of the mountpoint#4310

Merged
artonge merged 1 commit intostable30from
backport/4309/stable30
Jan 28, 2026
Merged

[stable30] fix(versions): Check the class of the file's storage instead of the mountpoint#4310
artonge merged 1 commit intostable30from
backport/4309/stable30

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot bot commented Jan 28, 2026

Backport of #4309

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added bug 3. to review Items that need to be reviewed php Items related to PHP updates and code issues feature: versions Items related to file versioning labels Jan 28, 2026
@backportbot backportbot bot added this to the Nextcloud 30 milestone Jan 28, 2026
…ountpoint

1. Create a groupfolder

2. Create a file inside that groupfolder

3. Share the file to Alice

4. As Alice, list the file version --> fails

If we check the mount point, we are not really checking the actual storage of the file.

Signed-off-by: Louis Chmn <louis@chmn.me>
@artonge artonge force-pushed the backport/4309/stable30 branch from 4057c30 to c6dba4b Compare January 28, 2026 15:03
@artonge artonge marked this pull request as ready for review January 28, 2026 15:04
@artonge
Copy link
Copy Markdown
Contributor

artonge commented Jan 28, 2026

The missing part was only introduced later.

@artonge artonge merged commit 8758c37 into stable30 Jan 28, 2026
49 of 51 checks passed
@artonge artonge deleted the backport/4309/stable30 branch January 28, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Items that need to be reviewed bug feature: versions Items related to file versioning php Items related to PHP updates and code issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants