ICEfaces
  1. ICEfaces
  2. ICE-5664

Review use of Wrappers rather than directly subclassing interfaces.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0-Alpha2
    • Fix Version/s: 2.0-Beta1, 2.0.0
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces 2 JSF 2

      Description

      We need to review our strategy of directly subclassing interfaces rather than using the supplied abstract wrappers for delegation:

      public class DOMRenderKit extends RenderKit vs public class DOMRenderKit extends RenderKitWrapper
      public class DOMResponseWriter extends ResponseWriter vs public class DOMResponseWriter extends ResponseWriterWrapper

      In many cases, using the wrapper would give us access to a delegate class where we could take advantage of existing code.

        Issue Links

          Activity

          Deryk Sinotte created issue -
          Deryk Sinotte made changes -
          Field Original Value New Value
          Salesforce Case []
          Fix Version/s 2.0-Beta [ 10231 ]
          Deryk Sinotte made changes -
          Link This issue blocks ICE-5679 [ ICE-5679 ]
          Ken Fyten made changes -
          Assignee Deryk Sinotte [ deryk.sinotte ]
          Deryk Sinotte made changes -
          Salesforce Case []
          Component/s Framework [ 10013 ]
          Assignee Priority P1
          Deryk Sinotte made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Fix Version/s 2.0.0 [ 10230 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Assignee Priority P1

            People

            • Assignee:
              Deryk Sinotte
              Reporter:
              Deryk Sinotte
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: