ICEfaces
  1. ICEfaces
  2. ICE-8949

Icefaces3 sample regression - showcase ace:autoCompleteEntry - using ' (apostrophe) char causes JS syntax error

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2, EE-3.2.0.BETA
    • Fix Version/s: EE-3.2.0.GA, 3.3
    • Component/s: ACE-Components, Sample Apps
    • Labels:
      None
    • Environment:
      Icefaces3 trunk revision#33280
      Server: tomcat6
      Browser: FF3.6, IE7 & GoogleChrome24
      Last Known Working Revision: Unknown
    • Assignee Priority:
      P1

      Description

      AutoComplete - All - (All browsers) - when using the ' (apostrophe) character on autocomplete, JS syntax error below occurs:

      Error: SyntaxError: missing ) after argument list
      Source File: http://localhost:8080/showcase/showcase.jsf?grp=aceMenu&exp=autoCompleteEntryBean
      Line: 0, Column: 60
      Source Code:
      completers["inputForm:autoComplete"];instance.updateField(''S-Hertogenbosch', true);})();


      To Reproduce:
      -Build and deploy showcase app on Tomcat7
      -Navigate to showcase page and go to any AutoComplete feature page
      -input ' (apostrophe) char to render autocompleteEntry (Note: In FF18, Chrome24 or IE9, must have error console visible in order to see this error)
      -Notice the JS syntax error occur within the error console

      Note: Moj 2.1.18 Library was used when this issue was detected

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #33302 Fri Feb 01 12:08:09 MST 2013 art.zambrano ICE-8949 escaped single quotes in updateField() call
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/autocompleteentry/AutoCompleteEntryRenderer.java

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Jerome Ruzol
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: