Changeset 36230

Timestamp:

2022-05-31T11:07:49+12:00 (23 months ago)

Author:

kjdon

Message:

previously we were removing metadata from the fragmenttablemodel by index - but the metadata arraylist is not in the same order as teh table rows - table ordered by language - so we need to remove the metadata by passing the object not the index

File:

• main/trunk/gli/src/org/greenstone/gatherer/cdm/TranslationView.java (modified) (2 diffs)

main/trunk/gli/src/org/greenstone/gatherer/cdm/TranslationView.java

@@ -518 +518 @@
             CollectionMeta metadata = fragment_table_model.getMetadata(index);
             if (metadata instanceof SearchMeta) {
-            CollectionDesignManager.searchmeta_manager.removeMetadata(metadata);     
+                CollectionDesignManager.searchmeta_manager.removeMetadata(metadata);     
             } else {
-            CollectionDesignManager.collectionmeta_manager.removeMetadata(metadata);
-            }
-            fragment_table_model.remove(index);
+                CollectionDesignManager.collectionmeta_manager.removeMetadata(metadata);
+            }
+                    fragment_table_model.remove(metadata);
             // If a remove occured then enable add
             add_button.setEnabled(true);
@@ -759 +759 @@
         }
 
-        public void remove(int index) {
-        metadata.remove(index);
-        fireTableRowsDeleted(index, index);
-        }
+          // we can't remove by index, as the table index doesn't relate to the index in the metadata list
+          public void remove(CollectionMeta meta) {
+            metadata.remove(meta);
+          }
+          
 
         public void setData(String feature_name, String feature_type, TreeSet languages, ArrayList metadata) {