svn commit: r168085 - /struts/shale/trunk/clay-plugin/src/conf/shale-clay.tld

2005-05-04 Thread craigmcc
Author: craigmcc
Date: Tue May  3 23:01:44 2005
New Revision: 168085

URL: http://svn.apache.org/viewcvs?rev=168085view=rev
Log:
Disambiguate the tag library URI (and the short name, which many tools use
as the default for the prefix).

PR:  Bugzilla #34712
Submitted By:  Manfred Klug manklu AT web.de

Modified:
struts/shale/trunk/clay-plugin/src/conf/shale-clay.tld

Modified: struts/shale/trunk/clay-plugin/src/conf/shale-clay.tld
URL: 
http://svn.apache.org/viewcvs/struts/shale/trunk/clay-plugin/src/conf/shale-clay.tld?rev=168085r1=168084r2=168085view=diff
==
--- struts/shale/trunk/clay-plugin/src/conf/shale-clay.tld (original)
+++ struts/shale/trunk/clay-plugin/src/conf/shale-clay.tld Tue May  3 23:01:44 
2005
@@ -23,8 +23,8 @@
 
   tlib-version1.0/tlib-version
   jsp-version1.2/jsp-version
-  short-namesh/short-name
-  urihttp://struts.apache.org/shale/core/uri
+  short-nameclay/short-name
+  urihttp://struts.apache.org/shale/clay-plugin/uri
   description/description
 
   listener



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



DO NOT REPLY [Bug 34712] - [Shale] Shale and Clay tags cannot be used on the same page

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=34712.
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=34712


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 08:05 ---
Fixed in nightly build 20050504.

I also disambiguated the short-name element, which many JSP oriented tools use
as the default prefix for the tag library when generating code.


-- 
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]



sunone 6.1 sp03

2005-05-04 Thread fernando
Hi

We are proving struts in webserver SUNONE 6.1 SP03

We download the struts-examples from struts home page.

We try this example in netbeans and it works fine.

Now, we make a .war archive, and install in SUNONE 6.1 SP03, and we have this 
problems:

1) If we go to http:/machine:port/https-agenda/ejemplo5, we have this problem

[29/Apr/2005:12:23:58] warning (22906): for host 10.95.26.94 trying to 
GET /https-agenda/ejemplo5/welcome.do, send-file repor
ts: HTTP4142: can't find /export/home/pdvoz/PDVOZ/WEB/https-
agenda/webapps/https-agenda/ejemplo5/welcome.do (File not found)

2) If we try http:/machine:port/https-agenda/ejemplo5/welcome.jsp, we have this 
problem:

[29/Apr/2005:12:24:02] failure (22906): for host 10.95.26.94 trying to 
GET /https-agenda/ejemplo5/welcome.jsp, service-j2ee reports: 
StandardWrapperValve[jsp]: WEB2792: Servlet.service() for servlet jsp threw 
exception org.apache.jasper.JasperException: WEB4059: can't find /tags/struts-
html (File not found)
...

In the startup, we have not any problem:

[29/Apr/2005:12:23:23] info (22905): CORE1116: Sun ONE Web Server 6.1SP3 
B08/17/2004 18:33
[29/Apr/2005:12:23:23] info (22906): CORE5076: Using [Java HotSpot(TM) Server 
VM, Version 1.4.2_06] from [Sun Microsystems Inc.]
[29/Apr/2005:12:23:25] info (22906): WEB0100: Loading web module in virtual 
server [https-agenda] at [/ejemplo5]
[29/Apr/2005:12:23:25] info (22906): WEB0100: Loading web module in virtual 
server [https-agenda] at [/search]
[29/Apr/2005:12:23:43] info (22906): Loading validation rules file from '/WEB-
INF/validator-rules.xml'
[29/Apr/2005:12:23:43] info (22906): Loading validation rules file from '/WEB-
INF/validator/validation.xml'
[29/Apr/2005:12:23:45] info (22906): HTTP3072: [LS ls1] http://agenda:18080 
ready to accept requests
[29/Apr/2005:12:23:45] info (22906): CORE3274: successful server startup


We have not touch any source file nor configuration file. 

Any idea?

Thank you very much

Regards

   Fernando


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



DO NOT REPLY [Bug 33355] - computeURLWithCharEncoding doesn't encodes the created by the response.encudeURL call

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33355.
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=33355





--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 09:53 ---
My examples was wrong.There is a good sample:
JSP code:

html:link action=/protected/CustomerDetailsAction paramId=listid 
paramName=element paramProperty=id
generated code:
a href=/mpi-ui/cad/protected/CustomerDetailsAction.do?
JSESSIONID=551432listid=16191+491612345/a

And this bug depends on the used the Servlet Container too. It is Only occurred 
if the conteiner uses the '' character to separate the jsessionid. Your result 
will be wrong to if you change the separator from ; to .
Because the encodeURL doesn't knows that PCDATA encoding have to be used in the 
result string- it generates encode free url. After it have to be convert it to 
the target context encoding (in this case this encoding is the PCDATA)
Therefore it seems to bea container problem. but it isn't a container problem. 
The second true solution is:
1; First call the encodeURL on the base url if needed
2; immediately convert the result to PCDATA,
3; add others parameters with amp; like in the current code.
4; remove the encodeURL call from the enc of method...





-- 
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]



build-tests.xml 1.3

2005-05-04 Thread rmanchu
hi
is it possible to include the build-tests.xml in the source distribution?
is it somewhere where i can get it?
help
riyaz
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: build-tests.xml 1.3

2005-05-04 Thread James Mitchell
From where are you getting your distribution?

--
James Mitchell
Software Engineer / Open Source Evangelist
Consulting / Mentoring / Freelance
EdgeTech, Inc.
678.910.8017
AIM:   jmitchtx
Yahoo: jmitchtx
MSN:   [EMAIL PROTECTED]

- Original Message - 
From: rmanchu [EMAIL PROTECTED]
To: Struts Dev Mailing List dev@struts.apache.org
Sent: Wednesday, May 04, 2005 5:01 AM
Subject: build-tests.xml 1.3


hi
is it possible to include the build-tests.xml in the source distribution?
is it somewhere where i can get it?
help
riyaz
-
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]


Re: build-tests.xml 1.3

2005-05-04 Thread rmanchu
http://svn.apache.org/builds/struts/maven/trunk/nightly/struts-core/
James Mitchell wrote:
From where are you getting your distribution?
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: build-tests.xml 1.3

