Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0
    • Fix Version/s: 1.2.0
    • Labels:
      None

      Description

      @Dependent beans must be interceptable, although implementations are not required to provide a client proxy for them.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        93d 22h 35m 1 Gurkan Erdogdu 31/Jan/10 14:15
        Closed Closed Reopened Reopened
        1m 56s 1 Gurkan Erdogdu 31/Jan/10 14:17
        Reopened Reopened Resolved Resolved
        1084d 14h 5m 1 Mark Struberg 20/Jan/13 04:22
        Resolved Resolved Closed Closed
        157d 2h 55m 1 Mark Struberg 26/Jun/13 08:18
        Mark Struberg made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Mark Struberg added a comment -

        shipped in OWB-1.2.0

        Show
        Mark Struberg added a comment - shipped in OWB-1 .2.0
        Mark Struberg made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Mark Struberg added a comment -

        After moving the Interceptor + Decorator creation to the InjectionTarget this finally works!

        Show
        Mark Struberg added a comment - After moving the Interceptor + Decorator creation to the InjectionTarget this finally works!
        Hide
        Mark Struberg added a comment -

        will get resolved while fixing OWB-344

        Show
        Mark Struberg added a comment - will get resolved while fixing OWB-344
        Mark Struberg made changes -
        Assignee Joe Bergmark [ bergmark ] Mark Struberg [ struberg ]
        Mark Struberg made changes -
        Fix Version/s 1.2.0 [ 12315461 ]
        Fix Version/s 1.1.0 [ 12315058 ]
        Hide
        Mark Struberg added a comment -

        This is still not working.
        Comment in the section

        Assert.assertTrue(DependentInterceptor.refCount == 1);

        in DependentInterceptorTest#testLifecycle() to see what I mean.

        Show
        Mark Struberg added a comment - This is still not working. Comment in the section Assert.assertTrue(DependentInterceptor.refCount == 1); in DependentInterceptorTest#testLifecycle() to see what I mean.
        Mark Struberg made changes -
        Fix Version/s 1.1.0 [ 12315058 ]
        Fix Version/s 1.0.0-GA [ 12315107 ]
        Mark Struberg made changes -
        Fix Version/s 1.0.0-GA [ 12315107 ]
        Fix Version/s 1.0.0-alpha-2 [ 12315104 ]
        Mark Struberg made changes -
        Affects Version/s 1.0.0-GA [ 12315107 ]
        Affects Version/s 1.0.0-alpha-1 [ 12313618 ]
        Hide
        Mark Struberg added a comment -

        moved to 1.0.0-GA

        Show
        Mark Struberg added a comment - moved to 1.0.0-GA
        Gurkan Erdogdu made changes -
        Fix Version/s 1.0.0-CR2 [ 12315104 ]
        Fix Version/s 1.0.0-CR1 [ 12313618 ]
        Affects Version/s 1.0.0-CR1 [ 12313618 ]
        Affects Version/s M3 [ 12313615 ]
        Gurkan Erdogdu made changes -
        Fix Version/s 1.0.0 [ 12313618 ]
        Fix Version/s M4 [ 12313616 ]
        Gurkan Erdogdu made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Hide
        Gurkan Erdogdu added a comment -

        For tracking

        Show
        Gurkan Erdogdu added a comment - For tracking
        Hide
        Gurkan Erdogdu added a comment -

        I have re-opened this issue. I have tried your concern in RI and it does not work either. I am re-opening this issue because of tracking.

        Show
        Gurkan Erdogdu added a comment - I have re-opened this issue. I have tried your concern in RI and it does not work either. I am re-opening this issue because of tracking.
        Gurkan Erdogdu made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Joe Bergmark made changes -
        Attachment publicFieldTest.patch [ 12430149 ]
        Hide
        Joe Bergmark added a comment -

        I do not believe the current commit completely solve all of the problems. There are two things that I believe should work with dependent scoped beans.

        1) public fields in the bean should work as expected. If we shield a bean with a proxy in the middle, then public field operations occur on the proxy, but the actual method calls flow through into the actual bean instance. This leads to strange behavior. I'll attach a testcase I wrote while I was working on this.

        2) To my knowledge dependent scoped beans don't require a default constructor. 3.1.3 allows for constructors annotated with @Inject, and I believe only 5.4.1 restricts this further. The committed change to LifecycleBean looks like the same problem I ran into in my first couple attempts.

        I'm still working on this area myself, just wanted to share some of my thoughts.

        Show
        Joe Bergmark added a comment - I do not believe the current commit completely solve all of the problems. There are two things that I believe should work with dependent scoped beans. 1) public fields in the bean should work as expected. If we shield a bean with a proxy in the middle, then public field operations occur on the proxy, but the actual method calls flow through into the actual bean instance. This leads to strange behavior. I'll attach a testcase I wrote while I was working on this. 2) To my knowledge dependent scoped beans don't require a default constructor. 3.1.3 allows for constructors annotated with @Inject, and I believe only 5.4.1 restricts this further. The committed change to LifecycleBean looks like the same problem I ran into in my first couple attempts. I'm still working on this area myself, just wanted to share some of my thoughts.
        Hide
        Gurkan Erdogdu added a comment -

        I have implemented this. I have committed it.

        Show
        Gurkan Erdogdu added a comment - I have implemented this. I have committed it.
        Gurkan Erdogdu made changes -
        Fix Version/s M4 [ 12313616 ]
        Affects Version/s M3 [ 12313615 ]
        Affects Version/s M4 [ 12313616 ]
        Joe Bergmark made changes -
        Field Original Value New Value
        Assignee Gurkan Erdogdu [ gerdogdu ] Joe Bergmark [ bergmark ]
        Hide
        Joe Bergmark added a comment -

        At the same time make them decoratable as well.

        Show
        Joe Bergmark added a comment - At the same time make them decoratable as well.
        Eric Covener created issue -

          People

          • Assignee:
            Mark Struberg
            Reporter:
            Eric Covener
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

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

                Development