Thanks. You are correct about this behavior. Now to find time to fix it! Is this something you might be able to help with?<br><br>I've been meaning to take a look at the deltaspike plugin. A test case would make this much easier to debug!<br>
<br>Thanks so much for finding this!<br>~Lincoln<br><br><div class="gmail_quote">On Wed, Feb 1, 2012 at 7:02 AM, Rudy De Busscher <span dir="ltr"><<a href="mailto:rdebusscher@gmail.com">rdebusscher@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Lincoln,<div><br></div><div>see
<a href="https://issues.jboss.org/browse/FORGE-471" target="_blank">https://issues.jboss.org/browse/FORGE-471</a> with a rather extended explanation.</div><div><br></div><div>Regards</div><div><span class="HOEnZb"><font color="#888888">Rudy</font></span><div>
<div class="h5"><br><br><div class="gmail_quote">On 31 January 2012 18:50, Lincoln Baxter, III <span dir="ltr"><<a href="mailto:lincolnbaxter@gmail.com" target="_blank">lincolnbaxter@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Rudy,<br><br>You're right: We don't yet have nested class support, and the top-level class is what should be returned, so I'm a bit surprised that this is what is happening. Do you know if this problem is caused by the JavaParser API or by the forge Resources API?<br>
<br>Care to file an issue? Thanks!<br><br><a href="https://issues.jboss.org/browse/FORGE" target="_blank">https://issues.jboss.org/browse/FORGE</a> <br><br>~Lincoln<br><br><div class="gmail_quote"><div><div>On Tue, Jan 31, 2012 at 10:06 AM, Rudy De Busscher <span dir="ltr"><<a href="mailto:rdebusscher@gmail.com" target="_blank">rdebusscher@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>Hi All,<div><br></div><div>Seems that nested classes aren't handled correctly by forge.</div>
<div><br></div><div>If you have a java source file like this</div>
<div><br></div><div><pre><span>public</span> class CustomProjectStageHolder <span>implements</span> ProjectStageHolder
{
<span>public</span> <span>static</span> <span>final</span> class CustomProjectStage <span>extends</span> ProjectStage
{
<span>private</span> <span>static</span> <span>final</span> <span>long</span> serialVersionUID = 1029094387976167179L;
}
<span>public</span> <span>static</span> <span>final</span> CustomProjectStage CustomProjectStage = <span>new</span> CustomProjectStage();
}</pre><pre><br></pre><pre><font face="arial, helvetica, sans-serif">And consider then following statements:</font></pre><pre><div><pre><font face="arial, helvetica, sans-serif">JavaResource javaResource = factory.getResourceFrom(new File("/path/to/file/RudyHolder.java")).reify(JavaResource.class);</font></pre>
<pre><font face="arial, helvetica, sans-serif">javaResource.getJavaSource().getQualifiedName();</font></pre><pre><font face="arial, helvetica, sans-serif"><br></font></pre><pre><font face="arial, helvetica, sans-serif">The javaResource is indeed pointing to the file, you can verify with javaResource.getFullyQualifiedName()</font></pre>
<pre><font face="arial, helvetica, sans-serif">But if you ask for the javaSource, as in the second statement, you get the nested class, CustomProjectStage in our example.</font></pre>
<pre><font face="arial, helvetica, sans-serif"><br></font></pre><pre><font face="arial, helvetica, sans-serif">Relation between JavaResource and JavaSource indicates that multiple classes per java file wasn't considered. And isn't common for entity classes etc that have to be handled by Forge.</font></pre>
<pre><font face="arial, helvetica, sans-serif">So when there are no plans in supporting this in Forge, isn't it logic to return the main class, instead of the nested class?</font></pre><pre><font face="arial, helvetica, sans-serif"><br>
</font></pre><pre><font face="arial, helvetica, sans-serif">Thx for the clarification.</font></pre><pre><font face="arial, helvetica, sans-serif"><br>
</font></pre><pre><font face="arial, helvetica, sans-serif">Regards</font></pre><span><font color="#888888"><pre><font face="arial, helvetica, sans-serif">Rudy</font></pre><pre><font face="arial, helvetica, sans-serif"><br>
</font></pre><pre>-- </pre></font></span></div></pre><span><font color="#888888">Rudy De Busscher<br><a href="http://www.c4j.be" target="_blank">http://www.c4j.be</a><br>
<br>
</font></span></div>
<br></div></div>_______________________________________________<br>
forge-users mailing list<br>
<a href="mailto:forge-users@lists.jboss.org" target="_blank">forge-users@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-users" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-users</a><br>
<br></blockquote></div><span><font color="#888888"><br><br clear="all"><br>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.com" target="_blank">http://ocpsoft.com</a><br><a href="http://scrumshark.com" target="_blank">http://scrumshark.com</a><br>
"Keep it Simple"<br>
</font></span><br>_______________________________________________<br>
forge-users mailing list<br>
<a href="mailto:forge-users@lists.jboss.org" target="_blank">forge-users@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-users" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-users</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Rudy De Busscher<br><a href="http://www.c4j.be" target="_blank">http://www.c4j.be</a><br><br>
</div></div></div>
<br>_______________________________________________<br>
forge-users mailing list<br>
<a href="mailto:forge-users@lists.jboss.org">forge-users@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-users" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-users</a><br>
<br></blockquote></div><br><br clear="all"><br>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.com">http://ocpsoft.com</a><br><a href="http://scrumshark.com">http://scrumshark.com</a><br>"Keep it Simple"<br>