Work in progress - more updates to the Alias Identifier editor

src/alias/AliasFactory.java

@@ -31,6 +31,102 @@ import alias.id.uniqueID.UniqueIDEditor;
 
 public class AliasFactory
 {
+	public static AliasID copyOf( AliasID id )
+	{
+		switch( id.getType() )
+		{
+			case ESN:
+				Esn originalESN = (Esn)id;
+				Esn copyESN = new Esn();
+				copyESN.setEsn( originalESN.getEsn() );
+				return copyESN;
+			case Fleetsync:
+				FleetsyncID originalFleetsyncID = (FleetsyncID)id;
+				FleetsyncID copyFleetsyncID = new FleetsyncID();
+				copyFleetsyncID.setIdent( originalFleetsyncID.getIdent() );
+				return copyFleetsyncID;
+			case LTRNetUID:
+				UniqueID originalUniqueID = (UniqueID)id;
+				UniqueID copyUniqueID = new UniqueID();
+				copyUniqueID.setUid( originalUniqueID.getUid() );
+				return copyUniqueID;
+			case LoJack:
+				LoJackFunctionAndID originalLoJackFunctionAndID = (LoJackFunctionAndID)id;
+				LoJackFunctionAndID copyLoJackFunctionAndID = new LoJackFunctionAndID();
+				copyLoJackFunctionAndID.setFunction( originalLoJackFunctionAndID.getFunction() );
+				copyLoJackFunctionAndID.setID( originalLoJackFunctionAndID.getID() );
+				return copyLoJackFunctionAndID;
+			case MDC1200:
+				MDC1200ID originalMDC1200ID = (MDC1200ID)id;
+				MDC1200ID copyMDC1200ID = new MDC1200ID();
+				copyMDC1200ID.setIdent( originalMDC1200ID.getIdent() );
+				return copyMDC1200ID;
+			case MIN:
+				Min originalMin = (Min)id;
+				Min copyMin = new Min();
+				copyMin.setMin( originalMin.getMin() );
+				return copyMin;
+			case MPT1327:
+				MPT1327ID originalMPT1327ID = (MPT1327ID)id;
+				MPT1327ID copyMPT1327ID = new MPT1327ID();
+				copyMPT1327ID.setIdent( originalMPT1327ID.getIdent() );
+				return copyMPT1327ID;
+			case NonRecordable:
+				return new NonRecordable();
+			case Priority:
+				Priority originalPriority = (Priority)id;
+				Priority copyPriority = new Priority();
+				copyPriority.setPriority( originalPriority.getPriority() );
+				return copyPriority;
+			case Site:
+				SiteID originalSiteID = (SiteID)id;
+				SiteID copySiteID = new SiteID();
+				copySiteID.setSite( originalSiteID.getSite() );
+				return copySiteID;
+			case Talkgroup:
+				TalkgroupID originalTalkgroupID = (TalkgroupID)id;
+				TalkgroupID copyTalkgroupID = new TalkgroupID();
+				copyTalkgroupID.setTalkgroup( originalTalkgroupID.getTalkgroup() );
+				return copyTalkgroupID;
+			case Status:
+				StatusID originalStatusID = (StatusID)id;
+				StatusID copyStatusID = new StatusID();
+				copyStatusID.setStatus( originalStatusID.getStatus() );
+				return copyStatusID;
+			default:
+		}
+
+		return null;
+	}
+	
+	public static AliasAction copyOf( AliasAction action )
+	{
+		if( action instanceof BeepAction )
+		{
+			return new BeepAction();
+		}
+		else if( action instanceof ClipAction )
+		{
+			ClipAction originalClip = (ClipAction)action;
+			ClipAction copyClip = new ClipAction();
+			copyClip.setInterval( originalClip.getInterval() );
+			copyClip.setPath( originalClip.getPath() );
+			copyClip.setPeriod( originalClip.getPeriod() );
+			return copyClip;
+		}
+		else if( action instanceof ScriptAction )
+		{
+			ScriptAction originalScript = (ScriptAction)action;
+			ScriptAction copyScript = new ScriptAction();
+			copyScript.setInterval( originalScript.getInterval() );
+			copyScript.setPeriod( originalScript.getPeriod() );
+			copyScript.setScript( originalScript.getScript() );
+			return copyScript;
+		}
+		
+		return null;
+	}
+	
 	public static Alias copyOf( Alias original )
 	{
 		Alias copy = new Alias( original.getName() );
@@ -41,107 +137,21 @@ public class AliasFactory
 		
 		for( AliasID id: original.getId() )
 		{
-			switch( id.getType() )
+			AliasID copyID = copyOf( id );
+			
+			if( copyID != null )
 			{
-				case ESN:
+				copy.addAliasID( copyID );
-					Esn originalESN = (Esn)id;
-					Esn copyESN = new Esn();
-					copyESN.setEsn( originalESN.getEsn() );
-					copy.addAliasID( copyESN );
-					break;
-				case Fleetsync:
-					FleetsyncID originalFleetsyncID = (FleetsyncID)id;
-					FleetsyncID copyFleetsyncID = new FleetsyncID();
-					copyFleetsyncID.setIdent( originalFleetsyncID.getIdent() );
-					copy.addAliasID( copyFleetsyncID );
-					break;
-				case LTRNetUID:
-					UniqueID originalUniqueID = (UniqueID)id;
-					UniqueID copyUniqueID = new UniqueID();
-					copyUniqueID.setUid( originalUniqueID.getUid() );
-					copy.addAliasID( copyUniqueID );
-					break;
-				case LoJack:
-					LoJackFunctionAndID originalLoJackFunctionAndID = (LoJackFunctionAndID)id;
-					LoJackFunctionAndID copyLoJackFunctionAndID = new LoJackFunctionAndID();
-					copyLoJackFunctionAndID.setFunction( originalLoJackFunctionAndID.getFunction() );
-					copyLoJackFunctionAndID.setID( originalLoJackFunctionAndID.getID() );
-					copy.addAliasID( copyLoJackFunctionAndID );
-					break;
-				case MDC1200:
-					MDC1200ID originalMDC1200ID = (MDC1200ID)id;
-					MDC1200ID copyMDC1200ID = new MDC1200ID();
-					copyMDC1200ID.setIdent( originalMDC1200ID.getIdent() );
-					copy.addAliasID( copyMDC1200ID );
-					break;
-				case MIN:
-					Min originalMin = (Min)id;
-					Min copyMin = new Min();
-					copyMin.setMin( originalMin.getMin() );
-					copy.addAliasID( copyMin );
-					break;
-				case MPT1327:
-					MPT1327ID originalMPT1327ID = (MPT1327ID)id;
-					MPT1327ID copyMPT1327ID = new MPT1327ID();
-					copyMPT1327ID.setIdent( originalMPT1327ID.getIdent() );
-					copy.addAliasID( copyMPT1327ID );
-					break;
-				case NonRecordable:
-					copy.addAliasID( new NonRecordable() );
-					break;
-				case Priority:
-					Priority originalPriority = (Priority)id;
-					Priority copyPriority = new Priority();
-					copyPriority.setPriority( originalPriority.getPriority() );
-					copy.addAliasID( copyPriority );
-					break;
-				case Site:
-					SiteID originalSiteID = (SiteID)id;
-					SiteID copySiteID = new SiteID();
-					copySiteID.setSite( originalSiteID.getSite() );
-					copy.addAliasID( copySiteID );
-					break;
-				case Talkgroup:
-					TalkgroupID originalTalkgroupID = (TalkgroupID)id;
-					TalkgroupID copyTalkgroupID = new TalkgroupID();
-					copyTalkgroupID.setTalkgroup( originalTalkgroupID.getTalkgroup() );
-					copy.addAliasID( copyTalkgroupID );
-					break;
-				case Status:
-				default:
-					throw new IllegalArgumentException( "Unrecognized alias "
-							+ "ID type: " + id.getType() );
 			}
 		}
 
 		for( AliasAction action: original.getAction() )
 		{
-			if( action instanceof BeepAction )
+			AliasAction copyAction = copyOf( action );
+			
+			if( copyAction != null )
 			{
-				copy.addAliasAction( new BeepAction() );
+				copy.addAliasAction( copyAction );
-			}
-			else if( action instanceof ClipAction )
-			{
-				ClipAction originalClip = (ClipAction)action;
-				ClipAction copyClip = new ClipAction();
-				copyClip.setInterval( originalClip.getInterval() );
-				copyClip.setPath( originalClip.getPath() );
-				copyClip.setPeriod( originalClip.getPeriod() );
-				copy.addAliasAction( copyClip );
-			}
-			else if( action instanceof ScriptAction )
-			{
-				ScriptAction originalScript = (ScriptAction)action;
-				ScriptAction copyScript = new ScriptAction();
-				copyScript.setInterval( originalScript.getInterval() );
-				copyScript.setPeriod( originalScript.getPeriod() );
-				copyScript.setScript( originalScript.getScript() );
-				copy.addAliasAction( copyScript );
-			}
-			else
-			{
-				throw new IllegalArgumentException( "Unrecognized alias "
-						+ "action: " + action.getClass() );
 			}
 		}
 		

src/alias/AliasIdentifierEditor.java

@@ -27,7 +27,6 @@ import net.miginfocom.swing.MigLayout;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import sample.Listener;
 import alias.AliasEvent.Event;
 
 public class AliasIdentifierEditor extends AliasConfigurationEditor
@@ -38,6 +37,10 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 
 	private static ListModel<AliasID> EMPTY_MODEL = new DefaultListModel<>();
 	private JList<AliasID> mAliasIDList = new JList<>( EMPTY_MODEL );
+	private JButton mNewIDButton;
+	private JButton mCloneIDButton;
+	private JButton mDeleteIDButton;
+	
 	private EditorContainer mEditorContainer = new EditorContainer();
 
 	private Alias mAlias;
@@ -73,6 +76,14 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 						AliasID selected = (AliasID)selectedItem;
 
 						mEditorContainer.setAliasID( selected );
+						
+						mCloneIDButton.setEnabled( true );
+						mDeleteIDButton.setEnabled( true );
+					}
+					else
+					{
+						mCloneIDButton.setEnabled( false );
+						mDeleteIDButton.setEnabled( false );
 					}
 				}
 			}
