DO NOT REPLY [Bug 34663] New: - DynaActionForm unable to find parameter method every second or third time.

2005-04-28 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=34663.
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=34663

   Summary: DynaActionForm unable to find parameter method every
second or third time.
   Product: Struts
   Version: 1.2.4
  Platform: PC
OS/Version: Windows XP
Status: NEW
  Severity: normal
  Priority: P2
 Component: Controller
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


Very strange error that happens every second or third time the form is 
submitted.  Otherwise it works?  I was orignially using 1.1 struts but just 
upgraded with the same issue to 1.2.4.  I thinking it's a timing or threading 
issue, because of the way it works then doesn't work.  The error message was a 
little better after going to struts 1.2.4.  It suggested This may be caused 
by whitespace in the label text.  Although this would be strange considering 
that it works then doesn't work on the next submit.  The other weird thing 
with this issue is that it seemed to work fine until I added a second 
DynaActionForm and a second method that is mapped in the same way as 
addcomponent action.


Here's the DynaActionForm in struts-config :

form-bean name=addComponentForm
   type=org.apache.struts.action.DynaActionForm
   form-property name=componentName type=java.lang.String 
initial= /
   form-property name=product type=java.lang.String /
 /form-bean

Here's the addcomponent action : 

action path=/addcomponent
 name=addComponentForm
 scope=session
 type=com.ihs.accumap.action.AdminFunctionsAction
 validate=true
 parameter=method
 input=/addComponent.jsp
  forward name=addedcomponent path=/addedcomponent.jsp/
  forward name=failedtoaddcomponent path=/addcomponent.jsp/
  forward name=failure path=/error.jsp/
/action

Here's most of the jsp : 

html:form action=/addcomponent
html:hidden name=testListForm property=product/ !-- we need the product 
name for this component as well --

table VALIGN=top cellpadding=3 cellspacing=1 WIDTH=%80 
bgcolor=cc align=center
tr
  tdEnter new component name : /td tdhtml:text size=25 
name=addComponentForm property=componentName//td
/tr
  td
  table align=right bgcolor=cc
  td align=right 
  html:submit property=method
bean:message key=function.addcomponent/
  /html:submit
  html:reset/
  /td
/tr
  /table
/td
  /tr
/table
/html:form
br/
table
tr
tdhtml:errors//td
/tr
tr
tdhtml:messages id=mess//td
/tr
/table
/body
/html:html

Here's the important bits out of my Action class: 

public class AdminFunctionsAction extends LookupDispatchAction
 {
  
protected Map getKeyMethodMap() {
  Map map = new HashMap();
  map.put(function.addtestcase, addTestCase);
  map.put(function.addproduct, addProduct);
  map.put(function.addcomponent, addComponent);
  map.put(function.deletecomponent, removeComponent );
  return map;
}


  
public ActionForward addComponent( ActionMapping mapping,
  ActionForm form,
  HttpServletRequest request,
  HttpServletResponse response)
  throws Exception {
  // do add
  
  DynaActionForm addCompForm = (DynaActionForm) form;

  try{
HttpSession session = request.getSession();
UserAccountsView user = this.getUser( session );


if( user.getUserName().length()  1 )
return  (mapping.findForward(needtoauthenticate));

ComponentService compService = this.getComponentService( session );
  
if ( compService.addComponent( addCompForm.get(componentName ) + , 
addCompForm.get(product) + , user ) )
  return mapping.findForward(addedcomponent );
else return mapping.findForward(failedtoaddcomponent);
  } catch( Exception x ){
x.printStackTrace();
System.out.println(Blew up trying to add component);
return mapping.findForward(failedtoaddcomponent);
  }finally{
addCompForm.set(componentName,  );
  }
}



Here's the error:

javax.servlet.ServletException: Request[/addcomponent] does not contain 
handler parameter named 'method'.  This may be caused by whitespace in the 
label text.
at org.apache.struts.actions.DispatchAction.unspecified
(DispatchAction.java:224)
at org.apache.struts.actions.DispatchAction.dispatchMethod
(DispatchAction.java:258)
at org.apache.struts.actions.LookupDispatchAction.execute
(LookupDispatchAction.java:162)
at org.apache.struts.action.RequestProcessor.processActionPerform

DO NOT REPLY [Bug 34663] - DynaActionForm unable to find parameter method every second or third time.

2005-04-28 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=34663.
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=34663


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |major




--- Additional Comments From [EMAIL PROTECTED]  2005-04-28 08:47 ---
oh yeah and here's the button text from my resource file...(you'll notice no 
white space).

