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

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

            People

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

              Dates

              • Created:
                Updated:
                Resolved: