ICEfaces
  1. ICEfaces
  2. ICE-2327

Component Showcase Selection demo doesn't handle empty country selection

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 1.6
          • Fix Version/s: 1.7
          • Component/s: Sample Apps
          • Labels:
            None
          • Environment:
            Component-showcase, Safari
          • Affects:
            Sample App./Tutorial

            Description

            The component-showcase sample app. fails with an NPE displayed on the browser if you select a country in the Selection demo, and then deselect it (no country selected after one previously was). Based on my testing only Safari allows you to toggle/deselect a selected item in the select list input component, but the app. should handle this possibility.

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Ken Fyten added a comment -

              Another one to review with the new Showcase.

              Show
              Ken Fyten added a comment - Another one to review with the new Showcase.
              Hide
              Permalink
              Patrick Corless added a comment -

              Refactoring effort for the selection demo fixed this issue. Countries and Cities can be selected with out having to worry about null pointers in the valueChangeListener method logic.

              Show
              Patrick Corless added a comment - Refactoring effort for the selection demo fixed this issue. Countries and Cities can be selected with out having to worry about null pointers in the valueChangeListener method logic.

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Ken Fyten
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: