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

        Mark Collette created issue -
        Mark Collette made changes -
        Field Original Value New Value
        Fix Version/s 1.6 [ 10031 ]
        Assignee Priority P1
        Assignee Mark Collette [ mark.collette ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #14082 Thu Jun 07 13:26:02 MDT 2007 mark.collette ICE-1656 : JSP Tags have wrong primitive attribute metadata
        Files Changed
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_properties/ice-dataTable-props.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-accordion-props.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-rowselector-props.xml
        Commit graph MODIFY /icefaces/trunk/icefaces/component-metadata/src/main/resources/conf/ice_cust_properties/cust-effect-props.xml
        Hide
        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
        Mark Collette made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 1.6DR#6 [ 10090 ]
        Fix Version/s 1.6 [ 10031 ]
        Ken Fyten made changes -
        Fix Version/s 1.6 [ 10031 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P1
        Assignee Mark Collette [ mark.collette ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: