[jboss-svn-commits] JBL Code SVN: r30695 - in labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client: packages and 1 other directory.

[jboss-svn-commits at lists.jboss.org]

Author: jervisliu
Date: 2009-12-15 21:45:08 -0500 (Tue, 15 Dec 2009)
New Revision: 30695

Modified:
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/messages/Constants.properties
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/packages/SnapshotView.java
Log:
GUVNOR-74: snapshot enhancements

Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/messages/Constants.properties
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/messages/Constants.properties	2009-12-16 02:36:58 UTC (rev 30694)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/messages/Constants.properties	2009-12-16 02:45:08 UTC (rev 30695)
@@ -803,7 +803,7 @@
 SnapshotDeleteConfirm=Are you sure you want to delete the snapshot labelled [{0}] from the package [{1}] ?
 SnapshotWasDeleted=Snapshot was deleted.
 CopySnapshotText=Copy snapshot {0}
-ExistingSnapshots=Existing Snapshots:
+ExistingSnapshots=To an existing snapshot or a new snapshot:
 NewSnapshotNameIs=New Snapshot name:
 CreatedSnapshot0ForPackage1=Created snapshot [{0}] for package [{1}]
 Snapshot0ForPackage1WasCopiedFrom2=Snapshot [{0}] for package [{1}] was copied from [{2}]

Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/packages/SnapshotView.java
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/packages/SnapshotView.java	2009-12-16 02:36:58 UTC (rev 30694)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/packages/SnapshotView.java	2009-12-16 02:45:08 UTC (rev 30695)
@@ -246,7 +246,6 @@
                                             final FormStylePopup copy = new FormStylePopup( "images/snapshot.png",
                                                                                             Format.format( constants.CopySnapshotText(),
                                                                                                            snapshotName ) );
-                                            final TextBox box = new TextBox();
                                             final List<RadioButton> options = new ArrayList<RadioButton>();
                                             VerticalPanel vert = new VerticalPanel();
                                             for ( int i = 0; i < snaps.length; i++ ) {
@@ -255,31 +254,44 @@
                                                 options.add( existing );
                                                 vert.add( existing );
                                             }
-                                            final RadioButton newBut = new RadioButton( "snapshotNameGroup",
-                                                                                        "NEW NAME" ); //NON-NLS
-                                            vert.add( newBut );
+                                            
+                    						HorizontalPanel newNameHorizontalPanel = new HorizontalPanel();
+                                            final TextBox newNameTextBox = new TextBox();
+                                    		final String newNameText = constants.NEW() + ": ";
 
-                                            copy.addAttribute( constants.ExistingSnapshots(),
+                    						final RadioButton newNameRadioButton = new RadioButton( "snapshotNameGroup",
+                    								newNameText);
+                    						newNameHorizontalPanel.add(newNameRadioButton);
+                    						newNameTextBox.setEnabled(false);
+                    						newNameRadioButton.addClickListener(new ClickListener() {
+                    							public void onClick(Widget w) {
+                    								newNameTextBox.setEnabled(true);
+                    							}
+                    						});
+
+                    						newNameHorizontalPanel.add(newNameTextBox);
+                    						options.add(newNameRadioButton);
+                    						vert.add(newNameHorizontalPanel);                    						
+
+                                            copy.addAttribute("to an existing snapshot or a new snapshot",
                                                                vert );
-                                            copy.addAttribute( constants.NewSnapshotNameIs(),
-                                                               box );
+
                                             Button ok = new Button( constants.OK() );
-                                            copy.addAttribute( "",
-                                                               ok );
+                                            copy.addAttribute( "", ok );
                                             ok.addClickListener( new ClickListener() {
                                                 public void onClick(Widget w) {
-                                                    if ( newBut.isChecked() ) {
+                                                    if ( newNameRadioButton.isChecked() ) {
                                                         if ( checkUnique( snaps,
-                                                                          box.getText() ) ) {
+                                                                          newNameTextBox.getText() ) ) {
                                                             serv.copyOrRemoveSnapshot( packageName,
                                                                                        snapshotName,
                                                                                        false,
-                                                                                       box.getText(),
+                                                                                       newNameTextBox.getText(),
                                                                                        new GenericCallback() {
                                                                                            public void onSuccess(Object data) {
                                                                                                copy.hide();
                                                                                                Window.alert( Format.format( constants.CreatedSnapshot0ForPackage1(),
-                                                                                                                            box.getText(),
+                                                                                                                            newNameTextBox.getText(),
                                                                                                                             packageName ) );
                                                                                            }
                                                                                        } );