Mailing-List: contact dev-help@zest.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@zest.apache.org
Message-ID: <5649F52E.70005@nosphere.org>
Date: Mon, 16 Nov 2015 16:24:30 +0100
From: Paul Merlin <paul@nosphere.org>
User-Agent: Postbox 3.0.11 (Macintosh/20140602)
MIME-Version: 1.0
To: dev@zest.apache.org
Subject: Re: Bug in @This injections??
References: 
 <CADmm+Ke+HokrqkkciMxd+vRS=s0a4sBMV=QHrp_jfrWuH1Tz9g@mail.gmail.com>
In-Reply-To: 
 <CADmm+Ke+HokrqkkciMxd+vRS=s0a4sBMV=QHrp_jfrWuH1Tz9g@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Hey,

Catching up ...

Niclas Hedhman a écrit :
> I think that "Private @This" doesn't work for Constructor injections at the
> moment.
>
> Not sure if that is because it can't be done, or because the implementation
> is lacking the feature.
>
> Looking at the ThisInjectionProviderFactory, I can't figure out how this is
> supposed to work at all. And if anyone has interest, please enlighten me of
> what we ned to do to support this properly. The previous decorator-feature
> I mention, depends on this :-(

Niclas Hedhman a écrit :
> So, for @This injection in fields, the type is somehow added to the
> Composite, but when it is injected in the Constructor, it seems to only
> take the public  Composite type interfaces...
 
I tried to understand/reproduce the issue but couldn't.
Also added some tests in ConstructorInjectionOfThisTest and
PrivateMixinTest by the way, with no luck. The latter existing test was
wrong as under-test Work fragment was declared as a *public* mixin.

/Paul