ICEfaces
  1. ICEfaces
  2. ICE-958

Columns component demo under component-showcase does two render/update cycle

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.6DR#1, 1.6
    • Component/s: Sample Apps
    • Labels:
      None
    • Environment:
      Operating System: All
      Platform: All

      Description

      Mark noticed that the "columns" demo under component-showcase, is causing two
      render/update cycles when the new value being entered for the columns
      inputText.

      Patrick noticed that if you update the columns, application responses in a
      sluggish manners.

        Activity

        Hide
        Adnan Durrani added a comment -

        I have created a test case : -r11389
        \svn\repo\support\iraptor\1000\977

        This test case have 500 items as child, and seem to be working with the
        adequate response time. This make me think that the problem could be
        application related.

        The component is also doing only one submit cycle.

        Show
        Adnan Durrani added a comment - I have created a test case : -r11389 \svn\repo\support\iraptor\1000\977 This test case have 500 items as child, and seem to be working with the adequate response time. This make me think that the problem could be application related. The component is also doing only one submit cycle.
        Hide
        Patrick Corless added a comment -

        Please take a look at this one. I suspect there could be application work being
        done on a getter. Please review the demo

        Show
        Patrick Corless added a comment - Please take a look at this one. I suspect there could be application work being done on a getter. Please review the demo
        Hide
        Arran McCullough added a comment -

        Changed to use a 2D array. The array is instantiated once and then is read by
        the getCellValue method. The getCellValue method was calculating the char
        value. This is now only calculated when the updateTableColumns method is called.

        Revision #12998

        Show
        Arran McCullough added a comment - Changed to use a 2D array. The array is instantiated once and then is read by the getCellValue method. The getCellValue method was calculating the char value. This is now only calculated when the updateTableColumns method is called. Revision #12998
        Hide
        Patrick Corless added a comment -

        The column headers labels do not appear anymore. Re-opening bug.

        Show
        Patrick Corless added a comment - The column headers labels do not appear anymore. Re-opening bug.
        Hide
        Patrick Corless added a comment -

        Also please add a cap to the number of columns. setting it high number will pin
        the server.

        • consider using a JSF validator to make sure the number stays under ~25 columns
        • or use a select one component.
        Show
        Patrick Corless added a comment - Also please add a cap to the number of columns. setting it high number will pin the server. consider using a JSF validator to make sure the number stays under ~25 columns or use a select one component.
        Hide
        Arran McCullough added a comment -

        Column headings are back, and a cap of 26 has been implemented.

        Revision #13007

        Show
        Arran McCullough added a comment - Column headings are back, and a cap of 26 has been implemented. Revision #13007

          People

          • Assignee:
            Unassigned
            Reporter:
            Adnan Durrani
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: