[shrinkwrap-issues] [JBoss JIRA] Commented: (SHRINKWRAP-241) remap targets for WebArchive add* methods

Sun Jan 9 06:40:49 EST 2011

    [ https://issues.jboss.org/browse/SHRINKWRAP-241?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12574260#comment-12574260 ] 

Aslak Knutsen commented on SHRINKWRAP-241:
------------------------------------------

WEB-INF/classes/META-INF i think is valid for persistence.xml

> remap targets for WebArchive add* methods
> -----------------------------------------
>
>                 Key: SHRINKWRAP-241
>                 URL: https://issues.jboss.org/browse/SHRINKWRAP-241
>             Project: ShrinkWrap
>          Issue Type: Feature Request
>          Components: api
>    Affects Versions: 1.0.0-alpha-11
>            Reporter: Dan Allen
>            Assignee: Andrew Rubinger
>            Priority: Blocker
>             Fix For: 1.0.0-alpha-12
>
>
> The Servlet specification specifically defines static and dynamic served from the web application as "web resources". Hence, the use of "addWebResource" to refer to WEB-INF is a mistake, and reopens the discussion of how each add* method should be mapped.
> Currently, the targets for the WebArchive add methods are:
> add() maps to /
> addResource() maps to /
> addWebResource() maps to /WEB-INF
> addManifestResource() maps to /WEB-INF/classes/META-INF
> (I know these are being refactor to "addAs". I'm just sticking with the terminology that's currently in place for the purpose of this discussion).
> To start, I think we should explicitly refer to WEB-INF by name, as in:
> addWebInfResource() maps to /WEB-INF
> Next, we should honor the "web resource" definition from the Servlet specification
> addWebResource() maps to /
> Now, because WebArchive is inheriting addResource() from ResourceArchive, we should honor the function of the target location, so:
> addResource() maps to /WEB-INF/classes
> Finally, we have to address add() inherited from Archive. In this case, I'm okay with sticking with add() as meaning the root of whatever archive we have:
> add() maps to /
> So, here's what an example might look like:
> WebArchive war = ShrinkWrap.create(WebArchive.class, "app.war")
>    .addWebInfResource(EmptyAsset.INSTANCE, "beans.xml")
>    .addWebResource(new StringAsset("<html><body>Hello World!</body></html>"), "index.html")
>    .addResource(new StringAsset("foo=bar"), "messages.properties);
> System.out.println(war.toString(true));
> prints:
> app.war
> /index.html
> /WEB-INF/beans.xml
> /WEB-INF/classes/messages.properties
> To me, this makes so much more sense. Hopefully you share the same opinion.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira