svn commit: r157506 - struts/core/trunk/doc/faqs/kickstart.xml

2005-03-14 Thread martinc
Author: martinc
Date: Mon Mar 14 20:26:21 2005
New Revision: 157506

URL: http://svn.apache.org/viewcvs?view=rev&rev=157506
Log:
Fix some link problems. Reported by Rajaneesh Acharya.

Modified:
struts/core/trunk/doc/faqs/kickstart.xml

Modified: struts/core/trunk/doc/faqs/kickstart.xml
URL: 
http://svn.apache.org/viewcvs/struts/core/trunk/doc/faqs/kickstart.xml?view=diff&r1=157505&r2=157506
==
--- struts/core/trunk/doc/faqs/kickstart.xml (original)
+++ struts/core/trunk/doc/faqs/kickstart.xml Mon Mar 14 20:26:21 2005
@@ -23,10 +23,9 @@
 Do I have to credit Struts on my own 
website?
   
   
-
-
-
-
+Why are you offering both Struts Shale and Struts 
Classic? Don't they compete for new development?
+Is Shale likely to become Struts 2.x?
+Is Struts Shale a fork?
   
   
 Where can I get a copy of Struts?
@@ -38,7 +37,6 @@
 Where can I get help with Struts?
 Is commercial support available for Struts?
 Are there ISPs that will host my Struts 
application?
-What Web sites are already Powered by 
Struts?
   
 What's the difference between Struts and Turbine?  
What's the difference between Struts and Expresso?
 Why aren't the Struts tags maintained as part of 
the Jakarta Taglibs project?
@@ -46,7 +44,7 @@
 What about JSTL and JavaServer Faces?
 Is there a particularly good IDE to use with 
Struts?
  
-Is there a digest for the mailing list?
+Is there a digest for the User list?
 Is there a Struts Newsgroup?
 Why didn't my posting show up on the user 
list?"
 How do I unsubscribe from the mailing 
list?



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33996] - ChainAction does not support non-default catalogs

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33996





--- Additional Comments From [EMAIL PROTECTED]  2005-03-15 04:29 ---
It might be OK to remove it from trunk, but it would be nice to fix it in the 
1.2.x branch, where it's currently broken. If we do remove it from trunk, it 
would be good to document how people would upgrade from 1.2.x to 1.3.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: class TagUtils, retrieving a message bundle, code modification re quest

2005-03-14 Thread Martin Cooper
I think the problem here is that you are trying to use modules and
bundles in a way that they were not designed to be used.

Modules, the way they are today, are intended to help split a Struts
application into multiple *independent* pieces. In your code, you
should never need to examine module prefixes, and you should be
accessing your resources using only the bundle names. Struts takes
care of ensuring that the correct bundles are in place, so that you
don't need to.

You seem to be trying to access one module's resources from a
different module. That's not something that the current module design
is set up for.

--
Martin Cooper


On Mon, 14 Mar 2005 12:14:53 +0100, Lesaint Sébastien
<[EMAIL PROTECTED]> wrote:
> 
> I originally wanted to send the message below to the dev-list, ended up on
> the user-list, so I'm posting a copy to the list it belongs to.
> 
> Seb'
> 
> -Message d'origine-
> De : Lesaint Sébastien
> Envoyé : lundi 14 mars 2005 10:43
> À : Struts Users Mailing List
> Objet : class TagUtils, retrieving a message bundle, code modification re
> quest
> 
> Hello,
> 
> I've been looking into the source code (of both 1.2.4 and 1.2.6, not cvs),
> and I have a question about the way a message bundle is looked up based on
> its name.
> 
> I understand from the code that the resource bundle is search:
>  - in the page context
>  - if not found, in the request context
>  - if not found, in the application context
> 
> This process make perfectly sense for me but I'm surprised how the search in
> the application context is done. From the code, we can see that the bundle
> is first searched by its name + the module prefix, then by its raw name.
> 
> I would have done it the other way around, for two reasons:
>  - If I specify in the code the correct bundle name with the module prefix
> appended, there is an extra search done (the one for "name + module prefix +
> module prefix")
>  - If I have two resources bundles, one in the default module, one in a
> module "foo", with the same name in the config file, let's say for pictures
> (so it would be called "ImagesResources"). If I am in the module foo, I
> can't get access to the bundle "ImageResources" of the default module
> because it will always find the bundle "ImagesResources/foo" first.
> 
> Before proposing reporting anything in bugzilla, I would like to know the
> opinion of the community on this issue (assuming that I managed to make it
> clear): should the lookup in application context be done first by raw name
> or by "name + module prefix"?
> 
> Seb'
> 
>

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 34013] New: - javascript struct validation

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=34013

   Summary: javascript struct validation
   Product: Struts
   Version: 1.1 Final
  Platform: Other
OS/Version: Windows XP
Status: NEW
  Severity: normal
  Priority: P2
 Component: Validator Framework
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


Getting javascript error :
'type' is null or not an object for struts validation
at around this area. Using internet explorer version 6. It says javascript1.1

if ((field.type == 'hidden' ||
field.type == 'text' ||
field.type == 'textarea' ||
field.type == 'select-one' ||
field.type == 'radio')  &&
field.disabled == false) {

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32165] - FacesRequestProcessor bug when using prefix mapped Struts servlet & extension mapped Faces servlet

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32165





--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:53 ---
It is true that a wrapper would be needed to address this use case, but there
are many related difficulties.  Indeed, we just removed the previous version of
the wrapper introduced to address behavior differences with MyFaces, as that is
no longer required.

I will look into a different wrapper application that addresses this use case,
but at the moment I am leaning towards documenting a restriction that Struts and
Faces need to be mapped the same way (extension mapped or path mapped) for the
integration library to work.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:50 ---
(Forgot to change bug resolution)


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33672] - struts-faces is incompatible with myfaces

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33672





--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:50 ---
Fixed in nightly build 20050315.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 32014] - HttpServletRequestWrapper in struts-faces broken for servlet 2.4

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32014


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:49 ---
Fixed in nightly build 20050315.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33950] - removing RequestWrapper

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33950


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 20:46 ---
Fixed in nightly build 20050315.


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r157452 - in struts/faces/trunk/core-library/src/java/org/apache/struts/faces: application/FacesRequestProcessor.java application/FacesTilesRequestProcessor.java util/HttpServletRequestWrapper.java

2005-03-14 Thread craigmcc
Author: craigmcc
Date: Mon Mar 14 11:46:14 2005
New Revision: 157452

URL: http://svn.apache.org/viewcvs?view=rev&rev=157452
Log:
Remove the problematic HttpServletRequestWrapper and references to it
in the request processor implementations.

Removed:

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/util/HttpServletRequestWrapper.java
Modified:

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java

struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java?view=diff&r1=157451&r2=157452
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesRequestProcessor.java
 Mon Mar 14 11:46:14 2005
@@ -128,11 +128,6 @@
 created = true;
 FacesContextFactory fcf = (FacesContextFactory)
 FactoryFinder.getFactory(FactoryFinder.FACES_CONTEXT_FACTORY);
-// HttpServletRequestWrapper wrapper = new 
HttpServletRequestWrapper(request, uri);
-// context = fcf.getFacesContext(servlet.getServletContext(), 
wrapper,
-//   response, lifecycle); 
-// Comment out the previous three lines and uncomment
-// the following two lines to test eliminating the wrapper
 context = fcf.getFacesContext(servlet.getServletContext(),
   request, response, lifecycle);
 }

Modified: 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java
URL: 
http://svn.apache.org/viewcvs/struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java?view=diff&r1=157451&r2=157452
==
--- 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java
 (original)
+++ 
struts/faces/trunk/core-library/src/java/org/apache/struts/faces/application/FacesTilesRequestProcessor.java
 Mon Mar 14 11:46:14 2005
@@ -41,7 +41,6 @@
 import org.apache.struts.config.ForwardConfig;
 import org.apache.struts.faces.Constants;
 import org.apache.struts.faces.component.FormComponent;
-import org.apache.struts.faces.util.HttpServletRequestWrapper;
 import org.apache.struts.tiles.TilesRequestProcessor;
 
 
@@ -129,13 +128,8 @@
 created = true;
 FacesContextFactory fcf = (FacesContextFactory)
 FactoryFinder.getFactory(FactoryFinder.FACES_CONTEXT_FACTORY);
-HttpServletRequestWrapper wrapper = new 
HttpServletRequestWrapper(request, uri);
-context = fcf.getFacesContext(servlet.getServletContext(), wrapper,
-  response, lifecycle); 
-// Comment out the previous three lines and uncomment
-// the following two lines to test eliminating the wrapper
-// context = fcf.getFacesContext(servlet.getServletContext(),
-//   request, response, lifecycle);
+context = fcf.getFacesContext(servlet.getServletContext(),
+  request, response, lifecycle);
 }
 
 // Create a new view root



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33996] - ChainAction does not support non-default catalogs

2005-03-14 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33996


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2005-03-14 15:26 ---
My sense is that ChainAction is now obsolete since you can use the struts-config
to directly configure a chain command to be executed as part of the request
process. 

I would argue for removing it completely, as it has never been formally
released, and I don't see any reason it is necessary.  I'll leave this ticket
open for a short while in case anyone can post a justification or preferred
resolution.

 

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



class TagUtils, retrieving a message bundle, code modification re quest

2005-03-14 Thread Lesaint Sébastien

I originally wanted to send the message below to the dev-list, ended up on
the user-list, so I'm posting a copy to the list it belongs to.

Seb'

-Message d'origine-
De : Lesaint Sébastien 
Envoyé : lundi 14 mars 2005 10:43
À : Struts Users Mailing List
Objet : class TagUtils, retrieving a message bundle, code modification re
quest

Hello,

I've been looking into the source code (of both 1.2.4 and 1.2.6, not cvs),
and I have a question about the way a message bundle is looked up based on
its name.

I understand from the code that the resource bundle is search:
 - in the page context
 - if not found, in the request context
 - if not found, in the application context

This process make perfectly sense for me but I'm surprised how the search in
the application context is done. From the code, we can see that the bundle
is first searched by its name + the module prefix, then by its raw name.

I would have done it the other way around, for two reasons:
 - If I specify in the code the correct bundle name with the module prefix
appended, there is an extra search done (the one for "name + module prefix +
module prefix")
 - If I have two resources bundles, one in the default module, one in a
module "foo", with the same name in the config file, let's say for pictures
(so it would be called "ImagesResources"). If I am in the module foo, I
can't get access to the bundle "ImageResources" of the default module
because it will always find the bundle "ImagesResources/foo" first.


Before proposing reporting anything in bugzilla, I would like to know the
opinion of the community on this issue (assuming that I managed to make it
clear): should the lookup in application context be done first by raw name
or by "name + module prefix"?

Seb'


Re: Nightly build script

2005-03-14 Thread Craig McClanahan
(Sorry for the late response -- got back from a week in Tokyo back to
back with TSSJS in Las Vegas yesterday).

There is indeed a cron job on cvs.apache.org that cleans out all files
older than five days.  I don't remember the details, but it's based on
the "find" command with some arguments that select the files of
interest and then executes an "rm" command on them.

Craig


On Thu, 10 Mar 2005 09:30:05 -0800, Martin Cooper <[EMAIL PROTECTED]> wrote:
> On Thu, 10 Mar 2005 10:35:44 -0500, Sean Schofield
> <[EMAIL PROTECTED]> wrote:
> > Thanks for the scripts.  I noticed there was nothing about the file
> > rotation in them (only keeping last 5 days or whatever.)  Do you know
> > if there is a chron script on the CVS server that takes care of that?
> 
> I don't know, and don't know how to check. I assume there must be.
> Craig or infra@ would know.
> 
> --
> Martin Cooper
> 
> 
> > sean
> >
> >
> > On Wed, 9 Mar 2005 11:29:07 -0800, Martin Cooper <[EMAIL PROTECTED]> wrote:
> > > OK, I've attached what I got from Craig some time ago. There are two
> > > scripts, and each builds multiple projects. Have fun! ;-)
> > >
> > > --
> > > Martin Cooper
> > >
> > > On Wed, 9 Mar 2005 14:05:28 -0500, Sean Schofield
> > > <[EMAIL PROTECTED]> wrote:
> > > > > Craig has the "master" script for Struts and Jakarta nightly builds,
> > > > > but that's based on the older Ant build system. We don't have the
> > > > > nightlies up and running (automatically) for the new Maven build
> > > > > system yet. (I have an older version of Craig's script, and can send
> > > > > you a copy of that if you need it.)
> > > >
> > > > I will take any version of the script you have.  If Craig is
> > > > monitoring this thread ... I'll take the new one when you get a chance
> > > > :-)
> > > >
> > > > > Craig's scripts build on his machine and then push the distributions
> > > > > up to cvs.a.o, which is indeed an ASF box. Once we have the nightlies
> > > > > running on ASF hardware (which is looking like it'll be soon after the
> > > > > infrathon right now), they will still be building on one box and then
> > > > > pushing up to cvs.a.o.
> > > >
> > > > That's  good news.  I haven't subscribed to the infra list yet so I
> > > > wasn't aware of any of those developments.
> > > >
> > > > I assume each project need its own script?  If so then Craig's CVS
> > > > script will be even more useful.
> > > >
> > > > > Martin Cooper
> > > >
> > > > sean
> > > > sean
> > > >
> > >
> > >
> > >
> >
> 
> -
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 
>

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]