2005-05-04 Thread James Mitchell
Ok, the Struts layout as it used to be (last year some time) is no longer 
the same monolithic take-it-or-leave-it Struts of yesteryears.  The Struts 
team has been working long and hard to reorganize the layout and (with all 
hope) much of the complexity that existed with the old layout.

The project was split into separate subprojects, and there is a common build 
subproject for build all (or any single piece) of Struts.

While Maven is the preferred choice for building Struts, the documentation 
generated during the build are not as helpful as they could be

Here are the docs that are generated by Maven:
http://svn.apache.org/builds/struts/maven/trunk/docs/
We hope to get the documentation back up to speed here in the next few 
weeks.

If you are still wanting to build/test Struts on your own, here's a guide:
http://wiki.apache.org/struts/StrutsMaintenanceMaven
--
James Mitchell
Software Engineer / Open Source Evangelist
Consulting / Mentoring / Freelance
EdgeTech, Inc.
678.910.8017
AIM:   jmitchtx
Yahoo: jmitchtx
MSN:   [EMAIL PROTECTED]

- Original Message - 
From: rmanchu [EMAIL PROTECTED]
To: Struts Developers List dev@struts.apache.org
Sent: Wednesday, May 04, 2005 7:04 AM
Subject: Re: build-tests.xml 1.3


http://svn.apache.org/builds/struts/maven/trunk/nightly/struts-core/
James Mitchell wrote:
From where are you getting your distribution?
-
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]


DO NOT REPLY [Bug 33355] - computeURLWithCharEncoding doesn't encodes the created by the response.encudeURL call

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33355.
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=33355


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|1.2.6 Beta  |Unknown




--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 15:33 ---
What servlet container is this problem occuring on?

-- 
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 33355] - computeURLWithCharEncoding doesn't encodes the created by the response.encudeURL call

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33355.
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=33355





--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 15:50 ---
(In reply to comment #3)
 What servlet container is this problem occuring on?

I uses Bea weblogic 8.1 sp3.
But IMHO the New Atlanta SevletExec is configurable for this mode.
It is possible that the Resin too.

For other examples pelase see: http://www.google.com/search?hl=enlr=q=%22%
26jsessionid%3D%22+parameterbtnG=Search

-- 
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 34445] - use charsets given by browser for form field encodings

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=34445.
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=34445





--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 16:30 ---
There was no intent to duplicate anything. It just appeared the easiest to use
the public ParameterParser. Agreed, if the FileItem interface 
could be enhanced with a getCharacterEncoding(), this would be more elegant. 
Martin, is there any new release scheduled anytime soon?

-- 
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]



[Struts Wiki] Trivial Update of WhyChain by JayBurgess

2005-05-04 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on Struts Wiki for change 
notification.

The following page has been changed by JayBurgess:
http://wiki.apache.org/struts/WhyChain

The comment on the change is:
Minor enhancements. Confirmed with Ted.

--
  
  In Struts 1.0, requests were processed using a lump of code in ActionServlet. 
To keep it organized, the code was subdivided in a series of methods. 
Unsurprisingly, each method in the series had a coherent task and was named 
after its task.
  
- In Struts 1.1, we pulled out that lump of code and turned it into a first 
class object. The original methods remained mainly the same, but now they were 
methods in a coherent object, rather than a set of related methods in a larger 
object.
+ In Struts 1.1, we pulled out that lump of code and turned it into a first 
class object called the RequestProcessor. The original methods remained mainly 
the same, but now they were methods in a coherent object, rather than a set of 
related methods in a larger object.
  
- Since it was a first-class object, we could subclass the request processor, 
and plugin a new one in, without touching the Servlet. (Users do not like to 
subclass ActionServlet.)
+ Since it was a first-class object, we could subclass this new request 
processor, and plug in a new one, without touching the Servlet. (Users do not 
like to subclass ActionServlet.)
  
  We were also able to use a different request processor for each module, if 
need be.
  
  The problem is that it was still an all-or-nothing object. Extensions like 
Tile and Workflow both needed to subclass the RequestProcessor in different 
ways for their own reasons.
  
- For Struts 1.3, we turned the 1.1 methods into Commands. Rather than 
subclassing a monolithic object, we could just replace Commands. We could also 
insert or remove Commands, if needed, to extend or streamline the request 
processing gauntlet.
+ For Struts 1.3, we turned the 1.1 request processor methods into Commands. 
Rather than subclassing a monolithic object, we could now just replace 
Commands. We could also insert or remove Commands, if needed, to extend or 
streamline the request processing gauntlet.
  
- Refactoring the RequestProcessor for CoR is about extending the request 
processing cycle. It's been a straight-line refactoring since the beginning. I 
don't believe anyone is trying to follow a particular pattern. We are just 
trying to provide a flexible approach extending the request processing cycle.
+ Refactoring the RequestProcessor for Chain of Responsibility (CoR) is about 
extending the request processing cycle using Commons Chain. It's been a 
straight-line refactoring since the beginning. I don't believe anyone is trying 
to follow a particular pattern. We are just trying to provide a flexible 
approach to extend the request processing cycle.
  
  Meanwhile, users are forever trying to chain Actions. Since most teams do 
not have a coherent business application framework of their own, they tend to 
piggyback one on Struts. The input and outputs to their business logic become 
embedded in Actions, and so they want to chain a copy and a delete to do a 
move.
  
  Another use of Commons Chain is that people can use it as the base of their 
own business application framework (and stop misusing Struts Actions).
  
- The idea behind CoR and Struts Chain is to solve both of these problems:
+ The idea behind CoR and Struts Chain/Commons Chain is to solve both of these 
problems:
  
   * A flexible processing layer for business applications
   * An extensible request processor for Struts

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



LookupDispatchAction question

2005-05-04 Thread Tom Viers
Hello,

I am using a LookupDispatchAction to consolidate CRUD actions for a
particular element in my application.
As far a I can tell, only one input can be specified for this lookup
action.

Is there a way to dynamically specify the input forward based on which
action method was dispatched?

I have tried calling setInput() on the mapping, but an
IllegalStateException is thrown Configuration is frozen.

Thanks in advance for the help,
Tom





Tom Viers
PICA Programming Team
[EMAIL PROTECTED]
ext 2148

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



Re: LookupDispatchAction question

2005-05-04 Thread Michael Jouravlev
If it is acceptable for you to specify input property in the action,
then you can forgo this property altogether. You need it only for
automatic error display. When input property is set, and validate is
set to true, and your validation method returns errors, Struts queues
errors into request object and forwards to your input path, without
calling action class.

Instead, you can do validation from the action class, and then
depending on result or on something else, you can forward to a needed
location. You can specify a whole bunch of them, be they input or not,
in your forward element.

You can also build new ActionForward object dynamically. Struts does
not allow to change static ActionForward objects, as you already
found.

Michael.

On 5/4/05, Tom Viers [EMAIL PROTECTED] wrote:
 Hello,
 
 I am using a LookupDispatchAction to consolidate CRUD actions for a
 particular element in my application.
 As far a I can tell, only one input can be specified for this lookup
 action.
 
 Is there a way to dynamically specify the input forward based on which
 action method was dispatched?
 
 I have tried calling setInput() on the mapping, but an
 IllegalStateException is thrown Configuration is frozen.
 
 Thanks in advance for the help,
 Tom
 
 Tom Viers
 PICA Programming Team
 [EMAIL PROTECTED]
 ext 2148
 
 -
 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]



Re: LookupDispatchAction question

2005-05-04 Thread Joe Germuska
This is actually a question better suited for the user list (in fact, 
a question similar came up last week!)

You've run into the real problem, which is that your code can't 
change an existing ActionConfig -- these are shared, so you'd have 
all kinds of synchronization issues in a multi-threaded servlet 
environment.

You may be able to apply the wildcard mapping: 
http://struts.apache.org/userGuide/building_controller.html#action_mapping_wildcards 
The wildcard matching creates an instantaneous copy of the matched 
ActionConfig and populates values by doing match replacements, which 
would probably handle your case.

Hope that helps,
Joe
At 11:33 AM -0400 5/4/05, Tom Viers wrote:
Hello,
I am using a LookupDispatchAction to consolidate CRUD actions for a
particular element in my application.
As far a I can tell, only one input can be specified for this lookup
action.
Is there a way to dynamically specify the input forward based on which
action method was dispatched?
I have tried calling setInput() on the mapping, but an
IllegalStateException is thrown Configuration is frozen.
Thanks in advance for the help,
Tom


Tom Viers
PICA Programming Team
[EMAIL PROTECTED]
ext 2148
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

--
Joe Germuska
[EMAIL PROTECTED]  
http://blog.germuska.com
Narrow minds are weapons made for mass destruction  -The Ex

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


Re: LookupDispatchAction question

2005-05-04 Thread Dakota Jack
I assume that you know that you can specify your ActionForward in your
Action's methods, e.g. return new ActionForward(myPage);?  You can
also specify all sorts of forwards in the mapping for success,
failure, etc.  Just do this instead for update, delete, etc. with the
CRUD formula.  You would be better, in my opinion, to jettison the
bloat in LookupDispatchAction for one of the solutions at
www.michaelmcgrady.com, my doppleganger.  ///;-)

Jack

On 5/4/05, Tom Viers [EMAIL PROTECTED] wrote:
 Hello,
 
 I am using a LookupDispatchAction to consolidate CRUD actions for a
 particular element in my application.
 As far a I can tell, only one input can be specified for this lookup
 action.
 
 Is there a way to dynamically specify the input forward based on which
 action method was dispatched?
 
 I have tried calling setInput() on the mapping, but an
 IllegalStateException is thrown Configuration is frozen.
 
 Thanks in advance for the help,
 Tom
 
 Tom Viers
 PICA Programming Team
 [EMAIL PROTECTED]
 ext 2148
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 
 


-- 
You can lead a horse to water but you cannot make it float on its back.
~Dakota Jack~

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



DO NOT REPLY [Bug 34714] - [Shale] Clay PropUtil.setProperty should not fail silently if the property could not be set.

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=34714.
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=34714





--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 19:24 ---
(In reply to comment #6)

 How about we make it an optional attribute of the component node.  Then 
 we could keep the attributes as a direct mapping to the JSF component. 
 
 !ELEMENT component (attributes?, converter?, validator*, actionListener*, 
 valueChangeListener*, element*)
 !ATTLIST component jsfid CDATA #REQUIRED
   extends CDATA #IMPLIED
   componentType CDATA #IMPLIED
   id CDATA #IMPLIED
 allowBody %Boolean; #IMPLIED


Yes. Excellent idea.

-- 
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 33355] - computeURLWithCharEncoding doesn't encodes the created by the response.encudeURL call

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33355.
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=33355





--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 20:36 ---
OK I did some googling myself and found the following...

http://www.mail-archive.com/struts-user@jakarta.apache.org/msg84489.html
http://www.caucho.com/support/resin-interest/0101/0428.html
http://www.caucho.com/support/resin-interest/0307/0161.html
http://archives.java.sun.com/cgi-bin/wa?A2=ind0001L=jsp-interestP=R52835

...all of which say that according to the servlet specification URL re-writing 
must use a path parameter, not a request parameter. What you're asking for is 
for struts to cater for the situation where the servlet container doesn't 
comply with this. As a general rule we don't change Struts to cater for other 
software which is *broken* and I think this should be closed as WONTFIX.

I also found a note in the Struts's weblogic 5.1 configuration page which seems 
relevant:

http://struts.apache.org/userGuide/installation-wls5.html

In the Addtional Recommendations section it says There are additional 
problems related to URL-based sessions, caused from BEA's way to encode the 
session id (J2EE-incompatible) as query-param.

However, according to these docs this should be fixed from 6.0 onwards...

http://e-docs.bea.com/wls/docs61/faq/plugin.html#252497

...so I don't understand why you're experiencing this problem?

Niall

-- 
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 33355] - computeURLWithCharEncoding doesn't encodes the created by the response.encudeURL call

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33355.
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=33355





--- Additional Comments From [EMAIL PROTECTED]  2005-05-04 20:42 ---
+1 to WONTFIX due to a noncompliant container.


-- 
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: Version 1.2.7

