Uploaded image for project: 'CDI Specification Issues'
  1. CDI Specification Issues
  2. CDI-28

Abstract methods on decorators

        Details

        • Type: Bug
        • Status: Closed (View Workflow)
        • Priority: Major
        • Resolution: Done
        • Affects Version/s: 1.0
        • Fix Version/s: 1.1.EDR
        • Component/s: Decorators
        • Labels:
          None

          Description

          If a method is abstract:

          • if it is a method of a decorated type, it has a generated implementation that calls the delegate
          • otherwise, definition error

            Gliffy Diagrams

              Issue Links

              is related to

              Bug - A problem which impairs or prevents the functions of the product. WELD-1021 Implicit implementation of abstract decorator method with at least one parameter is missing

              • Major - A request that should be considered seriously but is not a show stopper.
              • Resolved
              relates to

              Feature Request - A new feature of the product, which has yet to be developed. CDI-110 Provide support for binding an invocation handler to an interface or abstract class

              • Major - A request that should be considered seriously but is not a show stopper.
              • Open

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                gastaldi George Gastaldi added a comment -

                We should think about it when using Service Handlers

                Show
                gastaldi George Gastaldi added a comment - We should think about it when using Service Handlers
                Hide
                Permalink
                pmuir Pete Muir added a comment -

                Do you want to apply Gavin's change Marius?

                Show
                pmuir Pete Muir added a comment - Do you want to apply Gavin's change Marius?
                Hide
                Permalink
                marius.bogoevici Marius Bogoevici added a comment - - edited

                George, Service Handlers is a different story (with the potential of being complementary if you will). Here we just leave decorator implementors to forgo stuff like

                void decoratedMethod() {
                		 
                  return delegate.decoratedMethod();
                		 
                }

                and leave decoratedMethod() unimplemented instead.

                Show
                marius.bogoevici Marius Bogoevici added a comment - - edited George, Service Handlers is a different story (with the potential of being complementary if you will). Here we just leave decorator implementors to forgo stuff like void decoratedMethod() { return delegate.decoratedMethod(); } and leave decoratedMethod() unimplemented instead.
                Hide
                Permalink
                marius.bogoevici Marius Bogoevici added a comment -

                https://github.com/jboss/cdi/pull/26

                Show
                marius.bogoevici Marius Bogoevici added a comment - https://github.com/jboss/cdi/pull/26
                Hide
                Permalink
                pmuir Pete Muir added a comment -

                We should hold this one open for 14 days.

                Show
                pmuir Pete Muir added a comment - We should hold this one open for 14 days.

                  People

                  • Assignee:
                    marius.bogoevici Marius Bogoevici
                    Reporter:
                    pmuir Pete Muir
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved:

                      Development