New Files internal link GET param to avoid opening the file

[julien-nc]

This adds a openfile GET param to internal links (/f/FILE_ID) to avoid setting the openfile param in the redirect response.
This can be useful when the intention is to show the file's activity for example.
It is already possible to get there with links like https://my.nc.org/index.php/apps/files/?dir=/&scrollto=name.txt but it requires to know the file path.

[PVince81]

if the default behavior is to open the file, how about calling it "openfile=0" to disable it, and the default would be "openfile=1" ? or is that less intuitive ?

[julien-nc]

@PVince81 Yes, makes sense. Done and pushed.

[PVince81]

👍

[julien-nc]

Thank you both for the review. What do you think about backporting that?

[Pytal]

Thank you both for the review. What do you think about backporting that?

As long as this doesn't change existing functionality up to you :)

[skjnldsv]

Thank you both for the review. What do you think about backporting that?

As long as this doesn't change existing functionality up to you :)

Agreed!

[julien-nc]

/backport to stable23

[julien-nc]

/backport to stable22

[julien-nc]

/backport to stable21

[julien-nc]

CI failure is not related.

[lindenthal]

Thanks @eneiluj and @skjnldsv @Pytal and @PVince81
We really appreciate your effort. It is fun to develop open source software with the Nextcloud team!