2005-05-04 Thread Martin Cooper
On 5/3/05, Niall Pemberton [EMAIL PROTECTED] wrote:
 I had a look at the three outstanding bugs and IMO none of them should
 prevent a Struts release:
 
 1) Bug 33996 - ChainAction does not support non-default catalogs
 Seems to me that this should be N/A since either its an issue against a
 sandbox item (and sandbox problems shouldn't stop a Struts release) or its a
 1.3 issue and therefore shouldn't prevent a 1.2.x release.

OK, let's go with N/A.

 Also as a note - the release target in the 1.2.x ant build doesn't now
 include anything from the sandbox - whereas I believe 1.2.4 did include the
 contrib stuff.
 
 2)  Bug 33355 - computeURLWithCharEncoding doesn't encodes the 
 I'm having difficulty understanding this bugzilla ticket properly, but I
 tried this out and couldn't re-create the issues its talking about and
 therefore I don't think it should prevent a release.

Looks like a container issue, so WONTFIX.

 3) Bug 34445 - use charsets given by browser for form field encodings
 This is dependant on a release of Commons FileUpload and I think we should
 ship without it, rather than waiting for FileUpload - we can always do
 another 1.2.x version once FileUpload has been release.

Let's make this LATER, then.

And ... roll it!

--
Martin Cooper


 Niall
 
 - Original Message -
 From: Martin Cooper [EMAIL PROTECTED]
 Sent: Tuesday, May 03, 2005 5:22 PM
 
  The reason I didn't roll the release last weekend is because we still have
  three issues on the release page that need to be resolved, one way or
  another. One of these is in the sandbox, but since it's an issue that bit
  me in my day job, I was hoping we could resolve it, along with the other
  two.
 
 -
 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]



DO NOT REPLY [Bug 33355] - computeURLWithCharEncoding doesn't encodes the created by the response.encudeURL call

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=33355.
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=33355


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||WONTFIX




--- Additional Comments From [EMAIL PROTECTED]  2005-05-05 00:23 ---
Closing as WONTFIX.

-- 
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 29792] - Action method saveErrors lose messages on chained actions

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=29792.
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=29792


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
 Status|REOPENED|RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-05-05 00:36 ---
Can we leave this bug as FIXED please and create a new enhancement ticket for 
any new/additional requirements.

Thanks.

-- 
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 34730] - Javascript Validation Functions

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=34730.
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=34730


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Component|Validator Framework |Validator
Product|Struts  |Commons
Version|1.2.4   |Nightly Builds




--- Additional Comments From [EMAIL PROTECTED]  2005-05-05 00:39 ---
The JavaScript validation functions are part of Commons Validator. I'm re-
assigning this there.

-- 
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 34730] - [validator] Javascript Validation Functions

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=34730.
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=34730


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|dev@struts.apache.org   |commons-
   ||[EMAIL PROTECTED]
Summary|Javascript Validation   |[validator] Javascript
   |Functions   |Validation Functions




-- 
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]



[Struts Wiki] Update of StrutsRelease127 by NiallPemberton

2005-05-04 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on Struts Wiki for change 
notification.

The following page has been changed by NiallPemberton:
http://wiki.apache.org/struts/StrutsRelease127

The comment on the change is:
Complete Bug Review

--
  || 32595 || Scaffold !BaseAction.executeLogic() calls removed 
!ActionServlet.log(String, Integer) || sandbox || N/A ||
  || 32624 || Scaffold !BaseAction.getMessageResources calls removed 
!ActionServlet.getResources() || sandbox || N/A ||
  || 33132 || org.apache.struts.upload.!MultipartRequestWrapper doesn't 
implement servlet 2.4 api fully || upload || FIXED in 1.2.x ||
- || 33355 || computeURLWithCharEncoding doesn't encodes the  created by the 
response.encudeURL call || taglibs || ? ||
+ || 33355 || computeURLWithCharEncoding doesn't encodes the  created by the 
response.encudeURL call || taglibs || WONTFIX ||
  || 33254 || unable to specify maximum file size attribute greater than 1GB in 
struts-config.xml || upload || WORKSFORME ||
  || 33989 || ant test.tomcat.40 fails: taskdef class 
org.apache.cactus.ant.!RunServerTestsTask cannot be found, task name 
runservertests || test || FIXED in 1.2.x ||
- || 33996 || !ChainAction does not support non-default catalogs || sandbox || 
? ||
+ || 33996 || !ChainAction does not support non-default catalogs || sandbox || 
N/A (not released) ||
  || 34314 || Tiles Request Processor Does Not Catch !NoSuchDefinitionException 
|| tiles || FIXED ||
- || 34445 || use charsets given by browser for form field encodings || upload 
|| ? ||
+ || 34445 || use charsets given by browser for form field encodings || upload 
|| LATER (requires Commons File Upload release) ||
  || 34495 || Add Java Doc to may source files and add more Tapestry  like 
support || shale || N/A ||
  || 34496 || Add Java Doc to may source files and add more Tapestry  like 
support || shale || N/A ||
  || 34628 || html:multibox does not mark input status as checked when HTML 
sensitive characters are present in value. || taglibs || INVALID ||
@@ -44, +44 @@

  
  || '''#''' || '''Description''' || '''Status''' ||
  || 1. || Announce plan to dev@ list; link from roadmap page || Not Done ||
- || 2. || Review/Resolve Outstanding Bugs || Not Done ||
+ || 2. || Review/Resolve Outstanding Bugs || (./) ||
  || 3. || Update Release Notes || Not Done ||
  || 4. || Check Dependencies || Not Done ||
  || 5. || Update to version 1.2.7 build.xml, project.xml, and the MANIFEST.MF 
|| Not Done ||

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



Struts Release - contrib missing

2005-05-04 Thread Niall Pemberton
The last ga Struts release (i.e. version 1.2.4) was done from CVS and the
contents of the contrib folder were shipped in the source distribution.
Since then we have moved to SVN and re-organised the repository and the only
thing now included in the 1.2.x source distribution are the el tags.

Since I haven't used anything from contrib I'm pretty neutral about
including any of it, but given that some people may have done, do we want to
include any or all of the things that we previously shipped?

Struts 1.2.4 included the following in contrib (as well as Struts-el)

/artimus
/scaffold
/struts-chain
/struts-faces
/struts-jericho
/tag-doc
/workflow

For el I believe James did something smart in SVN so that when the
STRUTS_1_2_BRANCH is checked out it includes the el sub-project's
STRUTS_1_2_BRANCH - we would probably want something similar for any of the
remainder that we want to include?

I know some people have built stuff around struts-chain - so I think there
is a case for including that - from the sandbox  STRUTS_1_2_BRANCH -
anything else?

Niall



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



svn commit: r168218 - /struts/core/branches/STRUTS_1_2_BRANCH

2005-05-04 Thread niallp
Author: niallp
Date: Wed May  4 17:18:14 2005
New Revision: 168218

URL: http://svn.apache.org/viewcvs?rev=168218view=rev
Log:
Add svn:externals definition to include struts-chain from 
sandbox/STRUTS_1_2_BRANCH

Modified:
struts/core/branches/STRUTS_1_2_BRANCH/   (props changed)

Propchange: struts/core/branches/STRUTS_1_2_BRANCH/
--
--- svn:externals (original)
+++ svn:externals Wed May  4 17:18:14 2005
@@ -1 +1,2 @@
 contrib/el 
https://svn.apache.org/repos/asf/struts/el/branches/STRUTS_1_2_BRANCH/
+contrib/struts-chain  
https://svn.apache.org/repos/asf/struts/sandbox/branches/STRUTS_1_2_BRANCH/struts-chain



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



Re: Struts Release - contrib missing

2005-05-04 Thread Niall Pemberton
I've added an SVN externals definition so that struts-chain is included in
the core/STRUTS_1_2_BRANCH - its being pulled in from
sandbox/STRUTS_1_2_BRANCH/struts-chain

Niall

- Original Message - 
From: Niall Pemberton [EMAIL PROTECTED]
Sent: Thursday, May 05, 2005 12:29 AM


 The last ga Struts release (i.e. version 1.2.4) was done from CVS and
the
 contents of the contrib folder were shipped in the source distribution.
 Since then we have moved to SVN and re-organised the repository and the
only
 thing now included in the 1.2.x source distribution are the el tags.

 Since I haven't used anything from contrib I'm pretty neutral about
 including any of it, but given that some people may have done, do we want
to
 include any or all of the things that we previously shipped?

 Struts 1.2.4 included the following in contrib (as well as Struts-el)

 /artimus
 /scaffold
 /struts-chain
 /struts-faces
 /struts-jericho
 /tag-doc
 /workflow

 For el I believe James did something smart in SVN so that when the
 STRUTS_1_2_BRANCH is checked out it includes the el sub-project's
 STRUTS_1_2_BRANCH - we would probably want something similar for any of
the
 remainder that we want to include?

 I know some people have built stuff around struts-chain - so I think there
 is a case for including that - from the sandbox  STRUTS_1_2_BRANCH -
 anything else?



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



svn commit: r168223 - /struts/core/branches/STRUTS_1_2_BRANCH/build.xml

2005-05-04 Thread niallp
Author: niallp
Date: Wed May  4 18:03:40 2005
New Revision: 168223

URL: http://svn.apache.org/viewcvs?rev=168223view=rev
Log:
Fix 1.2.x Build - remove duplicate el directory from contrib

Modified:
struts/core/branches/STRUTS_1_2_BRANCH/build.xml

Modified: struts/core/branches/STRUTS_1_2_BRANCH/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/build.xml?rev=168223r1=168222r2=168223view=diff
==
--- struts/core/branches/STRUTS_1_2_BRANCH/build.xml (original)
+++ struts/core/branches/STRUTS_1_2_BRANCH/build.xml Wed May  4 18:03:40 2005
@@ -591,6 +591,9 @@
 /fileset
 /copy
 
+!-- delete duplicate el directory --
+delete dir=${stage.src.dir}/${upload.file.base}-src/contrib/el/
+
 mkdir dir=${upload.dir}/
 zip  zipfile=${upload.dir}/${upload.file.base}.zip
   basedir=${stage.bin.dir}/



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



svn commit: r168228 - /struts/core/branches/STRUTS_1_2_BRANCH/doc/userGuide/release-notes.xml

2005-05-04 Thread niallp
Author: niallp
Date: Wed May  4 18:28:17 2005
New Revision: 168228

URL: http://svn.apache.org/viewcvs?rev=168228view=rev
Log:
Update Version 1.2.7 Release Notes

Modified:
struts/core/branches/STRUTS_1_2_BRANCH/doc/userGuide/release-notes.xml

Modified: struts/core/branches/STRUTS_1_2_BRANCH/doc/userGuide/release-notes.xml
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/doc/userGuide/release-notes.xml?rev=168228r1=168227r2=168228view=diff
==
--- struts/core/branches/STRUTS_1_2_BRANCH/doc/userGuide/release-notes.xml 
(original)
+++ struts/core/branches/STRUTS_1_2_BRANCH/doc/userGuide/release-notes.xml Wed 
May  4 18:28:17 2005
@@ -185,6 +185,39 @@
  
thModification/ththRevision/ththBugzilla/ththDescription/th
  /tr/thead
  tr
