Program: 52717
Project: 52717
Buggy source:
protected Control createTransactionTable(Composite parent){
  Composite container=new Composite(parent,SWT.NONE);
  TableColumnLayout layout=new TableColumnLayout();
  container.setLayout(layout);
  transactions=new TableViewer(container,SWT.FULL_SELECTION | SWT.MULTI);
  ColumnEditingSupport.prepare(transactions);
  transactionsColumns=new ShowHideColumnHelper(AccountListView.class.getSimpleName() + "String_Node_Str",getPreferenceStore(),transactions,layout);
  Column column=new Column(Messages.ColumnDate,SWT.None,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return Values.Date.format(t.getDate());
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str","String_Node_Str","String_Node_Str").attachTo(column,SWT.DOWN);
  new DateEditingSupport(AccountTransaction.class,"String_Node_Str").addListener(this).attachTo(column);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnTransactionType,SWT.None,100);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return t.getType().toString();
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str"));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnAmount,SWT.RIGHT,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      long v=t.getAmount();
      if (EnumSet.of(Type.REMOVAL,Type.FEES,Type.TAXES,Type.BUY,Type.TRANSFER_OUT).contains(t.getType()))       v=-v;
      return Values.Money.format(Money.of(t.getCurrencyCode(),v),getClient().getBaseCurrency());
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str"));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.Balance,SWT.RIGHT,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      Money balance=transaction2balance.get(e);
      return balance != null ? Values.Money.format(balance,getClient().getBaseCurrency()) : null;
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(new AccountTransaction.ByDateAmountAndType()));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnSecurity,SWT.None,250);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return t.getSecurity() != null ? String.valueOf(t.getSecurity()) : null;
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str"));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnShares,SWT.RIGHT,80);
  column.setLabelProvider(new SharesLabelProvider(){
    @Override public Long getValue(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      if (t.getCrossEntry() instanceof BuySellEntry) {
        return ((BuySellEntry)t.getCrossEntry()).getPortfolioTransaction().getShares();
      }
 else       if (t.getType() == Type.DIVIDENDS && t.getShares() != 0) {
        return t.getShares();
      }
 else {
        return null;
      }
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  new ValueEditingSupport(AccountTransaction.class,"String_Node_Str",Values.Share){
    @Override public boolean canEdit(    Object element){
      AccountTransaction t=(AccountTransaction)element;
      return t.getType() == AccountTransaction.Type.DIVIDENDS;
    }
  }
.addListener(this).attachTo(column);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnPurchasePrice,SWT.RIGHT,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      if (t.getCrossEntry() instanceof BuySellEntry) {
        PortfolioTransaction pt=((BuySellEntry)t.getCrossEntry()).getPortfolioTransaction();
        return Values.Quote.format(pt.getGrossPricePerShare(),getClient().getBaseCurrency());
      }
 else       if (t.getType() == Type.DIVIDENDS && t.getShares() != 0) {
        long dividendPerShare=Math.round(t.getAmount() * Values.Share.divider() * Values.Quote.factorToMoney() / t.getShares());
        return Values.Quote.format(Quote.of(t.getCurrencyCode(),dividendPerShare),getClient().getBaseCurrency());
      }
 else {
        return null;
      }
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnOffsetAccount,SWT.None,120);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return t.getCrossEntry() != null ? t.getCrossEntry().getCrossOwner(t).toString() : null;
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnNote,SWT.None,200);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      return ((AccountTransaction)e).getNote();
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
    @Override public Image getImage(    Object e){
      String note=((AccountTransaction)e).getNote();
      return note != null && note.length() > 0 ? Images.NOTE.image() : null;
    }
  }
);
  ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str").attachTo(column);
  new StringEditingSupport(AccountTransaction.class,"String_Node_Str").addListener(this).attachTo(column);
  transactionsColumns.addColumn(column);
  transactionsColumns.createColumns();
  transactions.getTable().setHeaderVisible(true);
  transactions.getTable().setLinesVisible(true);
  transactions.setContentProvider(new SimpleListContentProvider());
  hookContextMenu(transactions.getTable(),this::fillTransactionsContextMenu);
  hookKeyListener();
  return container;
}

