Author: yzhishko Date: 2010-01-11 08:58:36 -0500 (Mon, 11 Jan 2010) New Revision: 19690 Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/TemplatesPreferencePage.java Log: https://jira.jboss.org/jira/browse/JBIDE-5648 - fixed Modified: trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/TemplatesPreferencePage.java =================================================================== --- trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/TemplatesPreferencePage.java 2010-01-11 09:22:32 UTC (rev 19689) +++ trunk/vpe/plugins/org.jboss.tools.vpe/src/org/jboss/tools/vpe/editor/preferences/TemplatesPreferencePage.java 2010-01-11 13:58:36 UTC (rev 19690) @@ -232,6 +232,7 @@ public void handleEvent(Event event) { Widget source = event.widget; + int selectIndex = tagsTable.getSelectionIndex(); if (source == addButton) { /* * Handle add event @@ -250,18 +251,23 @@ /* * Handle edit event */ - VpeAnyData data = (VpeAnyData) tagsList.get(tagsTable.getSelectionIndex()); - VpeEditAnyDialog editDialog = new VpeEditAnyDialog(getShell(), data); - editDialog.open(); - if(data.isChanged()) { - tagListWasChanged = true; + if (selectIndex > -1) { + VpeAnyData data = (VpeAnyData) tagsList.get(selectIndex); + VpeEditAnyDialog editDialog = new VpeEditAnyDialog(getShell(), data); + editDialog.open(); + if(data.isChanged()) { + tagListWasChanged = true; + } } } else if (source == removeButton) { /* * Handle remove event */ - tagsList.remove(tagsTable.getSelectionIndex()); - tagListWasChanged = true; + if (selectIndex > -1) { + tagsTable.remove(selectIndex); + tagsList.remove(selectIndex); + tagListWasChanged = true; + } } else { /* * Handle default event