ICEfaces
  1. ICEfaces
  2. ICE-7550

ACE:DataTable: JavaScript errors in IE7/8 when trying to use the filter & column-ordering features

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0.RC1
    • Fix Version/s: 3.0.RC2, 3.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Verified to be failing at: ICEfaces3 Revision# 26774
      Server: Tomcat6
      Browsers: IE7/8 (JS errors not present in IE9)
      Test pages located at: svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/datatable
    • Assignee Priority:
      P2

      Description

      IE8 - Filter:

      When navigating to the page this JS error is seen:
      Message: Object doesn't support this property or method
      Line: 26
      Char: 1
      Code: 0
      URI: http://localhost:8080/datatable/views/filterTest.jsf

      When Interacting with the filter this JS error is seen:
      Message: Object doesn't support this property or method
      Line: 81142033
      Char: 1
      Code: 0
      URI: http://localhost:8080/datatable/views/filterTest.jsf


      IE8 - Column Order:

      When navigating to the page this JS error is seen:
      Message: Object doesn't support this property or method
      Line: 26
      Char: 1
      Code: 0
      URI: http://localhost:8080/datatable/views/columnOrderTest.jsf

      When trying to re-order the columns this JS error is seen:
      Message: Object doesn't support this property or method
      Line: 80647817
      Char: 1
      Code: 0
      URI: http://localhost:8080/datatable/views/columnOrderTest.jsf

      Similar JS errors are seen in IE7.

        Activity

        Hide
        Nils Lundquist added a comment -

        This issue doesn't break any functionality, and the line numbers given for the errors are invalid locations.

        Given that this issue is browser specific, unintrusive and exceptionally difficult to debug without a location for error, I'm downgrading it until I have ton of free time to step through the entire data table JS with the debugger and see where the error is actually thrown.

        Show
        Nils Lundquist added a comment - This issue doesn't break any functionality, and the line numbers given for the errors are invalid locations. Given that this issue is browser specific, unintrusive and exceptionally difficult to debug without a location for error, I'm downgrading it until I have ton of free time to step through the entire data table JS with the debugger and see where the error is actually thrown.
        Hide
        Nils Lundquist added a comment -

        These features do not cause any bugs in the showcase.

        The problem is with the test case rather than the features.

        Show
        Nils Lundquist added a comment - These features do not cause any bugs in the showcase. The problem is with the test case rather than the features.
        Hide
        Ted Goddard added a comment -

        Saving the page output as source and editing to remove:

        <script type="text/javascript">tbl = new ice.ace.DataTable('tbl',{formId:'j_idt7',filterEvent:'change',paginator:new YAHOO.widget.Paginator(

        {rowsPerPage:5,totalRecords:10,initialPage:1,containers:['tbl_paginatortop','tbl_paginatorbottom'],alwaysVisible:false}

        )});</script>

        avoids the JavaScript error.

        Even the following script fails in this page, so it does not appear to be the YAHOO reference:

        tbl = new ice.ace.DataTable('tbl',

        {formId:'j_idt7',filterEvent:'change'}

        );

        Show
        Ted Goddard added a comment - Saving the page output as source and editing to remove: <script type="text/javascript">tbl = new ice.ace.DataTable('tbl',{formId:'j_idt7',filterEvent:'change',paginator:new YAHOO.widget.Paginator( {rowsPerPage:5,totalRecords:10,initialPage:1,containers:['tbl_paginatortop','tbl_paginatorbottom'],alwaysVisible:false} )});</script> avoids the JavaScript error. Even the following script fails in this page, so it does not appear to be the YAHOO reference: tbl = new ice.ace.DataTable('tbl', {formId:'j_idt7',filterEvent:'change'} );
        Hide
        Ted Goddard added a comment -

        Adding "var" in front of the tbl declaration seems to fix it:

        var tbl = new ice.ace.DataTable('tbl',

        {formId:'j_idt7',filterEvent:'change'}

        );

        runs on IE7 without any errors.

        Show
        Ted Goddard added a comment - Adding "var" in front of the tbl declaration seems to fix it: var tbl = new ice.ace.DataTable('tbl', {formId:'j_idt7',filterEvent:'change'} ); runs on IE7 without any errors.
        Hide
        Nils Lundquist added a comment -

        Fixed in r26826 - Prepended 'var' to JS global table instance declaration.

        Show
        Nils Lundquist added a comment - Fixed in r26826 - Prepended 'var' to JS global table instance declaration.
        Hide
        Carmen Cristurean added a comment -

        This has been verified with code revision 27135 in IE7, IE8, and no issues have been found.

        Show
        Carmen Cristurean added a comment - This has been verified with code revision 27135 in IE7, IE8, and no issues have been found.

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Cruz Miraback
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: