ICEfaces
  1. ICEfaces
  2. ICE-2048

panelCollapsible should be using primitive boolean type for expanded attribute instead of Boolean object type

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.6.1
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      panelCollapsible + expanded

      Description

      The panelCollapsible is using the Boolean object type for the expanded attribute, which does not align with the API of other components. In addition to that it is not convenient to use in the bean. To make a decision based on expanded state the following call needs to be made:

      if (component.isExpanded().booleanValue()) {
           ....
      }

      instead of

      if (component.isExpanded()) {
       ....
      }

        Activity

        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Adnan Durrani [ adnan.durrani ]
        Adnan Durrani made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 1.6.1 [ 10070 ]
        Resolution Fixed [ 1 ]
        Hide
        Adnan Durrani added a comment -

        fixed revision # 14617:

        The expanded attribute has been changed to use the primitive boolean type. This change might require some modification to the existing applications.

        1- bean.
        component.isExpanded().booleanValue() should be replaced with component.isExpanded()

        2- JSP
        <ice:panelCollapsible expanded="#

        {true}

        "> should be replaced with <ice:panelCollapsible expanded="true">

        Show
        Adnan Durrani added a comment - fixed revision # 14617: The expanded attribute has been changed to use the primitive boolean type. This change might require some modification to the existing applications. 1- bean. component.isExpanded().booleanValue() should be replaced with component.isExpanded() 2- JSP <ice:panelCollapsible expanded="# {true} "> should be replaced with <ice:panelCollapsible expanded="true">
        Adnan Durrani made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Adnan Durrani made changes -
        Field Original Value New Value
        Assignee Adnan Durrani [ adnan.durrani ]
        Adnan Durrani created issue -

          People

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

            Dates

            • Created:
              Updated:
              Resolved: