Skip to content

[stable33] Allow PDF Forms calculations#1409

Merged
danxuliu merged 3 commits intostable33from
backport/1325/stable33
Apr 5, 2026
Merged

[stable33] Allow PDF Forms calculations#1409
danxuliu merged 3 commits intostable33from
backport/1325/stable33

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot bot commented Apr 4, 2026

Backport of #1325

danxuliu and others added 2 commits April 5, 2026 02:01
@nextcloud/vue was already an implicit dependency, but version 9.x was
used, which is compatible only with Vue 3. As the PDF viewer uses Vue 2
the explicit dependency needs to be set to version 8.x instead, as
otherwise its components would not work (but this does not affect the
use of version 9.x by the dependencies themselves).

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
This PR adds a UI to configure the pdf viewer to allow calculations (secure in a sandbox).
Can be enabled/disabled through the admin settings globally, default is disabled.
Uses pdf.js´s sandbox feature, which should make this as secure as it can get.

Fixes #1265

Signed-off-by: Joly0 <13993216+Joly0@users.noreply.github.com>
@danxuliu danxuliu force-pushed the backport/1325/stable33 branch from 8ffb90d to 771279a Compare April 5, 2026 00:03
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliu danxuliu force-pushed the backport/1325/stable33 branch from dd98acf to 68270fd Compare April 5, 2026 00:13
@danxuliu danxuliu marked this pull request as ready for review April 5, 2026 00:14
@danxuliu danxuliu requested a review from szaimen as a code owner April 5, 2026 00:14
Copy link
Copy Markdown
Member

@danxuliu danxuliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works 👍

@danxuliu danxuliu merged commit 249fffa into stable33 Apr 5, 2026
43 checks passed
@danxuliu danxuliu deleted the backport/1325/stable33 branch April 5, 2026 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants