Tag List Report
The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
| Tag Class | Total number of occurrences | Tag strings used by tag class |
|---|---|---|
| @todo | 0 | @todo |
| TODO | 103 | TODO |
Each tag is detailed below:
TODO
Number of occurrences found in the code: 103
| org.apache.myfaces.application.ApplicationImpl | Line |
|---|---|
| this check should be performed by the expression factory. It is a requirement of the TCK | 1985 |
| Check if modification occurs often or not, might have to use a synchronized list instead. | 2800 |
| Check if modification occurs often or not, might have to use a synchronized list instead. | 2821 |
| org.apache.myfaces.application.ApplicationImplAnnotationTest | Line |
| need mock objects for VDL/VDLFactory remove from excludes list in pom.xml after complete | 58 |
| org.apache.myfaces.application.ApplicationImplJsfTest | Line |
| Make it work again | 162 |
| org.apache.myfaces.application.ApplicationImplTest | Line |
| need mock objects for VDL/VDLFactory remove from excludes list in pom.xml after complete | 63 |
| org.apache.myfaces.application.NavigationHandlerImpl | Line |
| cache results? | 957 |
| --No comment-- | 1161 |
| org.apache.myfaces.application.ResourceHandlerImpl | Line |
| Log using a localized message (which one?) | 630 |
| org.apache.myfaces.cdi.dependent.RequestDependentBeanStorage | Line |
| merge with AbstractBeanStorage if ViewDependentBeanStorage isn't needed (see MYFACES-3805) | 23 |
| org.apache.myfaces.cdi.util.CDIExternalArtifactResolver | Line |
| add serializable check again or remove this TODO once MYFACES-3805 is clarified | 152 |
| org.apache.myfaces.component.visit.PartialVisitContext | Line |
| log warning for trailing colon case } | 310 |
| smarter initial size? | 345 |
| org.apache.myfaces.config.DefaultFacesConfigurationProvider | Line |
| Find a better way without write the DOM and read it again and without rewrite commons-digester parser!. | 517 |
| org.apache.myfaces.config.FacesConfigurator | Line |
| configure flow object | 1527 |
| org.apache.myfaces.config.annotation.AllAnnotationLifecycleProvider | Line |
| @EJBs | 33 |
| where i find WebServiceRef? | 57 |
| org.apache.myfaces.config.annotation.AnnotationConfigurator | Line |
| Implement strategy pattern over this method. | 99 |
| MartinKoci MYFACES-3053 | 186 |
| org.apache.myfaces.config.annotation.NoInjectionAnnotationLifecycleProvider | Line |
| the servlet spec is not clear about searching in superclass?? | 118 |
| the servlet spec is not clear about searching in superclass?? May be only check non private fields and methods | 162 |
| org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProvider | Line |
| @Resources | 32 |
| local or global JNDI | 183 |
| local or global JNDI | 188 |
| local or global JNDI | 218 |
| local or global JNDI | 223 |
| org.apache.myfaces.config.impl.digester.DigesterFacesConfigUnmarshallerImpl | Line |
| validation set to false during implementation of 1.2 | 45 |
| org.apache.myfaces.config.impl.digester.elements.BehaviorImpl | Line |
| what about extensions and descriptionGroup elems? Not addressed in other config objects either. | 36 |
| org.apache.myfaces.context.servlet.PartialViewContextImpl | Line |
| process partial rendering https://issues.apache.org/jira/browse/MYFACES-2118 Collection<String> renderIds = getRenderIds(); | 435 |
| fix javascript side, so the field is not removed on ajax form update | 594 |
| Do not check for a specific instance, just render all children. if (comp instanceof javax.faces.component.html.HtmlBody) { comp.encodeAll (_facesContext); } } | 653 |
| org.apache.myfaces.context.servlet.ServletExternalContextImplTest | Line |
| Invalid test, because EL evaluation should be done before call these methods. | 68 |
| org.apache.myfaces.convert.ConverterUtils | Line |
| Move to util package and rename to better name | 23 |
| org.apache.myfaces.el.unified.FacesELContext | Line |
| decide if we need to implement our own FunctionMapperImpl and VariableMapperImpl instead of relying on Tomcat's version. this.functionMapper = new FunctionMapperImpl(); this.variableMapper = new VariableMapperImpl(); | 44 |
| org.apache.myfaces.event.SetPropertyActionListener | Line |
| use a Converter before calling setValue | 79 |
| org.apache.myfaces.lifecycle.CODIClientSideWindow | Line |
| only set internally and provide via restoreWindowId()? externalContext.getRequestMap().put(WINDOW_CONTEXT_ID_PARAMETER_KEY, windowId); | 169 |
| implement me | 337 |
| eval if we should also filter the other params | 343 |
| org.apache.myfaces.lifecycle.DefaultRestoreViewSupport | Line |
| CJH: I don't think this is correct, considering that getActionURL() expects everything to start with '/', and in the suffix case we only mess with the suffix and leave leading slashes alone. Please review... | 404 |
| org.apache.myfaces.lifecycle.DefaultRestoreViewSupportTest | Line |
| not testable unless static call to RendererUtils is removed | 206 |
| org.apache.myfaces.mc.test.core.AbstractMyFacesRequestTestCase | Line |
| check if this is correct | 115 |
| org.apache.myfaces.mc.test.core.mock.MockDefaultViewDeclarationLanguage | Line |
| VALIDATE - The spec is silent on the following line, but I feel bad if I don't set it | 106 |
| VALIDATE - Spec is silent on the return value when an error was sent | 117 |
| org.apache.myfaces.mc.test.core.runner.AbstractJsfRequestTestContainer | Line |
| check if this is correct | 119 |
| org.apache.myfaces.renderkit.ErrorPageWriterTest | Line |
| remove these two lines once myfaces-test goes alpha, see MYFACES-1155 | 65 |
| org.apache.myfaces.renderkit.NullReturningGetExpressionStringValueExpression | Line |
| Auto-generated method stub | 43 |
| Auto-generated method stub | 51 |
| Auto-generated method stub | 58 |
| Auto-generated method stub | 66 |
| Auto-generated method stub | 72 |
| Auto-generated method stub | 78 |
| Auto-generated method stub | 84 |
| org.apache.myfaces.renderkit.html.HtmlAjaxBehaviorRenderer | Line |
| we may need a proper type handling in this part lets leave it for now as it is quotes etc.. should be transferred directly and the rest is up to the toString properly implemented ANS: Both name and value should be quoted | 363 |
| org.apache.myfaces.renderkit.html.HtmlEscapeRendererTest | Line |
| remove these two lines once myfaces-test goes alpha, see MYFACES-1155 | 90 |
| org.apache.myfaces.renderkit.html.HtmlHeadRenderer | Line |
| Move constants to shared HTML class | 46 |
| org.apache.myfaces.renderkit.html.HtmlLabelRenderer | Line |
| Move to extended component | 181 |
| org.apache.myfaces.renderkit.html.HtmlScriptRenderer | Line |
| target check here | 123 |
| org.apache.myfaces.renderkit.html.HtmlTextRendererTest | Line |
| remove these two lines once myfaces-test goes alpha, see MYFACES-1155 | 75 |
| org.apache.myfaces.spi.ViewScopeProvider | Line |
| (Optional) provide something to cleanup the session when a view is discarded. | 27 |
| org.apache.myfaces.spi.impl.AllAnnotationInjectionProvider | Line |
| @EJBs | 33 |
| where i find WebServiceRef? | 57 |
| org.apache.myfaces.spi.impl.NoInjectionAnnotationInjectionProvider | Line |
| the servlet spec is not clear about searching in superclass?? | 131 |
| the servlet spec is not clear about searching in superclass?? May be only check non private fields and methods | 184 |
| org.apache.myfaces.spi.impl.ResourceAnnotationInjectionProvider | Line |
| @Resources | 32 |
| local or global JNDI | 183 |
| local or global JNDI | 188 |
| local or global JNDI | 218 |
| local or global JNDI | 223 |
| org.apache.myfaces.taglib.core.LoadBundleTag | Line |
| We should find a way to save loaded bundles in the state, because otherwise on the next request the bundle map will not be present before the render phase and value bindings that reference to the bundle will always log annoying "Variable 'xxx' could not be resolved" error messages. </p> | 49 |
| org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguage | Line |
| Auto-generated method stub | 1796 |
| this class and ViewHandlerImpl contain same constant <!--@@JSF_FORM_STATE_MARKER@@--> | 1916 |
| org.apache.myfaces.view.facelets.PostBuildComponentTreeOnRestoreViewEvent | Line |
| Remove it since after MYFACES-2389 this is not necessary anymore. Now, PostAddToViewEvent can be triggered inclusive on postback and restore view phase, but note RestoreViewExecutor call facesContext.setProcessingEvents(false) and cause the same effect than the check on UIComponent.setParent() (postback and restore view phase). | 27 |
| org.apache.myfaces.view.facelets.impl.FaceletCacheImpl | Line |
| Note MyFaces core has another type of Facelet for read composite component metadata. The reason behind do this in MyFaces is to retrieve some information related to insertChildren/insertFacet that can be used as metadata and use that information later to do the hack for composite components using templates, instead rely on component relocation. This is not handled by FaceletCache included here but in practice this should not be a problem, because the intention of this class is handle Facelet instances created using custom ResourceResolver stuff, and usually those pages are for views, not for composite components. Even if that is true, composite component metadata Facelet is smaller (only add cc:xxx stuff) that the other ones used for views or the one used to apply the composite component itself. | 35 |
| org.apache.myfaces.view.facelets.mock.MockRequestDispatcher | Line |
| Auto-generated method stub | 48 |
| Auto-generated method stub | 55 |
| org.apache.myfaces.view.facelets.tag.MetadataImpl | Line |
| PROFILE - Check if saving the _size worth it or if an enhanced for is better | 42 |
| org.apache.myfaces.view.facelets.tag.TagAttributesImpl | Line |
| PROFILE - Explore the possibility of using HashMap instead of sorted arrays. The footprint should be higher, but the instanciation and access speed should be faster Instanciation: from O(n log n) to O(1) Access: from O(log n) to O(1) | 33 |
| org.apache.myfaces.view.facelets.tag.composite.ExtensionHandler | Line |
| In theory the xml data inside this tag should be saved, but the spec does not say where and how this should be done. For now we just prevent execute any handler inside this tag. As soon JSR-276 is available, some behavior for this tag should be added. | 51 |
| org.apache.myfaces.view.facelets.tag.composite.SimpleComponentTagHandler | Line |
| Auto-generated constructor stub | 37 |
| org.apache.myfaces.view.facelets.tag.jsf.BehaviorTagHandlerDelegate | Line |
| check if the behavior could be applied to the current parent For run tests it is not necessary, so we let this one pending. | 154 |
| org.apache.myfaces.view.facelets.tag.jsf.ComponentHandler | Line |
| REFACTOR - "facelets.FACET_NAME" should be a constant somewhere, used to be in FacetHandler from real Facelets | 200 |
| org.apache.myfaces.view.facelets.tag.jsf.core.ActionListenerHandler | Line |
| Document me! | 204 |
| org.apache.myfaces.view.facelets.tag.jsf.core.EventHandler | Line |
| The spec is somewhat vague, but I think we're supposed to throw an exception here. The @NamedEvent javadocs say that if a short name is registered to more than one event class that we must throw an exception listing the short name and the classes in the list _when the application makes reference to it_. I believe processing this tag qualifies as the application "making reference" to the short name. Why the exception isn't thrown when processing the @NamedEvent annotation, I don't know. Perhaps follow up with the EG to see if this is correct. | 216 |
| org.apache.myfaces.view.facelets.tag.jsf.core.ResetValuesActionListenerHandler | Line |
| Document me! | 182 |
| org.apache.myfaces.view.facelets.tag.jsf.core.SetPropertyActionListenerHandler | Line |
| Document me! | 179 |
| org.apache.myfaces.view.facelets.tag.jsf.core.ValueChangeListenerHandler | Line |
| Document me! | 193 |
| org.apache.myfaces.view.facelets.tag.jsf.core.ValueChangeListenerImpl | Line |
| Auto-generated constructor stub | 30 |
| Auto-generated method stub | 35 |
| org.apache.myfaces.view.facelets.tag.jsf.core.ViewMetadataTestCase | Line |
| Auto-generated method stub | 80 |
| org.apache.myfaces.view.facelets.tag.jsf.html._HtmlOutputScript | Line |
| DOCUMENT ME! <p> This tag is backed using a javax.faces.component.UIOutput component instance. In other words, instances of this component class are created when it is resolved a Resource annotation, so there is no concrete class or specific tag handler for it, but there exists a concrete renderer for it. </p> | 27 |
| org.apache.myfaces.view.facelets.tag.jsf.html._HtmlOutputStylesheet | Line |
| DOCUMENT ME! This class is just for documentation purposes <p> This tag is backed using a javax.faces.component.UIOutput component instance. In other words, instances of this component class are created when it is resolved a Resource annotation, so there is no concrete class or specific tag handler for it, but there exists a concrete renderer for it. </p> | 27 |
| org.apache.myfaces.view.facelets.tag.jstl.core.MappedValueExpression | Line |
| Auto-generated method stub | 180 |
| Auto-generated method stub | 191 |
| org.apache.myfaces.view.facelets.tag.ui.CompositionHandler | Line |
| REFACTOR - This class could easily use a common parent with DecoratorHandler | 45 |
| org.apache.myfaces.view.facelets.tag.ui.DecorateHandler | Line |
| REFACTOR - This class could easily use a common parent with CompositionHandler | 56 |
| org.apache.myfaces.view.facelets.tag.ui.UIDebug | Line |
| Auto-generated method stub | 222 |
| org.apache.myfaces.view.jsp.JspViewDeclarationLanguageStrategy | Line |
| IMPLEMENT HERE | 38 |
| org.apache.myfaces.webapp.AbstractFacesInitializer | Line |
| is it possible to make a real cleanup? | 417 |
| org.apache.myfaces.webapp.Jsp21FacesInitializer | Line |
| this Class.forName will be removed when Tomcat fixes a bug also, we should then be able to remove jasper.jar from the deployment | 84 |
| org.apache.myfaces.webapp.Jsp21FacesInitializerTest | Line |
| adapt this test case for MyFaces 2.0, because currently it checks nothing! | 60 |
| add myfaces specific tests | 82 |
| org.apache.myfaces.webapp.MyFacesServlet | Line |
| who uses this method? | 53 |
| org.apache.myfaces.webapp.StartupServletContextListener | Line |
| who uses this method? | 192 |