ti/phase2/jars/core/src/java/org/apache/ti/pageflow/PageFlowRequestProcessor.java (12 lines):
	- line 119: /* TODO: re-add this -- it should be in Chain
	- line 125: // TODO: performance?
	- line 135: // TODO: re-add multipart support
	- line 238: /* TODO: re-add
	- line 329: /* TODO: re-add
	- line 352: /* TODO: re-add multipart support
	- line 428: /* TODO: re-add tokens, or something like it.
	- line 466: // TODO: re-add overloaded action support
	- line 471: // TODO: re-add class caching?
	- line 475: /* TODO: re-add overloaded action support
	- line 535: /* TODO: re-add Tiles support
	- line 555: /* TODO: re-add this.  It's code to customize XWork -- to handle shared flow and form-bean-specific actions.


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/xwork/PageFlowAction.java (10 lines):
	- line 226: // TODO: re-add this, in Chain
	- line 232: // TODO: re-add this
	- line 250: // TODO: re-add Locale-choosing, in Chain
	- line 254: // TODO: re-add roles, in Chain
	- line 259: // TODO: re-add login-required support, in Chain
	- line 261: // TODO: re-add action form creation (store in context), population and validation (in Chain)
	- line 361: // TODO: re-add Controls support
	- line 375: // TODO: re-add Controls support
	- line 443: // TODO: re-add overloaded action support
	- line 449: // TODO: re-add class caching?


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/FlowControllerGenerator.java (7 lines):
	- line 38: private static final boolean ALWAYS_GENERATE = true; // TODO: this turns stale checking off.  Do we need it?
	- line 70: e.printStackTrace(); // @TODO log
	- line 90: e.printStackTrace(); // TODO: log instead
	- line 129: // @TODO logger.info( "Writing Struts module: " + _strutsConfig.getStrutsConfigFile() );
	- line 132: // @TODO logger.info( _compilerJarName + " has been updated; writing Struts module "
	- line 136: // @TODO logger.info( "Struts module " + _strutsConfig.getStrutsConfigFile() + " is up-to-date." );
	- line 141: e.printStackTrace(); // @TODO get rid of this


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/InternalUtils.java (6 lines):
	- line 71: // TODO: re-add this
	- line 95: // TODO re-add
	- line 98: throw new UnsupportedOperationException("NYI -- HTML is: \n" + html); // TODO: NYI
	- line 164: /* TODO: re-add error/message support
	- line 563: /* TODO: re-add for page template support
	- line 622: /* TODO: re-add messages support


apps-test/sysclient-app/src/java/org/apache/struts/faces/sysclient/ContextTestCase.java (6 lines):
	- line 118: assertEquals("/struts-faces-systest1", span.asText()); // FIXME - Ant property?
	- line 122: String formRequestLocaleEC = span.asText(); // FIXME - validate this
	- line 136: assertEquals("/struts-faces-systest1", span.asText()); // FIXME - Ant property?
	- line 189: assertEquals("/struts-faces-systest1", span.asText()); // FIXME - Ant property?
	- line 192: String formRequestLocaleEC = span.asText(); // FIXME - validate this
	- line 206: assertEquals("/struts-faces-systest1", span.asText()); // FIXME - Ant property?


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/PageFlowManagedObject.java (5 lines):
	- line 34: implements Serializable /* TODO: re-enable through an abstraction: , HttpSessionBindingListener */ {
	- line 60: // TODO: re-add Controls support
	- line 71: // TODO: re-add Controls support
	- line 93: /* TODO: re-enable this through an abstraction
	- line 102: /* TODO: re-enable this through an abstraction


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/xwork/PageFlowResult.java (4 lines):
	- line 314: // TODO: enforce an annotation attribute that allows this; otherwise, throw.
	- line 495: // TODO: re-add an alternative way of displaying pages (rather than server forward)
	- line 574: // TODO: ACTION_EXTENSION can no longer be a constant -- we support arbitrary Servlet mappings
	- line 586: // TODO: re-add the previous functionality


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/PageFlowUtils.java (4 lines):
	- line 593: /* TODO: re-add some form of this, for portal/portlet support
	- line 695: // TODO: need ActionMapper to be reversible -- it should construct the URI.
	- line 716: // TODO: re-add the following line, using some abstraction
	- line 806: // TODO: re-add the following line, using some abstraction


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/ModuleConfig.java (4 lines):
	- line 64: // TODO: parse shared flow references
	- line 65: // TODO: parse multipart params
	- line 137: // TODO: we could also get this with:
	- line 144: throw new UnsupportedOperationException("NYI"); // TODO: NYI


flow/src/java/org/apache/struts/flow/core/javascript/fom/FOM_JavaScriptInterpreter.java (4 lines):
	- line 401: // FIXME: This might create a unwanted session
	- line 428: // FIXME: Where "session scope" should go when session is invalidated?
	- line 806: //FIXME: add a "tee" debugger that allows both to be used simultaneously
	- line 914: //FIXME: add a "tee" debugger that allows both to be used simultaneously


flow/src/java/org/apache/struts/flow/core/javascript/fom/template.js (4 lines):
	- line 35: // TODO: Debugging mode vs stop-on-error mode - runtime flag.
	- line 36: // TODO: Handle || (or) characters and backslashes.
	- line 37: // TODO: Add more modifiers.
	- line 141: var state    = { stack: [], line: 1 };                       // TODO: Fix line number counting.


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/genmodel/GenValidationModel.java (4 lines):
	- line 181: assert propertyName != null; // TODO: checker must enforce this
	- line 182: assert !propertyName.equals(""); // TODO: checker must enforce this
	- line 207: assert !propName.equals(""); // TODO: get checker to enforce this
	- line 305: assert rule.getMessageKey() == null; // TODO: checker should enforce


ti/phase2/jars/core/src/java/org/apache/ti/processor/ControllerActionInvocation.java (4 lines):
	- line 49: // TODO: DefaultActionInvocation should make the context-creation (currently in private init()) overridable.
	- line 70: // TODO: this should be optimized
	- line 122: // TODO: this should be optimized
	- line 161: // TODO: have to find out why this is necessary; shouldn't it be part of the base createAction()?


struts2-jst-plugin/src/main/resources/trimpath-template-1.0.38.js (4 lines):
	- line 35: // TODO: Debugging mode vs stop-on-error mode - runtime flag.
	- line 36: // TODO: Handle || (or) characters and backslashes.
	- line 37: // TODO: Add more modifiers.
	- line 167: var state    = { stack: [], line: 1 };                              // TODO: Fix line number counting.


mailreader-chain/src/java/org/apache/commons/chain/mailreader/struts/MailReaderAction.java (3 lines):
	- line 60: input = new HashMap(); // FIXME: Lame resolution
	- line 173: // TODO: Expose any output
	- line 174: // TODO: Expose any status messages,


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/httpservlet/internal/ServletPageFlowResult.java (3 lines):
	- line 40: /* TODO: re-add this
	- line 103: /* TODO: re-add
	- line 178: /* TODO: re-add this


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/DefaultContainerAdapter.java (3 lines):
	- line 76: // TODO: implement this based on parsing of web.xml
	- line 96: // TODO: have a configuration in struts-ti-config.xml to specify this; an alternative to having to have an adapter.
	- line 107: // TODO: have a configuration in struts-ti-config.xml to specify this; an alternative to having to have an adapter.


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/model/XWorkModuleConfigModel.java (3 lines):
	- line 92: /* TODO: re-add message bundle support
	- line 136: defaultMappingForThisPath.setComment(DUPLICATE_ACTION_COMMENT.replaceAll("\\{0\\}", mappingPath)); // @TODO I18N
	- line 199: /* TODO: re-add message bundle support


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/xwork/PageFlowActionContext.java (3 lines):
	- line 66: // TODO: currently these are initialized externally.  Instead we could have specific implementations of this class.
	- line 283: // TODO: implement this differently for portal/portlet support
	- line 292: // TODO: implement this differently for portal/portlet support


apps-test/sysclient-app/src/java/org/apache/struts/faces/sysclient/SimpleTestCase.java (2 lines):
	- line 126: // TODO: verify the "lang" attribute
	- line 127: // TODO: verify the "xml:lang" attribute


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/xmlhttprequest/XmlHttpRequestServlet.java (2 lines):
	- line 29: /* TODO: re-add backend XMLHttpRequest support
	- line 37: // TODO: does ErrorCRT really need to be an interceptor?


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/genmodel/GenXWorkModuleConfigModel.java (2 lines):
	- line 100: // TODO: comment
	- line 270: /* TODO: re-add message bundle support


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/PageFlowController.java (2 lines):
	- line 665: // TODO: re-evaluate whether path and namespace can be linked this way
	- line 683: /* TODO: re-enable this, through an abstraction


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/DefaultReloadableClassHandler.java (2 lines):
	- line 42: /* TODO: re-add this in the future
	- line 48: // TODO: make this configurable in struts-ti-config.xml.  You should be able to specify absolute files


struts2-jquery-plugin/src/main/resources/template/jquery-simple/form-close-validate.ftl (2 lines):
	- line 25: <#--TODO add tests for funky datatypes like Date obj returned from the datepicker
	- line 26: TODO consider ids that contain a period... valid for struts, invalid for jquery


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/genmodel/DefaultValidatorRuleFactory.java (2 lines):
	- line 96: assert value instanceof PrimitiveType : value.getClass().getName();  // TODO: checker enforces this
	- line 98: assert typeName != null : ((PrimitiveType) value).getKind().toString();  // TODO: checker enforces this


struts2-gxp-template-plugin/src/main/java/org/apache/struts2/gxp/template/GxpcUtil.java (2 lines):
	- line 84: // TODO Auto-generated catch block
	- line 87: // TODO Auto-generated catch block


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/xwork/NavigateToPageResult.java (2 lines):
	- line 96: // TODO: re-add logic for merging query strings (abstracted into a Servlet-specific version?)
	- line 280: return (_previousPageIndex > 0) ? "ti.NavigateTo.previousPage" : "ti.NavigateTo.currentPage"; // TODO: constant


flow/src/java/org/apache/struts/flow/core/ContinuationsManagerImpl.java (2 lines):
	- line 326: //TODO this looks like a security breach - should we throw?
	- line 490: // TODO: this avoids ConcurrentModificationException, still this is not


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/CompilerUtils.java (2 lines):
	- line 734: // TODO: This entire method is a workaround for a bug in APT where an annotation may not equal itelf.
	- line 1133: // TODO: there appears to be a bug in APT where both the key/value are contained in the key


ti/phase2/jars/core/src/java/org/apache/ti/script/common/ImplicitObjectUtil.java (2 lines):
	- line 89: /* TODO: re-add bundle support
	- line 129: // TODO: re-add bundle support


struts2-jquery-plugin/src/main/resources/template/jquery/form-close.ftl (2 lines):
	- line 22: TODO Need to check on elements like inputtransferselect and doubleselect
	- line 26: TODO check the logic in template/simple/form-close.ftl to see if we need to include it as well


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/genmodel/GenXWorkExceptionHandlerModel.java (2 lines):
	- line 50: // TODO I18N
	- line 99: // TODO: when we no longer support Struts 1.1, we can simply use the 'module' property


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/FlowController.java (2 lines):
	- line 531: /* TODO: re-enable this method
	- line 688: // TODO: re-add (need an abstraction for this)


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/genmodel/GenXWorkResultModel.java (2 lines):
	- line 116: // TODO: when we no longer support Struts 1.1, we can simply use the 'module' property
	- line 129: setComment("forward \"" + getName() + '"' + commentSuffix);  // @TODO I18N the comment


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/interceptor/action/InterceptorForward.java (2 lines):
	- line 54: /* TODO: re-add this
	- line 78: // TODO: re-add the ability to set redirect programmatically on a Forward; also setExternalRedirect


struts2-jquery-plugin/src/main/resources/template/jquery/form-close-validate.ftl (2 lines):
	- line 25: <#--TODO add tests for funky datatypes like Date obj returned from the datepicker
	- line 26: TODO consider ids that contain a period... valid for struts, invalid for jquery


apps-test/sysclient-app/src/java/org/apache/struts/faces/sysclient/LogonTestCase.java (2 lines):
	- line 133: // FIXME:  response string should really include "Username:"
	- line 143: // FIXME:  response string should really include "Password:"


artimus/WEB-INF/src/java/org/apache/artimus/article/Store.java (1 line):
	- line 127: // :FIXME: Should provide a batch insert method


ti/phase2/jars/core/src/java/org/apache/ti/processor/chain/pageflow/ChooseFormBean.java (1 line):
	- line 113: // TODO: make an interface for reset(), and call reset.


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/PageFlowChecker.java (1 line):
	- line 128: // TODO - rich - We're inferring a begin action if there isn't one.  Not sure how I feel about this yet. :)


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/JavaControlUtils.java (1 line):
	- line 28: /* TODO: re-add Controls support (as a Handler)


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/FacesBackingBean.java (1 line):
	- line 111: // TODO: should we add a compiler warning if this field isn't transient?  All this reinitialization logic is


ti/phase2/jars/core/src/java/org/apache/ti/util/internal/ServletUtils.java (1 line):
	- line 111: // TODO: make this class (or part of it) into a spring bean (SessionUtils or something like that)


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/model/AbstractResultContainer.java (1 line):
	- line 49: // TODO: logging


flow/src/java/org/apache/struts/flow/core/javascript/fom/FOM_Flow.java (1 line):
	- line 205: // FIXME: This method can return invalid continuation! Is it OK to do so?


artimus/WEB-INF/src/java/org/apache/artimus/wizard/http/Form.java (1 line):
	- line 237: int page = 0; // :FIXME:


mailreader-chain/src/webapp/WEB-INF/webtest.xml (1 line):
	- line 68: <!-- TODO: -->


ti/phase2/jars/core/src/java/org/apache/ti/script/el/tokens/MapKeyToken.java (1 line):
	- line 78: // @TODO: unicode escaping...


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/FacesBackingGenerator.java (1 line):
	- line 60: e.printStackTrace(); // TODO: log instead


artimus/WEB-INF/src/java/org/apache/artimus/article/Access.java (1 line):
	- line 80: // :FIXME: Should be a toplevel model method for this


ti/phase2/jars/core/src/java/org/apache/ti/processor/chain/pageflow/InitPageFlow.java (1 line):
	- line 78: // TODO: turn this whole thing into a chain


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/xwork/NavigateToActionResult.java (1 line):
	- line 88: return "ti.NavigateTo.previousAction";  // TODO: constant


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/SimpleSessionStorageHandler.java (1 line):
	- line 62: // TODO: create a Servlet version of this class that returns the actual session object


flow/src/java/org/apache/struts/flow/sugar/ScriptableList.java (1 line):
	- line 92: //TODO: optimize this :)


ti/phase2/jars/core/src/java/org/apache/ti/processor/chain/pageflow/PopulateData.java (1 line):
	- line 51: /* TODO: re-add multipart support


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/xwork/PageFlowPathResult.java (1 line):
	- line 50: /* TODO: re-add this sort of support for self-nesting a page flow


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/httpservlet/internal/PopulatePageFlowContext.java (1 line):
	- line 46: // TODO: re-add the ignoreIncludeServletPath() check, for page template support


ti/phase2/jars/core/src/java/org/apache/ti/script/el/ExpressionTerm.java (1 line):
	- line 118: // @TODO: error checking


src/main/java/org/apache/struts2/fileupload/DiskFileItemFactoryWrapper.java (1 line):
	- line 37: * TODO: Someone mentioned that google app engine won't let you


struts2-jquery-plugin/src/main/resources/template/jquery-simple/form-close.ftl (1 line):
	- line 22: TODO Need to check on elements like inputtransferselect and doubleselect


struts2-fileupload-plugin/src/main/java/org/apache/struts2/fileupload/DiskFileItemFactoryWrapper.java (1 line):
	- line 37: * TODO: Someone mentioned that google app engine won't let you


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/PageFlowConstants.java (1 line):
	- line 41: public static final String ACTION_EXTENSION = ".do";   // TODO: will need to read this from the Servlet mapping


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/FormData.java (1 line):
	- line 29: /* TODO: re-add the functionality in this class


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/DefaultModuleRegistrationHandler.java (1 line):
	- line 234: // TODO: This is a hack.  We write general module metadata to a special action.  We simply need params


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/grammar/SimpleActionGrammar.java (1 line):
	- line 156: // TODO: comment


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/model/XmlElementSupport.java (1 line):
	- line 192: public static String getTextContent(Element element) // TODO: move to a utils class, so XmlModelWriter is independentf


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/ProcessPopulate.java (1 line):
	- line 157: // TODO: re-add multipart request support


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/handler/Handlers.java (1 line):
	- line 188: // TODO: add a way to set custom props on HandlerConfig


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/ExpressionAwareMessageResources.java (1 line):
	- line 30: /* TODO: re-add expression-based message support


s2-jquery-showcase/src/main/webapp/WEB-INF/content/simple-theme/FormWithResetPostReqAjaxResp-input.jsp (1 line):
	- line 47: <%-- TODO what is the point of a reset button in an AJAX form if you don't use onreset= ? --%>


struts2-fileupload-plugin/src/main/java/org/apache/struts2/fileupload/UploadStatusHolder.java (1 line):
	- line 117: // TODO - implement proper logging


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/AdapterManager.java (1 line):
	- line 52: // TODO: this method could move to a more general place.


ti/phase2/jars/compiler-xdoclet/src/java/org/apache/ti/compiler/xdoclet/PageFlowDocletSubTask.java (1 line):
	- line 50: AnnotationTypeDeclaration[] decls = DeclarationImpl.getAllAnnotations();    // TODO: filter appropriately


struts2-jquery-plugin/src/main/java/org/apache/struts2/jquery/views/jsp/ui/JQueryFormTag.java (1 line):
	- line 20: // TODO the ajaxResult is making its way to the HTML, need to fix that


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/PageFlowBeanContext.java (1 line):
	- line 30: /* TODO: re-add this, under a controls-support module


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/ActionResult.java (1 line):
	- line 24: * @TODO - no such method PageFlowUtils#strutsLookup}.


s2-jquery-showcase/src/main/webapp/WEB-INF/content/simple-theme/SimpleFormPostNonAjaxResponse-input.jsp (1 line):
	- line 41: <!-- TODO on sjx:form validate="true" will call handleForm_${formId} as a callback -->


ti/phase2/jars/core/src/java/org/apache/ti/util/MessageResources.java (1 line):
	- line 22: // TODO: temporary stub interface for message resources


flow/src/java/org/apache/struts/flow/core/CompilingInterpreter.java (1 line):
	- line 74: * TODO - This is a little bit strange, the interpreter calls


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/model/XWorkResultModel.java (1 line):
	- line 86: // TODO: comment


jpa-mailreader/src/main/java/action/user/Create.java (1 line):
	- line 40: // FIXME: Can the email validations by applied to getUser?


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/validation/ValidatorRules.java (1 line):
	- line 26: /* TODO: re-add Validator support


ti/phase1/jars/legacy/src/java/org/apache/ti/legacy/PlugInInterceptor.java (1 line):
	- line 64: // TODO: we should be able to find the package name/namespace during init time, but for now it's passed in as param.


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/interceptor/action/ActionInterceptorContext.java (1 line):
	- line 242: // TODO: move some of the lazy-load logic in getActionInterceptors into here.


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/internal/InternalExpressionUtils.java (1 line):
	- line 86: /* TODO: re-add these in a Servlet-specific implementation


flow/src/java/org/apache/struts/flow/portlet/FlowPortlet.java (1 line):
	- line 93: // FIXME: commenting out for now


ti/phase2/jars/core/src/java/org/apache/ti/compiler/internal/RuntimeVersionChecker.java (1 line):
	- line 40: * TODO: in the future, we should have a way to (optionally) find the runtime jar(s) to check the versions.


struts2-jquery-plugin/src/main/resources/template/jquery-simple/form-validate.ftl (1 line):
	- line 23: TODO rather than going with traditional onclick handler, it would be nice to move to $().live("click", ...


struts2-api/src/main/java/org/apache/struts2/Messages.java (1 line):
	- line 36: // TODO: Use Object[] for args instead of String[].


ti/phase2/jars/core/src/java/org/apache/ti/processor/chain/ChainRequestProcessor.java (1 line):
	- line 205: // :FIXME: Why is this dependent on the container's version of


ti/phase2/jars/core/src/java/org/apache/ti/pageflow/httpservlet/HttpServletPageFlowController.java (1 line):
	- line 27: throw new UnsupportedOperationException("NYI"); // TODO: NYI