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

Shared expired forms disappear even when "access results" permission is granted #1991

Closed
RenWal opened this issue Mar 6, 2024 · 3 comments · Fixed by #2013
Closed

Shared expired forms disappear even when "access results" permission is granted #1991

RenWal opened this issue Mar 6, 2024 · 3 comments · Fixed by #2013
Assignees
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working
Milestone

Comments

@RenWal
Copy link

RenWal commented Mar 6, 2024

Please use the 👍 reaction to show that you are affected by the same issue. Please don't comment if you have no relevant information to add!

Describe the bug
When a form expires, all shares on the form stop working. The form disappears from the navigation bar of the share receivers. This includes shares with "access results" permission. Hence, it is impossible for collaborators to access the results of the form once the form expires.

It makes sense to hide the form from users that have received the share just to fill the form, but less so for users that are expected to work with the results.

A workaround is for the form creator to export the results to CSV and then share that CSV with all of their collaborators.

To Reproduce

  1. Create any form
  2. Share the form with any user and tick the checkbox for the "access results" permission
  3. Confirm that the user you shared the form with can indeed access the form and its results page
  4. Set the expiration date to a date in the past
  5. The user you shared the form with can no longer see it, so they can't access the results page anymore

Expected behavior
Users with the "access results" share permission should still be able to access the results when the form expires.

Nextcloud (please complete the following information):

  • Nextcloud-Version: 27.1.6 Enterprise
  • Forms-Version: 3.4.6
@RenWal RenWal added 0. Needs triage Pending approval or rejection. This issue is pending approval. bug Something isn't working labels Mar 6, 2024
@Chartman123
Copy link
Collaborator

@susnux could you have a look at this? If you can reproduce it, perhaps the fix can be integrated into your current PR regarding the form state...

@Chartman123 Chartman123 added 1. to develop Accepted and waiting to be taken care of and removed 0. Needs triage Pending approval or rejection. This issue is pending approval. labels Mar 6, 2024
@Chartman123
Copy link
Collaborator

I had a look at the FormsService.php... we currently always return false in isSharedFormShown as soon as the form has expired. I think we need to take canSeeResults into account there.

@susnux
Copy link
Collaborator

susnux commented Mar 17, 2024

@susnux could you have a look at this? If you can reproduce it, perhaps the fix can be integrated into your current PR regarding the form state...

Yes but follow up to not grow that one too big

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants