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

Optimize ChangeManager.createDocumentChange() implementation

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.1.0-core
    • 2.1.1-core
    • None
    • None

    Description

      Currently the implementation of org.apache.myfaces.trinidad.change ChangeManager.createDocumentChange( ComponentChange change) does not account for fact that the supplied ComponentChange implementation can also be implementing DocumentChange. Improvement is to do this check first, type cast the supplied component to DocumentChange and return.

      There are several ComponentChange implementations in Trinidad that actually also implement DocumentChange, so this is common usecase.

      Currently clients need to do this check outside of this call, which can be moved in here.

      Attachments

        1. TRINIDAD-2448_over_trunk.patch
          2 kB
          Prakash Udupa

        Activity

          People

            jeanne.waldman@oracle.com Jeanne Waldman
            pudupa Prakash Udupa
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - 1h
                1h
                Remaining:
                Remaining Estimate - 1h
                1h
                Logged:
                Time Spent - Not Specified
                Not Specified