<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>A description of Facet in inheritance and passing data from a plugin to a Facet is now in the docs: <a href="https://docs.jboss.org/author/display/FORGE/Enable+modular+functionality+with+Facets">https://docs.jboss.org/author/display/FORGE/Enable+modular+functionality+with+Facets</a> (bottom of the page).</div><div><br></div><div>Paul</div><br><div><div>On Mar 11, 2012, at 0:41 , Lincoln Baxter, III wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Hey Paul, Awesome! I did just push some changes to that respect. Sounds like yours is a lot better than mine. Sounds like we had very similar solutions :D but I think you probably also incorporated the facet inheritance approach? How is that working? Would be a good thing to write down as a recommended pattern if it works well!<br>
<br>Sorry, I should have told you I was starting some work on it.<br><br>I also fixed a few issues with the web.xml configuration. Looks like this was preserved in your merge, just want to make sure?<br><br>This dependency is not required and should be removed (I had done that in my commits:)<br>
<pre><span class="n">DependencyBuilder</span><span class="o">.</span><span class="na">create</span><span class="o">(</span><span class="s">"org.jboss.spec.javax.xml.bind:jboss-jaxb-api_2.2_spec"</span><span class="o">)</span></pre>
Additionally, this line was actually incorrectly modifying existing servlet mappings, which was why I corrected it.<br><br><div class="line" id="LC56"><span class="o"></span></div><blockquote style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">
<div class="line" id="LC57"><font size="1"> <span class="n">Node</span> <span class="n">servletClass</span> <span class="o">=</span> <span class="n">node</span><span class="o">.</span><span class="na">getOrCreate</span><span class="o">(</span><span class="s">"servlet-mapping/servlet-name="</span> <span class="o">+</span> <span class="n">JAXRS_SERVLET</span><span class="o">);</span></font></div>
</blockquote><br>Let's do a google hangout sometime and try to get our stuff re-coordinated?<br><br>~Lincoln<br><br><div class="gmail_quote">On Sat, Mar 10, 2012 at 11:46 AM, Paul Bakker <span dir="ltr"><<a href="mailto:paul.bakker.nl@gmail.com">paul.bakker.nl@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi!<div><br></div><div>I just pushed the new REST plugin to master. JAX-RS can now be configured by either the web.xml like before, or an Application class.</div>
<div>It's now also using the TreeVisiting API, but only if it finds that configuration is missing or is invalid. So if the configuration is not in sync because of refactoring for example, the plugin will notice this and fix it's configuration by scanning the project's classes. As long as the configuration is still correct, it will not rescan classes. By using the configuration API as a cache I hope the performance penalty of scanning classes will be minimized. We still should test this better on large projects however, but that's out of the scope of this issue.</div>
<span class="HOEnZb"><font color="#888888"><div><br></div><div>Paul</div></font></span><div><div class="h5"><div><br></div><div><br><div><div>On Mar 9, 2012, at 19:43 , Lincoln Baxter, III wrote:</div><br><blockquote type="cite">
Hey Paul,<br><br>How is this coming along? I only ask because it came up from the tools team that Forge has trouble detecting apps using the REST @Application() annotation.<br><br>I hope all is well!<br>~Lincoln<br><br><div class="gmail_quote">
On Mon, Feb 27, 2012 at 3:09 PM, 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">
Yep! Glad to be back, though :)<br><br>I think it might be a good idea to watch the performance of this operation, though. It could get expensive for large projects, so it might be a good idea to only do this check when necessary, and store the result in forge project scoped config. We are becoming an IDE :)<br>
<br>Will be complicated I think, or slow...<span><font color="#888888"><br><br>~Lincoln</font></span><div><div><br><br><div class="gmail_quote">On Mon, Feb 27, 2012 at 11:14 AM, Paul Bakker <span dir="ltr"><<a href="mailto:paul.bakker@luminis.eu" target="_blank">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">
<div style="word-wrap:break-word">
<div>Hi! Had a good time? </div>
<div>The new API looks really good to use here, didn't know we could do that… I wasn't too happy with using configuration for this, because it will break on refactoring, so it's much better is we can do it this way. I will look into it to use it.</div>
<div><br>
</div>
<div>Paul</div>
<div><br>
</div>
<div><br>
</div>
<br>
<div><div><div>
<div>On Feb 27, 2012, at 17:09 , Lincoln Baxter, III wrote:</div>
<br>
</div></div><blockquote type="cite"><div><div>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" target="_blank">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" target="_blank">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><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/" target="_blank">http://ocpsoft.com</a><br>
<a href="http://scrumshark.com/" target="_blank">http://scrumshark.com</a><br>
"Keep it Simple"<br></div></div>
_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org" target="_blank">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
</blockquote>
</div>
<br>
</div>
<br>_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org" target="_blank">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
<br></blockquote></div><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>
</div></div></blockquote></div><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>
_______________________________________________<br>forge-dev mailing list<br><a href="mailto:forge-dev@lists.jboss.org" target="_blank">forge-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
</blockquote></div><br></div></div></div></div><br>_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</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>
_______________________________________________<br>forge-dev mailing list<br><a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a><br>https://lists.jboss.org/mailman/listinfo/forge-dev<br></blockquote></div><br></body></html>