You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the problem you would like to solve
The FileManager classes are still used for TemporaryFiles, LibraryFiles and IssueFiles. That means we have two sets of code for reading and writing files to the files_dir directory.
Describe the solution you'd like
Use the new FileService class to read and write TemporaryFiles, LibraryFiles and IssueFiles. That will allow us to remove the FileManager classes and better support remote files in other parts of the system.
Housekeeping:1:TodoAny dependency management or refactor that would be nice to have some day.
1 participant
Converted from issue
This discussion was converted from issue #6704 on May 31, 2022 10:54.
Heading
Bold
Italic
Quote
Code
Link
Numbered list
Unordered list
Task list
Attach files
Mention
Reference
Menu
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
Describe the problem you would like to solve
The
FileManager
classes are still used for TemporaryFiles, LibraryFiles and IssueFiles. That means we have two sets of code for reading and writing files to thefiles_dir
directory.Describe the solution you'd like
Use the new
FileService
class to read and write TemporaryFiles, LibraryFiles and IssueFiles. That will allow us to remove theFileManager
classes and better support remote files in other parts of the system.Who is asking for this feature?
Developers
Beta Was this translation helpful? Give feedback.
All reactions