Remove superfluous folder in archived submissions
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mahara |
Fix Released
|
Medium
|
Robert Lyon |
Bug Description
Mahara 1.10+
When you archive submissions, the path to the actual file is like /var/lib/
I think we can safely remove the timestamp folder as each submission has a serialized number at the end. Disadvantages of the timestamp folder:
1. It is not clear unless you are a dev that the number is a timestamp.
2. Even if you know it's a timestamp, you'd need to calculate first if it's the one file in that folder that you need or not.
3. It doesn't add value because the file names are unique.
Advantages of removing the folder:
1. You can see all archived files from one user in one folder in chronological order. That makes it easier to review the files quickly and find the ones you are looking for.
2. One level less of hierarchy.
Changed in mahara: | |
assignee: | nobody → Robert Lyon (robertl-9) |
status: | Confirmed → In Progress |
Changed in mahara: | |
status: | In Progress → Fix Committed |
Changed in mahara: | |
status: | Fix Committed → Fix Released |
Patch for "master" branch: https:/ /reviews. mahara. org/4294