JTableHeaderLocation.columnName() doesn't work for reordered columns

[jedwards1211]

Hi Alex, this is my first contribution to FEST, woohoo :)

Here's the code I'm referring to (in https://github.com/alexruiz/fest-swing-1.x/blob/master/fest-swing/src/main/java/org/fest/swing/driver/JTableHeaderLocation.java):

  @RunsInCurrentThread
  private String columnName(@Nonnull JTableHeader tableHeader, int index) {
    return tableHeader.getTable().getModel().getColumnName(index);
  }

columnName(2) would return the name of the column at model index 2. Correct me if I'm wrong, but it should be returning the name of the column at view index 2, which may be different if the view columns have been reordered.

This should fix it:

  @RunsInCurrentThread
  private String columnName(@Nonnull JTableHeader tableHeader, int index) {
    return tableHeader.getTable().getColumnName(index);
  }

[manasouza]

Good point! I already been annoyed by this model/view JTable concepts....guess must be monitored cases when we have to apply convertColumnIndexToModel(int) and convertColumnIndexToView(int)....maybe we'll need two methods, one for each case...

[jedwards1211]

Well in this case, it must use the view index, or it may click a column
with a different name...
On Apr 14, 2013 10:07 AM, "Manassés Souza" [email protected] wrote:

Good point! I already been annoyed by this model/view JTable
concepts....guess must be monitored cases when we have to apply
convertColumnIndexToModel(int) and convertColumnIndexToView(int)....maybe
we'll need two methods, one for each case...

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/17#issuecomment-16352494
.