ICEfaces
  1. ICEfaces
  2. ICE-9198

REGRESSION: ace:dataTable - JS errors on table config panel test page

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.3
    • Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces3 trunk revision# 34435
      Browsers: All
    • Assignee Priority:
      P2

      Description

      To reproduce:
      1) Build / deploy test app located at: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dataTable
      2) Navigate to 'Config Panel Test'

      First JS error is seen:
      this.$table.offset() is undefined
      http://localhost:8080/dataTable/javax.faces.resource/util/ace-datatable.js.jsf?ln=icefaces.ace&v=3_3_0_130412
      Line 3

      3) Open table config panel
      Second JS error is seen:
      a is undefined
      http://localhost:8080/dataTable/javax.faces.resource/util/ace-datatable.js.jsf?ln=icefaces.ace&v=3_3_0_130412
      Line 3

      This issue can not be reproduced in Showcase sample.

        Activity

        Hide
        Nils Lundquist added a comment -

        Revision #35627
        Committed by nils.lundquist
        Moments ago
        ICE-9198 - Added delay to tableConfigPanel initialization to prevent initalizing before the table. This could still be subject to error and should be converted to lazy initialization eventually.

        Show
        Nils Lundquist added a comment - Revision #35627 Committed by nils.lundquist Moments ago ICE-9198 - Added delay to tableConfigPanel initialization to prevent initalizing before the table. This could still be subject to error and should be converted to lazy initialization eventually.
        Hide
        Cruz Miraback added a comment -

        Confirmed fixed using ICEfaces3 trunk revision# 35632.

        Show
        Cruz Miraback added a comment - Confirmed fixed using ICEfaces3 trunk revision# 35632.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: