Refactor: mass imports in ShUp

[michaelkain]

Hi @jcomedouteau, I hope you are doing fine. I have had a look on the ImportFromFolder and ImportFromCsvRunner in ShUp. The importData methods are both very long and quite similar as well. Could you please?

• switch from .csv to .xlsx
• fusion/refactor/align both methods and split the code into smaller pieces
• use only one Window (if possible), with a "study selection" only at the beginning (already present in ImportFromFolderWindow) and therefore delete the Window for CSV
• use one FileChooser to select either the folder (from folder) or a CSV file (from pacs)
• display a unified table with the content (list of exams) to import and the same monitoring on status
• adapt to the new CSV file format to support patient/study root query level and to allow the configuration of the modality
• rename the menu to "Mass import" and open the unique window for mass imports With kind regards, Michael

[michaelkain]

@jcomedouteau I am on it.

[michaelkain]

I will slowly unify all on ImportJobs

[jcomedouteau]

Hi Michael, Thanks for the review, I clearly need to do some refactoring, I don't know if you could make some advancement on this, but I can finish it ! Jean-Côme