function.addcomponent=AddComponent


I running on Tomcat version 4.1.x (latest).

-- 
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 34663] - DynaActionForm unable to find parameter method every second or third time.

2005-04-28 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=34663.
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=34663


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||INVALID




--- Additional Comments From [EMAIL PROTECTED]  2005-04-28 09:22 ---
The convention here is to post questions to the user list when you're having 
problems. Very often you'll find someone whoses experienced the same thing and 
can tell you whether its a bug or something you're doing. You'll also get a 
much wider audience that can help you. Since you haven't asked on the user list 
first, I'm closing this as INVALID. If you identify that it is actually a bug 
in Struts or get no joy from the user list then you can re-open this later. 
Thank You.

-- 
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: JDK Version for 1.3.x

2005-04-28 Thread David Graham
1.4.2 sounds good to me.

David

--- Don Brown [EMAIL PROTECTED] wrote:
 I can't find the thread, but 1.4 gets my vote.
 
 Don
 
 Niall Pemberton wrote:
  I remember the discussion about the JDK version for Struts 1.3 - but I
 can't
  remember if an actual decision was made. Have we decided on JDK 1.4 as
 a
  minimum for Struts 1.3?
  
  Niall
  
  
  
  -
  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 You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 

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



DO NOT REPLY [Bug 20034] - Invalid cache in InsertTag

2005-04-28 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=20034.
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=20034


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2005-04-28 18:33 ---
I decided against the patch I originally posted and have just added a line in 
the doStartTag() to clear the cached context value.

This has been fixed in the current development branch and the 1.2.x series.

-- 
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: r165162 - /struts/core/branches/STRUTS_1_2_BRANCH/doc/userGuide/release-notes.xml

2005-04-28 Thread niallp
Author: niallp
Date: Thu Apr 28 09:40:17 2005
New Revision: 165162

URL: http://svn.apache.org/viewcvs?rev=165162view=rev
Log:
Update 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=165162r1=165161r2=165162view=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 Thu 
Apr 28 09:40:17 2005
@@ -32,7 +32,7 @@
   Version 1.2.4 release. For detailed information concerning 
changes 
   see the a href=#DetailDetailed Changes/a section.
   /p
-  h32.1 Dependencies/h3
+  h3Dependencies/h3
   p
   Struts has changed its dependencies on the following software 
components:
   /p
@@ -184,6 +184,19 @@
  theadtr
  
thModification/ththRevision/ththBugzilla/ththDescription/th
  /tr/thead
+ 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
+ td align=centera 
href=http://issues.apache.org/bugzilla/show_bug.cgi?id=20034;20034/a/td
+ tdAdditional fix to Invalid cache in InsertTag for a 
problem on Orion./td
+ /tr
+ tr
+ td align=center2005-04-28/td
+ td align=centera 
href=http://svn.apache.org/viewcvs?rev=165097amp;view=rev;165097/a/td
+ td align=centera 
href=http://issues.apache.org/bugzilla/show_bug.cgi?id=33989;33989/a/td
+ tdFix cactus tests to work with Cactus 1.7./td
+ /tr
  tr
  td align=center2005-04-27/td
  td align=centera 
href=http://svn.apache.org/viewcvs?rev=164922amp;view=rev;164922/a



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



[Struts Wiki] Update of StrutsRelease127 by NiallPemberton

2005-04-28 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

--
  == Outstanding Bug Review ==
  
  || '''ID''' || '''Summary''' || '''Component''' || '''Status''' ||
- || 20034 || Invalid cache in !InsertTag || taglibs || ? ||
+ || 20034 || Invalid cache in !InsertTag || taglibs || FIXED ||
  || 23127 || Page attribute of img and image tags doesn't use pagePattern 
setting || taglibs || FIXED ||
  || 31230 || Multiple classes using deprecated !DefinitionsUtil class || Tiles 
|| Remove deprecations in 1.3.x ||
  || 31658 || !LogonAction does not check errors in appropriate place || 
!MailReader || FIXED ||
@@ -32, +32 @@

  || 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 || ? ||
  || 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 || ? ||
