ref #8147 Code cleanup
authorPatrick Plitzner <p.plitzner@bgbm.org>
Thu, 4 Apr 2019 16:05:36 +0000 (18:05 +0200)
committerPatrick Plitzner <p.plitzner@bgbm.org>
Thu, 4 Apr 2019 16:06:30 +0000 (18:06 +0200)
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/vocabulary/TermTranslationWizard.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/translation/TermTranslationWizardPage.java

index 18316c833183649ea60a42d6c29ee7254432f739..d5e2f8eac25b3d72d2fa456da53bffb946a367cd 100644 (file)
@@ -25,31 +25,18 @@ public class TermTranslationWizard extends Wizard {
 
     private final TermBase term;
 
-    /**
-     * <p>Constructor for TranslationWizard.</p>
-     *
-     * @param multilanguageText a {@link java.util.Map} object.
-     */
     public TermTranslationWizard(TermBase term) {
         setWindowTitle("Translation Editor");
         this.term = term;
         setNeedsProgressMonitor(true);
     }
 
-    /* (non-Javadoc)
-     * @see org.eclipse.jface.wizard.Wizard#addPages()
-     */
-    /** {@inheritDoc} */
     @Override
     public void addPages() {
         page = new TermTranslationWizardPage(term);
         addPage(page);
     }
 
-    /* (non-Javadoc)
-     * @see org.eclipse.jface.wizard.Wizard#performFinish()
-     */
-    /** {@inheritDoc} */
     @Override
     public boolean performFinish() {
         Representation representation = page.getTargetRepresentation();
@@ -67,9 +54,6 @@ public class TermTranslationWizard extends Wizard {
         return true;
     }
 
-    /**
-     * {@inheritDoc}
-     */
     @Override
     public boolean canFinish() {
         Representation representation = page.getTargetRepresentation();
index b0062dbe8b83d6717b3df6f742e79b8b84188f4d..502573a66bb7a110e2f146e134d41533f3e81db8 100644 (file)
@@ -33,12 +33,11 @@ import eu.etaxonomy.taxeditor.ui.element.RootElement;
  */
 public class TermTranslationWizardPage extends WizardPage implements IPropertyChangeListener{
 
-
         private static int TEXT_HEIGHT = 200;
 
         private final TermBase term;
 
-        MultilanguageTextElement source;
+        private MultilanguageTextElement source;
 
         private final CdmFormFactory formFactory;
 
@@ -46,11 +45,6 @@ public class TermTranslationWizardPage extends WizardPage implements IPropertyCh
 
         private RepresentationElement text_target;
 
-        /**
-         * <p>Constructor for TranslationWizardPage.</p>
-         *
-         * @param multilanguageText a {@link java.util.Map} object.
-         */
         public TermTranslationWizardPage(TermBase term) {
             super("TranslationWizardPage");
             //Assert.isNotNull(multilanguageText, "Multilanguage text may not be null");
@@ -61,57 +55,38 @@ public class TermTranslationWizardPage extends WizardPage implements IPropertyCh
             setPageComplete(false);
         }
 
-        /* (non-Javadoc)
-         * @see org.eclipse.jface.dialogs.IDialogPage#createControl(org.eclipse.swt.widgets.Composite)
-         */
-        /** {@inheritDoc} */
         @Override
         public void createControl(Composite parent) {
             Composite control = formFactory.createComposite(parent);
-//          control.setBackground(Display.getCurrent().getSystemColor(SWT.COLOR_GREEN));
             control.setLayout(LayoutConstants.LAYOUT(2, false));
 
             Composite leftContainer = formFactory.createComposite(control);
             leftContainer.setLayoutData(LayoutConstants.FILL(1, 4));
             leftContainer.setLayout(LayoutConstants.LAYOUT(2, false));
-//          leftContainer.setBackground(Display.getCurrent().getSystemColor(SWT.COLOR_BLUE));
             RootElement leftRoot = new RootElement(formFactory, leftContainer);
 
             Composite rightContainer = formFactory.createComposite(control);
             rightContainer.setLayoutData(LayoutConstants.FILL(1, 4));
             rightContainer.setLayout(LayoutConstants.LAYOUT(2, false));
-//          rightContainer.setBackground(Display.getCurrent().getSystemColor(SWT.COLOR_RED));
             RootElement rightRoot = new RootElement(formFactory, rightContainer);
 
             createLeftControls(leftRoot);
             createRightControls(rightRoot);
 
-
             // initialize with default language
             setSource(CdmStore.getDefaultLanguage());
 
-
             setControl(control);
             setPageComplete(true);
         }
 
-        /**
-         * @param defaultLanguage
-         */
         private void setSource(Language language) {
-
            Representation preferredRepresentation =term.getPreferredRepresentation(language);
-
             if(preferredRepresentation != null){
                text_source.setSelectedRepresentation(preferredRepresentation);
             }
         }
 
-
-        /**
-         *
-         * @param element
-         */
         private void createLeftControls(ICdmFormElement element){
             Representation preferredRepresentation= term.getPreferredRepresentation(PreferencesUtil.getGlobalLanguage());
 
@@ -120,43 +95,23 @@ public class TermTranslationWizardPage extends WizardPage implements IPropertyCh
 
         }
 
-        /**
-         *
-         * @param element
-         */
         private void createRightControls(ICdmFormElement element){
             text_target = formFactory.createRepresentationElement(element, null, term, TEXT_HEIGHT, SWT.NULL, false);
-
         }
 
-
-
-
-        /* (non-Javadoc)
-         * @see org.eclipse.jface.dialogs.DialogPage#dispose()
-         */
-        /** {@inheritDoc} */
         @Override
         public void dispose() {
             super.dispose();
         }
 
-        /**
-         * @return
-         *
-         */
         public Representation getTargetRepresentation() {
             return text_target.getSelectedRepresentation();
         }
 
-        /**
-        * {@inheritDoc}
-        */
         @Override
         public void propertyChange(org.eclipse.jface.util.PropertyChangeEvent event) {
             if (event.getSource() == text_target){
                 getWizard().getContainer().updateButtons();
             }
         }
-
 }