Fix setFullScreenMode on talk viewer opening

[skjnldsv]

Because Talk is doing some shenanigans to fake the Files Sidebar

Uncaught (in promise) TypeError: OCA.Files.Sidebar.setFullScreenMode is not a function
    at viewer-main.js:2
    at u (talk-main.js:1)
    at Generator._invoke (talk-main.js:1)
    at Generator.next (talk-main.js:1)
    at Rw (viewer-main.js:2)
    at a (viewer-main.js:2)
    at viewer-main.js:2
    at new Promise (<anonymous>)
    at viewer-main.js:2
    at a.openFile (viewer-main.js:2)

[skjnldsv]

/backport to stable24

[skjnldsv]

/rebase

[skjnldsv]

/compile amend /

[danxuliu]

Do you have reproduction steps? I have never seen that error (and I also tried to reproduce it without success), and it should not happen, as the fake sidebar has a setFullScreenMode function.

If the issue is caused by Talk I think it would be better to fix it in Talk instead (if it is possible, which I do not know :-) ).

[skjnldsv]

Maybe I was too much behind lol
In any case, this is good sanity check ;)

[danxuliu]

Maybe I was too much behind lol

More than a year, it is surprising that Talk even loaded :-O

In any case, this is good sanity check ;)

Then typeof OCA?.Files?.Sidebar?.setFullScreenMode === 'function'? Or that is too much? :-P

Anyway, fine by me to keep the old code, to check if setFullScreenMode exists or to check if setFullScreenMode is a function 👍

[backportbot-nextcloud]

The backport to stable24 failed. Please do this backport manually.