DO NOT REPLY [Bug 36028] - [Standalone Tiles] Refactor DefinitionsFactory and Create Reloadable Definitions

2005-08-04 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=36028





--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 07:29 ---
I agree that the *user* viewpoint of backwards compatibility is very important.
 Indeed, if/when Struts 1.x migrates to this version, it would likely be a good
thing to provide a deprecated org.apache.struts.tiles.Controller (and probably
ControllerSupport) implementation that simply delegate to the standalone
versions ... but would ease the transition for existing applications.

For the internal structure, we have a once-only opportunity to refactor the
internals, so this is a good time to look carefully at that stuff.  I'll be
reviewing Greg's proposed refactoring from that viewpoint.


-- 
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: r230400 - in /struts/sandbox/trunk/ti/src/java/org/apache/ti: config/ config/mapper/ processor/ processor/chain/ servlet/ util/

2005-08-04 Thread martinc
Author: martinc
Date: Thu Aug  4 22:13:54 2005
New Revision: 230400

URL: http://svn.apache.org/viewcvs?rev=230400&view=rev
Log:
Fix a bunch more Checkstyle complaints.

Modified:
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapper.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/BaseResult.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerActionInvocation.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ProcessorException.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/RequestProcessor.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ChainRequestProcessor.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionMapping.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionProxy.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ExecuteAction.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitXWork.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ProcessActionChain.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/SpringConfigRuleSet.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/servlet/StrutsTiServlet.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/util/ServletSourceResolver.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/util/SourceResolver.java

