Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.7
    • Component/s: None
    • Labels:
      None

      Description

      Hi,

      the setter for the attributes sortable, defaultSorted, groupBy are Boolean instead of boolean, which causes a exception if using this attributes.

      Regards,
      Martin Haimberger

        Activity

        Martin Haimberger created issue -
        Martin Haimberger made changes -
        Field Original Value New Value
        Assignee Martin Haimberger [ martin.haimberger ]
        Martin Haimberger made changes -
        Attachment HtmlColumns-patch_issue_1190.patch [ 12374955 ]
        Martin Haimberger made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Martin Marinschek added a comment -

        They should be Boolean - the code causing the exception (in the renderer?) is wrong.

        regards,

        Martin

        Show
        Martin Marinschek added a comment - They should be Boolean - the code causing the exception (in the renderer?) is wrong. regards, Martin
        Hide
        Martin Haimberger added a comment -

        Hi Martin,

        i have corrected the problem with patch v2. Now they are Boolean Objects (the convention is stays for a boolean and not a Boolean).

        regards,
        Martin Haimberger

        Show
        Martin Haimberger added a comment - Hi Martin, i have corrected the problem with patch v2. Now they are Boolean Objects (the convention is stays for a boolean and not a Boolean). regards, Martin Haimberger
        Martin Haimberger made changes -
        Attachment HtmlColumns-patch_issue_1190_v2.patch [ 12375062 ]
        Martin Haimberger made changes -
        Attachment HtmlColumns-patch_issue_1190.patch [ 12374955 ]
        Hide
        Leonardo Uribe added a comment -

        This exception is thrown on:

        javax.faces.component._ComponentAttributesMap.setComponentProperty(....)

        The code is here

        private void setComponentProperty(PropertyDescriptor propertyDescriptor, Object value)
        {
        Method writeMethod = propertyDescriptor.getWriteMethod();
        if (writeMethod == null)

        { throw new IllegalArgumentException("Component property " + propertyDescriptor.getName() + " is not writable"); }

        The problem was on the tag class, because it does not convert sortable to the proper value. With myfaces-builder-plugin, this tag class is now generated, so this problem was solved on TOMAHAWK-1259. I have tested and now this is not thrown.

        Show
        Leonardo Uribe added a comment - This exception is thrown on: javax.faces.component._ComponentAttributesMap.setComponentProperty(....) The code is here private void setComponentProperty(PropertyDescriptor propertyDescriptor, Object value) { Method writeMethod = propertyDescriptor.getWriteMethod(); if (writeMethod == null) { throw new IllegalArgumentException("Component property " + propertyDescriptor.getName() + " is not writable"); } The problem was on the tag class, because it does not convert sortable to the proper value. With myfaces-builder-plugin, this tag class is now generated, so this problem was solved on TOMAHAWK-1259 . I have tested and now this is not thrown.
        Leonardo Uribe made changes -
        Assignee Martin Haimberger [ martin.haimberger ] Leonardo Uribe [ lu4242 ]
        Fix Version/s 1.1.7-SNAPSHOT [ 12312361 ]
        Status Patch Available [ 10002 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Leonardo Uribe made changes -
        Fix Version/s 1.1.7-SNAPSHOT [ 12312361 ]
        Fix Version/s 1.1.7 [ 12313398 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        1m 1s 1 Martin Haimberger 07/Feb/08 10:07
        Patch Available Patch Available Closed Closed
        127d 8h 29m 1 Leonardo Uribe 13/Jun/08 19:36

          People

          • Assignee:
            Leonardo Uribe
            Reporter:
            Martin Haimberger
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development