+ td align=center2005-05-05/td
+ td align=centera 
href=http://svn.apache.org/viewcvs?rev=168223amp;view=rev;168223/a/td
+ td align=centerin/a/i/td
+ tdFix 1.2.x Build - remove duplicate el directory from 
contrib./td
+ /tr
+ tr
+ td align=center2005-05-05/td
+ td align=centera 
href=http://svn.apache.org/viewcvs?rev=168218amp;view=rev;168218/a/td
+ td align=centerin/a/i/td
+ tdFix 1.2.x build to include struts-chain in the contrib 
directory (from sandbox STRUTS_1_2_BRANCH)./td
+ /tr
+ tr
+ td align=center2005-05-03/td
+ td align=centera 
href=http://svn.apache.org/viewcvs?rev=168011amp;view=rev;168011/a
+ (a 
href=http://svn.apache.org/viewcvs.cgi?rev=168012amp;view=rev;168012/a)/td
+ td align=centerin/a/i/td
+ tdPrevent NullPointerException when toString() is called 
before dynaClass is initialized./td
+ /tr
+ tr
+ td align=center2005-04-29/td
+ td align=centera 
href=http://svn.apache.org/viewcvs?rev=165304amp;view=rev;165304/a
+ (a 
href=http://svn.apache.org/viewcvs.cgi?rev=165305amp;view=rev;165305/a)/td
+ td align=centerin/a/i/td
+ tdRetrieve FormTag stored in Request scope, rather than the 
TagSupport's findAncestorWithClass() ./td
+ /tr
+ tr
+ td align=center2005-04-29/td
+ td align=centera 
href=http://svn.apache.org/viewcvs?rev=165186amp;view=rev;165186/a
+ (a 
href=http://svn.apache.org/viewcvs.cgi?rev=165208amp;view=rev;165208/a)/td
+ td align=centerin/a/i/td
+ tdFixing value escape so that both backslashes and quotes 
are escaped correctly./td
+ /tr
+ tr
  td align=center2005-04-28/td
  td align=centera 
href=http://svn.apache.org/viewcvs?rev=165158amp;view=rev;165158/a
  (a 
href=http://svn.apache.org/viewcvs.cgi?rev=165160amp;view=rev;165160/a)/td



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



svn commit: r168234 - /struts/taglib/trunk/src/java/org/apache/struts/taglib/html/BaseHandlerTag.java

2005-05-04 Thread niallp
Author: niallp
Date: Wed May  4 18:44:58 2005
New Revision: 168234

URL: http://svn.apache.org/viewcvs?rev=168234view=rev
Log:
JavaDoc correction

Modified:

struts/taglib/trunk/src/java/org/apache/struts/taglib/html/BaseHandlerTag.java

Modified: 
struts/taglib/trunk/src/java/org/apache/struts/taglib/html/BaseHandlerTag.java
URL: 
http://svn.apache.org/viewcvs/struts/taglib/trunk/src/java/org/apache/struts/taglib/html/BaseHandlerTag.java?rev=168234r1=168233r2=168234view=diff
==
--- 
struts/taglib/trunk/src/java/org/apache/struts/taglib/html/BaseHandlerTag.java 
(original)
+++ 
struts/taglib/trunk/src/java/org/apache/struts/taglib/html/BaseHandlerTag.java 
Wed May  4 18:44:58 2005
@@ -698,9 +698,8 @@
 }
 
 /**
- *  Appends bean name with index in brackets for tags with
- *  'true' value in 'indexed' attribute.
- *  @param handlers The StringBuffer that output will be appended to.
+ *  Returns the index value for tags with 'true' value in 'indexed' 
attribute.
+ *  @return the index value.
  *  @exception JspException if 'indexed' tag used outside of iterate tag.
  */
 protected int getIndexValue() throws JspException {



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



svn commit: r168242 - /struts/core/trunk/src/share/org/apache/struts/actions/LocaleAction.java

2005-05-04 Thread niallp
Author: niallp
Date: Wed May  4 19:34:31 2005
New Revision: 168242

URL: http://svn.apache.org/viewcvs?rev=168242view=rev
Log:
JavaDoc correction

Modified:
struts/core/trunk/src/share/org/apache/struts/actions/LocaleAction.java

Modified: 
struts/core/trunk/src/share/org/apache/struts/actions/LocaleAction.java
URL: 
http://svn.apache.org/viewcvs/struts/core/trunk/src/share/org/apache/struts/actions/LocaleAction.java?rev=168242r1=168241r2=168242view=diff
==
--- struts/core/trunk/src/share/org/apache/struts/actions/LocaleAction.java 
(original)
+++ struts/core/trunk/src/share/org/apache/struts/actions/LocaleAction.java Wed 
May  4 19:34:31 2005
@@ -58,7 +58,8 @@
  * codecountry/code properties on the given form, constructs an
  * appropriate Locale object, and sets it as the Struts Locale for this
  * user's session.
- * Any codeActionForm, including a [EMAIL PROTECTED] DynaActionForm}, 
may be used.
+ * Any codeActionForm/code, including a 
+ * [EMAIL PROTECTED] org.apache.struts.action.DynaActionForm}, may be used.
  * /p
  * p
  * If a codepage/code property is also provided, then after



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



svn commit: r168243 - in /struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts: actions/LocaleAction.java taglib/html/BaseHandlerTag.java

2005-05-04 Thread niallp
Author: niallp
Date: Wed May  4 19:35:30 2005
New Revision: 168243

URL: http://svn.apache.org/viewcvs?rev=168243view=rev
Log:
Port JavaDoc corrections to 1.2.x branch

Modified:

struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/actions/LocaleAction.java

struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/BaseHandlerTag.java

Modified: 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/actions/LocaleAction.java
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/actions/LocaleAction.java?rev=168243r1=168242r2=168243view=diff
==
--- 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/actions/LocaleAction.java
 (original)
+++ 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/actions/LocaleAction.java
 Wed May  4 19:35:30 2005
@@ -58,7 +58,8 @@
  * codecountry/code properties on the given form, constructs an
  * appropriate Locale object, and sets it as the Struts Locale for this
  * user's session.
- * Any codeActionForm, including a [EMAIL PROTECTED] DynaActionForm}, 
may be used.
+ * Any codeActionForm/code, including a 
+ * [EMAIL PROTECTED] org.apache.struts.action.DynaActionForm}, may be used.
  * /p
  * p
  * If a codepage/code property is also provided, then after

Modified: 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/BaseHandlerTag.java
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/BaseHandlerTag.java?rev=168243r1=168242r2=168243view=diff
==
--- 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/BaseHandlerTag.java
 (original)
+++ 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/BaseHandlerTag.java
 Wed May  4 19:35:30 2005
