ref #9816: adapt tooltip text
[taxeditor.git] / eu.etaxonomy.taxeditor.editor / src / main / java / eu / etaxonomy / taxeditor / editor / descriptiveDataSet / matrix / CharacterMatrix.java
index e2403b91e8acb8cca4701a071838ac4012e71197..55950cda159450871475defb1241832064f42073 100644 (file)
@@ -49,6 +49,7 @@ import org.eclipse.nebula.widgets.nattable.coordinate.Range;
 import org.eclipse.nebula.widgets.nattable.copy.command.InternalCopyDataCommandHandler;
 import org.eclipse.nebula.widgets.nattable.copy.command.InternalPasteDataCommandHandler;
 import org.eclipse.nebula.widgets.nattable.data.IDataProvider;
+import org.eclipse.nebula.widgets.nattable.data.IRowDataProvider;
 import org.eclipse.nebula.widgets.nattable.data.ListDataProvider;
 import org.eclipse.nebula.widgets.nattable.export.command.ExportCommandHandler;
 import org.eclipse.nebula.widgets.nattable.extension.e4.selection.E4SelectionListener;
@@ -59,6 +60,7 @@ import org.eclipse.nebula.widgets.nattable.extension.glazedlists.tree.GlazedList
 import org.eclipse.nebula.widgets.nattable.freeze.CompositeFreezeLayer;
 import org.eclipse.nebula.widgets.nattable.freeze.FreezeHelper;
 import org.eclipse.nebula.widgets.nattable.freeze.FreezeLayer;
+import org.eclipse.nebula.widgets.nattable.grid.GridRegion;
 import org.eclipse.nebula.widgets.nattable.grid.command.ClientAreaResizeCommand;
 import org.eclipse.nebula.widgets.nattable.grid.data.DefaultColumnHeaderDataProvider;
 import org.eclipse.nebula.widgets.nattable.grid.data.DefaultCornerDataProvider;
@@ -74,11 +76,14 @@ import org.eclipse.nebula.widgets.nattable.group.ColumnGroupModel;
 import org.eclipse.nebula.widgets.nattable.layer.AbstractLayer;
 import org.eclipse.nebula.widgets.nattable.layer.DataLayer;
 import org.eclipse.nebula.widgets.nattable.layer.ILayer;
+import org.eclipse.nebula.widgets.nattable.layer.cell.ILayerCell;
 import org.eclipse.nebula.widgets.nattable.persistence.gui.PersistenceDialog;
 import org.eclipse.nebula.widgets.nattable.selection.SelectionLayer;
+import org.eclipse.nebula.widgets.nattable.selection.command.SelectRowsCommand;
 import org.eclipse.nebula.widgets.nattable.sort.SortHeaderLayer;
 import org.eclipse.nebula.widgets.nattable.sort.config.SingleClickSortConfiguration;
 import org.eclipse.nebula.widgets.nattable.style.theme.ModernNatTableThemeConfiguration;
+import org.eclipse.nebula.widgets.nattable.tooltip.NatTableContentTooltip;
 import org.eclipse.nebula.widgets.nattable.tree.ITreeRowModel;
 import org.eclipse.nebula.widgets.nattable.tree.TreeLayer;
 import org.eclipse.nebula.widgets.nattable.tree.command.TreeExpandToLevelCommand;
@@ -92,6 +97,7 @@ import org.eclipse.swt.layout.GridData;
 import org.eclipse.swt.layout.GridLayout;
 import org.eclipse.swt.widgets.Button;
 import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.Event;
 import org.eclipse.swt.widgets.Menu;
 
 import ca.odell.glazedlists.BasicEventList;
@@ -102,24 +108,29 @@ import eu.etaxonomy.cdm.api.application.CdmApplicationState;
 import eu.etaxonomy.cdm.api.service.IDescriptiveDataSetService;
 import eu.etaxonomy.cdm.api.service.IOccurrenceService;
 import eu.etaxonomy.cdm.api.service.UpdateResult;
+import eu.etaxonomy.cdm.api.service.config.RemoveDescriptionsFromDescriptiveDataSetConfigurator;
 import eu.etaxonomy.cdm.api.service.dto.FieldUnitDTO;
 import eu.etaxonomy.cdm.api.service.dto.RowWrapperDTO;
 import eu.etaxonomy.cdm.api.service.dto.SpecimenRowWrapperDTO;
+import eu.etaxonomy.cdm.api.service.dto.TaxonRowWrapperDTO;
 import eu.etaxonomy.cdm.common.monitor.IRemotingProgressMonitor;
+import eu.etaxonomy.cdm.model.common.CdmBase;
 import eu.etaxonomy.cdm.model.description.DescriptionBase;
 import eu.etaxonomy.cdm.model.description.DescriptiveDataSet;
-import eu.etaxonomy.cdm.model.description.Feature;
-import eu.etaxonomy.cdm.model.description.MeasurementUnit;
+import eu.etaxonomy.cdm.model.description.SpecimenDescription;
 import eu.etaxonomy.cdm.model.occurrence.FieldUnit;
-import eu.etaxonomy.cdm.model.term.TermNode;
-import eu.etaxonomy.cdm.model.term.TermTree;
+import eu.etaxonomy.cdm.persistence.dto.DescriptiveDataSetBaseDto;
+import eu.etaxonomy.cdm.persistence.dto.FeatureDto;
 import eu.etaxonomy.cdm.persistence.dto.SpecimenNodeWrapper;
 import eu.etaxonomy.cdm.persistence.dto.TermDto;
+import eu.etaxonomy.cdm.persistence.dto.TermNodeDto;
+import eu.etaxonomy.cdm.persistence.dto.TermTreeDto;
 import eu.etaxonomy.cdm.persistence.dto.UuidAndTitleCache;
 import eu.etaxonomy.taxeditor.editor.internal.TaxeditorEditorPlugin;
 import eu.etaxonomy.taxeditor.editor.l10n.Messages;
 import eu.etaxonomy.taxeditor.event.WorkbenchEventConstants;
 import eu.etaxonomy.taxeditor.model.MessagingUtils;
+import eu.etaxonomy.taxeditor.operation.IFeedbackGenerator;
 import eu.etaxonomy.taxeditor.session.ICdmEntitySession;
 import eu.etaxonomy.taxeditor.store.CdmStore;
 import eu.etaxonomy.taxeditor.workbench.WorkbenchUtility;
@@ -128,7 +139,6 @@ import eu.etaxonomy.taxeditor.workbench.WorkbenchUtility;
  * Character matrix editor for editing specimen/taxon descriptions in a table
  * @author pplitzner
  * @since Nov 26, 2017
- *
  */
 public class CharacterMatrix extends Composite {
 
@@ -150,6 +160,7 @@ public class CharacterMatrix extends Composite {
     static final String LABEL_TAXON_DESCRIPTION = "LABEL_TAXON_DESCRIPTION"; //$NON-NLS-1$
     static final String LABEL_DESCRIPTION_HAS_SUPPLEMENTAL_DATA = "LABEL_DESCRIPTION_HAS_SUPPLEMENTAL_DATA"; //$NON-NLS-1$
 
+    boolean isFirstCall = true;
     @Inject
     private UISynchronize sync;
 
@@ -158,11 +169,11 @@ public class CharacterMatrix extends Composite {
 
     private NatTable natTable;
 
-    private Collection<RowWrapperDTO<?>> rowsToSave = new HashSet<>();
+    private HashMap<UUID, RowWrapperDTO<?>> rowsToMerge = new HashMap<>();
 
-    private Map<Integer, Feature> indexToFeatureMap = new HashMap<>();
+    private Map<Integer, FeatureDto> indexToFeatureMap = new HashMap<>();
 
-    private Map<Feature, List<TermDto>> categoricalFeatureToStateMap = new HashMap<>();
+    private Map<UUID, List<TermDto>> categoricalFeatureToStateMap = new HashMap<>();
 
     private LinkedMap<String, String> propertyToLabelMap = new LinkedMap<>();
 
@@ -170,15 +181,13 @@ public class CharacterMatrix extends Composite {
 
     private Collection<SpecimenNodeWrapper> specimenCache = null;
 
-    private Map<Feature, CategoricalDataHistogram> featureToHistogramMap = new HashMap<>();
-
-    private Map<Feature, QuantitativeDataStatistics> featureToQuantDataStatisticsMap = new HashMap<>();
+    private Map<FeatureDto, CategoricalDataHistogram> featureToHistogramMap = new HashMap<>();
 
     private ListDataProvider<Object> bodyDataProvider;
 
     private FreezeLayer freezeLayer;
 
-    private List<Feature> features;
+    private List<FeatureDto> features;
 
     private CharacterMatrixPart part;
 
@@ -188,14 +197,21 @@ public class CharacterMatrix extends Composite {
 
     private MatrixBodyLayerStack bodyLayer;
 
+    private SelectionLayer selectionLayer;
+
     private boolean isTreeView = true;
 
     private CharacterMatrixToolbar toolbar;
 
     private DescriptionTreeFormat treeFormat;
-    private List<UUID> descriptionUuidsToDelete;
 
-    private List<SpecimenNodeWrapper> specimenToAdd;
+    private Map<RemoveDescriptionsFromDescriptiveDataSetConfigurator, List<UUID>> descriptionUuidsToDelete;
+
+    private List<SpecimenRowWrapperDTO> specimenToAdd;
+
+    private MouseEventMatcher mouseEventMatcher = null;
+
+    private Set<DescriptionBase<?>> descriptionsToSave = new HashSet<>();
 
     public CharacterMatrix(Composite parent, CharacterMatrixPart part) {
         super(parent, SWT.NONE);
@@ -205,21 +221,12 @@ public class CharacterMatrix extends Composite {
         createToolBar();
 
         natTable = new NatTable(this, false);
-
-        createBottomToolbar();
-
     }
 
     private void createToolBar(){
         toolbar = new CharacterMatrixToolbar(this, SWT.NONE);
     }
 
-    @SuppressWarnings("unused")
-    private void createBottomToolbar() {
-        new CharacterMatrixBottomToolbar(this, SWT.NONE);
-    }
-
-
     private void applyStyles(){
         ModernNatTableThemeConfiguration configuration = new ModernNatTableThemeConfiguration();
         // NOTE: Getting the colors and fonts from the GUIHelper ensures that
@@ -227,7 +234,6 @@ public class CharacterMatrix extends Composite {
         configuration.cHeaderBgColor = GUIHelper.getColor(211, 211, 211);
         configuration.rHeaderBgColor = GUIHelper.getColor(211, 211, 211);
         natTable.addConfiguration(configuration);
-
     }
 
     void toggleTreeFlat(boolean isTree, Button btnToggleFlat, Button btnToggleTree, Button btnCollapseAll, Button btnExpandAll, Button btnFreezeSuppInfo) {
@@ -252,6 +258,7 @@ public class CharacterMatrix extends Composite {
         /**
          * configuration
          */
+
         configureNatTable(treeView, configRegistry, topMostLayer);
 
         /**
@@ -262,8 +269,8 @@ public class CharacterMatrix extends Composite {
         //grab all space
         GridDataFactory.fillDefaults().grab(true, true).applyTo(natTable);
 
-        //update label to current data set
-        toolbar.getWsLabel().setText(getDescriptiveDataSet().getLabel());
+        //update label to current dataset
+        toolbar.getWsLabel().setText(getDescriptiveDataSet().getTitleCache());
         toolbar.getWsLabel().setLayoutData(new GridData(SWT.FILL, SWT.CENTER, true, false));
         toolbar.getWsLabel().getParent().layout();
 
@@ -286,59 +293,93 @@ public class CharacterMatrix extends Composite {
                 natTable.doCommand(new TreeExpandToLevelCommand(deepestTaxonLevel-2));
             }
         }
+        new NatTableContentTooltip(natTable, GridRegion.BODY) {
+               protected String getText(Event event) {
+                       int col = this.natTable.getColumnPositionByX(event.x);
+               int row = this.natTable.getRowPositionByY(event.y);
+
+               ILayerCell cell = this.natTable.getCellByPosition(col, row);
+               if (cell.getConfigLabels().getLabels().contains(CharacterMatrixConfigLabelAccumulator.NOT_EDITABLE)){
+                       Object o = bodyDataProvider.getRowObject(row);
+                       String object = "";
+                       if (o instanceof TaxonRowWrapperDTO){
+                               object = "taxon";
+                       }else {
+                               object = "specimen";
+                       }
+                               
+                       return "Multiple data exist. Editing only possible in factual data view of " + object;
+               }
+                       return null;
+               }
+       };
+    
 
         // clean up table state
         getNatTableState().remove(NatTable.INITIAL_PAINT_COMPLETE_FLAG);
         getNatTableState().remove(PersistenceDialog.ACTIVE_VIEW_CONFIGURATION_KEY);
     }
 
-    private List<Feature> initFeatureList(TermNode<Feature> node){
-        List<Feature> features = new ArrayList<>();
-        List<TermNode<Feature>> childNodes = node.getChildNodes();
-        for (TermNode<Feature> childNode : childNodes) {
-            features.add(childNode.getTerm());
-            features.addAll(initFeatureList(childNode));
+    private List<FeatureDto> initFeatureList(TermNodeDto node){
+        List<FeatureDto> features = new ArrayList<>();
+        List<TermNodeDto> childNodes = node.getChildren();
+        for (TermNodeDto childNode : childNodes) {
+            if (childNode != null){
+                features.add((FeatureDto) childNode.getTerm());
+                features.addAll(initFeatureList(childNode));
+            }
         }
         return features;
     }
 
     public void initDescriptiveDataSet(){
-        //get features/columns stored in descriptive data set
-        TermTree<Feature> tree = getDescriptiveDataSet().getDescriptiveSystem();
+        //get features/columns stored in descriptive dataset
+        TermTreeDto tree = getDescriptiveDataSet().getDescriptiveSystem();
         features = initFeatureList(tree.getRoot());
 
-        Set<Feature> duplicateFeatures = features.stream().filter(i -> Collections.frequency(features, i) >1)
+        Set<FeatureDto> duplicateFeatures = features.stream().filter(i -> Collections.frequency(features, i) >1)
         .collect(Collectors.toSet());
 
         if (!duplicateFeatures.isEmpty()) {
             throw new IllegalArgumentException("Duplicate features found: "
-                    + duplicateFeatures.stream().map(feature -> feature.getLabel()).collect(Collectors.joining(",")));
+                    + duplicateFeatures.stream().map(feature -> feature.getRepresentation_L10n()).collect(Collectors.joining(",")));
         }
 
         //init state data for categorical features
-        features.forEach(feature->fetchSupportedStates(feature));
+//        features.forEach(feature->fetchSupportedStates(feature));
+        fetchSupportedStates(features);
 
         descriptions = new BasicEventList<>();
 
     }
 
-    private void fetchSupportedStates(Feature feature) {
-        List<TermDto> supportedStates = CdmStore.getService(IDescriptiveDataSetService.class).getSupportedStatesForFeature(feature.getUuid());
-        categoricalFeatureToStateMap.put(feature, supportedStates);
+    private void fetchSupportedStates(List<FeatureDto> features) {
+        Set<UUID> featureUuids = new HashSet<>();
+        features.forEach(i->featureUuids.add(i.getUuid()));
+        categoricalFeatureToStateMap = CdmStore.getService(IDescriptiveDataSetService.class).getSupportedStatesForFeature(featureUuids);
+
     }
 
+
+
     private void createLayers(boolean treeView) {
+
         SortedList<Object> sortedList = new SortedList<>(descriptions, new MatrixRowComparator());
         // wrap the SortedList with the TreeList
         treeFormat = new DescriptionTreeFormat(getDescriptiveDataSet());
-        TreeList<Object> treeList = new TreeList(sortedList, treeFormat, TreeList.NODES_START_COLLAPSED);
+        @SuppressWarnings("unchecked")
+        TreeList<Object> treeList = new TreeList<Object>(sortedList, treeFormat, TreeList.NODES_START_COLLAPSED);
+        // wrap the SortedList with the TreeList
+//        treeFormat = new DescriptionTreeFormat(getDescriptiveDataSet());
+
         /**
          * data provider
          */
         SpecimenColumnPropertyAccessor columnPropertyAccessor = new SpecimenColumnPropertyAccessor(this);
         bodyDataProvider = treeView?new ListDataProvider<>(treeList, columnPropertyAccessor):new ListDataProvider<>(sortedList, columnPropertyAccessor);
 
-        configRegistry = new ConfigRegistry();
+        DataLayer bodyDataLayer = new DataLayer(bodyDataProvider);
+        bodyDataLayer.registerCommandHandler(new CopyPasteUpdateDataCommandHandler(bodyDataLayer));
 
 
         /**
@@ -372,9 +413,8 @@ public class CharacterMatrix extends Composite {
          *
 
          */
-        DataLayer bodyDataLayer = new DataLayer(bodyDataProvider);
-        bodyDataLayer.registerCommandHandler(new CopyPasteUpdateDataCommandHandler(bodyDataLayer));
 
+        configRegistry = new ConfigRegistry();
         //register labels
         CharacterMatrixConfigLabelAccumulator labelAccumulator = new CharacterMatrixConfigLabelAccumulator(this);
         bodyDataLayer.setConfigLabelAccumulator(labelAccumulator);
@@ -385,27 +425,30 @@ public class CharacterMatrix extends Composite {
         propertyToLabelMap.put(IDENTIFIER_COLUMN, Messages.CharacterMatrix_IDENTIFIER);
         propertyToLabelMap.put(COUNTRY_COLUMN, Messages.CharacterMatrix_COUNTRY);
         for(int i=0;i<features.size();i++){
-            Feature feature = features.get(i);
+            FeatureDto feature = features.get(i);
             initLabels(i, feature);
         }
 
         // layer for event handling of GlazedLists and PropertyChanges
-        GlazedListsEventLayer eventLayer = new GlazedListsEventLayer<>(bodyDataLayer, (EventList)bodyDataProvider.getList());
+        GlazedListsEventLayer<Object> eventLayer = new GlazedListsEventLayer<>(bodyDataLayer, (EventList<Object>)bodyDataProvider.getList());
 
-        GlazedListTreeData treeData = new GlazedListTreeData<>(treeList);
-        ITreeRowModel treeRowModel = new GlazedListTreeRowModel<>(treeData);
+        GlazedListTreeData<?> treeData = new GlazedListTreeData<>(treeList);
+        ITreeRowModel<?> treeRowModel = new GlazedListTreeRowModel<>(treeData);
 
         // assemble the column groups
         LinkedList<ColumnGroupWrapper> columnGroups = new LinkedList<>();
-        List<TermNode<Feature>> rootChildren = getDescriptiveDataSet().getDescriptiveSystem().getRootChildren();
+        List<TermNodeDto> rootChildren = getDescriptiveDataSet().getDescriptiveSystem().getRoot().getChildren();
         buildHeader(rootChildren, columnGroups);
 
         bodyLayer = new MatrixBodyLayerStack(eventLayer, columnGroups);
-        final SelectionLayer selectionLayer = bodyLayer.getSelectionLayer();
+        selectionLayer = bodyLayer.getSelectionLayer();
         freezeLayer = new FreezeLayer(selectionLayer);
         final CompositeFreezeLayer compositeFreezeLayer = new CompositeFreezeLayer(
                 freezeLayer, bodyLayer.getViewportLayer(), selectionLayer);
-        TreeLayer treeLayer = new TreeLayer(compositeFreezeLayer, treeRowModel);
+        TreeLayer treeLayer = null;
+        if (treeView){
+            treeLayer = new TreeLayer(compositeFreezeLayer, treeRowModel);
+        }
 
         topMostLayer = treeView?treeLayer:compositeFreezeLayer;
 
@@ -426,8 +469,8 @@ public class CharacterMatrix extends Composite {
             ColumnGroupHeaderLayer groupLayerLevel1 = null;
             ColumnGroupGroupHeaderLayer groupLayerLevel2 = null;
             groupLayerLevel1 = new ColumnGroupHeaderLayer(columnHeaderLayer, selectionLayer, groupLevel1.getModel());
-            for (Entry<TermNode<Feature>, TreeSet<Integer>> entry: groupLevel1.getColumnGroupToIndexMap().entrySet()) {
-                TermNode<Feature> group = entry.getKey();
+            for (Entry<TermNodeDto, TreeSet<Integer>> entry: groupLevel1.getColumnGroupToIndexMap().entrySet()) {
+                TermNodeDto group = entry.getKey();
                 TreeSet<Integer> indexList = entry.getValue();
                 int[] intArray = indexList.stream().mapToInt(Integer::intValue).toArray();
                 groupLayerLevel1.addColumnsIndexesToGroup(group.getTerm().getTitleCache(), intArray);
@@ -439,8 +482,8 @@ public class CharacterMatrix extends Composite {
             if(columnGroups.size()>1){
                 ColumnGroupWrapper groupLevel2 = columnGroups.get(1);
                 groupLayerLevel2 = new ColumnGroupGroupHeaderLayer(groupLayerLevel1, selectionLayer, groupLevel2.getModel());
-                for (Entry<TermNode<Feature>, TreeSet<Integer>> entry: groupLevel2.getColumnGroupToIndexMap().entrySet()) {
-                    TermNode<Feature> group = entry.getKey();
+                for (Entry<TermNodeDto, TreeSet<Integer>> entry: groupLevel2.getColumnGroupToIndexMap().entrySet()) {
+                    TermNodeDto group = entry.getKey();
                     TreeSet<Integer> indexList = entry.getValue();
                     int[] intArray = indexList.stream().mapToInt(Integer::intValue).toArray();
                     groupLayerLevel2.addColumnsIndexesToGroup(group.getTerm().getTitleCache(), intArray);
@@ -453,7 +496,7 @@ public class CharacterMatrix extends Composite {
         // add the SortHeaderLayer to the column header layer stack
         // as we use GlazedLists, we use the GlazedListsSortModel which
         // delegates the sorting to the SortedList
-        final SortHeaderLayer<DescriptionBase> sortHeaderLayer = new SortHeaderLayer<>(
+        final SortHeaderLayer<DescriptionBase<?>> sortHeaderLayer = new SortHeaderLayer<>(
                 topHeaderLayer,
                 new GlazedListsSortModel<>(
                         sortedList,
@@ -461,8 +504,6 @@ public class CharacterMatrix extends Composite {
                         configRegistry,
                         columnHeaderDataLayer));
 
-
-
         /**
          * row header layer
          */
@@ -471,7 +512,6 @@ public class CharacterMatrix extends Composite {
         RowHeaderLayer rowHeaderLayer = new RowHeaderLayer(rowHeaderDataLayer,
                 topMostLayer, selectionLayer);
 
-
         /**
          * corner layer
          */
@@ -479,38 +519,39 @@ public class CharacterMatrix extends Composite {
                 new DataLayer(new DefaultCornerDataProvider(columnHeaderDataProvider, rowHeaderDataProvider)),
                 rowHeaderLayer, sortHeaderLayer);
 
-
         /**
          * GRID layer (composition of all other layers)
          */
         GridLayer gridLayer = new GridLayer(topMostLayer, sortHeaderLayer, rowHeaderLayer, cornerLayer);
-
         natTable.setLayer(gridLayer);
-
     }
 
-    private TreeSet<Integer> recurseChildIndexes(TermNode<Feature> node){
+    private TreeSet<Integer> recurseChildIndexes(TermNodeDto node){
         TreeSet<Integer> childIndexes = new TreeSet<>();
-        if(node.getChildCount()>0){
-            List<TermNode<Feature>> childNodes = node.getChildNodes();
-            for (TermNode<Feature> childNode: childNodes) {
-                childIndexes.addAll(recurseChildIndexes(childNode));
+        if(node.getChildren().size() >0){
+            List<TermNodeDto> childNodes = node.getChildren();
+            for (TermNodeDto childNode: childNodes) {
+                if (childNode != null){
+                    childIndexes.addAll(recurseChildIndexes(childNode));
+                }
             }
         }
         childIndexes.add(features.indexOf(node.getTerm())+LEADING_COLUMN_COUNT);
         return childIndexes;
     }
 
-    private void buildHeader(List<TermNode<Feature>> nodes, LinkedList<ColumnGroupWrapper> columnGroups){
-        Map<TermNode<Feature>, TreeSet<Integer>> columnGroupToIndexMap = new HashMap<>();
-        List<TermNode<Feature>> childNodes = new ArrayList<>();
-        for (TermNode<Feature> node : nodes) {
-            TreeSet<Integer> childIndexes = recurseChildIndexes(node);
-            if(childIndexes.size()>1){
-                // filter out groups that only have one member
-                columnGroupToIndexMap.put(node, childIndexes);
+    private void buildHeader(List<TermNodeDto> nodes, LinkedList<ColumnGroupWrapper> columnGroups){
+        Map<TermNodeDto, TreeSet<Integer>> columnGroupToIndexMap = new HashMap<>();
+        List<TermNodeDto> childNodes = new ArrayList<>();
+        for (TermNodeDto node : nodes) {
+            if (node != null){
+                TreeSet<Integer> childIndexes = recurseChildIndexes(node);
+                if(childIndexes.size()>1){
+                    // filter out groups that only have one member
+                    columnGroupToIndexMap.put(node, childIndexes);
+                }
+                childNodes.addAll(node.getChildren());
             }
-            childNodes.addAll(node.getChildNodes());
         }
         if(!columnGroupToIndexMap.isEmpty()){
             columnGroups.addFirst(new ColumnGroupWrapper(new ColumnGroupModel(), columnGroupToIndexMap));
@@ -524,7 +565,7 @@ public class CharacterMatrix extends Composite {
         natTable.registerCommandHandler(new ExportCommandHandler(natTable));
 
         //selection listener
-        E4SelectionListener selectionListener = new CellSelectionListener(part.getSelectionService(),
+        E4SelectionListener<Object> selectionListener = new CellSelectionListener(part.getSelectionService(),
                 bodyLayer.getSelectionLayer(), bodyDataProvider, part);
         bodyLayer.getSelectionLayer().addLayerListener(selectionListener);
         selectionListener.setFullySelectedRowsOnly(false);
@@ -545,6 +586,11 @@ public class CharacterMatrix extends Composite {
         /**
          * CONFIGURATION
          */
+      //+++CONTEXT MENU+++
+        menuService.registerContextMenu(natTable, "eu.etaxonomy.taxeditor.editor.popupmenu.charactermatrix"); //$NON-NLS-1$
+        // get the menu registered by EMenuService
+        final Menu e4Menu = natTable.getMenu();
+
         natTable.setConfigRegistry(configRegistry);
 
         applyStyles();
@@ -565,7 +611,7 @@ public class CharacterMatrix extends Composite {
 
         // add the header menu configuration for adding the column header menu
         // with hide/show actions
-        natTable.addConfiguration(new CharacterMatrixHeaderMenuConfiguration(natTable));
+//        natTable.addConfiguration(new CharacterMatrixHeaderMenuConfiguration(natTable));
 
         // add custom configuration for data conversion and add column labels to viewport layer
         topMostLayer.addConfiguration(new CellEditorDataConversionConfiguration(this));
@@ -573,32 +619,44 @@ public class CharacterMatrix extends Composite {
         //copy&paste configuration
         natTable.addConfiguration(new CopyPasteEditBindings(bodyLayer.getSelectionLayer(), natTable.getInternalCellClipboard()));
 
-      //+++CONTEXT MENU+++
-        menuService.registerContextMenu(natTable, "eu.etaxonomy.taxeditor.editor.popupmenu.charactermatrix"); //$NON-NLS-1$
-        // get the menu registered by EMenuService
-        final Menu e4Menu = natTable.getMenu();
+
         // remove the menu reference from NatTable instance
         natTable.setMenu(null);
-        natTable.addConfiguration(
-                new AbstractUiBindingConfiguration() {
-            @Override
-            public void configureUiBindings(
-                    UiBindingRegistry uiBindingRegistry) {
-                // add e4 menu to NatTable
-                new PopupMenuBuilder(natTable, e4Menu)
-                    .build();
-
-                // register the UI binding for header, corner and body region
-                uiBindingRegistry.registerMouseDownBinding(
-                        new MouseEventMatcher(
-                                SWT.NONE,
-                                null,
-                                MouseEventMatcher.RIGHT_BUTTON),
-                        new PopupMenuAction(e4Menu));
-            }
-        });
+
+
+//        natTable.addConfiguration(new CharacterMatrixHeaderMenuConfiguration(natTable));
+
 
         natTable.configure();
+
+
+        natTable.addConfiguration(
+                    new AbstractUiBindingConfiguration() {
+                @Override
+                public void configureUiBindings(
+                        UiBindingRegistry uiBindingRegistry) {
+                    // add e4 menu to NatTable
+                    uiBindingRegistry.unregisterMouseDownBinding(mouseEventMatcher);
+                    if (isFirstCall){
+                           new PopupMenuBuilder(natTable, e4Menu)
+                           .withHideColumnMenuItem()
+                           .withShowAllColumnsMenuItem()
+                           .build();
+                           isFirstCall = false;
+                    }else{
+                       new PopupMenuBuilder(natTable, e4Menu)
+                           .build();
+                    }
+                    // register the UI binding for header, corner and body region
+                    mouseEventMatcher = new MouseEventMatcher(
+                            SWT.NONE,
+                            null,
+                            MouseEventMatcher.RIGHT_BUTTON);
+                    uiBindingRegistry.registerMouseDownBinding(mouseEventMatcher,
+                            new PopupMenuAction(e4Menu));
+                }
+            });
+
     }
 
     void freezeSupplementalColumns(boolean freeze){
@@ -619,30 +677,31 @@ public class CharacterMatrix extends Composite {
         }
     }
 
-    private void initLabels(int index, Feature feature) {
+    private void initLabels(int index, FeatureDto feature) {
         indexToFeatureMap.put(index+LEADING_COLUMN_COUNT, feature);
 
-        String label = feature.getLabel();
+        String label = feature.getTitleCache();
         String property = feature.getUuid().toString();
         //show unit for quantitative data
         if(feature.isSupportsQuantitativeData()){
-            Set<MeasurementUnit> recommendedMeasurementUnits = feature.getRecommendedMeasurementUnits();
+            Set<TermDto> recommendedMeasurementUnits = feature.getRecommendedMeasurementUnits();
 //            if(recommendedMeasurementUnits.size()>1){
 //                MessagingUtils.warningDialog(Messages.CharacterMatrix_INIT_PROBLEM, CharacterMatrix.class,
 //                        String.format(Messages.CharacterMatrix_INIT_PROBLEM_MESSAGE, feature.getLabel()));
 //            }
             if(recommendedMeasurementUnits.size()==1){
-                MeasurementUnit unit = recommendedMeasurementUnits.iterator().next();
+                TermDto unit = recommendedMeasurementUnits.iterator().next();
                 label += " ["+unit.getIdInVocabulary()+"]"; //$NON-NLS-1$ //$NON-NLS-2$
             }
         }
         propertyToLabelMap.put(property, label);
     }
 
-    public void loadDescriptions(UUID descriptiveDataSetUuid, boolean isInitialExpandToDeepestTaxonLevel) {
-        UUID monitorUuid =  CdmApplicationState.getLongRunningTasksService().monitGetRowWrapper(descriptiveDataSetUuid);
+    public void loadDescriptions(boolean isInitialExpandToDeepestTaxonLevel, boolean initialLoading) {
+       
+        UUID monitorUuid =  CdmApplicationState.getLongRunningTasksService().monitGetRowWrapper(this.getDescriptiveDataSet().getUuid());
 
-        final Collection<RowWrapperDTO> wrappers = new ArrayList<>();
+        final Collection<RowWrapperDTO<?>> wrappers = new ArrayList<>();
         String jobLabel = Messages.CharacterMatrix_LOAD_CHARACTER_DATA;
         Job job = Job.create(jobLabel, (ICoreRunnable) monitor -> {
             SubMonitor subMonitor = SubMonitor.convert(monitor);
@@ -654,7 +713,7 @@ public class CharacterMatrix extends Composite {
                         monitorUuid,
                         50,
                         null,
-                        (List)null,
+                        (List<IFeedbackGenerator>)null,
                         subMonitor);
             } catch (InterruptedException e) {
                 MessagingUtils.informationDialog(Messages.CharacterMatrix_LOADING_FAILED_TITLE,
@@ -663,7 +722,7 @@ public class CharacterMatrix extends Composite {
             }
             Object result = remotingMonitor.getResult();
             if(result instanceof Collection){
-                wrappers.addAll((Collection<RowWrapperDTO>) result);
+                wrappers.addAll((Collection<RowWrapperDTO<?>>) result);
             }
             if(result instanceof Exception){
                 MessagingUtils.errorDialog("Exception during description loading", this.getClass(), "An exception occured during loading", TaxeditorEditorPlugin.PLUGIN_ID, (Throwable) result, true);
@@ -690,8 +749,11 @@ public class CharacterMatrix extends Composite {
                                 );
                     }
                     descriptions.clear();
+//                    part.setDescriptiveDataSet(CdmStore.getService(IDescriptiveDataSetService.class).getDescriptiveDataSetDtoByUuid(part.getDescriptiveDataSet().getUuid()));
                     wrappers.stream().filter(row->row.getTaxonNode()!=null).forEach(wrapper->CharacterMatrix.this.descriptions.add(wrapper));
-                    loadingDone(isInitialExpandToDeepestTaxonLevel);
+                    if(initialLoading){
+                        loadingDone(isInitialExpandToDeepestTaxonLevel);
+                    }
                 });
             }
         });
@@ -712,13 +774,22 @@ public class CharacterMatrix extends Composite {
     private void loadingDone(boolean isInitialExpandToDeepestTaxonLevel) {
         this.part.loadingDone();
         createTable(isTreeView, freezeLayer.isFrozen(), isInitialExpandToDeepestTaxonLevel);
+
+    }
+
+    public List<TermDto> getSupportedStatesForCategoricalFeature(UUID featureUuid){
+        return categoricalFeatureToStateMap.get(featureUuid);
     }
 
-    public List<TermDto> getSupportedStatesForCategoricalFeature(Feature feature){
-        return categoricalFeatureToStateMap.get(feature);
+    public Set<DescriptionBase<?>> getDescriptionsToSave() {
+        return descriptionsToSave;
     }
 
-    public Map<Integer, Feature> getIndexToFeatureMap() {
+    public void addDescriptionToSave(DescriptionBase<?> descriptionToSave) {
+        this.descriptionsToSave.add(descriptionToSave);
+    }
+
+    public Map<Integer, FeatureDto> getIndexToFeatureMap() {
         return indexToFeatureMap;
     }
 
@@ -730,34 +801,40 @@ public class CharacterMatrix extends Composite {
         part.setDirty();
     }
 
-    public List<UUID> getDescriptionsToDelete() {
+    public Map<RemoveDescriptionsFromDescriptiveDataSetConfigurator, List<UUID>> getDescriptionsToDelete() {
         return descriptionUuidsToDelete;
     }
 
-    public void addDescriptionToDelete(UUID descriptionToDelete) {
+    public void addDescriptionToDelete(UUID descriptionToDelete, RemoveDescriptionsFromDescriptiveDataSetConfigurator config) {
         if (descriptionUuidsToDelete == null){
-            descriptionUuidsToDelete = new ArrayList();
+            descriptionUuidsToDelete = new HashMap<>();
+        }
+        if (descriptionUuidsToDelete.get(config) != null){
+            descriptionUuidsToDelete.get(config).add(descriptionToDelete);
+        }else{
+            List<UUID> uuidList = new ArrayList<>();
+            uuidList.add(descriptionToDelete);
+            this.descriptionUuidsToDelete.put(config, uuidList);
         }
 
-        this.descriptionUuidsToDelete.add(descriptionToDelete);
 
     }
 
-    public List<SpecimenNodeWrapper> getSpecimenToAdd() {
+    public List<SpecimenRowWrapperDTO> getSpecimenToAdd() {
         return specimenToAdd;
     }
 
-    public void addSpecimenToAdd(Collection<SpecimenNodeWrapper> specimenToAdd) {
+    public void addSpecimenToAdd(Collection<SpecimenRowWrapperDTO> specimenToAdd) {
         if (this.specimenToAdd == null){
-            this.specimenToAdd = new ArrayList();
+            this.specimenToAdd = new ArrayList<>();
         }
 
         this.specimenToAdd.addAll(specimenToAdd);
         this.specimenCache.removeAll(specimenToAdd);
     }
-    public void addSpecimenToAdd(SpecimenNodeWrapper specimenToAdd) {
+    public void addSpecimenToAdd(SpecimenRowWrapperDTO specimenToAdd) {
         if (this.specimenToAdd == null){
-            this.specimenToAdd = new ArrayList();
+            this.specimenToAdd = new ArrayList<>();
         }
 
         this.specimenToAdd.add(specimenToAdd);
@@ -776,11 +853,11 @@ public class CharacterMatrix extends Composite {
         return descriptions;
     }
 
-    public DescriptiveDataSet getDescriptiveDataSet() {
+    public DescriptiveDataSetBaseDto getDescriptiveDataSet() {
         return part.getDescriptiveDataSet();
     }
 
-    public void setDescriptiveDataSet(DescriptiveDataSet dataSet) {
+    public void setDescriptiveDataSet(DescriptiveDataSetBaseDto dataSet) {
         part.setDescriptiveDataSet(dataSet);
     }
 
@@ -794,7 +871,7 @@ public class CharacterMatrix extends Composite {
         //map descriptions on a list of uuids of the described specimen
         !this.descriptions.stream()
         .filter(rowWrapper->rowWrapper instanceof SpecimenRowWrapperDTO)
-        .map(specimenRowWrapper->((SpecimenRowWrapperDTO) specimenRowWrapper).getSpecimen().getUuid())
+        .map(specimenRowWrapper->((SpecimenRowWrapperDTO) specimenRowWrapper).getSpecimenDto().getUuid())
         .collect(Collectors.toList())
         //and check if the specimen to add is already contained
         .contains(wrapper.getUuidAndTitleCache().getUuid())
@@ -802,12 +879,23 @@ public class CharacterMatrix extends Composite {
         .collect(Collectors.toList());
     }
 
-    public void addRowToSave(RowWrapperDTO<?> row){
-        rowsToSave.add(row);
+//    public void addRowToSave(RowWrapperDTO<?> row){
+//        rowsToMerge.put(row.getDescription().getDescription().getUuid(), row);
+//    }
+//
+//    public HashMap<UUID,RowWrapperDTO<?>> getRowsToSave() {
+//        return rowsToMerge;
+//    }
+
+    public HashMap<UUID, RowWrapperDTO<?>> getRowsToMerge() {
+        return rowsToMerge;
     }
 
-    public Collection<RowWrapperDTO<?>> getRowsToSave() {
-        return rowsToSave;
+    public void putRowToMerge(RowWrapperDTO<?> rowToMerge) {
+        if (this.rowsToMerge == null){
+            this.rowsToMerge = new HashMap<>();
+        }
+        this.rowsToMerge.put(rowToMerge.getDescription().getDescriptionUuid(), rowToMerge);
     }
 
     public Properties getNatTableState() {
@@ -826,19 +914,17 @@ public class CharacterMatrix extends Composite {
         return new File(WorkbenchUtility.getBaseLocation(), CHARACTER_MATRIX_STATE_PROPERTIES);
     }
 
-    public List<Feature> getFeatures() {
+    public List<FeatureDto> getFeatures() {
         return features;
     }
 
-    public Map<Feature, CategoricalDataHistogram> getFeatureToHistogramMap() {
+    public Map<FeatureDto, CategoricalDataHistogram> getFeatureToHistogramMap() {
         return featureToHistogramMap;
     }
 
-    public Map<Feature, QuantitativeDataStatistics> getFeatureToQuantDataStatisticsMap() {
-        return featureToQuantDataStatisticsMap;
-    }
 
-    public ICdmEntitySession getCdmEntitiySession(){
+
+    public ICdmEntitySession getCdmEntitySession(){
         return part.getCdmEntitySession();
     }
 
@@ -852,17 +938,17 @@ public class CharacterMatrix extends Composite {
     }
 
     public void addRowsToMatrix(Collection<SpecimenNodeWrapper> wrappers){
-        addSpecimenToAdd(wrappers);
-        List<UUID> specimenUuids = new ArrayList();
+
+        List<UUID> specimenUuids = new ArrayList<>();
         wrappers.forEach(wrapper -> specimenUuids.add(wrapper.getUuidAndTitleCache().getUuid()));
 //        List<SpecimenOrObservationBase> specimens = CdmStore.getService(IOccurrenceService.class).load(specimenUuids, null);
-        Map<UUID, UuidAndTitleCache<FieldUnit>> specimenMap = new HashMap();
+        Map<UUID, UuidAndTitleCache<FieldUnit>> specimenMap = new HashMap<>();
 
         for (UUID specimenUuid: specimenUuids ){
             try{
                 FieldUnitDTO fieldUnitDto = CdmStore.getService(IOccurrenceService.class).loadFieldUnitDTO(specimenUuid);
                 if (fieldUnitDto != null){
-                    UuidAndTitleCache<FieldUnit> fieldUnit = new UuidAndTitleCache(FieldUnit.class, fieldUnitDto.getUuid(), null, fieldUnitDto.getTitleCache());
+                    UuidAndTitleCache<FieldUnit> fieldUnit = new UuidAndTitleCache<>(FieldUnit.class, fieldUnitDto.getUuid(), null, fieldUnitDto.getLabel());
                     specimenMap.put(specimenUuid, fieldUnit);
                 }
             }catch(Exception e){
@@ -872,60 +958,114 @@ public class CharacterMatrix extends Composite {
         }
         for (SpecimenNodeWrapper wrapper: wrappers){
             SpecimenRowWrapperDTO rowWrapper = CdmStore.getService(IDescriptiveDataSetService.class).createSpecimenRowWrapper(wrapper.getUuidAndTitleCache().getUuid(), wrapper.getTaxonNode().getUuid(), getDescriptiveDataSet().getUuid());
+            addSpecimenToAdd(rowWrapper);
 //            SpecimenRowWrapperDTO rowWrapper = new SpecimenRowWrapperDTO(wrapper.getUuidAndTitleCache(), wrapper.getType(), wrapper.getTaxonNode(), specimenMap.get(wrapper.getUuidAndTitleCache().getUuid()), null, null);
             descriptions.add(rowWrapper);
         }
-        loadingDone(false);
+
 
         setDirty();
     }
 
-    public void addSpecimensToDescriptiveDataSet(){
+    public HashMap<UUID, DescriptionBase<?>> addSpecimensToDescriptiveDataSet(){
         if (specimenToAdd == null || specimenToAdd.isEmpty()){
-            return;
+            return new HashMap<>();
         }
-        UUID monitorUuid =  CdmApplicationState.getLongRunningTasksService().addRowWrapperToDataset(specimenToAdd, getDescriptiveDataSet().getUuid());
-        specimenToAdd = null;
-        String jobLabel = "Add specimens to matrix";
-        Job job = Job.create(jobLabel, (ICoreRunnable) monitor -> {
-            SubMonitor subMonitor = SubMonitor.convert(monitor);
-            subMonitor.beginTask(jobLabel, IProgressMonitor.UNKNOWN);
-            IRemotingProgressMonitor remotingMonitor;
-            try {
-                remotingMonitor = CdmStore.getProgressMonitorClientManager()
-                        .pollMonitor(jobLabel,
-                                monitorUuid,
-                                50,
-                                null,
-                                (List)null,
-                                subMonitor);
-                Object resultObject = remotingMonitor.getResult();
-                if(resultObject instanceof Exception){
-                    MessagingUtils.errorDialog("Adding specimens failed", this, "Adding specimens was not successfull", TaxeditorEditorPlugin.PLUGIN_ID, (Exception)resultObject, true, true);
-                }
-                else if(resultObject instanceof UpdateResult){
-                    UpdateResult result = (UpdateResult)resultObject;
-                    if(!result.getExceptions().isEmpty()){
-                        MessagingUtils.warningDialog(Messages.CharacterMatrixBottomToolbar_ERROR_ROW_CREATION_TITLE, this,
-                                String.format(Messages.CharacterMatrixBottomToolbar_ERROR_ROW_CREATION_MESSAGE, result.getExceptions()
-                                        .stream().map(ex->ex.toString())
-                                        .collect(Collectors.joining("\n"))));
-                    }
-                    DescriptiveDataSet dataSet = (DescriptiveDataSet) result.getCdmEntity();
-                    dataSet = this.getCdmEntitiySession().load(dataSet, true);
-                    // update local dataset
-                    this.setDescriptiveDataSet(dataSet);
-                    this.loadDescriptions(getDescriptiveDataSet().getUuid(), false);
+        UpdateResult result = CdmStore.getService(IDescriptiveDataSetService.class).addRowWrapperToDataset(specimenToAdd, getDescriptiveDataSet().getUuid());
+        if(!result.getExceptions().isEmpty()){
+            MessagingUtils.warningDialog(Messages.CharacterMatrixBottomToolbar_ERROR_ROW_CREATION_TITLE, this,
+                    String.format(Messages.CharacterMatrixBottomToolbar_ERROR_ROW_CREATION_MESSAGE, result.getExceptions()
+                            .stream().map(ex->ex.toString())
+                            .collect(Collectors.joining("\n"))));
+        }
+        DescriptiveDataSet dataSet = (DescriptiveDataSet) result.getCdmEntity();
+        HashMap<UUID, DescriptionBase<?>> resultMap = new HashMap<>();
+        for (CdmBase updated: result.getUpdatedObjects()){
+            if (updated instanceof SpecimenDescription){
+                resultMap.put(updated.getUuid(), (DescriptionBase<?>)updated);
+            }
 
 
-                }
-            } catch (InterruptedException e) {
-                return;
-            }
-            monitor.done();
-        });
+        }
+        dataSet = this.getCdmEntitySession().load(dataSet, true);
+        // update local dataset
+        DescriptiveDataSetBaseDto dto = DescriptiveDataSetBaseDto.fromDescriptiveDataSet(dataSet);
+        this.setDescriptiveDataSet(dto);
+
+        //these descriptions are already updated
+        for (SpecimenRowWrapperDTO row: specimenToAdd){
+            this.rowsToMerge.remove(row.getDescription().getDescriptionUuid());
+        }
+        specimenToAdd.clear();
+        return resultMap;
+
+    }
+    public void setSelectedObject(Object selectedObject){
+
+        int rowIndex = ((IRowDataProvider<Object>)bodyDataProvider).indexOfRowObject(selectedObject);
+       int rowPosition = selectionLayer.getRowPositionByIndex(rowIndex);
+       natTable.doCommand( new SelectRowsCommand(natTable, 1, rowPosition, false, false));
+    }
+
+    public TreeList<Object> updateDescriptions(boolean treeView){
+       SortedList<Object> sortedList = new SortedList<>(descriptions, new MatrixRowComparator());
+        // wrap the SortedList with the TreeList
+        treeFormat = new DescriptionTreeFormat(getDescriptiveDataSet());
+        TreeList<Object> treeList = new TreeList<Object>(sortedList, treeFormat, TreeList.NODES_START_COLLAPSED);
+        // wrap the SortedList with the TreeList
+//        treeFormat = new DescriptionTreeFormat(getDescriptiveDataSet());
+
+        /**
+         * data provider
+         */
+        SpecimenColumnPropertyAccessor columnPropertyAccessor = new SpecimenColumnPropertyAccessor(this);
+        bodyDataProvider = treeView?new ListDataProvider<>(treeList, columnPropertyAccessor):new ListDataProvider<>(sortedList, columnPropertyAccessor);
+        return treeList;
+
+    }
+
+
+    @Inject
+    @Optional
+    private void updateView(@UIEventTopic(WorkbenchEventConstants.ADD_TO_MERGE_ROWS) UUID uuid) {
+
+    }
+
+    @Override
+    public void dispose () {
+        super.dispose();
+        if (descriptions != null){
+               descriptions.dispose();
+        }
+        descriptions = null;
+        this.natTable.dispose();
+        this.natTable = null;
+        categoricalFeatureToStateMap = null;
+        rowsToMerge.clear();
+        rowsToMerge = null;
+        indexToFeatureMap = null;
+        propertyToLabelMap = null;
+        specimenCache = null;
+        featureToHistogramMap = null;
+        bodyDataProvider = null;
+        freezeLayer = null;
+        features = null;
+        topMostLayer.dispose();
+        topMostLayer = null;
+        configRegistry = null;
+        bodyLayer.dispose();
+        bodyLayer = null;
+        toolbar.dispose();
+        toolbar = null;
+        treeFormat = null;
+        descriptionUuidsToDelete = null;
+        specimenToAdd = null;
+        mouseEventMatcher = null;
+        descriptionsToSave = null;
+        menuService = null;
+        sync = null;
+        part = null;
 
-        job.schedule();
     }
 
 }