Project

General

Profile

Actions

Bug #18699

closed

Problem with File Upload to Group Library

Added by Matt Gold 8 months ago. Updated 8 months ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
Group Library
Target version:
Start date:
2023-08-29
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

Yesterday, I uploaded a file with the following name to a group library:

F'23 Gold, M (ENGL 89600, DHUM 78000).pdf

It was uploaded here -
https://commons.gc.cuny.edu/groups/knowledge-infrastructures-2/library/

I've attached the notification message I received of the file upload here.

After the file was uploaded, a student contacted me to say he couldn't download it. I tried and I also wasn't able to download, either from the link in the notification email or directly from the library. in each case, clicking on the file redirects me to the home page of the commons. FYI, I was logged into the CAC at the time.

I suspect that there was an issue with the special characters in the file name, but let me know what you think.


Files

Actions #1

Updated by Boone Gorges 8 months ago

  • Category name set to Group Library
  • Assignee changed from Raymond Hoh to Boone Gorges

I see a document called 'Library Reserve List' at that URL, but the link points to https://commons.gc.cuny.edu/?get_group_doc=3120/library-reserve+list.pdf, not https://commons.gc.cuny.edu?get_group_doc=3120/F%2723+Gold%2C+M+%28ENGL+89600%2C+DHUM+78000%29.pdf, which is what I saw in the email text. Did you replace the upload with a renamed version? This is not to dismiss the possibility of a bug, I'm just trying to understand what we're looking at.

Actions #2

Updated by Matt Gold 8 months ago

Yes, I deleted the problematic file and uploaded a new one.

I recommend that to test, you upload a PDF with the name I shared above to a test group to see if you can replicate the problem. Sorry for deleting, but I wanted to solve this ASAP for my students

Actions #3

Updated by Boone Gorges 8 months ago

  • Status changed from Assigned to Resolved
  • Target version set to 2.1.14

Thanks for the additional information. Your hunch that the issue was with characters in the filename was correct. The apostrophe was being slashed by PHP and not properly unslashed before locating the intended file. This is fixed in https://github.com/cuny-academic-commons/cac/commit/18d3915b071099b7f9ac662ae675b207b23da65a and deployed as a hotfix.

Actions #4

Updated by Matt Gold 8 months ago

Thank you!!

Actions

Also available in: Atom PDF