MyFaces Core
  1. MyFaces Core
  2. MYFACES-1598

JSF 1.2 TLD compliance: attributes "binding" and "converter" with wrong deferred-type

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2.0
    • Fix Version/s: 1.2.0
    • Component/s: JSR-252
    • Labels:
      None

      Description

      For "binding" the deferred-type must be 'javax.faces.component.UIComponent'
      For "converter" the deferred-type must be 'javax.faces.convert.Converter'

      Now the deferred-type tags in the tld for these attributes are empty (hence expeting java.jang.Object)

      1. MYFACES-1598.patch
        0.8 kB
        Tim McConnell
      2. MYFACES-1598-ADFFACES.patch
        6 kB
        Tim McConnell

        Activity

        Hide
        Tim McConnell added a comment -

        Two patches are provided to resolve this defect:

        1- A change for MYFACES proper to change the deferred type for the "binding" (the "converter" deferred type was already correctly specified, but requires #2 below).
        2- A change for ADFFACES to invlude the "type" value for "binding" and "converter" property names.

        Show
        Tim McConnell added a comment - Two patches are provided to resolve this defect: 1- A change for MYFACES proper to change the deferred type for the "binding" (the "converter" deferred type was already correctly specified, but requires #2 below). 2- A change for ADFFACES to invlude the "type" value for "binding" and "converter" property names.
        Hide
        Tim McConnell added a comment -

        I opened a ADFFACES plugin companion defect (ADFFACES-477) for the ADFFACES patch, which can now be ignored. The MYFACES patch still needs to be applied though but I would prefer that the ADFFACES patch be reviewed and applied first to ensure I'm fixing this in the proper manner. Thanks.

        Show
        Tim McConnell added a comment - I opened a ADFFACES plugin companion defect ( ADFFACES-477 ) for the ADFFACES patch, which can now be ignored. The MYFACES patch still needs to be applied though but I would prefer that the ADFFACES patch be reviewed and applied first to ensure I'm fixing this in the proper manner. Thanks.
        Hide
        Bruno Aranda added a comment -

        Path applied. Thanks Tim!

        Show
        Bruno Aranda added a comment - Path applied. Thanks Tim!

          People

          • Assignee:
            Tim McConnell
            Reporter:
            Bruno Aranda
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development