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

draft copyediting file copied from a discussion can not be accessed #10584

Open
bozana opened this issue Nov 6, 2024 · 2 comments
Open

draft copyediting file copied from a discussion can not be accessed #10584

bozana opened this issue Nov 6, 2024 · 2 comments
Assignees
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Milestone

Comments

@bozana
Copy link
Collaborator

bozana commented Nov 6, 2024

S. https://forum.pkp.sfu.ca/t/assoc-type-preventing-file-download-by-copyeditor-ojs-3-3-0-13/93501

See also this related meta issue: #10438.

When using the function Upload/Select File for Draft Files in Copyediting tab the users can select files from Copyediting Discussions.

  1. Also the copyeditor that is not a participant of (and thus has no access to) the discussion, that the discussion file belongs to, can select the discussion file and add it to the Draft Files area.
  2. The discussion file is copied to the Draft Files area, but it keeps the assoc_type = 520 (which means discussion file).
    It results that the copyeditor that has no access to the discussion and discussion file see the new file in the Draft Files area, but he/she cannot download the file.

I believe here are two things to consider:

  • The users that have no access to discussion files should not be able to select them from the menu Upload/Select File.
  • I would think that a file that is selected and displayed in the Draft Files area should not get the assoc_type = 520 i.e. that is should be accessed by everybody with the access to that area.

Tested on OJS 3.3. Needs to be tested for 3.4 and main.

@bozana bozana added the Bug:1:Low A bug that does not have a severe consequence or affects a small number of users. label Nov 6, 2024
@bozana bozana added this to the 3.3.0-20 milestone Nov 6, 2024
@luizbgomide
Copy link
Contributor

Actually it wasn't the copyeditor that selected the file from the discussion (it was another editor), but we was supposed to access files in the Draft Section.

@bozana
Copy link
Collaborator Author

bozana commented Nov 6, 2024

Yes, that could also be, but also a person not involved in the discussion - that does not have access to the discussion and file - could select the file and copy to the draft section... which is the second problem...

@asmecher asmecher modified the milestones: 3.3.0-20, 3.3.0-21 Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Projects
None yet
Development

No branches or pull requests

4 participants