[ICE-7985] Rowselector delault isKeyboardNavigationEnabled returns invalid value/ - ICEsoft JIRA Issue Tracker

Details

Workaround Description:

Hide
Patch
public boolean isKeyboardNavigationEnabled() {
        if (keyboardNavigationEnabled != null) {
            return keyboardNavigationEnabled.booleanValue();
        }
        ValueBinding vb = getValueBinding("keyboardNavigationEnabled");
        Boolean boolVal = vb != null ?
                (Boolean) vb.getValue(getFacesContext()) : null;
         return boolVal != null ? boolVal.booleanValue() : false;
    }

Show
Patch public boolean isKeyboardNavigationEnabled() {         if (keyboardNavigationEnabled != null) {             return keyboardNavigationEnabled.booleanValue();         }         ValueBinding vb = getValueBinding("keyboardNavigationEnabled");         Boolean boolVal = vb != null ?                 (Boolean) vb.getValue(getFacesContext()) : null;          return boolVal != null ? boolVal.booleanValue() : false;     }

According to tld keyboardNavigationEnabled should be disabled by default for backward compatibility

But in file http://anonsvn.icesoft.org/repo/icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/ext/RowSelector.java
we can see
public boolean isKeyboardNavigationEnabled() {
(...)
return boolVal != null ? boolVal.booleanValue() : true;
}

Correct line should return false.

Hide

Permalink

Krashan Brahmanjara added a comment - 17/Apr/12 3:27 AM

This patch can be invalidated after adding patch from ~~ICE-8013~~

Show

Krashan Brahmanjara added a comment - 17/Apr/12 3:27 AM This patch can be invalidated after adding patch from ICE-8013