+ || 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 || 
? ||
  || 34314 || Tiles Request Processor Does Not Catch !NoSuchDefinitionException 
|| tiles || FIXED ||
  || 34445 || use charsets given by browser for form field encodings || upload 
|| ? ||

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



[Struts Wiki] Update of StrutsRelease127 by NiallPemberton

2005-04-28 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

--
  
  || '''Dependency''' || '''Version''' || '''Status''' ||
  || Commons !BeanUtils || 1.7.0 || Released ||
- || Commons Collections || 2.1.1 || Released ||
  || Commons Digester || 1.6 || Released ||
  || Commons !FileUpload || 1.0 || Released ||
  || Commons Logging || 1.0.4 || Released ||

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



RE: JDK Version for 1.3.x

2005-04-28 Thread Pilgrim, Peter

How about Java 5.0?

Just jokkin' man

--
Peter Pilgrim
Operations/IT - Credit Suisse First Boston, 
Floor 15, 5 Canada Square, London E14 4QJ, United Kingdom
Tel: +44-(0)207-883-4497


 -Original Message-
 From: David Graham [mailto:[EMAIL PROTECTED]
 Sent: 28 April 2005 14:36
 To: Struts Developers List
 Subject: Re: JDK Version for 1.3.x
 
 
 1.4.2 sounds good to me.
 
 David
 
 --- Don Brown [EMAIL PROTECTED] wrote:
  I can't find the thread, but 1.4 gets my vote.
  
  Don
  
  Niall Pemberton wrote:
   I remember the discussion about the JDK version for 
 Struts 1.3 - but I
  can't
   remember if an actual decision was made. Have we decided 
 on JDK 1.4 as
  a
   minimum for Struts 1.3?
   
   Niall
   
   
   
   
 -
   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 You Yahoo!?
 Tired of spam?  Yahoo! Mail has the best spam protection around 
 http://mail.yahoo.com 
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 
 

==
This message is for the sole use of the intended recipient. If you received
this message in error please delete it and notify us. If this message was
misdirected, CSFB does not waive any confidentiality or privilege. CSFB
retains and monitors electronic communications sent through its network.
Instructions transmitted over this system are not binding on CSFB until they
are confirmed by us. Message transmission is not guaranteed to be secure.
==


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



Re: JDK Version for 1.3.x

2005-04-28 Thread Niall Pemberton
Actually that was discussed...

http://www.mail-archive.com/dev@struts.apache.org/msg02909.html

The thing was we took a decision on the Servlet Spec but the JDK aspect was
just discussed.

- Original Message - 
From: Pilgrim, Peter [EMAIL PROTECTED]
Sent: Thursday, April 28, 2005 6:10 PM

 How about Java 5.0?



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



[Struts Wiki] Update of StrutsMaintenanceSvnApps by James Mitchell

2005-04-28 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 James Mitchell:
http://wiki.apache.org/struts/StrutsMaintenanceSvnApps

New page:
Apps Strategy (overview):
---
The strategy for building apps here is that Maven will build 
the variations of mailreader apps by repeating a process that, 
for each mailreader app:

a) copies the shared base file structure and files
   to the target dir