@@ -83,9 +94,9 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 		
 		add( mEditorContainer, "span,grow" );
 
-		final JButton newButton = new JButton( "New ..." );
+		mNewIDButton = new JButton( "New ..." );
 		
-		newButton.addMouseListener( new MouseAdapter()
+		mNewIDButton.addMouseListener( new MouseAdapter()
 		{
 			@Override
 			public void mouseClicked( MouseEvent e )
@@ -111,9 +122,57 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 			}
 		} );
 		
-		add( newButton );
+		add( mNewIDButton );
-		add( new JButton( "Copy" ) );
+		
-		add( new JButton( "Delete" ), "wrap" );
+		mCloneIDButton = new JButton( "Clone" );
+		mCloneIDButton.setEnabled( false );
+		mCloneIDButton.addActionListener( new ActionListener()
+		{
+			@Override
+			public void actionPerformed( ActionEvent e )
+			{
+				AliasID selected = mAliasIDList.getSelectedValue();
+				
+				if( selected != null )
+				{
+					AliasID clone = AliasFactory.copyOf( selected );
+
+					if( clone != null )
+					{
+						addAliasIDToList( clone );
+					}
+				}
+			}
+		} );
+		add( mCloneIDButton );
+		
+		mDeleteIDButton = new JButton( "Delete" );
+		mDeleteIDButton.setEnabled( false );
+		mDeleteIDButton.addActionListener( new ActionListener()
+		{
+			@Override
+			public void actionPerformed( ActionEvent e )
+			{
+				final AliasID selected = mAliasIDList.getSelectedValue();
+				
+				if( selected != null )
+				{
+					int choice = JOptionPane.showConfirmDialog( mDeleteIDButton, 
+						"Do you want to delete [" + selected.toString() + "]", 
+						"Are you sure?", JOptionPane.YES_NO_OPTION );
+					
+					if( choice == JOptionPane.YES_OPTION )
+					{
+						mAlias.removeAliasID( selected );
+						
+						mAliasModel.broadcast( new AliasEvent( mAlias, Event.CHANGE ) );
+
+						setAlias( mAlias );
+					}
+				}
+			}
+		} );
+		add( mDeleteIDButton, "wrap" );
 	}
 	
 	@Override
@@ -154,6 +213,11 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 	@Override
 	public void save()
 	{
+		if( mEditorContainer.isModified() )
+		{
+			mEditorContainer.save();
+		}
+		
 		mAliasModel.broadcast( new AliasEvent( mAlias, Event.CHANGE ) );
 	}
 	
@@ -170,6 +234,24 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 			add( mEditor );
 		}
 		
+		public boolean isModified()
+		{
+			if( mEditor != null )
+			{
+				return mEditor.isModified();
+			}
+			
+			return false;
+		}
+		
+		public void save()
+		{
+			if( mEditor != null )
+			{
+				mEditor.save();
+			}
+		}
+		
 		public void setAliasID( AliasID aliasID )
 		{
 			if( mEditor != null )
@@ -199,6 +281,25 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 		}
 	}
 	
+	private void addAliasIDToList( final AliasID id )
+	{
+		if( id != null && mAlias != null )
+		{
+			mAlias.addAliasID( id );
+			
+			EventQueue.invokeLater( new Runnable()
+			{
+				@Override
+				public void run()
+				{
+					setAlias( mAlias );
+					
+					mAliasIDList.setSelectedValue( id, true );
+				}
+			} );
+		}
+	}
+	
 	public class AddAliasIdentifierItem extends JMenuItem
 	{
 		private static final long serialVersionUID = 1L;
@@ -218,21 +319,7 @@ public class AliasIdentifierEditor extends AliasConfigurationEditor
 				{
 					final AliasID id = AliasFactory.getAliasID( mAliasIDType );
 					
-					if( id != null && mAlias != null )
+					addAliasIDToList( id );
-					{
-						mAlias.addAliasID( id );
-						
-						EventQueue.invokeLater( new Runnable()
-						{
-							@Override
-							public void run()
-							{
-								setAlias( mAlias );
-								
-								mAliasIDList.setSelectedValue( id, true );
-							}
-						} );
-					}
 				}
 			} );
 		}

src/alias/id/priority/PriorityEditor.java

@@ -19,6 +19,8 @@ package alias.id.priority;
 
 import java.awt.Color;
 import java.awt.Cursor;
+import java.awt.event.ActionEvent;
+import java.awt.event.ActionListener;
 import java.awt.event.MouseAdapter;
 import java.awt.event.MouseEvent;
 
@@ -65,6 +67,14 @@ public class PriorityEditor extends ComponentEditor<AliasID>
 		setLayout( new MigLayout( "fill,wrap 2", "[right][left]", "[][]" ) );
 
 		mDoNotMonitorCheckBox.setToolTipText( HELP_TEXT );
+		mDoNotMonitorCheckBox.addActionListener( new ActionListener()
+		{
+			@Override
+			public void actionPerformed( ActionEvent e )
+			{
+				setModified( true );
+			}
+		} );
 		add( mDoNotMonitorCheckBox, "span,align center" );
 
 		mPrioritySlider = new JSlider( JSlider.HORIZONTAL,
@@ -83,6 +93,7 @@ public class PriorityEditor extends ComponentEditor<AliasID>
 			public void stateChanged( ChangeEvent e )
 			{
 				mPrioritySliderLabel.setText( "Priority: " + mPrioritySlider.getValue() );
+				setModified( true );
 			}
 		} );
 		mPrioritySlider.setToolTipText( HELP_TEXT );

src/alias/id/status/StatusID.java

@@ -43,7 +43,7 @@ public class StatusID extends AliasID
 	
 	public String toString()
 	{
-		return "Status: " + mStatus;
+		return "Status: " + String.format( "%03d", mStatus );
 	}
 
 	@Override

src/alias/id/status/StatusIDEditor.java

@@ -113,7 +113,7 @@ public class StatusIDEditor extends ComponentEditor<AliasID>
 		
 		if( statusID != null )
 		{
-			mTextField.setText( String.valueOf( statusID.getStatus() ) );
+			mTextField.setText( String.format( "%03d", statusID.getStatus() ) );
 		}
 		
 		setModified( false );