Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug with Denied Permissions #17

Closed
jason-vetter-drasgow opened this issue Jul 18, 2023 · 2 comments
Closed

Bug with Denied Permissions #17

jason-vetter-drasgow opened this issue Jul 18, 2023 · 2 comments

Comments

@jason-vetter-drasgow
Copy link

Here are 2 examples regarding access control permissions on a directory:
Note: In these examples, AllowedPermissions is set to FileManagerPermissions.ReadOnly

#1: In the Preview pane, there is no Print button and there is no Download button.
AccessControl.DeniedPermissions = FileManagerPermissions.Download Or FileManagerPermissions.Copy Or FileManagerPermissions.Print

#2: In the Preview pane, there is a Print button and a Download button.
AccessControl.DeniedPermissions = FileManagerPermissions.Download Or FileManagerPermissions.Copy

There should not be a Print button in case #2.
It seems that when Print is available, download automatically becomes available.

@GleamTech
Copy link
Owner

That's not a bug, when you can print, it seems you can print to PDF in printer dialog, that's why download as PDF button is enabled. But download as original button will be disabled

@GleamTech
Copy link
Owner

FYI, with v9.1.0, we changed this behavior:

  • Improved: Disabling FileManagerPermissions.Download permission will also disable DocumentViewerPermissions.DownloadAsPdf.
    In the past we allowed DownloadAsPdf as the user could already print the document and save as PDF but it makes more sense
    to disable it to prevent confusion as it has "Download" word in it.
    So currently;
    Disabling FileManagerPermissions.Download -> disables DocumentViewerPermissions.Download and DocumentViewerPermissions.DownloadAsPdf
    Disabling FileManagerPermissions.Print -> disables DocumentViewerPermissions.Print and DocumentViewerPermissions.SelectText

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants