Merge pull request nextcloud#10129 from nextcloud/backport/10067/stable-33.0

nilsding · web-flow · commit 62a76fd2d9bd · 2026-06-09T13:44:41.000+02:00
[stable-33.0] fix(filedetails): use correct path for fetching file tags
diff --git a/src/gui/filedetails/filedetails.cpp b/src/gui/filedetails/filedetails.cpp
@@ -177,7 +177,6 @@ FileTagModel *FileDetails::fileTagModel() const
 void FileDetails::updateFileTagModel()
 {
     const auto localPath = _fileRecord.path();
-    const auto relPath = localPath.mid(_folder->cleanPath().length() + 1);
     QString serverPath = _folder->remotePathTrailingSlash() + _fileRecord.path();
  
     if (const auto vfsMode = _folder->vfs().mode(); _fileRecord.isVirtualFile() && vfsMode == Vfs::WithSuffix) {
@@ -186,7 +185,7 @@ void FileDetails::updateFileTagModel()
         }
     }
 
-    _fileTagModel = std::make_unique<FileTagModel>(relPath, _folder->accountState()->account());
+    _fileTagModel = std::make_unique<FileTagModel>(serverPath, _folder->accountState()->account());
     Q_EMIT fileTagModelChanged();
 }
 

Original file line number	Diff line number	Diff line change
`@@ -177,7 +177,6 @@ FileTagModel *FileDetails::fileTagModel() const`
`177`	`177`	`void FileDetails::updateFileTagModel()`
`178`	`178`	`{`
`179`	`179`	`const auto localPath = _fileRecord.path();`
`180`		`- const auto relPath = localPath.mid(_folder->cleanPath().length() + 1);`
`181`	`180`	`QString serverPath = _folder->remotePathTrailingSlash() + _fileRecord.path();`
`182`	`181`
`183`	`182`	`if (const auto vfsMode = _folder->vfs().mode(); _fileRecord.isVirtualFile() && vfsMode == Vfs::WithSuffix) {`
`@@ -186,7 +185,7 @@ void FileDetails::updateFileTagModel()`
`186`	`185`	`}`
`187`	`186`	`}`
`188`	`187`
`189`		`- _fileTagModel = std::make_unique<FileTagModel>(relPath, _folder->accountState()->account());`
	`188`	`+ _fileTagModel = std::make_unique<FileTagModel>(serverPath, _folder->accountState()->account());`
`190`	`189`	`Q_EMIT fileTagModelChanged();`
`191`	`190`	`}`
`192`	`191`