Uploaded image for project: 'MyFaces Trinidad'
  1. MyFaces Trinidad
  2. TRINIDAD-1659

Maven 2.0 plugin needs to add support for the new JSF 2.0 faces-config element <partial-view-context-factory> element

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • New Feature
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.0.0-plugins
    • 2.0.0-plugins
    • Plugins
    • None
    • Windows XP / JSF 2.0

    Description

      1. When tryjng to register a new factory - PartialViewContextFactory - in faces-config-base.xml in trinidad-impl, I noticed that the generated faces-config.xml did not have the factory element I added. It should look like this (see last element)

      <factory>
      <faces-context-factory>org.apache.myfaces.trinidadinternal.context.FacesContextFactoryImpl</faces-context-factory>
      <render-kit-factory>org.apache.myfaces.trinidadinternal.renderkit.CoreRenderKitFactory</render-kit-factory>
      <partial-view-context-factory>org.apache.myfaces.trinidadinternal.context.PartialViewContextFactoryImpl</partial-view-context-factory>
      </factory>

      2. Turns out the transform20.xsl used to parse and generate the faces-config was not looking for this new element. Code changes were required in transform20.xsl (maven-faces-plugin\src\main\resources\org\apache\myfaces\trinidadbuild\plugin\faces\resources)

      3. Making the necessary updates in the file, compiling and using the plugin JARS to build trinidad, worked. AS in the generated faces-config.xml did contain the entry I added for the <partial-view-context-factory>

      4. See attached patch

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            matzew Matthias Wessendorf
            pasubra Pavitra Subramaniam
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment