ICEfaces
  1. ICEfaces
  2. ICE-8045

component-showcase - Autocomplete server-side error in IE if using keyboard navigation

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 3.1.0.RC1
          • Fix Version/s: 3.1, EE-3.0.0.GA_P01
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            Icefaces3/trunk revision# 28883
            Server: tomcat6
            Browser: IE7/8 (may be in other IE browsers)
          • Assignee Priority:
            P1

            Description

            In the component-showcase sample app - Autocomplete page:
            The tests pass, however there is a java.lang.NumberFormatException that occurs in the tomcat log when selecting a value from the autocomplete list if using the keyboard (not occurring in the other browsers, or when selecting the value with the mouse).

            java.lang.NumberFormatException: For input string: "undefined"
                    at java.lang.NumberFormatException.forInputString(NumberFormatException.
            java:48)
                    at java.lang.Integer.parseInt(Integer.java:447)
                    at java.lang.Integer.parseInt(Integer.java:497)
                    at com.icesoft.faces.component.ext.KeyEvent.getKeyCode(KeyEvent.java:95)

                    at com.icesoft.faces.component.selectinputtext.SelectInputText.queueEven
            tIfEnterKeyPressed(SelectInputText.java:584)
                    at com.icesoft.faces.component.selectinputtext.SelectInputText.decode(Se
            lectInputText.java:123)
                    at javax.faces.component.UIComponentBase.processDecodes(UIComponentBase.
            java:1181)
                    at javax.faces.component.UIInput.processDecodes(UIInput.java:662)
                    at javax.faces.component.UIComponentBase.processDecodes(UIComponentBase.
            java:1176)
                ..............

              Activity

              Ascending order - Click to sort in descending order
              Carmen Cristurean created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Salesforce Case []
              Fix Version/s 3.1 [ 10312 ]
              Assignee Priority P3
              Assignee yip.ng [ yip.ng ]
              yip.ng made changes -
              Attachment screenshot-01.png [ 14648 ]
              Hide
              Permalink
              yip.ng added a comment - - edited

              This is caused by the keycode becoming undefined when it should have the value for the Enter key: 13. See screenshot-01.png.

              Show
              yip.ng added a comment - - edited This is caused by the keycode becoming undefined when it should have the value for the Enter key: 13. See screenshot-01.png.
              yip.ng made changes -
              Attachment screenshot-02.png [ 14649 ]
              Hide
              Permalink
              yip.ng added a comment -

              This is caused by the framework serialization and submit. See screenshot-02.png.

              Show
              yip.ng added a comment - This is caused by the framework serialization and submit. See screenshot-02.png.
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P3 P2
              Affects Version/s 3.1.0.RC1 [ 10337 ]
              Affects Version/s 3.1 [ 10312 ]
              Assignee yip.ng [ yip.ng ] Mircea Toma [ mircea.toma ]
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P2 P1
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #29958 Thu Jul 12 14:04:43 MDT 2012 mircea.toma ICE-8045 Use Event.srcElement property to detect the event implementation, if IE or Netscape type. Previously the detection relied on the presence of Event.preventDefault() method but since some of the third party libraries (Prototype) try to normalize the API the correct detection of the event type fails.
              Files Changed
              Commit graph MODIFY /projects/bridge-support/src/main/javascript/event.js
              Hide
              Permalink
              Mircea Toma added a comment -

              The issue is caused by the false detection of the event type. Before submit the captured events are wrapped so that they provide a uniform API, regardless of the browser type. The detection of IE events was relying on the missing of the Event.preventDefault() method but Prototype is creating this method, trying to provide a common API based on the Netscape event API.

              The fix modifies the IE event detection by using Event.srcElement property instead of Event.preventDefault() method.

              Show
              Mircea Toma added a comment - The issue is caused by the false detection of the event type. Before submit the captured events are wrapped so that they provide a uniform API, regardless of the browser type. The detection of IE events was relying on the missing of the Event.preventDefault() method but Prototype is creating this method, trying to provide a common API based on the Netscape event API. The fix modifies the IE event detection by using Event.srcElement property instead of Event.preventDefault() method.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s EE-3.0.0.GA_P01 [ 10327 ]
              Ken Fyten made changes -
              Salesforce Case []
              Security Private [ 10001 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Carmen Cristurean
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: