Hi Rudy, 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? Care to file an issue? Thanks! https://issues.jboss.org/browse/FORGE ~Lincoln On Tue, Jan 31, 2012 at 10:06 AM, Rudy De Busscher <rdebusscher(a)gmail.com&gt;wrote: -- Lincoln Baxter, III http://ocpsoft.com http://scrumshark.com "Keep it Simple"

Thanks. You are correct about this behavior. Now to find time to fix it! Is this something you might be able to help with? I've been meaning to take a look at the deltaspike plugin. A test case would make this much easier to debug! Thanks so much for finding this! ~Lincoln On Wed, Feb 1, 2012 at 7:02 AM, Rudy De Busscher <rdebusscher(a)gmail.com&gt;wrote: -- Lincoln Baxter, III http://ocpsoft.com http://scrumshark.com "Keep it Simple"

Sure, let's talk about that. I think the quick fix is good enough for now. We can come up with a strategy to handle the nesting properly later. Is there a JIRA for this? BTW. The Forge Java Parser uses the Eclipse parser under the covers, and if you really need to, you can use the method: parserObject.getInternal(); Which should return the related Eclipse parser object. Does that help? Let's discuss further on the forge-dev list. ~Lincoln On Thu, Feb 2, 2012 at 3:43 AM, Rudy De Busscher <rdebusscher(a)gmail.com&gt;wrote: > Lincoln, > @deltaspike plugin, I have been making some kind of prototype that creates > the correct files. Now I need to make the checks and update files. But > your input is welcome. > @fix There are 2 options: > - Quick fix that just handles the main class and ignores nested classes > - Correct handling, but that needs a lot of refactoring to handle multiple > classes per file. > For deltaspike, I was writing something based on the eclipse parser to > have full access to everyhting. > Thus depends what forge project wants and I could try to help but have > limited time. We can discuss this maybe outside of this thread. > regards > Rudy > On 1 February 2012 21:24, Lincoln Baxter, III <lincolnbaxter(a)gmail.com&gt;wrote: >> Thanks. You are correct about this behavior. Now to find time to fix it! >> Is this something you might be able to help with? >>> I've been meaning to take a look at the deltaspike plugin. A test case >> would make this much easier to debug! >>> Thanks so much for finding this! >> ~Lincoln >>>> On Wed, Feb 1, 2012 at 7:02 AM, Rudy De Busscher <rdebusscher(a)gmail.com&gt;wrote: >>>> Hi Lincoln, >>>>> see https://issues.jboss.org/browse/FORGE-471 with a rather extended >>> explanation. >>>>> Regards >>> Rudy >>>>>>> On 31 January 2012 18:50, Lincoln Baxter, III <lincolnbaxter(a)gmail.com&gt;wrote: >>>>>> Hi Rudy, >>>>>>> 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? >>>>>>> Care to file an issue? Thanks! >>>>>>> https://issues.jboss.org/browse/FORGE >>>>>>> ~Lincoln >>>>>>> On Tue, Jan 31, 2012 at 10:06 AM, Rudy De Busscher < >>>> rdebusscher(a)gmail.com&gt; wrote: >>>>>>>> Hi All, >>>>>>>>> Seems that nested classes aren't handled correctly by forge. >>>>>>>>> If you have a java source file like this >>>>>>>>> public class CustomProjectStageHolder implements ProjectStageHolder >>>>> { >>>>> public static final class CustomProjectStage extends ProjectStage >>>>> { >>>>> private static final long serialVersionUID = 1029094387976167179L; >>>>> } >>>>>>>>> public static final CustomProjectStage CustomProjectStage = new CustomProjectStage(); >>>>> } >>>>>>>>>>>>> And consider then following statements: >>>>>>>>> JavaResource javaResource = factory.getResourceFrom(new File("/path/to/file/RudyHolder.java")).reify(JavaResource.class); >>>>>>>>> javaResource.getJavaSource().getQualifiedName(); >>>>>>>>>>>>> The javaResource is indeed pointing to the file, you can verify with javaResource.getFullyQualifiedName() >>>>>>>>> But if you ask for the javaSource, as in the second statement, you get the nested class, CustomProjectStage in our example. >>>>>>>>>>>>> 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. >>>>>>>>> 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? >>>>>>>>>>>>>>>>>>>>>>>>> Thx for the clarification. >>>>>>>>>>>>> Regards >>>>>>>>> Rudy >>>>>>>>>>>>>>>>>>>>>>>>>>>>> -- >>>>>>>>> Rudy De Busscher >>>>> http://www.c4j.be >>>>>>>>>>>>> _______________________________________________ >>>>> forge-users mailing list >>>>> forge-users(a)lists.jboss.org >>>>> https://lists.jboss.org/mailman/listinfo/forge-users >>>>>>>>>>>>>>>>>> -- >>>> Lincoln Baxter, III >>>> http://ocpsoft.com >>>> http://scrumshark.com >>>> "Keep it Simple" >>>>>>> _______________________________________________ >>>> forge-users mailing list >>>> forge-users(a)lists.jboss.org >>>> https://lists.jboss.org/mailman/listinfo/forge-users >>>>>>>>>>>>> -- >>> Rudy De Busscher >>> http://www.c4j.be >>>>>>> _______________________________________________ >>> forge-users mailing list >>> forge-users(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/forge-users >>>>>>>> -- >> Lincoln Baxter, III >> http://ocpsoft.com >> http://scrumshark.com >> "Keep it Simple" >>> _______________________________________________ >> forge-users mailing list >> forge-users(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/forge-users >>> -- > Rudy De Busscher > http://www.c4j.be > _______________________________________________ > forge-users mailing list > forge-users(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/forge-users -- Lincoln Baxter, III http://ocpsoft.com http://scrumshark.com "Keep it Simple"