Author: dazarov
Date: 2008-02-21 07:59:53 -0500 (Thu, 21 Feb 2008)
New Revision: 6495
Modified:
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/OpenSeamComponentDialog.java
Log:
http://jira.jboss.com/jira/browse/JBIDE-518
Modified:
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/OpenSeamComponentDialog.java
===================================================================
---
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/OpenSeamComponentDialog.java 2008-02-21
12:19:25 UTC (rev 6494)
+++
trunk/seam/plugins/org.jboss.tools.seam.ui/src/org/jboss/tools/seam/ui/wizard/OpenSeamComponentDialog.java 2008-02-21
12:59:53 UTC (rev 6495)
@@ -14,6 +14,7 @@
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
+import java.io.IOException;
import java.util.Comparator;
import java.util.Iterator;
@@ -32,6 +33,7 @@
import org.eclipse.swt.widgets.Control;
import org.eclipse.swt.widgets.Shell;
import org.eclipse.ui.IMemento;
+import org.eclipse.ui.WorkbenchException;
import org.eclipse.ui.XMLMemento;
import org.eclipse.ui.dialogs.FilteredItemsSelectionDialog;
import org.jboss.tools.seam.core.ISeamComponent;
@@ -60,8 +62,6 @@
setListLabelProvider(new SeamComponentLabelProvider());
setDetailsLabelProvider(new SeamComponentLabelProvider());
- //validateHistory();
-
XMLMemento memento = loadMemento();
if (memento != null)
getSelectionHistory().load(memento);
@@ -131,9 +131,16 @@
reader = new FileReader(file);
memento = XMLMemento.createReadRoot(reader);
- reader.close();
- } catch (Exception ex) {
- ex.printStackTrace();
+ }catch (IOException ex) {
+ SeamCorePlugin.getPluginLog().logError(ex);
+ }catch (WorkbenchException ex){
+ SeamCorePlugin.getPluginLog().logError(ex);
+ }finally{
+ try{
+ reader.close();
+ }catch(IOException ex){
+ SeamCorePlugin.getPluginLog().logError(ex);
+ }
}
}
return memento;
@@ -148,9 +155,14 @@
writer = new FileWriter(file);
xmlMemento.save(writer);
- writer.close();
- } catch (Exception ex) {
- ex.printStackTrace();
+ }catch (IOException ex) {
+ SeamCorePlugin.getPluginLog().logError(ex);
+ }finally{
+ try{
+ writer.close();
+ }catch(IOException ex){
+ SeamCorePlugin.getPluginLog().logError(ex);
+ }
}
}