mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-08-20 06:35:50 +02:00
Code Issues Wiki Activity

[235223] Duplicate Filter Strings

Browse source
This commit is contained in:
Kevin Doyle 2008-08-05 04:07:01 +00:00
parent 95e3a8cbaa
commit 895465bd08
1 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/ui/filters/SystemChangeFilterPane.java
View file

@ -14,6 +14,7 @@
* Contributors: * Contributors:
* David Dykstal (IBM) - [226561] add API markup to javadoc * David Dykstal (IBM) - [226561] add API markup to javadoc
* David McKnight(IBM) - [239257] Tooltip for Filter Pool label is incorrect * David McKnight(IBM) - [239257] Tooltip for Filter Pool label is incorrect
* Kevin Doyle (IBM) - [235223] Duplicate Filter Strings
*******************************************************************************/ *******************************************************************************/
package org.eclipse.rse.ui.filters; package org.eclipse.rse.ui.filters;
@ -796,8 +797,10 @@ public class SystemChangeFilterPane extends SystemBaseForm
{ {
getMessageLine().clearMessage(); getMessageLine().clearMessage();
applyPressed();
// if error, do not change selection and we keep pending changes state // if error, do not change selection and we keep pending changes state
if (getFilterStringEditPane(getShell()).verify() != null) if (getMessageLine().getErrorMessage() != null)
{ {
e.doit = false; // dang, this doesn't work! e.doit = false; // dang, this doesn't work!
sm.setChangesMade(); sm.setChangesMade();
@ -807,10 +810,6 @@ public class SystemChangeFilterPane extends SystemBaseForm
applyButton.setEnabled(false); // d45795 applyButton.setEnabled(false); // d45795
return; return;
} }
// no errors
saveFilterString(getFilterStringEditPane(getShell()).getFilterString(),
getFilterStringEditPane(getShell()).getCurrentSelectionIndex());
} }
else { else {
getMessageLine().clearMessage(); getMessageLine().clearMessage();