Fix Bug 9787 - Carbide gets its SDK list out of sync with devices.xml
authortimkelly
Tue, 08 Sep 2009 14:52:55 -0500
changeset 483 34f888c29813
parent 482 b17329432d21
child 485 6a656b7f160f
Fix Bug 9787 - Carbide gets its SDK list out of sync with devices.xml
core/com.nokia.carbide.cpp.sdk.ui/src/com/nokia/carbide/cpp/internal/sdk/ui/SDKPreferencePage.java
--- a/core/com.nokia.carbide.cpp.sdk.ui/src/com/nokia/carbide/cpp/internal/sdk/ui/SDKPreferencePage.java	Fri Sep 04 11:39:11 2009 -0500
+++ b/core/com.nokia.carbide.cpp.sdk.ui/src/com/nokia/carbide/cpp/internal/sdk/ui/SDKPreferencePage.java	Tue Sep 08 14:52:55 2009 -0500
@@ -369,6 +369,7 @@
 			if (sdkPropDlg.open() == SDKPropertiesDialog.OK){
 				sdkListTableViewer.refresh();
 				setSelectedSDKInfoText(sdk);
+				rescanSDKs(false);
 			}
 		} else {
 			MessageDialog.openError(getShell(), Messages.getString("SDKPreferencePage.No_SDK_Selected"), Messages.getString("SDKPreferencePage.No_selected_SDK_detected")); //$NON-NLS-1$ //$NON-NLS-2$
@@ -403,6 +404,10 @@
 	}
 	
 	private void rescanNowButtonAction(){
+		rescanSDKs(true);
+	}
+	
+	private void rescanSDKs(boolean scanForNewPlugins){
 		sdkListTableViewer.getTable().clearAll();
 		sdkListTableViewer.refresh();
 		sdkList.clear();
@@ -418,7 +423,9 @@
 			setSelectedSDKInfoText(sdk);
 		}
 		
-		NewPluginChecker.checkForNewlyInstalledPlugins(SDKUIPlugin.getDefault().getWorkbench());
+		if (scanForNewPlugins){
+			NewPluginChecker.checkForNewlyInstalledPlugins(SDKUIPlugin.getDefault().getWorkbench());
+		}
 	}
 	
 	/**