Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

[Bug] Closing a file preview calls a missing Files Sidebar method

Aperta Adatta ai principianti
  • Qualcosa non va?
#3,446 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 8 giorni

Nessuno ha ancora preso questa issue.

  • #3447 di @skjnldsv — chiusa senza merge

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
75/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
javascript
Ambito
frontend

Direzione di ricerca

Leggi src/views/Viewer.vue intorno al gestore di chiusura alle righe 967–979, poi controlla come beforeOpen() verifica il metodo della barra laterale. Conferma che il percorso di chiusura gestisce l’assenza del metodo setFullScreenMode e raggiunge comunque la pulizia della modalità a schermo intero; il controllo mirato Node VM descritto nell’issue copre le API mancanti e il metodo legacy invocabile. Il lavoro è completato quando la chiusura dell’anteprima non genera eccezioni in questi casi.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

3. to review bug
Environment
  • Nextcloud Server 35.0.2, official Apache Docker image
  • Bundled Viewer 8.0.0
  • Chromium with a 390 × 844 mobile viewport and touch emulation
  • Reproduced with custom CSS disabled and by accessing Nextcloud directly, without proxy HTML injection or modified JavaScript
Steps to reproduce
  1. Open Files and preview a PNG image.
  2. Close the preview using its close button.
  3. Inspect the browser console.
Actual result

Viewer.close() throws TypeError: OCA.Files.Sidebar.setFullScreenMode is not a function.

Image preview and download work. A PDF can be opened after the close transition finishes, so this report does not claim that subsequent previews are consistently blocked.

Expected result

Closing the preview completes without an exception when this optional Files Sidebar method is unavailable.

Suspected cause

The close handler checks that OCA.Files.Sidebar exists, but does not check whether setFullScreenMode is callable. beforeOpen() already checks for the method's presence. The same unsafe close call is still present in current master:
https://github.com/nextcloud/viewer/blob/master/src/views/Viewer.vue#L967-L979

A callable guard around the close-time invocation should preserve compatibility with both Sidebar APIs and allow the following fullscreen cleanup to run.

Minimal proposed fix and focused check

Against viewer commit d0d68b960285c820cc69e7eb05d68d9a98555230:

- if (OCA?.Files?.Sidebar) {
+ if (typeof OCA?.Files?.Sidebar?.setFullScreenMode === 'function') {
    OCA.Files.Sidebar.setFullScreenMode(false)
  }

A Node VM check using the actual extracted close() method reproduces the exception on the baseline (and skipped fullscreen exit), then passes with a missing method, missing Files API, and callable legacy method. The legacy method receives false with its original receiver. This is a focused method check, not a full Vue build or integration test.

Lingua principale
Nessun dato sulla lingua
Stelle
122
Fork
75
Merge medio
9g 5h
PR unite (30g)
22

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di nextcloud/viewer

Tutte le issue di nextcloud/viewer

Issue simili

Altre issue su Web Dev

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.