Uploaded image for project: 'MyFaces Core'
  1. MyFaces Core
  2. MYFACES-3589

Allow multiple resolutions of cc into the same EL expression

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.1.8
    • 2.0.15, 2.1.9
    • None
    • None

    Description

      See MYFACES-2651.
      In particular, there is still a case in which the proposed solution won't work.
      Consider the component test 1 slightly modified:

      <cc:interface name="test1">
      <cc:attribute name="test" required="true" />
      </cc:interface>
      <cc:implementation>
      test1
      <t2:test2 test="#

      {myLibrary:myFunction(...) or cc.attrs.test}

      " />
      </cc:implementation>

      In that case the regex expression looking for cc.attrs will exclude the expression because cc.attrs is preceded by a left parenthesis (as stated in CompositeComponentELUtils).

      Changing to
      <t2:test2 test="#

      {cc.attrs.test or myLibrary:myFunction(...)}

      " />
      will work.

      Attachments

        1. ASF.LICENSE.NOT.GRANTED--MYFACES-2561.patch
          21 kB
          marco fago
        2. ASF.LICENSE.NOT.GRANTED--MYFACES-2561-2.patch
          42 kB
          marco fago
        3. ASF.LICENSE.NOT.GRANTED--test.xhtml
          0.5 kB
          marco fago
        4. ASF.LICENSE.NOT.GRANTED--test1.xhtml
          0.9 kB
          marco fago
        5. ASF.LICENSE.NOT.GRANTED--test2.xhtml
          0.8 kB
          marco fago

        Issue Links

          Activity

            People

              jakobkorherr Jakob Korherr
              marco.fago marco fago
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: