Xcos: remove block default values 87/1387/3
Clément DAVID [Wed, 21 Jul 2010 08:54:47 +0000 (10:54 +0200)]
Remove constructors with value
Remove on block loading value setup

Change-Id: I8f9c12d04fcd6a25c9119074a4a342a760694c73

15 files changed:
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/AfficheBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/BasicBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/BlockFactory.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/SplitBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/TextBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/actions/EditBlockFormatAction.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/ContextUpdate.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/EventInBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/EventOutBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/ExplicitInBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/ExplicitOutBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/ImplicitInBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/block/io/ImplicitOutBlock.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/io/scicos/LabelElement.java
scilab/modules/xcos/src/java/org/scilab/modules/xcos/palette/PaletteBlockCtrl.java

index 9762b72..2de0a72 100644 (file)
@@ -118,8 +118,8 @@ public final class AfficheBlock extends BasicBlock {
                }               
        }
        
-       private Timer printTimer;  
-       private UpdateValueListener updateAction;
+       private final Timer printTimer;  
+       private final UpdateValueListener updateAction;
        
        /** Default constructor */
        public AfficheBlock() {
@@ -128,19 +128,17 @@ public final class AfficheBlock extends BasicBlock {
                updateAction = new UpdateValueListener(this);
                printTimer = new Timer(DEFAULT_TIMER_RATE, updateAction);
                printTimer.setRepeats(false);
-               
-               setValue("0.0");
        }
-
+       
        /**
-        * Constructor with label
-        * 
-        * @param value
-        *            the default value.
+        * Set the default values
+        * @see org.scilab.modules.xcos.block.BasicBlock#setDefaultValues()
         */
-       protected AfficheBlock(String value) {
-               this();
-               setValue(value);
+       @Override
+       protected void setDefaultValues() {
+               super.setDefaultValues();
+               
+               setValue("0.0");
        }
 
        /**
index f94dbb4..3182e1e 100644 (file)
@@ -295,7 +295,6 @@ public class BasicBlock extends ScilabGraphUniqueObject implements Serializable
        protected BasicBlock(String label) {
                this();
                setDefaultValues();
-               setValue(label);
        }
 
        /**
index 711b517..47de68a 100644 (file)
@@ -40,7 +40,7 @@ public final class BlockFactory {
         */
        public static enum BlockInterFunction {
                /** @see TextBlock */
-               TEXT_f(new TextBlock(XcosMessages.DOTS)),
+               TEXT_f(new TextBlock()),
                /** @see SuperBlock */
                SUPER_f(new SuperBlock()),
                /** @see SuperBlock */
index 21c7ee5..5d64c6b 100644 (file)
@@ -43,18 +43,6 @@ public final class SplitBlock extends BasicBlock {
        public SplitBlock() {
                super();
        }
-
-       /**
-        * @param label
-        *            block label
-        */
-       protected SplitBlock(String label) {
-               // SPLIT_f <-> lsplit
-               // CLKSPLIT_f <-> split
-               // IMPSPLIT_F <-> limpsplit
-               this();
-               setValue(label);
-       }
        
        /**
         * Connect the splitblock to a source and 2 targets.
index 660a33e..cfaf820 100644 (file)
@@ -21,6 +21,7 @@ import org.scilab.modules.types.scilabTypes.ScilabString;
 import org.scilab.modules.types.scilabTypes.ScilabType;
 import org.scilab.modules.xcos.block.actions.BlockParametersAction;
 import org.scilab.modules.xcos.block.actions.RegionToSuperblockAction;
+import org.scilab.modules.xcos.utils.XcosMessages;
 
 import com.mxgraph.util.mxConstants;
 
@@ -73,15 +74,6 @@ public final class TextBlock extends BasicBlock {
        }
 
        /**
-        * Constructor with a default text
-        * @param label The default text
-        */
-       protected TextBlock(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
@@ -89,6 +81,8 @@ public final class TextBlock extends BasicBlock {
                super.setDefaultValues();
                setInterfaceFunctionName(INTERFUNCTION_NAME);
                setStyle(INTERFUNCTION_NAME);
+               
+               setValue(XcosMessages.DOTS);
        }
     
     /**
index 7f467ca..e7cf3bd 100644 (file)
@@ -100,7 +100,7 @@ public class EditBlockFormatAction extends DefaultAction {
                /*
                 * Create and show the dialog
                 */
-               EditFormatDialog dialog = createDialog(selectedCell, (Frame) window);
+               EditFormatDialog dialog = createDialog(selectedCell, window);
                dialog.setName(name);
                dialog.setVisible(true);
        }
@@ -166,11 +166,11 @@ public class EditBlockFormatAction extends DefaultAction {
                        textColor = mxUtils.parseColor(working);
                }
                
-               working = (String) selectedCell.getValue();
-               if (working == null) {
+               Object current = selectedCell.getValue();
+               if (current == null) {
                        text = "";
                } else {
-                       text = mxUtils.getBodyMarkup(working, false);
+                       text = mxUtils.getBodyMarkup(current.toString(), false);
                }
                
                EditFormatDialog dialog = new EditFormatDialog(window);
index e514ae0..67ad5a1 100644 (file)
@@ -244,15 +244,6 @@ public abstract class ContextUpdate extends BasicBlock {
        }
 
        /**
-        * @param label
-        *            block labek
-        */
-       protected ContextUpdate(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
index 617cb99..985d43a 100644 (file)
@@ -28,15 +28,6 @@ public final class EventInBlock extends ContextUpdate {
        }
 
        /**
-        * @param label
-        *            block label
-        */
-       protected EventInBlock(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
index 0ca75bd..1f1f041 100644 (file)
@@ -28,14 +28,6 @@ public final class EventOutBlock extends ContextUpdate {
        }
 
        /**
-        * @param label block label
-        */
-       protected EventOutBlock(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
index ea6b4dd..aa1ea35 100644 (file)
@@ -28,14 +28,6 @@ public final class ExplicitInBlock extends ContextUpdate {
        }
 
        /**
-        * @param label block label
-        */
-       protected ExplicitInBlock(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
index 7e903a8..f6f62b6 100644 (file)
@@ -28,14 +28,6 @@ public final class ExplicitOutBlock extends ContextUpdate {
        }
 
        /**
-        * @param label block label
-        */
-       protected ExplicitOutBlock(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
index a266e57..d462116 100644 (file)
@@ -28,14 +28,6 @@ public final class ImplicitInBlock extends ContextUpdate {
        }
 
        /**
-        * @param label block label
-        */
-       protected ImplicitInBlock(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
index e4dfc85..851918b 100644 (file)
@@ -29,14 +29,6 @@ public final class ImplicitOutBlock extends ContextUpdate {
        }
 
        /**
-        * @param label block label
-        */
-       protected ImplicitOutBlock(String label) {
-               this();
-               setValue(label);
-       }
-
-       /**
         * Initialize the block with the default values
         */
        @Override
index 9a4c4fb..628e17e 100644 (file)
@@ -35,10 +35,10 @@ public class LabelElement extends AbstractElement<TextBlock> {
        private ScilabMList data;
        
        /** Element used to decode/encode Scicos model part into a BasicBlock*/
-       private BlockModelElement modelElement = new BlockModelElement();
+       private final BlockModelElement modelElement = new BlockModelElement();
        
        /** Element used to decode/encode Scicos model part into a BasicBlock*/
-       private BlockGraphicElement graphicElement = new BlockGraphicElement();
+       private final BlockGraphicElement graphicElement = new BlockGraphicElement();
        
        /**
         * Default constructor
index d0ec0ab..3f5a2cc 100644 (file)
@@ -151,7 +151,6 @@ public final class PaletteBlockCtrl {
                        
                        if (block.getStyle().compareTo("") == 0) {
                                block.setStyle(block.getInterfaceFunctionName());
-                               block.setValue(block.getInterfaceFunctionName());
                        }
                } else {
                        block = BlockFactory.createBlock(BlockInterFunction.TEXT_f);