Hey Paul! <br><br>Back from Vaca.<br><br>My first thought here is that this is excellent use of the Configuration API, but I think currently the way this is written, it depends on the Configuration in order to recognize the fact that there is an application class serving as a REST activator?<br>
<br>We could use the JavaParser TreeVisiting API that was just introduced, in order to search project sources and make this determination.<br><br><a href="https://github.com/forge/core/blob/master/shell-api/src/main/java/org/jboss/forge/project/facets/JavaSourceFacet.java#L165">https://github.com/forge/core/blob/master/shell-api/src/main/java/org/jboss/forge/project/facets/JavaSourceFacet.java#L165</a><br>
<br>Thoughts?<br>~Lincoln<br><br><div class="gmail_quote">On Wed, Feb 15, 2012 at 5:48 PM, Paul Bakker <span dir="ltr"><<a href="mailto:paul.bakker@luminis.eu">paul.bakker@luminis.eu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Lincoln,<br>
<br>
I made some progress on refactoring the rest stuff, plus I added the option to use an Application class instead of web.xml. Because it changed quite a lot and this is the first time we use the idea of having "nested" facets it would probably be good if you review before I merge to master.<br>
<br>
<a href="https://github.com/forge/core/commit/ec0275a821c6bb3ccf690b55d66816073ba0c328" target="_blank">https://github.com/forge/core/commit/ec0275a821c6bb3ccf690b55d66816073ba0c328</a><br>
<br>
Let me know what you think :-)<br>
<span class="HOEnZb"><font color="#888888"><br>
Paul<br>
</font></span></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>