From: Katja Luther <k.luther@bgbm.org>
Date: Wed, 2 Dec 2020 17:28:21 +0000 (+0100)
Subject: ref #9199: layout issues and code cleaning
X-Git-Tag: 5.18.0~1^2~16
X-Git-Url: https://dev.e-taxonomy.eu/gitweb/taxeditor.git/commitdiff_plain/387aef2a775a74295ca4c91fd1d0839c02656210

ref #9199: layout issues and code cleaning
---

diff --git a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/common/ExternalLinksSection.java b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/common/ExternalLinksSection.java
index ab4e30c35..93441847f 100755
--- a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/common/ExternalLinksSection.java
+++ b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/common/ExternalLinksSection.java
@@ -141,11 +141,6 @@ public class ExternalLinksSection extends AbstractEntityCollectionSection<Origin
     }
 
 
-    public String getEmptySectionString() {
-        return emptySectionString;
-    }
-
-
     public void setEmptySectionString(String emptySectionString) {
         this.emptySectionString = emptySectionString;
     }
diff --git a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedNomenclaturalSourceElement.java b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedNomenclaturalSourceElement.java
index 18494ff37..bb08cef7b 100755
--- a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedNomenclaturalSourceElement.java
+++ b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedNomenclaturalSourceElement.java
@@ -38,7 +38,10 @@ public class AdvancedNomenclaturalSourceElement extends AdvancedSourceElement {
     protected void createControls(ICdmFormElement formElement, OriginalSourceBase entity, int style) {
         Display display = Display.getCurrent();
         Color background = display.getSystemColor(SWT.COLOR_WIDGET_BACKGROUND);
+        text_originaleNameString = formFactory.createTextWithLabelElement(
+                formElement, "Original Information", entity != null?entity.getOriginalNameString():null, SWT.NULL);
 
+        text_originaleNameString.setBackground(background);
 
         if (PreferencesUtil.getBooleanValue(PreferencePredicate.ShowNamespaceInSource.getKey())){
             text_idNamespace = formFactory.createTextWithLabelElement(formElement, "ID Namespace", entity != null?entity.getIdNamespace():null, style);
@@ -49,11 +52,6 @@ public class AdvancedNomenclaturalSourceElement extends AdvancedSourceElement {
             text_idInSource.setBackground(background);
 
         }
-
-        text_originaleNameString = formFactory.createTextWithLabelElement(
-                formElement, "Original Information", entity != null?entity.getOriginalNameString():null, SWT.NULL);
-
-        text_originaleNameString.setBackground(background);
         Label label = formFactory.createEmptyCell(getLayoutComposite());
     }
 
diff --git a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedSourceElement.java b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedSourceElement.java
index ae8dd82e0..7578a92e1 100755
--- a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedSourceElement.java
+++ b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/AdvancedSourceElement.java
@@ -45,6 +45,11 @@ public class AdvancedSourceElement extends AbstractCdmDetailElement<OriginalSour
     protected void createControls(ICdmFormElement formElement, OriginalSourceBase entity, int style) {
         Display display = Display.getCurrent();
         Color background = display.getSystemColor(SWT.COLOR_WIDGET_BACKGROUND);
+        text_originaleNameString = formFactory.createTextWithLabelElement(
+                formElement, "Original Information", entity != null?entity.getOriginalNameString():null, SWT.NULL);
+
+        text_originaleNameString.setBackground(background);
+
         if (entity instanceof DescriptionElementSource){
             select_nameUsedInSource = formFactory.createSelectionElement(TaxonName.class, formElement, "Name in Source", entity != null? ((DescriptionElementSource)entity).getNameUsedInSource(): null, EntitySelectionElement.DELETABLE, style);
             select_nameUsedInSource.setBackground(background);
@@ -60,20 +65,6 @@ public class AdvancedSourceElement extends AbstractCdmDetailElement<OriginalSour
 
         }
 
-        text_originaleNameString = formFactory.createTextWithLabelElement(
-                formElement, "Original Information", entity != null?entity.getOriginalNameString():null, SWT.NULL);
-
-        text_originaleNameString.setBackground(background);
-
-
-
-//        externalLinks = formFactory.createExternalLinksSection(getConversationHolder(), formElement, StoreUtil.getSectionStyle(ExternalLinksSection.class, DescriptionElementSource.class.getCanonicalName()));
-//        externalLinks.setLayoutData(LayoutConstants.FILL_HORIZONTALLY(2, 3));
-//        externalLinks.setEntity(entity);
-//
-//        externalLinks.setBackground(background);
-//        addControl(externalLinks);
-//        addElement(externalLinks);
     }
 
     @Override
diff --git a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/NomenclaturalSourceElement.java b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/NomenclaturalSourceElement.java
index fa19bb626..02c734584 100755
--- a/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/NomenclaturalSourceElement.java
+++ b/eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/reference/NomenclaturalSourceElement.java
@@ -24,7 +24,6 @@ import eu.etaxonomy.taxeditor.store.StoreUtil;
 import eu.etaxonomy.taxeditor.ui.element.CdmFormFactory;
 import eu.etaxonomy.taxeditor.ui.element.ICdmFormElement;
 import eu.etaxonomy.taxeditor.ui.element.LayoutConstants;
-import eu.etaxonomy.taxeditor.ui.element.TextWithLabelElement;
 import eu.etaxonomy.taxeditor.ui.section.common.ExternalLinksSection;
 import eu.etaxonomy.taxeditor.ui.selection.EntitySelectionElement;
 import eu.etaxonomy.taxeditor.ui.selection.EntitySelectionElementWithAbbreviatedTitle;
@@ -36,19 +35,10 @@ import eu.etaxonomy.taxeditor.ui.selection.EntitySelectionElementWithAbbreviated
 public class NomenclaturalSourceElement extends OriginalSourceElement {
 
     private EntitySelectionElementWithAbbreviatedTitle<Reference> selection_NomRef;
-
-//    protected EntitySelectionElement<Reference> selection_Ref;
-//    protected TextWithLabelElement microReference;
-//    protected OriginalSourceAdvancedSection advancedSection;
     protected String label = "Source";
     protected CdmBase cdmEntity;
 
-    protected TextWithLabelElement text_idInSource;
-    protected TextWithLabelElement text_idNamespace;
-    protected TextWithLabelElement text_originaleNameString;
-    protected TextWithLabelElement text_cdmsource;
     protected EntitySelectionElement<TaxonName> select_nameUsedInSource;
-
     protected ExternalLinksSection externalLinks;
 
 
@@ -84,7 +74,6 @@ public class NomenclaturalSourceElement extends OriginalSourceElement {
         }
 
         microReference = formFactory.createTextWithLabelElement(formElement, "Details", entity != null? entity.getCitationMicroReference(): "", style);
-//        microReference.setIndent(15);
 
         Label sep = formFactory.createLabel(formElement.getLayoutComposite(), "");
         sep.setLayoutData(LayoutConstants.FILL_HORIZONTALLY(2, 1));
@@ -92,10 +81,11 @@ public class NomenclaturalSourceElement extends OriginalSourceElement {
         select_nameUsedInSource = formFactory.createSelectionElement(TaxonName.class, formElement, "Original Spelling", entity != null? ((DescriptionElementSource)entity).getNameUsedInSource(): null, EntitySelectionElement.ALL, style);
 
         externalLinks = formFactory.createExternalLinksSection(getConversationHolder(), formElement, "Protologue/Original Publication", StoreUtil.getSectionStyle(ExternalLinksSection.class, DescriptionElementSource.class.getCanonicalName()));
+        externalLinks.setEmptySectionString("No protologue yet.");
         externalLinks.setLayoutData(LayoutConstants.FILL_HORIZONTALLY(2, 3));
         externalLinks.setEntity(entity);
         externalLinks.setFont(AbstractUtility.getFont(Resources.COLOR_FONT_DEFAULT));
-        externalLinks.setEmptySectionString("No protologue yet.");
+
 
         addControl(externalLinks);
         addElement(externalLinks);
@@ -109,8 +99,6 @@ public class NomenclaturalSourceElement extends OriginalSourceElement {
         advancedSection.setFont(AbstractUtility.getFont(Resources.COLOR_FONT_DEFAULT));
         addControl(advancedSection);
         addElement(advancedSection);
-
-
         this.getLayoutComposite().layout();
 
     }