Author: kstam
Date: Fri Sep  9 06:55:41 2011
New Revision: 1167018

URL: http://svn.apache.org/viewvc?rev=1167018&view=rev
Log:
JUDDI-520, adding a confirmation box on delete

Modified:
    
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/JUDDIPublisher.java
    
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/MenuBarPanel.java
    
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/PublisherPanel.java
    
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/UDDISubscription.java

Modified: 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/JUDDIPublisher.java
URL: 
http://svn.apache.org/viewvc/juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/JUDDIPublisher.java?rev=1167018&r1=1167017&r2=1167018&view=diff
==============================================================================
--- 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/JUDDIPublisher.java
 (original)
+++ 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/JUDDIPublisher.java
 Fri Sep  9 06:55:41 2011
@@ -160,6 +160,8 @@ public class JUDDIPublisher implements E
        public void deletePublisher() {
                if (publisherPanel!=null) {
                        publisherPanel.deletePublisher(getToken());
+               } else {
+                       Window.alert("Please select a publisher first.");
                }
        }
        

Modified: 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/MenuBarPanel.java
URL: 
http://svn.apache.org/viewvc/juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/MenuBarPanel.java?rev=1167018&r1=1167017&r2=1167018&view=diff
==============================================================================
--- 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/MenuBarPanel.java
 (original)
+++ 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/MenuBarPanel.java
 Fri Sep  9 06:55:41 2011
@@ -17,6 +17,7 @@
 package org.apache.juddi.portlets.client;
 
 import com.google.gwt.core.client.GWT;
+import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.ui.ClickListener;
 import com.google.gwt.user.client.ui.FlowPanel;
 import com.google.gwt.user.client.ui.HorizontalPanel;
@@ -64,7 +65,7 @@ public class MenuBarPanel extends FlowPa
                        syncButton.setTitle("Refresh subscriptions");
                        pushPanel.add(syncButton);
                        newButton.setTitle("Create New Subscription");
-                       deleteButton.setTitle("Delete Subscription");
+                       deleteButton.setTitle("Delete Selected Subscription");
                        saveButton.setTitle("Save Subscription");
                }
                
@@ -75,7 +76,7 @@ public class MenuBarPanel extends FlowPa
                        managerButton.setTitle("Restart ClerkManager");
                        pushPanel.add(managerButton);
                        newButton.setTitle("Create New Publisher");
-                       deleteButton.setTitle("Delete Publisher");
+                       deleteButton.setTitle("Delete Selected Publisher");
                        saveButton.setTitle("Save Publisher");
                        
                }

Modified: 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/PublisherPanel.java
URL: 
http://svn.apache.org/viewvc/juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/PublisherPanel.java?rev=1167018&r1=1167017&r2=1167018&view=diff
==============================================================================
--- 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/PublisherPanel.java
 (original)
+++ 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/PublisherPanel.java
 Fri Sep  9 06:55:41 2011
@@ -135,20 +135,22 @@ public class PublisherPanel extends Flow
        
        protected void deletePublisher(String token){
                if (publisher!=null) {
-                       juddiApiService.deletePublisher(token, 
publisher.getAuthorizedName(), new AsyncCallback<JUDDIApiResponse>() 
-                       {
-                               public void onFailure(Throwable caught) {
-                                       Window.alert("Error: " + 
caught.getMessage());
-                               }
-       
-                               public void onSuccess(JUDDIApiResponse 
response) {
-                                       if (response.isSuccess()) {
-                                               
JUDDIPublisher.getInstance().hidePublisher();
-                                       } else {
-                                               Window.alert("error: " + 
response.getMessage());
+                       if (Window.confirm("Are you sure you want to delete 
Publisher '" + publisher.getAuthorizedName() + "'?")) {
+                               juddiApiService.deletePublisher(token, 
publisher.getAuthorizedName(), new AsyncCallback<JUDDIApiResponse>() 
+                               {
+                                       public void onFailure(Throwable caught) 
{
+                                               Window.alert("Error: " + 
caught.getMessage());
                                        }
-                               }
-                       });
+               
+                                       public void onSuccess(JUDDIApiResponse 
response) {
+                                               if (response.isSuccess()) {
+                                                       
JUDDIPublisher.getInstance().hidePublisher();
+                                               } else {
+                                                       Window.alert("error: " 
+ response.getMessage());
+                                               }
+                                       }
+                               });
+                       }
                }
        }
        

Modified: 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/UDDISubscription.java
URL: 
http://svn.apache.org/viewvc/juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/UDDISubscription.java?rev=1167018&r1=1167017&r2=1167018&view=diff
==============================================================================
--- 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/UDDISubscription.java
 (original)
+++ 
juddi/trunk/juddi-console/uddi-portlets/src/main/java/org/apache/juddi/portlets/client/UDDISubscription.java
 Fri Sep  9 06:55:41 2011
@@ -19,6 +19,7 @@ package org.apache.juddi.portlets.client
 import org.apache.juddi.portlets.client.model.Subscription;
 
 import com.google.gwt.core.client.EntryPoint;
+import com.google.gwt.user.client.Window;
 import com.google.gwt.user.client.ui.DockPanel;
 import com.google.gwt.user.client.ui.RootPanel;
 
@@ -117,7 +118,11 @@ public class UDDISubscription implements
        
        public void deleteSubscription() {
                if (detailPanel!=null) {
-                       detailPanel.deleteSubscription(getToken());
+                       if (Window.confirm("Are you sure you want to delete 
Subscription?")) {
+                               detailPanel.deleteSubscription(getToken());
+                       }
+               } else {
+                       Window.alert("Please select a subscription first.");
                }
        }
        



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to