Skip to content

Commit 06de17a

Browse files
authored
Merge pull request #3021 from nextcloud/backport/2988/stable29
[stable29] fix(Viewer): Provide `id` instead of `fileid` to `File` constructor
2 parents aeb426c + 09cac99 commit 06de17a

File tree

5 files changed

+15
-5
lines changed

5 files changed

+15
-5
lines changed

css/main-CoOrSg1H.chunk.css

Lines changed: 10 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

css/viewer-main.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
/* extracted by css-entry-points-plugin */
2-
@import './main-D-tsloeb.chunk.css';
2+
@import './main-CoOrSg1H.chunk.css';
33
@import './previewUtils-DG0nQhh6.chunk.css';

js/viewer-main.mjs

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

js/viewer-main.mjs.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/views/Viewer.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1124,13 +1124,13 @@ export default defineComponent({
11241124
11251125
async onDelete() {
11261126
try {
1127-
const fileid = this.currentFile.fileid
1127+
const fileid = Number.parseInt(this.currentFile.fileid)
11281128
const url = this.currentFile.source ?? this.currentFile.davPath
11291129
11301130
// Fake node to emit the event until Viewer is migrated to the new Node API.
11311131
const node = new NcFile({
11321132
source: url,
1133-
fileid,
1133+
id: fileid,
11341134
mime: this.currentFile.mime,
11351135
owner: this.currentFile.ownerId,
11361136
root: url.includes('remote.php/dav') ? davGetRootPath() : undefined,

0 commit comments

Comments
 (0)