UI-Component Sets

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 ClassTotal number of occurrencesTag strings used by tag class
@todo0@todo
TODO98TODO

Each tag is detailed below:

TODO

Number of occurrences found in the code: 98

org.apache.myfaces.application.ApplicationImplLine
this check should be performed by the expression factory. It is a requirement of the TCK1955
Check if modification occurs often or not, might have to use a synchronized list instead.2700
Check if modification occurs often or not, might have to use a synchronized list instead.2721
org.apache.myfaces.application.ApplicationImplAnnotationTestLine
need mock objects for VDL/VDLFactory remove from excludes list in pom.xml after complete54
org.apache.myfaces.application.ApplicationImplTestLine
need mock objects for VDL/VDLFactory remove from excludes list in pom.xml after complete63
org.apache.myfaces.application.NavigationHandlerImplLine
cache results?348
--No comment--531
org.apache.myfaces.application.ResourceHandlerImplLine
Log using a localized message (which one?)427
org.apache.myfaces.application.ViewHandlerImplLine
JSF 2.0 - need to make sure calculateViewId follows the new algorithm from 7.5.299
org.apache.myfaces.application.jsp.JspViewHandlerImplTestLine
--No comment--349
--No comment--358
--No comment--367
--No comment--376
--No comment--385
org.apache.myfaces.component.visit.PartialVisitContextLine
log warning for trailing colon case311
smarter initial size?346
org.apache.myfaces.config.annotation.AllAnnotationLifecycleProviderLine
@EJBs33
where i find WebServiceRef?57
org.apache.myfaces.config.annotation.AnnotationConfiguratorLine
MartinKoci MYFACES-3053167
All annotations scanned at startup must be configured here!848
org.apache.myfaces.config.annotation.NoInjectionAnnotationLifecycleProviderLine
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 methods162
org.apache.myfaces.config.annotation.ResourceAnnotationLifecycleProviderLine
@Resources32
local or global JNDI183
local or global JNDI188
local or global JNDI218
local or global JNDI223
org.apache.myfaces.config.impl.digester.DigesterFacesConfigUnmarshallerImplLine
validation set to false during implementation of 1.243
org.apache.myfaces.config.impl.digester.elements.BehaviorLine
what about extensions and descriptionGroup elems? Not addressed in other config objects either.36
org.apache.myfaces.context.servlet.PartialViewContextImplLine
process partial rendering https://issues.apache.org/jira/browse/MYFACES-2118 Collection<String> renderIds = getRenderIds();432
Do not check for a specific instance, just render all children. if (comp instanceof javax.faces.component.html.HtmlBody) { comp.encodeAll (_facesContext); } }619
org.apache.myfaces.context.servlet.ServletExternalContextImplTestLine
Invalid test, because EL evaluation should be done before call these methods.68
org.apache.myfaces.convert.ConverterUtilsLine
Move to util package and rename to better name23
org.apache.myfaces.el.unified.FacesELContextLine
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.SetPropertyActionListenerLine
use a Converter before calling setValue71
org.apache.myfaces.lifecycle.DefaultRestoreViewSupportLine
Remove this hack and use VisitHints.SKIP_ITERATION in JSF 2.1117
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...375
org.apache.myfaces.lifecycle.DefaultRestoreViewSupportTestLine
not testable unless static call to RendererUtils is removed213
org.apache.myfaces.mc.test.core.AbstractMyFacesRequestTestCaseLine
check if this is correct98
org.apache.myfaces.mc.test.core.MockDefaultViewDeclarationLanguageLine
VALIDATE - The spec is silent on the following line, but I feel bad if I don't set it105
VALIDATE - Spec is silent on the return value when an error was sent116
org.apache.myfaces.renderkit.ErrorPageWriterTestLine
remove these two lines once myfaces-test goes alpha, see MYFACES-115565
org.apache.myfaces.renderkit.NullReturningGetExpressionStringValueExpressionLine
Auto-generated method stub43
Auto-generated method stub51
Auto-generated method stub58
Auto-generated method stub66
Auto-generated method stub72
Auto-generated method stub78
Auto-generated method stub84
org.apache.myfaces.renderkit.html.HtmlAjaxBehaviorRendererLine
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 quoted280
org.apache.myfaces.renderkit.html.HtmlEscapeRendererTestLine
remove these two lines once myfaces-test goes alpha, see MYFACES-115590
org.apache.myfaces.renderkit.html.HtmlHeadRendererLine
Move constants to shared HTML class44
org.apache.myfaces.renderkit.html.HtmlLabelRendererLine
Move to extended component183
org.apache.myfaces.renderkit.html.HtmlScriptRendererLine
target check here123
org.apache.myfaces.renderkit.html.HtmlTextRendererTestLine
remove these two lines once myfaces-test goes alpha, see MYFACES-115575
org.apache.myfaces.taglib.core.LoadBundleTagLine
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.ViewDeclarationLanguageFactoryImplLine
It would be nice to be able to preinitialize the factory. However, since it requires access to the ExternalContext it may not be possible, depending on the loading order in the FactoryFinder. Could use ideas here. -= SL =-76
org.apache.myfaces.view.facelets.DefaultFaceletsStateManagementStrategyLine
This object should be application scoped and shared between jsp and facelets187
Why is necessary enable event processing? ANS: On RestoreViewExecutor, setProcessingEvents is called first to false and then to true when postback. Since we need listeners registered to PostAddToViewEvent event to be handled, we should enable it again. We are waiting a response from EG about the behavior of those listeners, because for partial state saving we need this listeners be called from here and relocate components properly, but for now we have to let this code as is.277
ask the EG the this is a spec violation if we do it that way see Section 2.2.1369
Why is necessary enable event processing? ANS: On RestoreViewExecutor, setProcessingEvents is called first to false and then to true when postback. Since we need listeners registered to PostAddToViewEvent event to be handled, we should enable it again. We are waiting a response from EG about the behavior of those listeners (see comment on vdl.buildView). -= Leonardo Uribe =- I think enable event processing in this point does not have any side effect. Enable it allows programatically add components when binding is set with pss enabled. That feature works without pss, so we should preserve backward behavior. Tomahawk t:aliasBean example creating components on binding requires this to work. context.setProcessingEvents(true); try { view.visitTree(VisitContext.createVisitContext(context), new RestoreStateCallback()); } finally { context.setProcessingEvents(oldContextEventState); }371
not sure the best way to handle dynamic adds/removes as mandated by the spec.622
org.apache.myfaces.view.facelets.FaceletViewDeclarationLanguageLine
Move to shared project1447
Auto-generated method stub1557
this class and ViewHandlerImpl contain same constant <!--@@JSF_FORM_STATE_MARKER@@-->1675
VALIDATE - Is _buildBeforeRestore relevant at all for 2.0? -= SL =- ANS: buildBeforeRestore evolved to partial state saving, so this logic is now on StateManagerStrategy implementation -= Leo U =-1789
org.apache.myfaces.view.facelets.PostBuildComponentTreeOnRestoreViewEventLine
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.component.UIRepeatLine
Auto-generated method stub1560
org.apache.myfaces.view.facelets.mock.MockRequestDispatcherLine
Auto-generated method stub48
Auto-generated method stub55
org.apache.myfaces.view.facelets.tag.MetadataImplLine
PROFILE - Check if saving the _size worth it or if an enhanced for is better42
org.apache.myfaces.view.facelets.tag.TagAttributesImplLine
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.ExtensionHandlerLine
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.SimpleComponentTagHandlerLine
Auto-generated constructor stub41
org.apache.myfaces.view.facelets.tag.jsf.BehaviorTagHandlerDelegateLine
check if the behavior could be applied to the current parent For run tests it is not necessary, so we let this one pending.153
org.apache.myfaces.view.facelets.tag.jsf.ComponentHandlerLine
REFACTOR - "facelets.FACET_NAME" should be a constant somewhere, used to be in FacetHandler from real Facelets200
org.apache.myfaces.view.facelets.tag.jsf.core.ActionListenerHandlerLine
Document me!203
org.apache.myfaces.view.facelets.tag.jsf.core.AjaxHandlerLine
The spec is not clear about how to deal with composite component instances. The default one proposed here is use a different stack on DefaultFaceletContext.applyCompositeComponent, so components inside composite:implementation tag will not be affected by f:ajax outsider handlers.204
org.apache.myfaces.view.facelets.tag.jsf.core.EventHandlerLine
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.198
org.apache.myfaces.view.facelets.tag.jsf.core.SetPropertyActionListenerHandlerLine
Document me!179
org.apache.myfaces.view.facelets.tag.jsf.core.ValueChangeListenerHandlerLine
Document me!192
org.apache.myfaces.view.facelets.tag.jsf.core.ValueChangeListenerImplLine
Auto-generated constructor stub30
Auto-generated method stub35
org.apache.myfaces.view.facelets.tag.jsf.core.ViewMetadataTestCaseLine
Auto-generated method stub82
org.apache.myfaces.view.facelets.tag.jsf.html._HtmlOutputScriptLine
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._HtmlOutputStylesheetLine
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.MappedValueExpressionLine
Auto-generated method stub180
Auto-generated method stub191
org.apache.myfaces.view.facelets.tag.ui.CompositionHandlerLine
REFACTOR - This class could easily use a common parent with DecoratorHandler46
org.apache.myfaces.view.facelets.tag.ui.DecorateHandlerLine
REFACTOR - This class could easily use a common parent with CompositionHandler56
org.apache.myfaces.view.facelets.tag.ui.UIDebugLine
Auto-generated method stub220
org.apache.myfaces.view.jsp.JspViewDeclarationLanguageStrategyLine
IMPLEMENT HERE38
org.apache.myfaces.webapp.AbstractFacesInitializerLine
is it possible to make a real cleanup?350
org.apache.myfaces.webapp.Jsp21FacesInitializerLine
this Class.forName will be removed when Tomcat fixes a bug also, we should then be able to remove jasper.jar from the deployment83
org.apache.myfaces.webapp.Jsp21FacesInitializerTestLine
adapt this test case for MyFaces 2.0, because currently it checks nothing!60
add myfaces specific tests82
org.apache.myfaces.webapp.MyFacesServletLine
who uses this method?53
org.apache.myfaces.webapp.StartupServletContextListenerLine
who uses this method?184