ICEfaces
  1. ICEfaces
  2. ICE-1656

JSP Tags have wrong primitive attribute metadata

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.6DR#5
          • Fix Version/s: 1.6DR#6, 1.6
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            WinXP SP2, Firefox 1.5.0.11, JBoss 4.0.4 GA, JSP component-showcase

            Description

            In icefaces\component-metadata\src\main\resources\conf there are a bunch of xml files for the components, with <property-class>java.lang.Boolean</property-class> sections. It should be "boolean" not "java.lang.Boolean". This might affect all primitives, not just boolean. These attributes work with Facelets but not JSP.

              Activity

              Hide
              Permalink
              Mark Collette added a comment -

              I searched for all primitive wrappers: java.lang.

              {Boolean, Byte, Character, Double, Float, Integer, Long, Number, Short, Void}

              and only found some Boolean and Integer ones. I tested them before and after making the change.

              java.lang.Boolean -> boolean
              ice:accordion.toggleOnClick
              ice:effect.fire
              ice:effect.autoReset
              ice:effect.transitory
              ice:effect.submit
              ice:dataTable.scrollable

              java.lang.Integer -> int
              ice:effect.sequenceNumber
              ice:rowSelector.clickedRow

              Subversion 14082
              icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-accordion-props.xml
              icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-effect-props.xml
              icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-rowselector-props.xml
              icefaces\component-metadata\src\main\resources\conf\ice_properties\ice-dataTable-props.xml

              Show
              Mark Collette added a comment - I searched for all primitive wrappers: java.lang. {Boolean, Byte, Character, Double, Float, Integer, Long, Number, Short, Void} and only found some Boolean and Integer ones. I tested them before and after making the change. java.lang.Boolean -> boolean ice:accordion.toggleOnClick ice:effect.fire ice:effect.autoReset ice:effect.transitory ice:effect.submit ice:dataTable.scrollable java.lang.Integer -> int ice:effect.sequenceNumber ice:rowSelector.clickedRow Subversion 14082 icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-accordion-props.xml icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-effect-props.xml icefaces\component-metadata\src\main\resources\conf\ice_cust_properties\cust-rowselector-props.xml icefaces\component-metadata\src\main\resources\conf\ice_properties\ice-dataTable-props.xml

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Mark Collette
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: