refactored NumberWithLabelElement to a single constructor with java.lang.Number as...
authorCherian Mathew <c.mathew@bgbm.org>
Wed, 10 Jul 2013 14:28:00 +0000 (14:28 +0000)
committerCherian Mathew <c.mathew@bgbm.org>
Wed, 10 Jul 2013 14:28:00 +0000 (14:28 +0000)
12 files changed:
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/element/CdmFormFactory.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/element/NumberWithLabelElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/element/PartialElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/element/PointElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/description/StatisticalMeasurementValueElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/key/PolytomousKeyNodeDetailElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/media/ImageFileElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/media/MediaRepresentationPartElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/name/NameDetailElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/occurrence/FieldObservationDetailElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/occurrence/GatheringEventDetailElement.java
eu.etaxonomy.taxeditor.store/src/main/java/eu/etaxonomy/taxeditor/ui/section/occurrence/GeneralDetailElement.java

index 1ff3d6d68a2ed42b22309a0ec5ee62fb2cf0999e..604d2c2c7b0ef6f79c5d2f3fdc95f721f6a52dca 100644 (file)
@@ -642,42 +642,16 @@ public class CdmFormFactory extends FormToolkit {
         * @return a {@link eu.etaxonomy.taxeditor.ui.element.NumberWithLabelElement}
         *         object.
         */
-       public NumberWithLabelElement createIntegerTextWithLabelElement(
+       public NumberWithLabelElement createNumberTextWithLabelElement(
                        ICdmFormElement parentElement, String labelString,
-                       Integer initialInteger, int style) {
+                       Number initialNumber, int style) {
                NumberWithLabelElement element = new NumberWithLabelElement(this,
-                               parentElement, labelString, initialInteger, style);
+                               parentElement, labelString, initialNumber, style);
                adapt(element);
                parentElement.addElement(element);
                return element;
        }
 
-       /**
-        * <p>
-        * createFloatTextWithLabelElement
-        * </p>
-        * 
-        * @param parentElement
-        *            a {@link eu.etaxonomy.taxeditor.ui.element.ICdmFormElement}
-        *            object.
-        * @param labelString
-        *            a {@link java.lang.String} object.
-        * @param initialFloat
-        *            a float.
-        * @param style
-        *            a int.
-        * @return a {@link eu.etaxonomy.taxeditor.ui.element.NumberWithLabelElement}
-        *         object.
-        */
-       public NumberWithLabelElement createFloatTextWithLabelElement(
-                       ICdmFormElement parentElement, String labelString,
-                       float initialFloat, int style) {
-               NumberWithLabelElement element = new NumberWithLabelElement(this,
-                               parentElement, labelString, initialFloat, style);
-               adapt(element);
-               parentElement.addElement(element);
-               return element;
-       }
 
        /**
         * <p>
index eab43db6772d2cbfd646e48ef84aa47a030f5540..3b58a586b90f7ada47f4ce445c73d64109437bb4 100644 (file)
@@ -37,47 +37,23 @@ public class NumberWithLabelElement extends TextWithLabelElement {
         * @param toolkit a {@link eu.etaxonomy.taxeditor.ui.element.CdmFormFactory} object.
         * @param parentElement a {@link eu.etaxonomy.taxeditor.ui.element.ICdmFormElement} object.
         * @param labelString a {@link java.lang.String} object.
-        * @param initialInteger a {@link java.lang.Integer} object.
+        * @param initialNumber a {@link java.lang.Number} object.
         * @param style a int.
         */
        public NumberWithLabelElement(CdmFormFactory toolkit,
                        ICdmFormElement parentElement, String labelString,
-                       Integer initialInteger, int style) {
+                       Number initialNumber, int style) {
                super(toolkit, parentElement, labelString, null, null, style);
-               setInteger(initialInteger);
+               setNumber(initialNumber);
        }
        
-       /**
-        * <p>Constructor for NumberWithLabelElement.</p>
-        *
-        * @param toolkit a {@link eu.etaxonomy.taxeditor.ui.element.CdmFormFactory} object.
-        * @param parentElement a {@link eu.etaxonomy.taxeditor.ui.element.ICdmFormElement} object.
-        * @param labelString a {@link java.lang.String} object.
-        * @param initialFloat a {@link java.lang.Float} object.
-        * @param style a int.
-        */
-       public NumberWithLabelElement(CdmFormFactory toolkit,
-                       ICdmFormElement parentElement, String labelString,
-                       Float initialFloat, int style) {
-               super(toolkit, parentElement, labelString, null, null, style);
-               setFloat(initialFloat);
-       }
-       
-       /**
-        * <p>setInteger</p>
-        *
-        * @param number a {@link java.lang.Integer} object.
-        */
-       public void setInteger(Integer number) {
-               super.setText(getStringRepresentation(number));
-       }
        
        /**
-        * <p>setFloat</p>
+        * <p>setNumber</p>
         *
-        * @param number a {@link java.lang.Float} object.
+        * @param number a {@link java.lang.Number} object.
         */
-       public void setFloat(Float number) {
+       public void setNumber(Number number) {
                super.setText(getStringRepresentation(number));
        }
        
@@ -101,6 +77,16 @@ public class NumberWithLabelElement extends TextWithLabelElement {
                return new Float(text);
        }
        
+       /**
+        * <p>getDouble</p>
+        *
+        * @return a {@link java.lang.Float} object.
+        */
+       public Float getDouble(){
+               String text = super.getText();
+               return new Float(text);
+       }
+       
        private String getStringRepresentation(Object number){
                if(number != null){
                        return number.toString();
index efcc8541c23fb5fdeaf708a8d7436a868decba81..72b26fcfcc05d890a115df9ca7732d737de9b565 100644 (file)
@@ -49,13 +49,13 @@ public class PartialElement extends AbstractCdmFormElement implements ISelectabl
                label = formFactory.createLabel(getLayoutComposite(), labelString);
                addControl(label);
                
-               number_year = formFactory.createIntegerTextWithLabelElement(this, "Year", null, style);
+               number_year = formFactory.createNumberTextWithLabelElement(this, "Year", null, style);
                number_year.setLimits(4, -9999, 9999);
                
-               number_month = formFactory.createIntegerTextWithLabelElement(this, "Month", null, style);               
+               number_month = formFactory.createNumberTextWithLabelElement(this, "Month", null, style);                
                number_month.setLimits(2, 1, 12);
                
-               number_day = formFactory.createIntegerTextWithLabelElement(this, "Day", null, style);
+               number_day = formFactory.createNumberTextWithLabelElement(this, "Day", null, style);
                number_day.setLimits(2, 1, 31);
                
                
@@ -71,9 +71,9 @@ public class PartialElement extends AbstractCdmFormElement implements ISelectabl
                this.partial = partial;
                
                if(partial != null){
-                       number_day.setInteger(TimePeriod.getPartialValue(partial, TimePeriod.DAY_TYPE));
-                       number_month.setInteger(TimePeriod.getPartialValue(partial, TimePeriod.MONTH_TYPE));
-                       number_year.setInteger(TimePeriod.getPartialValue(partial, TimePeriod.YEAR_TYPE));
+                       number_day.setNumber(TimePeriod.getPartialValue(partial, TimePeriod.DAY_TYPE));
+                       number_month.setNumber(TimePeriod.getPartialValue(partial, TimePeriod.MONTH_TYPE));
+                       number_year.setNumber(TimePeriod.getPartialValue(partial, TimePeriod.YEAR_TYPE));
                }
        }
 
index 6ed881b074fd789475e4bb170731b730c09d7d66..c6fbd056f10c04187faa9b57b33ed3117047f402 100644 (file)
@@ -74,7 +74,7 @@ public class PointElement extends AbstractCdmFormElement implements
                text_longitudeParsed = formFactory.createTextWithLabelElement(
                                formElement, "", null, style);
                text_longitudeParsed.setEnabled(false);
-               number_errorRadius = formFactory.createIntegerTextWithLabelElement(
+               number_errorRadius = formFactory.createNumberTextWithLabelElement(
                                formElement, "Error Radius (m)", null, style);
                combo_referenceSystem = formFactory
                                .createTermComboElement(ReferenceSystem.class,
@@ -179,7 +179,7 @@ public class PointElement extends AbstractCdmFormElement implements
                        text_latitudeParsed.setText(latitude);
                        text_longitude.setText(longitude);
                        text_longitudeParsed.setText(longitude);
-                       number_errorRadius.setInteger(point.getErrorRadius());
+                       number_errorRadius.setNumber(point.getErrorRadius());
                        combo_referenceSystem.setSelection(point.getReferenceSystem());
                }
        }
index bddaddeebed5cdd630c468b0c1abd213e752e3d1..714a8a0345945c65a735333d655e6244c7a20b21 100644 (file)
@@ -57,7 +57,7 @@ public class StatisticalMeasurementValueElement extends
        @Override
        public void setEntity(StatisticalMeasurementValue entity) {
                this.entity = entity;
-               number_value.setFloat(entity.getValue());
+               number_value.setNumber(entity.getValue());
                combo_type.setSelection(entity.getType());
                section_modifiers.setEntity(entity);
        }
@@ -65,7 +65,7 @@ public class StatisticalMeasurementValueElement extends
        /** {@inheritDoc} */
        @Override
        public void createControls(ICdmFormElement element, int style) {
-               number_value = formFactory.createFloatTextWithLabelElement(element, "Value", 0, style);
+               number_value = formFactory.createNumberTextWithLabelElement(element, "Value", 0, style);
                combo_type = formFactory.createTermComboElement(StatisticalMeasure.class, element, "Statistical Measure", null, style);
                section_modifiers = (ModifierSection) formFactory.createEntityDetailSection(EntityDetailType.MODIFIER, getConversationHolder(), element, Section.TWISTIE);
                section_modifiers.setLayoutData(LayoutConstants.FILL_HORIZONTALLY(2, 1));
index c21c24900d40a42f3540325d3bfce1a74cd5e26c..e5627d0456f362d576c365930789ec39faf8ce97 100644 (file)
@@ -43,7 +43,7 @@ public class PolytomousKeyNodeDetailElement extends
                        PolytomousKeyNode entity, int style) {
                if (!entity.isLeaf()) {
                        NumberWithLabelElement number_nodeNumber = formFactory
-                                       .createIntegerTextWithLabelElement(formElement,
+                                       .createNumberTextWithLabelElement(formElement,
                                                        "Node Number", entity.getNodeNumber(), style);
                        number_nodeNumber.setEnabled(false);
                }
index 6d790a5ef585765111a9a53ed85c3fcbfba9e560..098620fd70d003c00b96a7bc129492c53277f732 100644 (file)
@@ -72,9 +72,9 @@ public class ImageFileElement extends MediaRepresentationPartElement<ImageFile>
        public void createControls(ICdmFormElement formElement, int style) {
                super.createControls(formElement, style);
                
-               text_height = formFactory.createIntegerTextWithLabelElement(formElement, "Height", null, style);
+               text_height = formFactory.createNumberTextWithLabelElement(formElement, "Height", null, style);
                text_height.setEnabled(false);
-               text_width = formFactory.createIntegerTextWithLabelElement(formElement, "Width", null, style);
+               text_width = formFactory.createNumberTextWithLabelElement(formElement, "Width", null, style);
                text_width.setEnabled(false);
                element_image = formFactory.createImageElement(formElement, null, style);
                element_keyValue = formFactory.createKeyValueViewerElement(formElement, "Key", "Value", null);
@@ -84,8 +84,8 @@ public class ImageFileElement extends MediaRepresentationPartElement<ImageFile>
        @Override
        public void setEntity(ImageFile entity) {
                super.setEntity(entity);
-               text_height.setInteger(entity.getHeight());
-               text_width.setInteger(entity.getWidth());
+               text_height.setNumber(entity.getHeight());
+               text_width.setNumber(entity.getWidth());
                try {
                        loadImage(entity.getUri(), false);              
                } catch (Exception e) {
@@ -113,10 +113,10 @@ public class ImageFileElement extends MediaRepresentationPartElement<ImageFile>
                        // But then, I don't think we will handle such large files in the near future
                        getEntity().setSize((int) imageInfo.getLength());
                        
-                       text_height.setInteger(imageInfo.getHeight());
+                       text_height.setNumber(imageInfo.getHeight());
                        getEntity().setHeight(imageInfo.getHeight());
                                
-                       text_width.setInteger(imageInfo.getWidth());
+                       text_width.setNumber(imageInfo.getWidth());
                        getEntity().setWidth(imageInfo.getWidth());
                }
                
@@ -152,8 +152,8 @@ public class ImageFileElement extends MediaRepresentationPartElement<ImageFile>
        protected void handleException(Exception e) {
                super.handleException(e);
                element_image.unloadImage(postRunnable);
-               text_height.setInteger(0);
-               text_width.setInteger(0);
+               text_height.setNumber(0);
+               text_width.setNumber(0);
                
                element_image.loadImage(new Runnable(){
                        public void run() {
index 45d44c50733af89948b20e8da1900c82f2cb64ef..a5ef673414987ed2965ce762c3d0cc092dd6459b 100644 (file)
@@ -61,7 +61,7 @@ public class MediaRepresentationPartElement<T extends MediaRepresentationPart> e
        @Override
        public void createControls(ICdmFormElement element, int style) {
                text_uri = formFactory.createTextWithLabelElement(element, "URI", null, style);
-               text_size = formFactory.createIntegerTextWithLabelElement(element, "Size", null, style);
+               text_size = formFactory.createNumberTextWithLabelElement(element, "Size", null, style);
                text_size.setEnabled(false);
                
                element_messages = formFactory.createLabel(element, "");
index 7d2c60b4d1f4b8a67126dd6a4abc9293531c9d5d..0b4d91728e2750b88ae0ce417796878e14d1af92 100644 (file)
@@ -141,8 +141,8 @@ public class NameDetailElement extends AbstractIdentifiableEntityDetailElement<N
        private void createZoologicalNameParts(ICdmFormElement formElement, NonViralName nonViralName, int style){
                ZoologicalName zoologicalName = (ZoologicalName) nonViralName;
                text_breed = formFactory.createTextWithLabelElement(formElement, "Breed", zoologicalName.getBreed(), style);
-               text_publicationYear = formFactory.createIntegerTextWithLabelElement(formElement, "Publication Year", zoologicalName.getPublicationYear(), style);
-               text_originalPublicationYear = formFactory.createIntegerTextWithLabelElement(formElement, "Orig. Publication Year", zoologicalName.getOriginalPublicationYear(), style);
+               text_publicationYear = formFactory.createNumberTextWithLabelElement(formElement, "Publication Year", zoologicalName.getPublicationYear(), style);
+               text_originalPublicationYear = formFactory.createNumberTextWithLabelElement(formElement, "Orig. Publication Year", zoologicalName.getOriginalPublicationYear(), style);
        }
        
        private void createGenusOrUninomialControls(ICdmFormElement element, NonViralName nonViralName, int style){             
index 482d8c6a38075faa681e5c2a10e8918f739dae99..5c54f7847e6fd36e12db7a63fe4dd24ca04db29f 100644 (file)
@@ -64,7 +64,7 @@ public class FieldObservationDetailElement extends AbstractCdmDetailElement<Deri
 //             text_fieldObjectDefinition = formFactory.createTextWithLabelElement(formElement, "Field Object Definition", entity.getFieldObjectDefinition(CdmStore.getDefaultLanguage()), style);
                text_fieldNotes = formFactory.createTextWithLabelElement(formElement, "Field Notes", entity.getFieldNotes(), style);
                
-               number_individualCount = formFactory.createIntegerTextWithLabelElement(formElement, "Individual Count", entity.getIndividualCount(), style);
+               number_individualCount = formFactory.createNumberTextWithLabelElement(formElement, "Individual Count", entity.getIndividualCount(), style);
                //FIXME:3.3MC
                combo_lifeStage = formFactory.createTermComboElement(DefinedTerm.class, formElement, "Life Stage", entity.getLifeStage(), style);
                combo_sex = formFactory.createTermComboElement(DefinedTerm.class, formElement, "Sex", entity.getSex(), style);
index 235110c7a7927f637ce2ed0c2f772f19e75dcf0e..44002562e76bade1018044598bf4b4325fa53a35 100644 (file)
@@ -74,15 +74,15 @@ public class GatheringEventDetailElement extends
                        DerivedUnitFacade entity, int style) {
                // collectingArea
                number_absoluteElevationError = formFactory
-                               .createIntegerTextWithLabelElement(formElement,
+                               .createNumberTextWithLabelElement(formElement,
                                                "Elevation Error", entity.getAbsoluteElevationError(),
                                                style);
                number_absoluteElevationMinimum = formFactory
-                               .createIntegerTextWithLabelElement(formElement,
+                               .createNumberTextWithLabelElement(formElement,
                                                "Elevation Minimum (m)",
                                                entity.getAbsoluteElevationMinimum(), style);
                number_absoluteElevationMaximum = formFactory
-                               .createIntegerTextWithLabelElement(formElement,
+                               .createNumberTextWithLabelElement(formElement,
                                                "Elevation Maximum (m)",
                                                entity.getAbsoluteElevationMaximum(), style);
 
@@ -91,11 +91,11 @@ public class GatheringEventDetailElement extends
                                style);
                //FIXME:3.3MC
                number_distanceToGround = formFactory
-                               .createIntegerTextWithLabelElement(formElement,
+                               .createNumberTextWithLabelElement(formElement,
                                                "Distance To Ground (m)", entity.getDistanceToGround(),
                                                style);
                number_distanceToWaterSurface = formFactory
-                               .createIntegerTextWithLabelElement(formElement,
+                               .createNumberTextWithLabelElement(formElement,
                                                "Distance To Water Surface (m)",
                                                entity.getDistanceToWaterSurface(), style);
                // exactLocation
index 30f43dd00f4dad403647906fbbcb280a36ee594c..a20485029060e4de8b4058020f91ccc54373563d 100644 (file)
@@ -77,7 +77,7 @@ public class GeneralDetailElement extends
                                                entity.getLocality(), style);           
                element_point = formFactory.createPointElement(formElement,
                                entity.getExactLocation(), style);
-               number_elevation = formFactory.createIntegerTextWithLabelElement(
+               number_elevation = formFactory.createNumberTextWithLabelElement(
                                formElement, "Absolute Elevation (m)",
                                entity.getAbsoluteElevation(), style);
                element_date = formFactory.createTimePeriodElement(formElement, "Date",