Fixed version (ground truth):
protected Control createTransactionTable(Composite parent){
  Composite container=new Composite(parent,SWT.NONE);
  TableColumnLayout layout=new TableColumnLayout();
  container.setLayout(layout);
  transactions=new TableViewer(container,SWT.FULL_SELECTION | SWT.MULTI);
  ColumnEditingSupport.prepare(transactions);
  transactionsColumns=new ShowHideColumnHelper(AccountListView.class.getSimpleName() + "String_Node_Str",getPreferenceStore(),transactions,layout);
  Column column=new Column(Messages.ColumnDate,SWT.None,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return Values.Date.format(t.getDate());
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str","String_Node_Str","String_Node_Str").attachTo(column,SWT.DOWN);
  new DateEditingSupport(AccountTransaction.class,"String_Node_Str").addListener(this).attachTo(column);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnTransactionType,SWT.None,100);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return t.getType().toString();
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str"));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnAmount,SWT.RIGHT,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      long v=t.getAmount();
      if (t.getType().isNegative())       v=-v;
      return Values.Money.format(Money.of(t.getCurrencyCode(),v),getClient().getBaseCurrency());
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str"));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.Balance,SWT.RIGHT,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      Money balance=transaction2balance.get(e);
      return balance != null ? Values.Money.format(balance,getClient().getBaseCurrency()) : null;
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(new AccountTransaction.ByDateAmountAndType()));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnSecurity,SWT.None,250);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return t.getSecurity() != null ? String.valueOf(t.getSecurity()) : null;
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  column.setSorter(ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str"));
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnShares,SWT.RIGHT,80);
  column.setLabelProvider(new SharesLabelProvider(){
    @Override public Long getValue(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      if (t.getCrossEntry() instanceof BuySellEntry) {
        return ((BuySellEntry)t.getCrossEntry()).getPortfolioTransaction().getShares();
      }
 else       if (t.getType() == Type.DIVIDENDS && t.getShares() != 0) {
        return t.getShares();
      }
 else {
        return null;
      }
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  new ValueEditingSupport(AccountTransaction.class,"String_Node_Str",Values.Share){
    @Override public boolean canEdit(    Object element){
      AccountTransaction t=(AccountTransaction)element;
      return t.getType() == AccountTransaction.Type.DIVIDENDS;
    }
  }
.addListener(this).attachTo(column);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnPurchasePrice,SWT.RIGHT,80);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      if (t.getCrossEntry() instanceof BuySellEntry) {
        PortfolioTransaction pt=((BuySellEntry)t.getCrossEntry()).getPortfolioTransaction();
        return Values.Quote.format(pt.getGrossPricePerShare(),getClient().getBaseCurrency());
      }
 else       if (t.getType() == Type.DIVIDENDS && t.getShares() != 0) {
        long dividendPerShare=Math.round(t.getAmount() * Values.Share.divider() * Values.Quote.factorToMoney() / t.getShares());
        return Values.Quote.format(Quote.of(t.getCurrencyCode(),dividendPerShare),getClient().getBaseCurrency());
      }
 else {
        return null;
      }
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnOffsetAccount,SWT.None,120);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      AccountTransaction t=(AccountTransaction)e;
      return t.getCrossEntry() != null ? t.getCrossEntry().getCrossOwner(t).toString() : null;
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
  }
);
  transactionsColumns.addColumn(column);
  column=new Column(Messages.ColumnNote,SWT.None,200);
  column.setLabelProvider(new ColumnLabelProvider(){
    @Override public String getText(    Object e){
      return ((AccountTransaction)e).getNote();
    }
    @Override public Color getForeground(    Object element){
      return colorFor((AccountTransaction)element);
    }
    @Override public Image getImage(    Object e){
      String note=((AccountTransaction)e).getNote();
      return note != null && note.length() > 0 ? Images.NOTE.image() : null;
    }
  }
);
  ColumnViewerSorter.create(AccountTransaction.class,"String_Node_Str").attachTo(column);
  new StringEditingSupport(AccountTransaction.class,"String_Node_Str").addListener(this).attachTo(column);
  transactionsColumns.addColumn(column);
  transactionsColumns.createColumns();
  transactions.getTable().setHeaderVisible(true);
  transactions.getTable().setLinesVisible(true);
  transactions.setContentProvider(new SimpleListContentProvider());
  hookContextMenu(transactions.getTable(),this::fillTransactionsContextMenu);
  hookKeyListener();
  return container;
}