@@ -698,9 +698,8 @@
 }
 
 /**
- *  Appends bean name with index in brackets for tags with
- *  'true' value in 'indexed' attribute.
- *  @param handlers The StringBuffer that output will be appended to.
+ *  Returns the index value for tags with 'true' value in 'indexed' 
attribute.
+ *  @return the index value.
  *  @exception JspException if 'indexed' tag used outside of iterate tag.
  */
 protected int getIndexValue() throws JspException {



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



svn commit: r168248 - /struts/core/branches/STRUTS_1_2_BRANCH

2005-05-04 Thread niallp
Author: niallp
Date: Wed May  4 19:58:07 2005
New Revision: 168248

URL: http://svn.apache.org/viewcvs?rev=168248view=rev
Log:
Add svn:externals definition to include struts-faces from faces/trunk as per 
Craig's request

Modified:
struts/core/branches/STRUTS_1_2_BRANCH/   (props changed)

Propchange: struts/core/branches/STRUTS_1_2_BRANCH/
--
--- svn:externals (original)
+++ svn:externals Wed May  4 19:58:07 2005
@@ -1,2 +1,3 @@
 contrib/el 
https://svn.apache.org/repos/asf/struts/el/branches/STRUTS_1_2_BRANCH/
 contrib/struts-chain  
https://svn.apache.org/repos/asf/struts/sandbox/branches/STRUTS_1_2_BRANCH/struts-chain
+contrib/struts-faces  https://svn.apache.org/repos/asf/struts/faces/trunk



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



Re: Struts Release - contrib missing

2005-05-04 Thread Niall Pemberton
Also now added an SVN externals definition so that struts-faces is included
in the core/STRUTS_1_2_BRANCH - its being pulled in from faces/trunk
sub-project, following a conversation with Craig McClanahan.

So as it stands now the source distro for 1.2.x will include a contrib
folder with the following in:

/struts-chain
/struts-el
/struts-faces

Niall

 - Original Message - 
 From: Niall Pemberton [EMAIL PROTECTED]
 Sent: Thursday, May 05, 2005 12:29 AM


  The last ga Struts release (i.e. version 1.2.4) was done from CVS and
 the
  contents of the contrib folder were shipped in the source
distribution.
  Since then we have moved to SVN and re-organised the repository and the
 only
  thing now included in the 1.2.x source distribution are the el tags.
 
  Since I haven't used anything from contrib I'm pretty neutral about
  including any of it, but given that some people may have done, do we
want
 to
  include any or all of the things that we previously shipped?
 
  Struts 1.2.4 included the following in contrib (as well as Struts-el)
 
  /artimus
  /scaffold
  /struts-chain
  /struts-faces
  /struts-jericho
  /tag-doc
  /workflow
 
  For el I believe James did something smart in SVN so that when the
  STRUTS_1_2_BRANCH is checked out it includes the el sub-project's
  STRUTS_1_2_BRANCH - we would probably want something similar for any of
 the
  remainder that we want to include?
 
  I know some people have built stuff around struts-chain - so I think
there
  is a case for including that - from the sandbox  STRUTS_1_2_BRANCH -
  anything else?



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



Struts Chain

2005-05-04 Thread Niall Pemberton
Joe,

I just updated the STRUTS_1_2_BRANCH so that struts-chain is included in the
1.2.x source distribution (from sandbox/STRUTS_1_2_BRANCH/struts-chain) - is
there any



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



Re: Struts Chain 1.2.7 Version

2005-05-04 Thread Niall Pemberton
Joe,

I just updated the STRUTS_1_2_BRANCH so that struts-chain is included in the
1.2.x source distribution (from sandbox/STRUTS_1_2_BRANCH/struts-chain) - is
there any merit in including any of your recent Chain changes in 1.2.7?

http://svn.apache.org/viewcvs?rev=165627view=rev
http://svn.apache.org/viewcvs?rev=165626view=rev
http://svn.apache.org/viewcvs?rev=165624view=rev

Niall



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



svn commit: r168253 - /struts/core/branches/STRUTS_1_2_BRANCH/build.xml

2005-05-04 Thread martinc
Author: martinc
Date: Wed May  4 20:44:26 2005
New Revision: 168253

URL: http://svn.apache.org/viewcvs?rev=168253view=rev
Log:
Minor cleanup.

Modified:
struts/core/branches/STRUTS_1_2_BRANCH/build.xml

Modified: struts/core/branches/STRUTS_1_2_BRANCH/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/build.xml?rev=168253r1=168252r2=168253view=diff
==
--- struts/core/branches/STRUTS_1_2_BRANCH/build.xml (original)
+++ struts/core/branches/STRUTS_1_2_BRANCH/build.xml Wed May  4 20:44:26 2005
@@ -276,7 +276,6 @@
 --
 
 mkdir dir=${libdir}/
-mkdir dir=${libdir}/tld/
 
 get dest=${libdir}/commons-beanutils.jar
 usetimestamp=true ignoreerrors=true
@@ -333,9 +332,6 @@
 property name=servlet.jarvalue=${libdir}/servlet.jar/
 property name=jstl.jar   
value=${libdir}/jstl-1.0.6.jar/
 property name=jstl-standard.jar  
value=${libdir}/standard-1.0.6.jar/
-
-property name=mailreader.inherit value=true/
-property name=jstl.tld.dir   value=${libdir}/tld/
 
 
 /target



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



svn commit: r168254 - /struts/core/branches/STRUTS_1_2_BRANCH/doc/roadmap.xml

2005-05-04 Thread martinc
Author: martinc
Date: Wed May  4 20:51:27 2005
New Revision: 168254

URL: http://svn.apache.org/viewcvs?rev=168254view=rev
Log:
Roadmap now points to 1.2.7 wiki page.

Modified:
struts/core/branches/STRUTS_1_2_BRANCH/doc/roadmap.xml

Modified: struts/core/branches/STRUTS_1_2_BRANCH/doc/roadmap.xml
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/doc/roadmap.xml?rev=168254r1=168253r2=168254view=diff
==
--- struts/core/branches/STRUTS_1_2_BRANCH/doc/roadmap.xml (original)
+++ struts/core/branches/STRUTS_1_2_BRANCH/doc/roadmap.xml Wed May  4 20:51:27 
2005
@@ -416,7 +416,7 @@
 
 
 li
-a href=http://wiki.apache.org/struts/StrutsRelease126;Release Plan 
1.2.6/a
+a href=http://wiki.apache.org/struts/StrutsRelease127;Release Plan 
1.2.7/a
 /li
 
 li



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



svn commit: r168255 - in /struts/core/branches/STRUTS_1_2_BRANCH: build.xml conf/share/MANIFEST.MF project.xml

2005-05-04 Thread martinc
Author: martinc
Date: Wed May  4 21:00:32 2005
New Revision: 168255

URL: http://svn.apache.org/viewcvs?rev=168255view=rev
Log:
Update for 1.2.7 build.

Modified:
struts/core/branches/STRUTS_1_2_BRANCH/build.xml
struts/core/branches/STRUTS_1_2_BRANCH/conf/share/MANIFEST.MF
struts/core/branches/STRUTS_1_2_BRANCH/project.xml

Modified: struts/core/branches/STRUTS_1_2_BRANCH/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/build.xml?rev=168255r1=168254r2=168255view=diff
==
--- struts/core/branches/STRUTS_1_2_BRANCH/build.xml (original)
+++ struts/core/branches/STRUTS_1_2_BRANCH/build.xml Wed May  4 21:00:32 2005
@@ -147,7 +147,7 @@
 property name=project.name value=struts/
 
 !-- Version of the project --
-property name=project.version value=1.2.7-dev/
+property name=project.version value=1.2.7/
 
 !-- == Derived Properties  --
 

Modified: struts/core/branches/STRUTS_1_2_BRANCH/conf/share/MANIFEST.MF
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/conf/share/MANIFEST.MF?rev=168255r1=168254r2=168255view=diff
==
--- struts/core/branches/STRUTS_1_2_BRANCH/conf/share/MANIFEST.MF (original)
+++ struts/core/branches/STRUTS_1_2_BRANCH/conf/share/MANIFEST.MF Wed May  4 
21:00:32 2005
@@ -2,11 +2,11 @@
 Extension-Name: Struts Framework
 Specification-Title: Struts Framework
 Specification-Vendor: The Apache Software Foundation
-Specification-Version: 1.2.7-dev
+Specification-Version: 1.2.7
 Implementation-Title: Struts Framework
 Implementation-Vendor: The Apache Software Foundation
 Implementation-Vendor-Id: org.apache
-Implementation-Version: 1.2.7-dev
+Implementation-Version: 1.2.7
 Class-Path: 
   commons-beanutils.jar
   commons-collections.jar

Modified: struts/core/branches/STRUTS_1_2_BRANCH/project.xml
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/project.xml?rev=168255r1=168254r2=168255view=diff
==
--- struts/core/branches/STRUTS_1_2_BRANCH/project.xml (original)
+++ struts/core/branches/STRUTS_1_2_BRANCH/project.xml Wed May  4 21:00:32 2005
@@ -3,7 +3,7 @@
   pomVersion3/pomVersion
   namestruts/name
   idstruts/id
-  currentVersion1.2.7-dev/currentVersion
+  currentVersion1.2.7/currentVersion
   organization
 nameThe Apache Software Foundation/name
 urlhttp://struts.apache.org//url



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



svn commit: r168258 - /struts/el/branches/STRUTS_1_2_BRANCH/project.xml

2005-05-04 Thread martinc
Author: martinc
Date: Wed May  4 21:02:46 2005
New Revision: 168258

URL: http://svn.apache.org/viewcvs?rev=168258view=rev
Log:
Update for 1.2.7 build.

Modified:
struts/el/branches/STRUTS_1_2_BRANCH/project.xml

Modified: struts/el/branches/STRUTS_1_2_BRANCH/project.xml
URL: 
http://svn.apache.org/viewcvs/struts/el/branches/STRUTS_1_2_BRANCH/project.xml?rev=168258r1=168257r2=168258view=diff
==
--- struts/el/branches/STRUTS_1_2_BRANCH/project.xml (original)
+++ struts/el/branches/STRUTS_1_2_BRANCH/project.xml Wed May  4 21:02:46 2005
@@ -4,7 +4,7 @@
   namestruts-el/name
   idstruts-el/id
   groupIdstruts/groupId
-  currentVersion1.2.7-dev/currentVersion
+  currentVersion1.2.7/currentVersion
   inceptionYear2003/inceptionYear
   packageorg.apache.strutsel/package
   shortDescriptionExtension/adaptation of Struts tag libraries to use JSP 
1.2 expression language./shortDescription



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



[Struts Wiki] Update of StrutsRelease127 by MartinCooper

2005-05-04 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on Struts Wiki for change 
notification.

The following page has been changed by MartinCooper:
http://wiki.apache.org/struts/StrutsRelease127

The comment on the change is:
Starting the process of rolling the 1.2.7 build.

--
  == Preparation Checklist ==
  
  || '''#''' || '''Description''' || '''Status''' ||
- || 1. || Announce plan to dev@ list; link from roadmap page || Not Done ||
+ || 1. || Announce plan to dev@ list; link from roadmap page || (./) ||
  || 2. || Review/Resolve Outstanding Bugs || (./) ||
- || 3. || Update Release Notes || Not Done ||
+ || 3. || Update Release Notes || (./) ||
- || 4. || Check Dependencies || Not Done ||
+ || 4. || Check Dependencies || (./) ||
- || 5. || Update to version 1.2.7 build.xml, project.xml, and the MANIFEST.MF 
|| Not Done ||
+ || 5. || Update to version 1.2.7 build.xml, project.xml, and the MANIFEST.MF 
|| (./) ||
  
  The Commons [http://jakarta.apache.org/commons/releases/prepare.html 
Preparation Guide] is a helpful preparation backgrounder, but Commons
  uses the beta/release-candidate/final process.

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



DO NOT REPLY [Bug 34714] - [Shale] Clay PropUtil.setProperty should not fail silently if the property could not be set.

2005-05-04 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
http://issues.apache.org/bugzilla/show_bug.cgi?id=34714.
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=34714





--- Additional Comments From [EMAIL PROTECTED]  2005-05-05 06:05 ---
Created an attachment (id=14937)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=14937action=view)
allowBody patch