b) copies **/*.* (from the app) over top of (a)
c) generates any docs/javadocs/etc and wars the artifact
(would also run the full test suite)

This process would repeat for each one-off mailreaderlike so:

{{{
 -- default --
apps/trunk/mailreader/blank
apps/trunk/mailreader/shared
apps/trunk/mailreader/el
apps/trunk/mailreader/shale
apps/trunk/mailreader/jericho
apps/trunk/mailreader/resources (impl using commons-resources)
apps/trunk/mailreader/chain (this demonstrates chain for a wizard)
apps/trunk/mailreader/i18n  (using commons-i18n)
apps/trunk/mailreader/tiles (this is not the tiles doc app)

 -- optional --
apps/trunk/mailreader/faces (the integration library)
apps/trunk/mailreader/j2ee  (mostly xdoclet generated ejb 3.0
 against HSSQLDB)
apps/trunk/mailreader/spring
apps/trunk/mailreader/hibernate
apps/trunk/mailreader/spring-hibernate (it's a beautiful thing)
apps/trunk/mailreader/resources-hibernate  (HSSQLDB with Hibernate)
apps/trunk/mailreader/resources-iBatis (HSSQLDB with iBatis)
apps/trunk/mailreader/resources-jdbc   (HSSQLDB with JDBC)


There's a main project.xml in apps/trunk/
Each mailreader app project.xml extends that and overrides only what it
needs to for that specific app.  This is more or less why Maven was 
invented in the first place (from what I've read).

This way we don't have to have full blown webapp structures that 
duplicate 95% of the code.  Only the files that are actually different
than the shared will exist in these one off'swhat do we call
these anyway?  Mailreader app demo?  (MAD)

So, in the end, if you were to add some new button or page to 
the shared app, they would all inherit it.  And if something changes 
that breaks any one of the MADs (LOL), the extensive test coverage
would catch thatyes I'm volunteering again ;)

What are the current targets (other than stock maven ones)

mailreader:template - produces the basic folders and project.xml/maven.xml that
  would allow for an alternate implementation of the 
  mailreader application
  


Details:
---
 [-] apps
  ' [-] trunk
  '  '   maven.xml   (reactor!!!)
  '  '   project.xml (parent config)
  '  '
  '  ' [-] blank
  '  '  maven.xml   
  '  '  project.xml (parent config)
  '  '
  '  '[-] src
  '  '   [+] java (nothing in here right now)
  '  '   [+] resources
  '  '   [-] webapp
  '  '  [+] WEB-INF
  '  '[-] test
  '  '   
  '  '
  ' [-]  mailreader
  '  ' [-] src
  '  '[+] java
  '  '[+] resources
  '  '[+] test
  '  '[+] webapp
  '  '
  ' [-]  examples
  '  ' (exercise, upload, validator)
  '  ' (i feel this should be renamed to 'modules')
  '  ' [-] src
  '  '[+] java
  '  '[+] resources
  '  '[+] test
  '  '[+] webapp
  '  '
  ' [-]  test
  '  '  (really need to get this back in order
  '  '   or figure out a new strategy..more on
  '  '   this later)
  '  ' [-] src
  '  '[+] java
  '  '[+] resources
  '  '[+] test
  '  '[+] webapp
  '  '
  ' [-] tile-documentation
  '  ' (this should probably go to tiles/trunk)
  '  ' [-] src
  '  '[+] java
  '  '[+] resources
  '  '[+] test
  '  '[+] webapp

}}}

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



Struts 1.3 Taglib JavascriptValidatorTag bug?

2005-04-28 Thread Corey Probst
Hi.  I have a question about the createDynamicJavascript() function in
the JavascriptValidatorTag class in Struts 1.3.  I am using the
nightly builds (not building from source) and I noticed that my
regular expression mask doesn't work anymore.  I'm using the following
regular expression... ^\d{5}(\-\d{4})?$ for zipcode + option 4.

The problem:
In the function, code was added to escape the double quotes right
after we escape the backward slash.  But it uses the original value
instead of the escaped value, like this...

String varValueEscaped = ValidatorUtils.replace(varValue, \\, );
varValueEscaped = ValidatorUtils.replace(varValue, \, \\\);

I think it should be...1
String varValueEscaped = ValidatorUtils.replace(varValue, \\, );
varValueEscaped = ValidatorUtils.replace(varValueEscaped, \, \\\);

So we don't lose the escaped backward slashes.

If I'm missing something, let me know.  I didn't know if I should
enter a bug or just send a note to the list since 1.3 isn't released
yet.

Thanks,
Corey

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



svn commit: r165186 - /struts/taglib/trunk/src/java/org/apache/struts/taglib/html/JavascriptValidatorTag.java

2005-04-28 Thread mrdon
Author: mrdon
Date: Thu Apr 28 12:30:34 2005
New Revision: 165186

URL: http://svn.apache.org/viewcvs?rev=165186view=rev
Log:
Fixed escaping slashes being lost when escaping quotes - thanks Corey

Modified:

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

Modified: 
struts/taglib/trunk/src/java/org/apache/struts/taglib/html/JavascriptValidatorTag.java
URL: 
http://svn.apache.org/viewcvs/struts/taglib/trunk/src/java/org/apache/struts/taglib/html/JavascriptValidatorTag.java?rev=165186r1=165185r2=165186view=diff
==
--- 
struts/taglib/trunk/src/java/org/apache/struts/taglib/html/JavascriptValidatorTag.java
 (original)
+++ 
struts/taglib/trunk/src/java/org/apache/struts/taglib/html/JavascriptValidatorTag.java
 Thu Apr 28 12:30:34 2005
@@ -518,7 +518,7 @@
 }
 
 String varValueEscaped = ValidatorUtils.replace(varValue, 
\\, );
-varValueEscaped = ValidatorUtils.replace(varValue, \, 
\\\);
+varValueEscaped = ValidatorUtils.replace(varValueEscaped, 
\, \\\);
 
 if (Var.JSTYPE_INT.equalsIgnoreCase(jsType)) {
 results.append(



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



Re: Struts 1.3 Taglib JavascriptValidatorTag bug?

2005-04-28 Thread Don Brown
Good catch - it is fixed in changeset 165186.  Thanks for pointing it out.
Don
Corey Probst wrote:
Hi.  I have a question about the createDynamicJavascript() function in
the JavascriptValidatorTag class in Struts 1.3.  I am using the
nightly builds (not building from source) and I noticed that my
regular expression mask doesn't work anymore.  I'm using the following
regular expression... ^\d{5}(\-\d{4})?$ for zipcode + option 4.
The problem:
In the function, code was added to escape the double quotes right
after we escape the backward slash.  But it uses the original value
instead of the escaped value, like this...
String varValueEscaped = ValidatorUtils.replace(varValue, \\, );
varValueEscaped = ValidatorUtils.replace(varValue, \, \\\);
I think it should be...1
String varValueEscaped = ValidatorUtils.replace(varValue, \\, );
varValueEscaped = ValidatorUtils.replace(varValueEscaped, \, \\\);
So we don't lose the escaped backward slashes.
If I'm missing something, let me know.  I didn't know if I should
enter a bug or just send a note to the list since 1.3 isn't released
yet.
Thanks,
Corey
-
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]


[Struts Wiki] Update of StrutsMaintenance by James Mitchell

2005-04-28 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 James Mitchell:
http://wiki.apache.org/struts/StrutsMaintenance

--
   * StrutsMaintenanceMaven - Notes on Strut's maven build
   * StrutsMaintenanceGump - Notes on Strut's gump metadata
   * StrutsMaintenanceValidWhenParser - Notes on changing ValidWhenParser.g
+  * StrutsMaintenanceSvnApps - Notes on structure and strategy for developing 
and building apps under the current layout.
  

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



svn commit: r165208 - /struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/JavascriptValidatorTag.java

2005-04-28 Thread mrdon
Author: mrdon
Date: Thu Apr 28 14:41:45 2005
New Revision: 165208

URL: http://svn.apache.org/viewcvs?rev=165208view=rev
Log:
Fixing value escape so that both backslashes and quotes are escaped
correctly

Modified:

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

Modified: 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/JavascriptValidatorTag.java
URL: 
http://svn.apache.org/viewcvs/struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/JavascriptValidatorTag.java?rev=165208r1=165207r2=165208view=diff
==
--- 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/JavascriptValidatorTag.java
 (original)
+++ 
struts/core/branches/STRUTS_1_2_BRANCH/src/share/org/apache/struts/taglib/html/JavascriptValidatorTag.java
 Thu Apr 28 14:41:45 2005
@@ -518,7 +518,7 @@
 }
 
 String varValueEscaped = ValidatorUtils.replace(varValue, 
\\, );
-varValueEscaped = ValidatorUtils.replace(varValue, \, 
\\\);
+varValueEscaped = ValidatorUtils.replace(varValueEscaped, 
\, \\\);
 
 if (Var.JSTYPE_INT.equalsIgnoreCase(jsType)) {
 results.append(



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



DispatchAction: use name instead of value

2005-04-28 Thread Michael Jouravlev
What about allowing to use name instead of value to select
dispatch method? This frees up value, which really is a relief, since
pushbuttons display text from value attribute.  Current naming
convention requires to use the same name... er, the same value for
button and for dispatched method, which may be not convenient.

Using name works because only clicked input is submitted, other do
not. Of course, inputs that are linked to methods, would need to have
special prefix to distinguish them from other request parameters.

Action.Parameter can be reused to specify prefix.

Michael.

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



DO NOT REPLY [Bug 34660] - [Shale] Clay was not setting properties correctly

2005-04-28 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=34660.
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=34660





--- Additional Comments From [EMAIL PROTECTED]  2005-04-29 00:01 ---
Created an attachment (id=14874)
 -- (http://issues.apache.org/bugzilla/attachment.cgi?id=14874action=view)
simple junit test case


-- 
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: JDK Version for 1.3.x

2005-04-28 Thread Ted Husted
On 4/28/05, Pilgrim, Peter [EMAIL PROTECTED] wrote:
 How about Java 5.0?
 
 Just jokkin' man

If someone had an itch, I'd wager that a Struts Tiger codebase, that
leveraged the new language attributes, would be a good way to get
started on Struts 2.x.

-Ted.

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



svn commit: r165229 - /struts/shale/trunk/use-cases/src/web/WEB-INF/faces-config.xml

2005-04-28 Thread dgeary
Author: dgeary
Date: Thu Apr 28 16:24:30 2005
New Revision: 165229

URL: http://svn.apache.org/viewcvs?rev=165229view=rev
Log:
Reinstated some navigation cases that don't use dialogs and fixed the class 
name of the validator Test managed bean

Modified:
struts/shale/trunk/use-cases/src/web/WEB-INF/faces-config.xml

Modified: struts/shale/trunk/use-cases/src/web/WEB-INF/faces-config.xml
URL: 
http://svn.apache.org/viewcvs/struts/shale/trunk/use-cases/src/web/WEB-INF/faces-config.xml?rev=165229r1=165228r2=165229view=diff
==
--- struts/shale/trunk/use-cases/src/web/WEB-INF/faces-config.xml (original)
+++ struts/shale/trunk/use-cases/src/web/WEB-INF/faces-config.xml Thu Apr 28 
16:24:30 2005
@@ -112,7 +112,7 @@
   managed-bean
 managed-bean-namevalidate$test/managed-bean-name
 managed-bean-class
-  org.apache.shale.usecases.validate.Test
+  org.apache.shale.usecases.validator.Test
 /managed-bean-class
 managed-bean-scoperequest/managed-bean-scope
   /managed-bean
@@ -199,10 +199,6 @@
   navigation-rule
 from-view-id*/from-view-id
 navigation-case
