diff options
-rw-r--r-- | forms/qa/integration/forms/BooleanValidator.java | 4 | ||||
-rw-r--r-- | forms/qa/integration/forms/DateValidator.java | 4 | ||||
-rw-r--r-- | forms/qa/integration/forms/FormComponent.java | 20 | ||||
-rw-r--r-- | forms/qa/integration/forms/ListSelectionValidator.java | 4 | ||||
-rw-r--r-- | forms/qa/integration/forms/NumericValidator.java | 4 | ||||
-rw-r--r-- | forms/qa/integration/forms/TextValidator.java | 11 | ||||
-rw-r--r-- | forms/qa/integration/forms/TimeValidator.java | 4 |
7 files changed, 13 insertions, 38 deletions
diff --git a/forms/qa/integration/forms/BooleanValidator.java b/forms/qa/integration/forms/BooleanValidator.java index 730c45f65eb9..8201fe99f5ae 100644 --- a/forms/qa/integration/forms/BooleanValidator.java +++ b/forms/qa/integration/forms/BooleanValidator.java @@ -61,9 +61,7 @@ public class BooleanValidator extends integration.forms.ControlValidator return false; boolean value = ((Boolean)Value).booleanValue(); - if ( m_preventChecked && ( value ) ) - return false; - return true; + return !(m_preventChecked && ( value )); } catch( java.lang.Exception e ) { diff --git a/forms/qa/integration/forms/DateValidator.java b/forms/qa/integration/forms/DateValidator.java index 055e77952314..255ec6f6f6f3 100644 --- a/forms/qa/integration/forms/DateValidator.java +++ b/forms/qa/integration/forms/DateValidator.java @@ -54,9 +54,7 @@ public class DateValidator extends integration.forms.ControlValidator if ( isDedicatedInvalidDate( dateValue ) ) return false; - if ( !isNextMonthsDate( dateValue ) ) - return false; - return true; + return isNextMonthsDate( dateValue ); } catch( java.lang.Exception e ) { diff --git a/forms/qa/integration/forms/FormComponent.java b/forms/qa/integration/forms/FormComponent.java index 89b8f91b83c7..eb7467a55e59 100644 --- a/forms/qa/integration/forms/FormComponent.java +++ b/forms/qa/integration/forms/FormComponent.java @@ -103,9 +103,7 @@ public class FormComponent /* ------------------------------------------------------------------ */ public String[] getElementNames() { - if ( m_nameAccess != null ) - return m_nameAccess.getElementNames(); - return new String[]{}; + return ( m_nameAccess != null ) ? m_nameAccess.getElementNames() : new String[]{}; } @@ -113,9 +111,7 @@ public class FormComponent /* ------------------------------------------------------------------ */ public int getCount() { - if ( m_indexAccess != null ) - return m_indexAccess.getCount(); - return 0; + return ( m_indexAccess != null ) ? m_indexAccess.getCount() : 0; } /* ------------------------------------------------------------------ */ @@ -149,17 +145,13 @@ public class FormComponent /* ------------------------------------------------------------------ */ public FormComponent getParent() { - if ( m_child != null ) - return new FormComponent( m_child.getParent() ); - return new FormComponent(); + return ( m_child != null ) ? new FormComponent( m_child.getParent() ) : new FormComponent(); } /* ------------------------------------------------------------------ */ public String getName() { - if ( m_named != null ) - return m_named.getName(); - return ""; + return ( m_named != null ) ? m_named.getName() : ""; } /* ------------------------------------------------------------------ */ @@ -167,8 +159,6 @@ public class FormComponent { XServiceInfo si = UnoRuntime.queryInterface( XServiceInfo.class, m_component ); - if ( si != null ) - return si.getImplementationName(); - return ""; + return ( si != null ) ? si.getImplementationName() : ""; } } diff --git a/forms/qa/integration/forms/ListSelectionValidator.java b/forms/qa/integration/forms/ListSelectionValidator.java index ce94a1ed7a86..164110d30f82 100644 --- a/forms/qa/integration/forms/ListSelectionValidator.java +++ b/forms/qa/integration/forms/ListSelectionValidator.java @@ -40,9 +40,7 @@ public class ListSelectionValidator extends integration.forms.ControlValidator try { short[] selectionIndexes = (short[])Value; - if ( selectionIndexes.length > 2 ) - return false; - return true; + return selectionIndexes.length <= 2; } catch( java.lang.Exception e ) { diff --git a/forms/qa/integration/forms/NumericValidator.java b/forms/qa/integration/forms/NumericValidator.java index 25b3ea88e23b..06370cfb30e6 100644 --- a/forms/qa/integration/forms/NumericValidator.java +++ b/forms/qa/integration/forms/NumericValidator.java @@ -48,9 +48,7 @@ public class NumericValidator extends integration.forms.ControlValidator return false; if ( !isProperRange( value ) ) return false; - if ( !isProperDigitCount( value ) ) - return false; - return true; + return isProperDigitCount( value ); } catch( java.lang.Exception e ) { diff --git a/forms/qa/integration/forms/TextValidator.java b/forms/qa/integration/forms/TextValidator.java index d60c1022fa8b..0ed59dd41cfa 100644 --- a/forms/qa/integration/forms/TextValidator.java +++ b/forms/qa/integration/forms/TextValidator.java @@ -44,9 +44,7 @@ public class TextValidator extends integration.forms.ControlValidator String value = (String)Value; if ( containsZs( value ) ) return false; - if ( !isProperChunks( value ) ) - return false; - return true; + return isProperChunks( value ); } catch( java.lang.Exception e ) { @@ -61,10 +59,7 @@ public class TextValidator extends integration.forms.ControlValidator private boolean containsZs( String value ) { - if ( ( value.indexOf( 'Z' ) != -1 ) - || ( value.indexOf( 'z' ) != -1 ) - ) - return true; - return false; + return ( value.indexOf( 'Z' ) != -1 ) + || ( value.indexOf( 'z' ) != -1 ); } } diff --git a/forms/qa/integration/forms/TimeValidator.java b/forms/qa/integration/forms/TimeValidator.java index 673cf6723749..8ff19464ab3f 100644 --- a/forms/qa/integration/forms/TimeValidator.java +++ b/forms/qa/integration/forms/TimeValidator.java @@ -52,9 +52,7 @@ public class TimeValidator extends integration.forms.ControlValidator com.sun.star.util.Time.class, Value); if ( isInvalidTime( timeValue ) ) return false; - if ( !isFullHour( timeValue ) ) - return false; - return true; + return isFullHour( timeValue ); } catch( java.lang.Exception e ) { |