Ticket #1833 (new defect)

Opened 6 years ago

Last modified 6 years ago

Drag and Drop: support .xls and .csv files

Reported by: bdezonia Owned by: curtis
Priority: major Milestone: imagej2-b10-compatibility
Component: Core Version:
Severity: serious Keywords:
Cc: Blocked By:
Blocking: #1821

Description

In IJ1 dragged and dropped .xls and .csv files open as results tables. Make this work in IJ2. (I believe it was before a recent refactor was done).

Change History

comment:1 Changed 6 years ago by curtis

I did not see any support for .xls or .csv files while working on the drag-and-drop branch (apologies if I missed it somehow).

The closest thing I can think of is that in bfd6e895 the LUT service's API was refactored. As part of that change, the logic which could decode a ColorTable from a text LUT was disabled, due to the fact that the TableLoader class only operates on URLs.

However, I am not certain any of that is relevant to results tables.

It looks like nothing uses the TableLoader now. But it is certainly what would be used to support .xls and .csv...

In a  recent thread to imagej-devel I inquired whether we should really be doing all this stuff for drag-and-drop specifically, rather than as I/O handler plugins for use with the new IOService#load(File) method. In line with that, I propose that a new IOHandler be created which uses the TableLoader to create a ResultsTable from the data. A generic FileDragAndDropHandler which uses IOService#load(File) will take care of the rest, and this ticket is solved!

Note: See TracTickets for help on using tickets.