Add FIXME comment
authorPatrick Plitzner <p.plitzner@bgbm.org>
Wed, 3 Feb 2016 16:59:49 +0000 (17:59 +0100)
committerPatrick Plitzner <p.plitzner@bgbm.org>
Tue, 16 Feb 2016 14:14:49 +0000 (15:14 +0100)
eu.etaxonomy.taxeditor.bulkeditor/src/main/java/eu/etaxonomy/taxeditor/annotatedlineeditor/AnnotatedLineEditor.java

index e1208b01c5083936c05f265642af25d8bcf989df..e7640f011c211fac76387e65b040e4cc0c0d4575 100644 (file)
@@ -164,6 +164,9 @@ public class AnnotatedLineEditor extends TextEditor implements IConversationEnab
         // to create / save the entity may have been closed
         if(entity != null && ((AnnotatedLineDocumentProvider) getDocumentProvider()).getEntityCreator(getEditorInput()).savesEntity()) {
             getConversationHolder().bind();
+            //FIXME: why do we use IOccurrenceService here? is this generic?
+            //because this method is only invoked by the handler which is used in the
+            //specimen bulk editor. This is unsafe and should be refactored!
             Object object = CdmStore.getService(IOccurrenceService.class).load(((CdmBase)entity).getUuid());
             entity = HibernateProxyHelper.deproxy(object);
         }