ICEfaces
  1. ICEfaces
  2. ICE-6890

selectInputText rows=0 should display all in popup list instead of none

    Details

      Description

      According to TLD doc for rows attribute: "If this value is set to zero, all available rows in the underlying data model will be displayed." Seems there is a bug. Zero rows will display nothing.

        Activity

        Hide
        yip.ng added a comment - - edited

        Done. "while" condition simplified and sped up as well. Also, wrong spelling of variable names corrected.

        Note that when the popup list is dynamically provided by the user, "all rows" means all matched rows provided by the user, not all rows in the original data list.

        Revision: 24632


        Modified : /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/selectinputtext/SelectInputTextRenderer.java

        Revision: 24633


        Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/selectinputtext/SelectInputTextRenderer.java

        Revision: 24642


        Modified : /icefaces2/branches/icefaces-2.0.x-maintenance/icefaces/compat/components/src/main/java/com/icesoft/faces/component/selectinputtext/SelectInputTextRenderer.java

        Show
        yip.ng added a comment - - edited Done. "while" condition simplified and sped up as well. Also, wrong spelling of variable names corrected. Note that when the popup list is dynamically provided by the user, "all rows" means all matched rows provided by the user, not all rows in the original data list. Revision: 24632 Modified : /icefaces/trunk/icefaces/component/src/com/icesoft/faces/component/selectinputtext/SelectInputTextRenderer.java Revision: 24633 Modified : /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/selectinputtext/SelectInputTextRenderer.java Revision: 24642 Modified : /icefaces2/branches/icefaces-2.0.x-maintenance/icefaces/compat/components/src/main/java/com/icesoft/faces/component/selectinputtext/SelectInputTextRenderer.java

          People

          • Assignee:
            yip.ng
            Reporter:
            yip.ng
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: