From: Patrick Plitzner <p.plitzner@bgbm.org>
Date: Wed, 3 Feb 2016 16:59:49 +0000 (+0100)
Subject: Add FIXME comment
X-Git-Tag: 4.0.0^2~150
X-Git-Url: https://dev.e-taxonomy.eu/gitweb/taxeditor.git/commitdiff_plain/198106815dadfac611b4f20fe0ca8a6674a30b1f

Add FIXME comment
---

diff --git a/eu.etaxonomy.taxeditor.bulkeditor/src/main/java/eu/etaxonomy/taxeditor/annotatedlineeditor/AnnotatedLineEditor.java b/eu.etaxonomy.taxeditor.bulkeditor/src/main/java/eu/etaxonomy/taxeditor/annotatedlineeditor/AnnotatedLineEditor.java
index e1208b01c..e7640f011 100644
--- a/eu.etaxonomy.taxeditor.bulkeditor/src/main/java/eu/etaxonomy/taxeditor/annotatedlineeditor/AnnotatedLineEditor.java
+++ b/eu.etaxonomy.taxeditor.bulkeditor/src/main/java/eu/etaxonomy/taxeditor/annotatedlineeditor/AnnotatedLineEditor.java
@@ -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);
         }