Changed the allowBody to be a component node attribute in the clay xml
configuration DTD.  Removed the code with an assumed name in the attributes
collection.  Added the new attribute to the ComponentBean and extend the logic
in the ComponentConfigBean to resolve inheritance on the new attribute.
  
This change will allow the original suggested patch to PropUtils.

I also did some more cleanup work.  I added documentation to the DTD under the
class path that is used by the digester when network connectivity is not
available.  Added some comments to the base clay view-confg.xml

M src/java/org/apache/shale/clay/config/Globals.java
M src/java/org/apache/shale/clay/config/beans/ComponentBean.java
M src/java/org/apache/shale/clay/config/beans/ComponentConfigBean.java
M src/java/org/apache/shale/clay/config/resources/clay-config_1_0.dtd
M src/java/org/apache/shale/clay/parser/builder/InputTextareaBuilder.java
M src/java/org/apache/shale/clay/parser/builder/SelectItemBuilder.java
M src/java/org/apache/shale/clay/parser/builder/Builder.java
M src/java/org/apache/shale/clay/parser/builder/SelectItemsBuilder.java
M src/conf/view-config.dtd
M src/conf/view-config.xml


-- 
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]



[Struts Wiki] Update of StrutsRelease127 by MartinCooper

2005-05-04 Thread Apache Wiki
Dear Wiki user,

You have subscribed to a wiki page or wiki category on Struts Wiki for change 
notification.

The following page has been changed by MartinCooper:
http://wiki.apache.org/struts/StrutsRelease127

--
  === Testing Summary ===
  
  || '''#''' || '''Description''' || '''Completed''' ||
- || 1. || Run Unit Test targets  || No ||
+ || 1. || Run Unit Test targets  || (./) ||
  || 2. || Run Cactus Tests (see below) || No ||
  || 3. || Play test bundled applications (TC 5.0.x) || No ||
  

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