Modified: struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java?rev=230400&r1=230399&r2=230400&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java Thu 
Aug  4 22:13:54 2005
@@ -49,14 +49,14 @@
 protected void crawl(File src, String srcName, File dest, String destName, 
List stack) throws IOException {
 File[] kids = src.listFiles();
 boolean controllerFound = false;
-for (int x=0; xhttp://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java?rev=230400&r1=230399&r2=230400&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java 
Thu Aug  4 22:13:54 2005
@@ -55,7 +55,7 @@
 template = velocity.getTemplate(templateName);
 } catch (ResourceNotFoundException ex) {
 log.error("Unable to locate template to process javadoc tags", ex);
-}catch (Exception ex) {
+} catch (Exception ex) {
 log.error("Unable to intialize velocity", ex);
 }
 

Modified: 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapper.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapper.java?rev=230400&r1=230399&r2=230400&view=diff
==
--- 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapper.java 
(original)
+++ 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapper.java 
Thu Aug  4 22:13:54 2005
@@ -20,7 +20,7 @@
 import org.apache.commons.chain.web.WebContext;
 
 /**
- * Handles creation of ActionMapping and reconstruction of URI's from one
+ * Handles creation of ActionMapping and reconstruction of URI's from one.
  */
 public interface ActionMapper {
 ActionMapping getMapping(WebContext ctx);

Modified: 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java?rev=230400&r1=230399&r2=230400&view=diff
==
--- 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java
 (original)
+++ 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java
 Thu Aug  4 22:13:54 2005
@@ -38,7 +38,7 @@
 
 public ActionMapping getMapping(WebContext ctx) {
 
-HttpServletRequest request = ((ServletWebContext)ctx).getRequest();
+HttpServletRequest request = ((

svn commit: r230396 - in /struts/sandbox/trunk/ti: ./ src/example/ src/example/WEB-INF/ src/example/WEB-INF/src/ src/java/ src/java/org/apache/ti/config/ src/java/org/apache/ti/processor/chain/ src/te

2005-08-04 Thread martinc
Author: martinc
Date: Thu Aug  4 21:39:37 2005
New Revision: 230396

URL: http://svn.apache.org/viewcvs?rev=230396&view=rev
Log:
Fix line ends and set eol-style and keyword properties.

Modified:
struts/sandbox/trunk/ti/LICENSE.txt   (props changed)
struts/sandbox/trunk/ti/NOTICE.txt   (contents, props changed)
struts/sandbox/trunk/ti/README.txt   (contents, props changed)
struts/sandbox/trunk/ti/project.xml   (contents, props changed)
struts/sandbox/trunk/ti/src/example/WEB-INF/src/build.xml   (props changed)
struts/sandbox/trunk/ti/src/example/WEB-INF/web.xml   (props changed)
struts/sandbox/trunk/ti/src/example/index.jsp   (props changed)

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/spring-config-servlet.xml 
  (props changed)
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/velocity.properties   
(contents, props changed)

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/chain-config-servlet.xml
   (props changed)
struts/sandbox/trunk/ti/src/java/xwork-default.xml   (props changed)

struts/sandbox/trunk/ti/src/test/org/apache/ti/processor/chain/chain-config.xml 
  (props changed)

Propchange: struts/sandbox/trunk/ti/LICENSE.txt
--
svn:eol-style = native

Modified: struts/sandbox/trunk/ti/NOTICE.txt
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/NOTICE.txt?rev=230396&r1=230395&r2=230396&view=diff
==
--- struts/sandbox/trunk/ti/NOTICE.txt (original)
+++ struts/sandbox/trunk/ti/NOTICE.txt Thu Aug  4 21:39:37 2005
@@ -1,8 +1,8 @@
-NOTICE
-
-This software consists of voluntary contributions made by many
-individuals on behalf of the Apache Software Foundation
-
-Portions of this software are Copyright(c) 2002-2003 by 
-OpenSymphony and provided under the Open Symphony License 
+NOTICE
+
+This software consists of voluntary contributions made by many
+individuals on behalf of the Apache Software Foundation
+
+Portions of this software are Copyright(c) 2002-2003 by 
+OpenSymphony and provided under the Open Symphony License 
 [http://opensymphony.com/xwork/license.action].

Propchange: struts/sandbox/trunk/ti/NOTICE.txt
--
svn:eol-style = native

Modified: struts/sandbox/trunk/ti/README.txt
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/README.txt?rev=230396&r1=230395&r2=230396&view=diff
==
--- struts/sandbox/trunk/ti/README.txt (original)
+++ struts/sandbox/trunk/ti/README.txt Thu Aug  4 21:39:37 2005
@@ -1,80 +1,80 @@
-OVERVIEW
-
-Struts Ti is a simplified Model 2 framework for developing webapps which
-allows the developer better access to the underlying servlet/portlet
-environment. 
-
-Struts Ti serves a niche of web applications that don’t want the
-additional complexity of server-side components and verbose
-configuration, yet want the structure and controller features of a
-modern web framework. Struts Ti builds on the directions of Struts 1.x,
-yet re-implements the framework to provide a clean slate for the next
-generation of Struts Ti. It aims to combine the simplicity of Ruby on
-Rails and NanoWeb, the refinement of WebWork 2, the tool-friendly
-authoring and Page Flow of Beehive, and the lessons learned from Struts 1.x.
-
-The key word for Struts Ti is simplicity. Ideally, Struts Ti should
-approach Ruby on Rails levels of easy of use, yet scale up to large
-applications providing a smooth transition to JSF/Shale if desired.
-
-KEY FEATURES
-
-* POJO-based action that combines an Action and ActionForm in a
-similar manner to JSF backing beans and WebWork 2 Commands
-* Intelligent defaults utilizing naming and placement conventions
-to require minimal, if any, configuration per page, however it will be
-possible to override everything on a global and per-action basis
-* Configuration can be “assumed” or declared through annotations,
-xml or properties files, or any other pluggable mechanism
-* Pluggable EL for data binding defaulting to JSP 2.0 EL but
-allowing OGNL or even BeanUtils
-* Integration of a dialog or page flow capability drawing from
-Beehive, Spring’s web flow, and Shale’s Dialogs.
-* Per-Action optional interceptor chain ala WebWork 2
-* Built-in dependency injection support
-
-DESIGN GOALS
-
-* No servlet dependency in core framework, portlet and JSF support
-out of the box
-* Spring-based dependency injection in core to allow for pluggability
-* No bias to any view technology
-* Ability to layer Struts 1.x compatibility on top
-* Highly toolable
-* Smooth integration into a portal/portlet environment
-* AJAX-friendly
-
-IMPLEMENTATION
-
-* Built on the backbone of commons-chain
-* No restriction on multiple Se

svn commit: r230395 - in /struts/sandbox/trunk/ti/src/java/org/apache/ti: config/ config/mapper/ processor/ processor/chain/ servlet/ util/

2005-08-04 Thread martinc
Author: martinc
Date: Thu Aug  4 21:28:45 2005
New Revision: 230395

URL: http://svn.apache.org/viewcvs?rev=230395&view=rev
Log:
Fix a couple of hundred Checkstyle complaints.

Modified:
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapping.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/BaseResult.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerAction.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerActionInvocation.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ProcessorException.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/RequestProcessor.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/SpringActionProxyFactory.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/SpringObjectFactory.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ChainRequestProcessor.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionMapping.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionProxy.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ExecuteAction.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitXWork.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ProcessActionChain.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/SpringConfigRuleSet.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/servlet/ServletDispatcherResult.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/servlet/StrutsTiServlet.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/util/ServletSourceResolver.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/util/SourceResolver.java

Modified: 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java?rev=230395&r1=230394&r2=230395&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java 
Thu Aug  4 21:28:45 2005
@@ -31,27 +31,27 @@
 private ProcessTags processTags;
 private File srcdir;
 private File destdir;
-
+
 private static final Log log = LogFactory.getLog(ProcessTagsTask.class);
-
+
 public void setSrcdir(File file) {
 this.srcdir = file;
 }
-
+
 public void setDestdir(File file) {
 this.destdir = file;
 }
-
+
 public void execute() {
 XDocletParser parser = new XDocletParser();
 parser.init();
 ProcessTags pt = new ProcessTags();
 pt.setXDocletParser(parser);
-
+
 try {
 pt.process(srcdir, "Controller.java", destdir, "xwork.xml");
 } catch (IOException ex) {
 ex.printStackTrace();
 }
 }
-}
+}

Modified: 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java?rev=230395&r1=230394&r2=230395&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java 
Thu Aug  4 21:28:45 2005
@@ -35,7 +35,7 @@
 import xjavadoc.filesystem.ReaderFile;
 
 /**
- *  Processes xdoclet-style tags and uses a velocity template to generate 
+ *  Processes xdoclet-style tags and uses a velocity template to generate
  *  content.
  */
 public class XDocletParser {
@@ -44,10 +44,10 @@
 private Template template;
 private Map parameters;
 private static final Log log = LogFactory.getLog(XDocletParser.class);
-
+
 public void init() {
 VelocityEngine velocity = new VelocityEngine();
-
+
 Properties props = new Properties();
 try {
 props.load(getClass().getResourceAsStream("velocity.properties"));
@@ -57,46 +57,46 @@
 log.error("Unable to locate template to process javadoc tags", ex);
 }catch (Exception ex) {
 log.error("Unable to intialize velocity", ex);
-}
-
+}
+
 }
-
+
 public void setParameters(Map m

DO NOT REPLY [Bug 36037] - Allow query strings and anchors when returning forwards from Action

2005-08-04 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=36037





--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 06:00 ---
Created an attachment (id=15897)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=15897&action=view)
Very simple sample webapp to demonstrate the patch


-- 
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 36037] - Allow query strings and anchors when returning forwards from Action

2005-08-04 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=36037





--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 05:57 ---
Created an attachment (id=15896)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=15896&action=view)
The patch file, covers all four altered classes


-- 
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 36037] New: - Allow query strings and anchors when returning forwards from Action

2005-08-04 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=36037

   Summary: Allow query strings and anchors when returning forwards
from Action
   Product: Struts
   Version: 1.3.0
  Platform: Other
OS/Version: other
Status: NEW
  Severity: normal
  Priority: P3
 Component: Unknown
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


This was prompted by a recent question on the User's list, although I have 
seen similar questions come up many times...

At the end of a typical Action, a developer does:

return mapping.findForward("someForward");

This will, typically, forward to some JSP.  It is sometimes desireable to pass 
parameters to that JSP, or to the next Action if the developer chooses to 
forward to a mapping.  It is also sometimes desireable to be able to send an 
anchor along so that the browser will jump to the named anchor on the returned 
page.

Both of these can be accomplished by "cloning" the returned forward and 
manually setting the path appropriately.  However, I believe this is more work 
than it really needs to be.  Even though it's not a big deal to do, we've 
probably all done it at some point, it does require a developer to know the 
internals of Struts a little more than they maybe should (i.e., they have to 
understand that a ForwardConfig is frozen and has to be cloned to alter its 
fields), and I believe this modification removes that neccessity.

The attached patch enables this capability in a more developer-friendly way, 
namely that they can simply append the query string and/or anchor to the 
forward name requested, like so:

return mapping.findForward("someMapping?myParam=myVal#myAnchor");

In most cases this will probably be constructed dynamically in the Action, but 
the result is the same.

The patch affects four classes: RequestProcessor, PerformForward, 
ActionMapping and ActionForward.  ActionForward simply has two new fields and 
associated bean methods, "queryString" and "anchor".  RequestProcessor and 
PerformForward have identical new code, namely two lines that simply append 
the query string and anchor to the URI forwarded or redirected to.  
ActionMapping is where the more involved (relatively-speaking) changes are, 
but then only in the findForward() method (as well as one new import).

Attached is also a dirt-simple sample app to demonstrate this working.  Should 
just be able to drop it in any container and give it a wirl.

It is my hope that enough people find this useful (and find no problems with 
it of course!) to warrant being committed for the 1.3 release.  I think more 
than a few people will thank us for it :)

-- 
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: r230389 - in /struts/sandbox/trunk/ti/src/java/org/apache/ti: config/ config/mapper/ processor/ processor/chain/ servlet/ util/

2005-08-04 Thread martinc
Author: martinc
Date: Thu Aug  4 20:39:20 2005
New Revision: 230389

URL: http://svn.apache.org/viewcvs?rev=230389&view=rev
Log:
Fix up copyright headers.

Modified:
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapper.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ActionMapping.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ProcessorException.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ChainRequestProcessor.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionMapping.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionProxy.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ExecuteAction.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitXWork.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ProcessActionChain.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/SpringConfigRuleSet.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/servlet/StrutsTiServlet.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/util/ServletSourceResolver.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/util/SourceResolver.java

Modified: struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java?rev=230389&r1=230388&r2=230389&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java Thu 
Aug  4 20:39:20 2005
@@ -1,20 +1,20 @@
-/*
- * Copyright 2004-2005 The Apache Software Foundation.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- * $Header:$
- */
+/*
+ * $Id$
+ *
+ * Copyright 2005 The Apache Software Foundation.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package org.apache.ti.config;
 
 import java.io.File;

Modified: 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java?rev=230389&r1=230388&r2=230389&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java 
Thu Aug  4 20:39:20 2005
@@ -1,20 +1,20 @@
-/*
- * Copyright 2004-2005 The Apache Software Foundation.
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- *
- * $Header:$
- */
+/*
+ * $Id$
+ *
+ * Copyright 2005 The Apache Software Foundation.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you ma

svn commit: r230388 - in /struts/sandbox/trunk/ti/src: java/org/apache/ti/config/ java/org/apache/ti/config/mapper/ java/org/apache/ti/processor/ java/org/apache/ti/processor/chain/ java/org/apache/ti

2005-08-04 Thread martinc
Author: martinc
Date: Thu Aug  4 20:21:15 2005
New Revision: 230388

URL: http://svn.apache.org/viewcvs?rev=230388&view=rev
Log:
Clean up imports.

Modified:
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/config/mapper/ServletActionMapper.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/BaseResult.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerAction.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerActionInvocation.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/ProcessorException.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/RequestProcessor.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/SpringActionProxyFactory.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/SpringObjectFactory.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ChainRequestProcessor.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionMapping.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/CreateActionProxy.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ExecuteAction.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitControllerContext.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/InitXWork.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/ProcessActionChain.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/processor/chain/SpringConfigRuleSet.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/servlet/ServletDispatcherResult.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/servlet/StrutsTiServlet.java

struts/sandbox/trunk/ti/src/java/org/apache/ti/util/ServletSourceResolver.java
struts/sandbox/trunk/ti/src/java/org/apache/ti/util/SourceResolver.java
struts/sandbox/trunk/ti/src/test/org/apache/ti/config/TestProcessTags.java
struts/sandbox/trunk/ti/src/test/org/apache/ti/config/TestXDocletParser.java

struts/sandbox/trunk/ti/src/test/org/apache/ti/config/mapper/TestServletActionMapper.java
struts/sandbox/trunk/ti/src/test/org/apache/ti/processor/chain/Bean.java

struts/sandbox/trunk/ti/src/test/org/apache/ti/processor/chain/TestSpringConfigRuleSet.java

Modified: struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java?rev=230388&r1=230387&r2=230388&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTags.java Thu 
Aug  4 20:21:15 2005
@@ -17,11 +17,16 @@
  */
 package org.apache.ti.config;
 
-import org.apache.commons.logging.*;
-import java.util.*;
-import java.io.*;
-import xjavadoc.*;
-import xjavadoc.filesystem.*;
+import java.io.File;
+import java.io.FileReader;
+import java.io.FileWriter;
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.Iterator;
+import java.util.List;
+
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 /**
  *  Crawls a directory, processing all found Java source files.

Modified: 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java?rev=230388&r1=230387&r2=230388&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/apache/ti/config/ProcessTagsTask.java 
Thu Aug  4 20:21:15 2005
@@ -17,10 +17,11 @@
  */
 package org.apache.ti.config;
 
-import org.apache.commons.logging.*;
-import org.apache.tools.ant.*;
-import java.util.*;
-import java.io.*;
+import java.io.File;
+import java.io.IOException;
+
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
 
 /**
  *  Ant task that wraps ProcessTags

Modified: 
struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java
URL: 
http://svn.apache.org/viewcvs/struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java?rev=230388&r1=230387&r2=230388&view=diff
==
--- struts/sandbox/trunk/ti/src/java/org/apache/ti/config/XDocletParser.java 
(original)
+++ struts/sandbox/trunk/ti/src/java/org/

DO NOT REPLY [Bug 36024] - Unsure struts issue

2005-08-04 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=36024





--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 04:45 ---
*** Bug 36034 has been marked as a duplicate of this bug. ***

-- 
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 36034] - Unsure struts issue

2005-08-04 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=36034


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 04:45 ---


*** This bug has been marked as a duplicate of 36024 ***

-- 
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 36024] - Unsure struts issue

2005-08-04 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=36024


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|RESOLVED|CLOSED




--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 04:01 ---
(In reply to comment #1)
> Please ask questions on the User list before filing a bug report. Your first
> example is illegal JSP syntax, and is not related to Struts. The value of a 
tag
> attribute must be either a string literal or an expression; you cannot mix 
the
> two, as you are trying to do.


Thanks a lot.


-- 
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 36034] New: - Unsure struts issue

2005-08-04 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=36034

   Summary: Unsure struts issue
   Product: Struts
   Version: 1.1 Final
  Platform: PC
OS/Version: Windows XP
Status: UNCONFIRMED
  Severity: normal
  Priority: P2
 Component: Custom Tags
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


Hi,

We have an issue when we use the struts tag [select] and nested jsp expression.

The jsp source code looks like the following
Pay attention to the jsp expression
*



*
The issue is that the html source result from server side looks like
*

...
...

*
Yes. The jsp expression is not translated to its value.

But when we replace the onchange method as 
onchange='<%="jsMethod('" + jspExpress + "')"=>'
It can be translated.

We are not sure whether the select tag format is correct.
Can you help us or give us some suggestion?

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]



svn commit: r230366 - /struts/shale/trunk/build.xml

2005-08-04 Thread wsmoak
Author: wsmoak
Date: Thu Aug  4 17:36:06 2005
New Revision: 230366

URL: http://svn.apache.org/viewcvs?rev=230366&view=rev
Log:
Reorganize directory structures for the 'release' target

Modified:
struts/shale/trunk/build.xml

Modified: struts/shale/trunk/build.xml
URL: 
http://svn.apache.org/viewcvs/struts/shale/trunk/build.xml?rev=230366&r1=230365&r2=230366&view=diff
==
--- struts/shale/trunk/build.xml (original)
+++ struts/shale/trunk/build.xml Thu Aug  4 17:36:06 2005
@@ -281,7 +281,7 @@
   
 
 
-  
 
 
@@ -488,75 +488,78 @@
 
   
 
+  
+  
 
-  
+
+
+  
+
+  
 
 
 
 
-
-
-
-
+
+
+
+
 
 
-
-
+
+
   
 
-
+
   
 
 
 
-
-
+
+
   
 
-
+
   
 
 
 
-
-
+
+
   
 
-
+
   
 
 
 
-
-
+
+
   
 
-
+
   
 
 
 
-
-
-
-
-
-
+ longfile="gnu">
+  
+
+
+  
+
   
 
 



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



DO NOT REPLY [Bug 36028] - [Standalone Tiles] Refactor DefinitionsFactory and Create Reloadable Definitions

2005-08-04 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=36028





--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 02:19 ---
I've preserved compatibility from a user standpoint.  Only those who've
customized DefinitionsFactory or other interfaces will be affected by my 
changes.

-- 
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 36028] - [Standalone Tiles] Refactor DefinitionsFactory and Create Reloadable Definitions

2005-08-04 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=36028





--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 01:06 ---
Are you referring to backwards compatability for users or backwards 
compatability for people who have customized Tiles by extending or modifying 
classes? I would imagine compatability for users is sufficient rather than for 
people that have customized tiles itself. 

-- 
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 36028] - [Standalone Tiles] Refactor DefinitionsFactory and Create Reloadable Definitions

2005-08-04 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=36028





--- Additional Comments From [EMAIL PROTECTED]  2005-08-05 00:08 ---
We definitely need to maintain backwards compatibility, as this version of Tiles
will one day (hopefully pretty soon) be retrofitted to Struts.

-- 
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 36028] - [Standalone Tiles] Refactor DefinitionsFactory and Create Reloadable Definitions

2005-08-04 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=36028





--- Additional Comments From [EMAIL PROTECTED]  2005-08-04 18:07 ---
Created an attachment (id=15892)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=15892&action=view)
Standalone Tiles Patch

This patch should be considered a first attempt.  I submit it to ask for
feedback on how to improve on it.  I like the basic structure of the
refactoring but I don't think the synchronization is very good.  Also I didn't
know how to handle backwards compatibility.  Since this is a new project do we
need to deprecate old things or can we just start from scratch.  If better
backwards compatibility is needed I'm happy to work that in.  If not, then
there's probably more refactoring to do to get rid of some orphaned objects.

-- 
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 36028] New: - [Standalone Tiles] Refactor DefinitionsFactory and Create Reloadable Definitions

2005-08-04 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=36028

   Summary: [Standalone Tiles] Refactor DefinitionsFactory and
Create Reloadable Definitions
   Product: Struts
   Version: 1.3.0
  Platform: Other
OS/Version: other
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: Sandbox
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


This is an enhancement request to refactor the Tiles DefinitionsFactory 
structure and provide support 
for reloadable Tiles definitions.  It is applicable to the standalone Tiles 
project.  The reason for the 
refactoring is as follows:  The current structure for DefinitionsFactory is 
overly complicated and the 
desire is to simplify it and separate concerns.  This refactoring externalizes 
the reading of data from 
config files and the structure of definitions objects from the ServletContext 
management.  It also 
provides a servlet Filter to allow Tiles Definitions to be reloaded.  Patch to 
be submitted shortly.

-- 
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: [Proposal] Struts Ti

2005-08-04 Thread Don Brown

Dakota Jack wrote:


How are you going to allow IoC dependency injection and build on the
backbone of commons-chain?  Those seem to be inherently incompatible. 
Isn't the chain necessary only where you use a Template-Method Pattern

rather than a Strategy Pattern, as the recommended relationship
between the view and the controller by GoF, and accordingly have IoC
problems?


In Struts Ti, they are used for two entirely different purposes: 
commons-chain is used when it needs to define a business process in a 
flexible, extendable manner (do this, then do this, then this, etc), and 
Spring when I want to abstract the creation of objects from the code 
(provide a Servlet-specific implementation without the target code 
knowing or caring).  Commons-chain is important as it lets the developer 
inject code into Ti processes or handle a decision differently.  Spring 
lets me load an entirely different object set based on environment 
(servlet, portlet, etc) so I can keep the core clean of platform 
dependencies.


In Struts 1.3, we use one big chain commons-chain (two if you include 
the exception handling) to handle the request, but in Ti, I'm looking 
into using smaller, single-purposed chains to abstract key decisions 
like Action creation to allow Ti to ship with support for WebWork 
actions, Page Flow controllers, or classic Struts Actions, yet the 
developer can easily add chain commands to add support for their 
preferred Action implementation.


Don





At this point, the project site, code, and more detailed design
discussions are on my personal server:

https://www.twdata.org/projects/struts-ti




This looks promising.  I will be following with a lot of interest.  Good show!




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



Re: [Proposal] Struts Ti

2005-08-04 Thread Dakota Jack
This all sounds good, Don, and I have been an admirer of your "horse
sense" for some time.  I especially like the emphasis on KISS.  I have
a few comments infra.


On 8/2/05, Don Brown <[EMAIL PROTECTED]> wrote:
> I'd been waiting to announce/propose this until I could write up a
> decent proposal and have the code in a better state, but with all the
> talk about Struts 2.0, "good" now is better than "better" tomorrow I
> suppose.  The following is a proposal for Struts Ti, a possible
> successor to Struts classic:

I like the name Struts-Titanium too.  That has the right feel for what
would be a good framework's attributes: light and strong.

> 
> Struts Ti is a simplified Model 2 framework for developing webapps which
> allows the developer better access to the underlying servlet/portlet
> environment. ..
> The key word for Struts Ti is simplicity. Ideally, Struts Ti should
> approach Ruby on Rails levels of easy of use, yet scale up to large
> applications providing a smooth transition to JSF/Shale if desired.

GREAT!  

>  * POJO-based action that combines an Action and ActionForm in a
> similar manner to JSF backing beans and WebWork 2 Commands

Allowing multiple ActionForms per Action would be a blessing,  This
would allow you, where you have the following flow ActionA --> FormA
--> ActionB --> FormB --> ActionC and allow ActionB to share FormA
with ActionA and FormB with ActionC.

>  * Spring-based dependency injection in core to allow for pluggability

GREAT!

>  * Built on the backbone of commons-chain

How are you going to allow IoC dependency injection and build on the
backbone of commons-chain?  Those seem to be inherently incompatible. 
Isn't the chain necessary only where you use a Template-Method Pattern
rather than a Strategy Pattern, as the recommended relationship
between the view and the controller by GoF, and accordingly have IoC
problems?

> At this point, the project site, code, and more detailed design
> discussions are on my personal server:
> 
> https://www.twdata.org/projects/struts-ti
> 

This looks promising.  I will be following with a lot of interest.  Good show!

-- 
"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 36024] - Unsure struts issue

2005-08-04 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=36024


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution||INVALID




--- Additional Comments From [EMAIL PROTECTED]  2005-08-04 15:37 ---
Please ask questions on the User list before filing a bug report. Your first
example is illegal JSP syntax, and is not related to Struts. The value of a tag
attribute must be either a string literal or an expression; you cannot mix the
two, as you are trying to do.

-- 
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: [Proposal] Struts Ti

2005-08-04 Thread Ted Husted
On 8/3/05, Frank W. Zammetti <[EMAIL PROTECTED]> wrote:
> With that thought in mind, I personally see the next evolution almost
> really being two frameworks that work in tandem... and maybe even it is
> *literally* that... one the usual server-side framework, but now with a
> client-side component that is a fair ways beyond anything we have today.
>   They have to be intimately tied together to really make it worth it,
> but I see them both being fairly complex endeavors.

I would go so far as to suggest three layers, the client-side (Ajax),
the server-side (Struts), and the business-side (XWork or Nexus).

In practice, many web application frameworks end up doing double duty.
Since there are few business logic frameworks, developers tend to put
features into presentation frameworks that don't need to be there. We
just don't have another place to put them :)

I've been working on a view-agnostic business-logic framework in the
sandbox, based on Commons CoR, but, alas, all the code is still in C#.

* http://opensource.atlassian.com/confluence/oss/display/OVR/WhitePaper


> We're seeing a lot of experimentation in that direction... ASP.Net, JSF,
> all the various Java frameworks out there, more open-source projects
> than you can count... but I'm not sure any of them are quite "right"
> yet.  

And, unlike ASP.NET and JSF, the open source frameworks represent the
technologies that front-line developers actually *want* to use.
Otherwise, we would not have built them :)

-Ted.

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



Re: [Proposal] Struts Ti

2005-08-04 Thread Ted Husted
On 8/2/05, netsql <[EMAIL PROTECTED]> wrote:
> Michael, in esence, one could just cut and paste the code from
> "org.struts.api" to C#, fixing syntax as needed.

Struts Classic would be hard port, because of all the HTTP references.
There are .NET equivalents, but the syntax is very diffeent, and the
paradigm would not be so useful for ASP.NET applications (ASP classic,
maybe).

Commons Chain of Responsiblity, on the other hand, was a very easy
port. CoR is a very nifty way to call business logic from the
code-behinds, especially when you throw Spring.Web into the mix.

> 
> Then in esence people that use it in one or the other have the same
> signature. Some users may never know how it's implemented. The issue is
> more interesting on iBatis, where they have "smilar" usage, but not same.

Once you get past the config, they are very much alike. Mainly, there
are feature-set differences where one implementation is running ahead
or behind the other. The expected feature-set will probably be
resolved in the 3.x timeframe. My own goal is to be able to reuse the
iBATIS mapping files, as is, between implementations.

Some Struts-like frameworks with multiple implementations, like
FuseBox, can also share configuration files between implementations.

-T.

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



Re: 1.3.0 Release - Next Steps

2005-08-04 Thread Ted Husted
On 8/1/05, Ted Husted <[EMAIL PROTECTED]> wrote:
> Thanks Hubert. I'm still getting my Java development environment dusted off :)

OK, making progress. I've got the latest Java's installed (1.3.1_15,
1.4.2_07,  1.5.0_04) and updated Injellij to version 5 (thanks,
JetBrains!).

Now, onto to the code :) 

For 1.3.0, I think we should concentrate on bug-fixes and
infrastructure. There have been a lot of internal changes, and job one
should be to get a clean build. Problem reports  with working patches
will be applied. If a problem report doesn't have a patch or obvious
fix, then we may have to leave it open, until someone comes up with a
fix.

For 1.3.1, we could move the focus to enhancements, again apply those
with working patches. If people *test* the patch for an enhancement,
and vote for it, I would give the patch higher priority.

It is unlikely that an enhancement request without a patch will be
implemented, and we might consider moving such requests to a "wish
list" or "job jar" wiki page.

-T.

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



DO NOT REPLY [Bug 36024] New: - Unsure struts issue

2005-08-04 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=36024

   Summary: Unsure struts issue
   Product: Struts
   Version: 1.1 Final
  Platform: PC
OS/Version: Windows XP
Status: UNCONFIRMED
  Severity: normal
  Priority: P2
 Component: Custom Tags
AssignedTo: dev@struts.apache.org
ReportedBy: [EMAIL PROTECTED]


Hi,

We have an issue when we use the struts tag [select] and nested jsp expression.

The jsp source code looks like the following
Pay attention to the jsp expression
*



*
The issue is that the html source result from server side looks like
*

...
...

*
Yes. The jsp expression is not translated to its value.

But when we replace the onchange method as 
onchange='<%="jsMethod('" + jspExpress + "')"=>'
It can be translated.

We are not sure whether the select tag format is correct.
Can you help us or give us some suggestion?

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]



[Struts Wiki] Update of "StrutsTools" by YannCebron

2005-08-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 YannCebron:
http://wiki.apache.org/struts/StrutsTools

--
  
  http://www.common-controls.com/en/index.php - The Common-Controls form a 
Java™ Presentation Framework based on Java Servlets, Java Serverpages (JSP™) 
and  Struts. The Presentation Framework contains the most common control 
elements like lists, trees, tabfolders, menus and forms, which are required for 
the development of J2EE™ applications with HTML frontends.
  
+ http://www.intellij.org/twiki/bin/view/Main/StrutsPluginDevelopment - Struts 
Plugin for IntelliJ IDEA 5 provides inspections to check 
struts-config.xml-files and Struts specific code on-the-fly (also available via 
the integrated plugin manager).
+ 

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