<div dir="ltr">Discussion moved to pull request: <a href="https://github.com/windup/windup/pull/228">https://github.com/windup/windup/pull/228</a></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Sep 6, 2014 at 9:01 AM, Ondrej Zizka <span dir="ltr"><<a href="mailto:ozizka@redhat.com" target="_blank">ozizka@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Problem with WindupConfigModel is that *currently* it needs context<br>
initialized, since it is initialized like this:<br>
<br>
WindupConfigurationModel config =<br>
GraphService.getConfigurationModel(context);<br>
<br>
And at the same time this config "DTO" needs to be passed into the<br>
top-level API (currently WindupProcessor).<br>
<br>
I can see 2 solutions:<br>
<br>
A) Continue using WCM, implement WINDUP-147 GraphService.merge()<br>
to store classes implementing the model interface<br>
so that supports adjacent models (needed for the scan packages)<br>
<br>
B) Have WindupConfig, and copy stuff to WCM (which is the same as A,<br>
only done manually)<br>
<br>
C) What I am currently doing, an ugly hack - passing final variables<br>
into GraphLifecycleListener<br>
<br>
Ondra<br>
<br>
_______________________________________________<br>
windup-dev mailing list<br>
<a href="mailto:windup-dev@lists.jboss.org">windup-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/windup-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/windup-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>"Simpler is better."
</div>