⚠	NOTICE! This is a static HTML version of a legacy ImageJ Trac ticket. The ImageJ project now uses GitHub Issues for issue tracking. Please file all new issues there.

Ticket #1646 (closed defect: moved)

Opened 2013-02-01T14:57:36-06:00

Last modified 2013-03-20T11:48:50-05:00

Improve Table API

Reported by:	curtis	Owned by:	curtis
Priority:	minor	Milestone:	imagej2-unscheduled
Component:	Data Model	Version:	2.0.0-beta6
Severity:	minor	Keywords:
Cc:		Blocked By:
Blocking:	#1713

Description

I believed the Table API (e.g., for Results tables) was pretty thorough, but when trying to use it to implement a new IJ2 List Shortcuts command, I discovered that there are no appendRow methods. These would be a nice convenience, rather than having to explicitly call set(String colHeader, int row, Object value) for each cell of the row.

Change History

comment:1 Changed 2013-03-19T13:30:17-05:00 by curtis

Well, there *are* appendRow methods, but none that take the actual row data. So the current ListShortcuts implementation does:

shortcuts.appendRow();
shortcuts.set(hotKeyHeader, row, shortcut.toString());
shortcuts.set(commandHeader, row, info.getTitle());

When what would be nicer would be:

shortcuts.appendRow(shortcut.toString(), info.getTitle());

Of course, while this is more succinct, it also loses the column-safety, assuming columns are in a particular order.

comment:2 Changed 2013-03-20T11:45:16-05:00 by bdezonia

Cc 1628 added

comment:3 Changed 2013-03-20T11:47:44-05:00 by bdezonia

Cc 1628 removed

comment:4 Changed 2013-03-20T11:48:50-05:00 by bdezonia

Blocking 1713 added

comment:5 Changed 2014-08-12T01:17:02-05:00 by curtis

Status changed from new to closed
Resolution set to moved

Migrated to https://github.com/imagej/imagej-common/issues/26