-  from-outcomeusecases$toplevel/from-outcome
-  to-view-id/usecases.jsp/to-view-id
-/navigation-case
-navigation-case
   from-outcomelogon$authenticated/from-outcome
   to-view-id/usecases.jsp/to-view-id
 /navigation-case
@@ -226,10 +222,9 @@
   from-outcomelogon$unauthenticated/from-outcome
   to-view-id/usecases.jsp/to-view-id
 /navigation-case
-navigation-case
-  from-outcomevalidate$test/from-outcome
-  to-view-id/validator/test.jsp/to-view-id
-/navigation-case
+  /navigation-rule
+--
+  navigation-rule
 navigation-case
   from-outcomevalidate$test/from-outcome
   to-view-id/validator/test.jsp/to-view-id
@@ -238,8 +233,22 @@
   from-outcomevalidate$thankYou/from-outcome
   to-view-id/validator/thankYou.jsp/to-view-id
 /navigation-case
+navigation-case
+  from-outcomeusecases$toplevel/from-outcome
+  to-view-id/usecases.jsp/to-view-id
+/navigation-case
+  /navigation-rule
+
+  navigation-rule
+navigation-case
+  from-outcomeusecases$toplevel/from-outcome
+  to-view-id/usecases.jsp/to-view-id
+/navigation-case
+navigation-case
+  from-outcomevalidate$test/from-outcome
+  to-view-id/validator/test.jsp/to-view-id
+/navigation-case
   /navigation-rule
---
 
 
   !-- === Logon and Profile Edit Dialog === 
--



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



[Struts Wiki] Update of FrontPage by HubertRabago

2005-04-28 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 HubertRabago:
http://wiki.apache.org/struts/FrontPage

--
  === Editing This Wiki ===
  In order to give credit where credit is due, you are now required to log in 
before you can edit the pages here.  Do this by selecting the UserPreferences 
link and setting up an account.  Then, use this link to log in, and pages will 
be editable.
  
- == Steven, Welcome to the Struts Wiki ==
+ == Welcome to the Struts Wiki ==
  
  The goal of the Struts project is to provide an open source framework for 
building web applications. The core of the Struts framework is a flexible 
control layer based on standard technologies such as Java Servlets, JavaBeans, 
resource bundles, and the Extensible Markup Language (XML).
  

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